[PATCH] openbsc[master]: dyn TS: fix: properly run an lchan activation timeout

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Wed Aug 24 16:56:33 UTC 2016


Review at  https://gerrit.osmocom.org/759

dyn TS: fix: properly run an lchan activation timeout

Actually schedule an activation timer for the activation part of a dyn TS
switchover. It needs to be restarted because the channel release procedure in
the first part of a switchover actually removes the activation timer.

Change-Id: Ibf50d13ba10298464a8b07e34716763161438990
---
M openbsc/src/libbsc/abis_rsl.c
1 file changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/59/759/1

diff --git a/openbsc/src/libbsc/abis_rsl.c b/openbsc/src/libbsc/abis_rsl.c
index 841aa7b..1e9cecd 100644
--- a/openbsc/src/libbsc/abis_rsl.c
+++ b/openbsc/src/libbsc/abis_rsl.c
@@ -2487,6 +2487,13 @@
 	lchan->dyn.rqd_ref = NULL;
 	lchan->dyn.rqd_ta = 0;
 
+	/* During switchover, we have received a release ack, which means that
+	 * the act_timer has been stopped. Start the timer again so we mark
+	 * this channel broken if the activation ack comes too late. */
+	lchan->act_timer.cb = lchan_act_tmr_cb;
+	lchan->act_timer.data = lchan;
+	osmo_timer_schedule(&lchan->act_timer, 4, 0);
+
 	rc = rsl_chan_activate_lchan(lchan, act_type, ho_ref);
 	if (rc) {
 		LOGP(DRSL, LOGL_ERROR,

-- 
To view, visit https://gerrit.osmocom.org/759
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibf50d13ba10298464a8b07e34716763161438990
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>



More information about the gerrit-log mailing list