Change in osmo-bsc[master]: lchan_fsm: Improve timeout logging line in state WAIT_RLL_RTP_ESTABLISH

pespin gerrit-no-reply at
Fri Jul 9 11:32:40 UTC 2021

pespin has submitted this change. ( )

Change subject: lchan_fsm: Improve timeout logging line in state WAIT_RLL_RTP_ESTABLISH

lchan_fsm: Improve timeout logging line in state WAIT_RLL_RTP_ESTABLISH

This allows quickly finding out whether the timeout is happening due to
RLL not established or whether RTP is not ready. In essence, it
indicates whether issue may be coming from MGW or from MS/BTS side.

If coming from MGW, the timer T3101 is in general not a problem and the
issue should be related to some misbehaving.
If coming from MS/BTS, T3101 may require tunning (it could be a
misbehaving of the MS/BTS too, or simply bad signal).

Related: SYS#5526
Change-Id: Ib655f71aec584962c70d84a4405d996505dff53c
M src/osmo-bsc/lchan_fsm.c
1 file changed, 8 insertions(+), 1 deletion(-)

  laforge: Looks good to me, but someone else must approve
  dexter: Looks good to me, approved
  Jenkins Builder: Verified

diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index e3b6c83..841415f 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -1712,7 +1712,14 @@
 		lchan->release.in_error = true;
 		lchan->release.rsl_error_cause = RSL_ERR_INTERWORKING;
 		lchan->release.rr_cause = bsc_gsm48_rr_cause_from_rsl_cause(lchan->release.rsl_error_cause);
-		lchan_fail("Timeout");
+		if (fi->state  == LCHAN_ST_WAIT_RLL_RTP_ESTABLISH) {
+			lchan_fail("Timeout (rll_ready=%s,voice_require=%s,voice_ready=%s)",
+				   (lchan->sapis[0] != LCHAN_SAPI_UNUSED) ? "yes" : "no",
+				   lchan-> ? "yes" : "no",
+				   lchan_rtp_established(lchan) ? "yes" : "no");
+		} else {
+			 lchan_fail("Timeout");
+		}
 		return 0;

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ib655f71aec584962c70d84a4405d996505dff53c
Gerrit-Change-Number: 24890
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier at>
Gerrit-Reviewer: laforge <laforge at>
Gerrit-Reviewer: neels <nhofmeyr at>
Gerrit-Reviewer: pespin <pespin at>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list