fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/42910?usp=email )
(
1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: common: flush paging queue in nm_bts_fsm's NOTINSTALLED on_enter ......................................................................
common: flush paging queue in nm_bts_fsm's NOTINSTALLED on_enter
paging_reset() exists to flush stale paging records from the queue, but was never called anywhere. Stale paging records can accumulate when the OML link goes down: osmo-bts does not exit, so the paging queue survives into the next BSC session.
Call it alongside bts_cbch_reset() and bts_asci_notification_reset() in st_op_disabled_notinstalled_on_enter(), which fires after all TRXs are confirmed closed and before a new OML connection is accepted.
Change-Id: I109ab282986b68b68ba5c11859c44b771c0416fd AI-Assisted: yes (Claude) --- M src/common/nm_bts_fsm.c 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved laforge: Looks good to me, approved osmith: Looks good to me, but someone else must approve
diff --git a/src/common/nm_bts_fsm.c b/src/common/nm_bts_fsm.c index 5c4a534..e5bfb76 100644 --- a/src/common/nm_bts_fsm.c +++ b/src/common/nm_bts_fsm.c @@ -37,6 +37,7 @@ #include <osmo-bts/phy_link.h> #include <osmo-bts/cbch.h> #include <osmo-bts/notification.h> +#include <osmo-bts/paging.h>
#define X(s) (1 << (s))
@@ -69,6 +70,7 @@ TALLOC_FREE(bts->mo.nm_attr); bts_cbch_reset(bts); bts_asci_notification_reset(bts); + paging_reset(bts->paging_state); if (bts->c0_power_red_db > 0) bts_set_c0_pwr_red(bts, 0);