diff options
author | Pau Espin Pedrol <pespin@sysmocom.de> | 2020-12-02 21:50:40 +0100 |
---|---|---|
committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2020-12-03 16:31:36 +0100 |
commit | dd9abff60b6d450058c88c03c2acc968e8cee8c0 (patch) | |
tree | 2922e703ff76598de51e1eeeeea1496d9d5ea5cd /src/osmo-bsc/nm_bts_sm_fsm.c | |
parent | 64c422858db9388e210875dc681f2d1952e0d0bb (diff) |
nm_bts_sm_fsm: Fix peer_has_no_avstate_offline not applied for nanobts
The condition was set in st_op_disabled_notinstalled_on_enter():
"""
site_mgr->peer_has_no_avstate_offline = (bts->type == GSM_BTS_TYPE_NANOBTS);
"""
However, at startup of the FSM the oneneter func of the default initial
state is not called. In any case, if called it would be too early since
the type is not known yet (because its parsed later on at the VTY with
the "type" command, that's after the "bts X" node is called and the
bts_sm is allocated.
So we need to make sure to always enable it, also for nanobts.
Change-Id: Ic6049a44ae3fca1b8e968fe800c268f579e7cad4
Diffstat (limited to 'src/osmo-bsc/nm_bts_sm_fsm.c')
-rw-r--r-- | src/osmo-bsc/nm_bts_sm_fsm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/osmo-bsc/nm_bts_sm_fsm.c b/src/osmo-bsc/nm_bts_sm_fsm.c index e5b5a3cbe..815eea660 100644 --- a/src/osmo-bsc/nm_bts_sm_fsm.c +++ b/src/osmo-bsc/nm_bts_sm_fsm.c @@ -78,8 +78,8 @@ static void st_op_disabled_notinstalled(struct osmo_fsm_inst *fi, uint32_t event "have your .cfg with 'type nanobts'. Otherwise, you probably " "are using an old osmo-bts; automatically adjusting OML " "behavior to be backward-compatible.\n"); - site_mgr->peer_has_no_avstate_offline = true; } + site_mgr->peer_has_no_avstate_offline = true; nm_bts_sm_fsm_state_chg(fi, NM_BTS_SM_ST_OP_ENABLED); return; } |