<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24890">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">lchan_fsm: Improve timeout logging line in state WAIT_RLL_RTP_ESTABLISH<br><br>This allows quickly finding out whether the timeout is happening due to<br>RLL not established or whether RTP is not ready. In essence, it<br>indicates whether issue may be coming from MGW or from MS/BTS side.<br><br>If coming from MGW, the timer T3101 is in general not a problem and the<br>issue should be related to some misbehaving.<br>If coming from MS/BTS, T3101 may require tunning (it could be a<br>misbehaving of the MS/BTS too, or simply bad signal).<br><br>Related: SYS#5526<br>Change-Id: Ib655f71aec584962c70d84a4405d996505dff53c<br>---<br>M src/osmo-bsc/lchan_fsm.c<br>1 file changed, 8 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/90/24890/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c</span><br><span>index 6350481..5a6d27c 100644</span><br><span>--- a/src/osmo-bsc/lchan_fsm.c</span><br><span>+++ b/src/osmo-bsc/lchan_fsm.c</span><br><span>@@ -1704,7 +1704,14 @@</span><br><span>                lchan->release.in_error = true;</span><br><span>           lchan->release.rsl_error_cause = RSL_ERR_INTERWORKING;</span><br><span>            lchan->release.rr_cause = bsc_gsm48_rr_cause_from_rsl_cause(lchan->release.rsl_error_cause);</span><br><span style="color: hsl(0, 100%, 40%);">-              lchan_fail("Timeout");</span><br><span style="color: hsl(120, 100%, 40%);">+              if (fi->state  == LCHAN_ST_WAIT_RLL_RTP_ESTABLISH) {</span><br><span style="color: hsl(120, 100%, 40%);">+                       lchan_fail("Timeout (rll_ready=%s,voice_require=%s,voice_ready=%s)",</span><br><span style="color: hsl(120, 100%, 40%);">+                                   (lchan->sapis[0] != LCHAN_SAPI_UNUSED) ? "yes" : "no",</span><br><span style="color: hsl(120, 100%, 40%);">+                                 lchan->activate.info.requires_voice_stream ? "yes" : "no",</span><br><span style="color: hsl(120, 100%, 40%);">+                             lchan_rtp_established(lchan) ? "yes" : "no");</span><br><span style="color: hsl(120, 100%, 40%);">+          } else {</span><br><span style="color: hsl(120, 100%, 40%);">+                       lchan_fail("Timeout");</span><br><span style="color: hsl(120, 100%, 40%);">+             }</span><br><span>            return 0;</span><br><span>    }</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24890">change 24890</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/24890"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ib655f71aec584962c70d84a4405d996505dff53c </div>
<div style="display:none"> Gerrit-Change-Number: 24890 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>