Attention is currently required from: osmith, laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28059 )
Change subject: paging: Implement upper bound of 60s for dynamic T3113 ......................................................................
Patch Set 3:
(1 comment)
File src/osmo-bsc/paging.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/28059/comment/98c4f120_559225d4 PS3, Line 394: to, estimated_to);
I think it's not obvious why in most cases you would have "expires in 5 seconds (estimated 5)". […]
I think the easiest is leaving it as this. In generalthe expire is the same as the estimated, so nothing confusing. In the event they are different, one will probably see several of them in near time, and the expire will be always 60 with a higher estimate, so it's easy to infer it tops at 60. Later on if we make the 60 configurable (if we deem necessary) we can update the log to tell the user to tweak the VTY option.