fixeria submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

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
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(-)

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 change 42910. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I109ab282986b68b68ba5c11859c44b771c0416fd
Gerrit-Change-Number: 42910
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>