Attention is currently required from: daniel. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/30363 )

Change subject: paging: Split paging queue into 2 queues: initial and retrans ......................................................................

Patch Set 1:

(1 comment)

File src/osmo-bsc/paging.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/30363/comment/7c1293f2_d2b0c38e PS1, Line 263: bts_pag_st->initial_req_list_len--;

You already adjust the *_list_len accordingly in paging_remove_request(). […]

So you basically suggesting to move the -- and ++ lines inside paging_req_timeout_trans() within a "if (req->attempts==0)" conditional block, right?

So same behavior in the end, just different place. I can do that, it may be clearer for people when reading the lifecycle of requests across the different lists.