diff options
author | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-09-22 18:15:12 +0200 |
---|---|---|
committer | pespin <pespin@sysmocom.de> | 2021-09-23 12:07:29 +0000 |
commit | 130e2e59c65aba55a2402ad8c09afc00a7c1c14a (patch) | |
tree | 66ccecc63f6a578e719155f0e5b33fc9ebdc9387 /include | |
parent | 49979bb79444545756f52a028587fa092f9e1bbd (diff) |
bts_trx: Drop non-executed path in trx_link_estab()
This function is only called during sign_link_up() e1inp callback, hence
only the link!=NULL condition (UP) is ever executed. Let's drop the DOWN
path and make it a function only used to trigger events when link
becomes up, similar to what bts_link_estab() does with OML.
Here it becomes clear the NM_EV_RSL_DOWN was never sent. It's not much
of an issue though since it would only make transition RCARRIER/BBTRANSC
Enabled->DisabledOffline. However, since due to libosmo-abis limitation
we receive a sign_link_down() for the entire line when 1 of its links
goes down, we don't care much since we go for shutdown of the entire BTS
anyway. Ideally, libosmo-abis would support simply telling us 1 of the
links in the line went down and if it was not OML and not RSL TRX==C0,
then we could keep on running and simply disable the related TRX.
Change-Id: Iac553c68339c0da32fd313676995747eb4344087
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions