pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/32239 )

Change subject: nm: Dispatch NM_EV_SW_ACT in cascade to BTS SiteMgr children ......................................................................

nm: Dispatch NM_EV_SW_ACT in cascade to BTS SiteMgr children

Change-Id: I97445812bb1b6de450411aceaeece2427027ae67 Related: OS#5994 --- M src/common/bts.c M src/common/nm_bts_sm_fsm.c 2 files changed, 11 insertions(+), 2 deletions(-)

Approvals: laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified

diff --git a/src/common/bts.c b/src/common/bts.c index 9c48a1d..37a3023 100644 --- a/src/common/bts.c +++ b/src/common/bts.c @@ -457,8 +457,6 @@

/* BTS SITE MGR becomes Offline (tx SW ACT Report), BTS, NSE, etc. is DEPENDENCY */ osmo_fsm_inst_dispatch(bts->site_mgr.mo.fi, NM_EV_SW_ACT, NULL); - osmo_fsm_inst_dispatch(bts->mo.fi, NM_EV_SW_ACT, NULL); - osmo_fsm_inst_dispatch(bts->gprs.nse.mo.fi, NM_EV_SW_ACT, NULL);

/* All other objects start off-line until the BTS Model code says otherwise */ for (i = 0; i < bts->num_trx; i++) { diff --git a/src/common/nm_bts_sm_fsm.c b/src/common/nm_bts_sm_fsm.c index f775265..8752a90 100644 --- a/src/common/nm_bts_sm_fsm.c +++ b/src/common/nm_bts_sm_fsm.c @@ -69,6 +69,7 @@ case NM_EV_SW_ACT: oml_mo_tx_sw_act_rep(&site_mgr->mo); nm_bts_sm_fsm_state_chg(fi, NM_BTS_SM_ST_OP_DISABLED_OFFLINE); + ev_dispatch_children(site_mgr, event); return; default: OSMO_ASSERT(0);