dexter has uploaded this change for review. (
https://gerrit.osmocom.org/c/osmo-bsc/+/28916 )
Change subject: abis_om2000: om2k_trx_s_done_onenter() does not send signal
......................................................................
abis_om2000: om2k_trx_s_done_onenter() does not send signal
the function om2k_trx_s_done_onenter() updates the administrative state
of the TRX oml MO but it does not notify the update to other entities
using S_NM_STATECHG
Change-Id: Iabf9f3a1a345c5d53d9a4d02fa2d6d13ddfd86ae
Related: OS#5634
---
M src/osmo-bsc/abis_om2000.c
1 file changed, 12 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/16/28916/1
diff --git a/src/osmo-bsc/abis_om2000.c b/src/osmo-bsc/abis_om2000.c
index ac5b401..fa6c067 100644
--- a/src/osmo-bsc/abis_om2000.c
+++ b/src/osmo-bsc/abis_om2000.c
@@ -2225,9 +2225,20 @@
static void om2k_trx_s_done_onenter(struct osmo_fsm_inst *fi, uint32_t prev_state)
{
struct om2k_trx_fsm_priv *otfp = fi->priv;
+ struct nm_statechg_signal_data nsd;
+
+ memset(&nsd, 0, sizeof(nsd));
+
+ nsd.bts = otfp->trx->bts;
+ nsd.obj = &otfp->trx;
+ nsd.old_state = otfp->trx->mo.nm_state;
+ nsd.new_state = otfp->trx->mo.nm_state;
+ nsd.om2k_mo = &otfp->trx->rbs2000.trxc.om2k_mo.addr;
/* See e1_config:bts_isdn_sign_link() / OS#4914 */
- otfp->trx->mo.nm_state.administrative = NM_STATE_UNLOCKED;
+ nsd.new_state.administrative = NM_STATE_UNLOCKED;
+ otfp->trx->mo.nm_state.administrative = nsd.new_state.administrative;
+ osmo_signal_dispatch(SS_NM, S_NM_STATECHG, &nsd);
abis_om2000_fsm_becomes_enabled(otfp->trx);
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/28916
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Iabf9f3a1a345c5d53d9a4d02fa2d6d13ddfd86ae
Gerrit-Change-Number: 28916
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newchange