aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStefan Sperling <ssperling@sysmocom.de>2018-04-16 13:56:56 +0200
committerStefan Sperling <ssperling@sysmocom.de>2018-04-16 13:56:56 +0200
commit92a661e6a68dc4dd8a23cca6f9ee0d50545f439c (patch)
treea97bfdeaaf7a60789b57850b954dc777f48447f3
parent9ab2c124859a5b5666183f1efba487f0955dacdf (diff)
trigger acc ramping on state-changed-event reports
Trigger ACC ramping not only when an Administrative State Change ACK is received from a BTS, but also when an administrative state change is reported for TRX 0 in a State Changed Event Report. This should allow ACC ramping to work with any BTS which reports an administrative state change to 'unlock' using either of these OML messages. Tested with a sysmobts and a nanobts. The sysmobts only reports TRX locked/unlock changes in Administrative State Change ACKs, not via State Changed Event Reports. The nanobts is known to send both of these OML messages in quick succession, so do not re-trigger ramping if it's already in progress. This is a port of osmo-bsc commit b06c7a253752ecb67fd20cdf0b069688b561af0e Change-Id: I6443635b822b6cd776f6dc8a6ee73ab09e865b04 Related: OS#2591
-rw-r--r--openbsc/src/libbsc/acc_ramp.c12
1 files changed, 10 insertions, 2 deletions
diff --git a/openbsc/src/libbsc/acc_ramp.c b/openbsc/src/libbsc/acc_ramp.c
index 03ba80bee..a86645818 100644
--- a/openbsc/src/libbsc/acc_ramp.c
+++ b/openbsc/src/libbsc/acc_ramp.c
@@ -144,7 +144,8 @@ static int acc_ramp_nm_sig_cb(unsigned int subsys, unsigned int signal, void *ha
struct acc_ramp *acc_ramp = handler_data;
struct gsm_bts_trx *trx = NULL;
- if (signal != S_NM_STATECHG_ADM)
+ /* Handled signals map to an Administrative State Change ACK, or a State Changed Event Report. */
+ if (signal != S_NM_STATECHG_ADM && signal != S_NM_STATECHG_OPER)
return 0;
if (nsd->obj_class != NM_OC_RADIO_CARRIER)
@@ -163,7 +164,14 @@ static int acc_ramp_nm_sig_cb(unsigned int subsys, unsigned int signal, void *ha
/* Trigger or abort ACC ramping based on the new 'RF lock' state of this TRX. */
switch (nsd->new_state->administrative) {
case NM_STATE_UNLOCKED:
- acc_ramp_trigger(acc_ramp);
+ /*
+ * Do not re-trigger ACC ramping if ramping is already in progress.
+ * A BTS might send several "unlock" change events: One in the Administrative
+ * State Change ACK, and/or another in a State Changed Event Report.
+ * For instance, the nanobts is known to send both.
+ */
+ if (!osmo_timer_pending(&acc_ramp->step_timer))
+ acc_ramp_trigger(acc_ramp);
break;
case NM_STATE_LOCKED:
case NM_STATE_SHUTDOWN: