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);
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42910?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I109ab282986b68b68ba5c11859c44b771c0416fd
Gerrit-Change-Number: 42910
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
