Attention is currently required from: laforge, pespin. osmith 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:

(2 comments)

Patchset:

PS3: Somewhat unrelated, but tests/paging/paging_test.ok is hard to review as it has so many lines. Maybe it makes sense to change the test in another patch so it outputs something less verbose?

File src/osmo-bsc/paging.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/28059/comment/f3283d15_bf9e44e5 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)". Both times are estimates, but "to" is cut off at 60 seconds.

So maybe log a separate message in case the bound is not hit, which only has the number once?

(And maybe the message could be reworded to make it more clear in case the upper bound is reached? But I don't have a good suggestion so maybe just leave as is.)