<p>neels <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/14700">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, but someone else must approve
  neels: Looks good to me, approved
  osmith: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">remove double BSSMAP Clear on HO failure<br><br>If a handover fails when the new lchan is already fully established, osmo-bsc<br>so far caused two BSSMAP Clear Requests to be sent out to the MSC: one caused<br>by detaching the lchan from the gscon, one from returning the gscon back to<br>ST_ACTIVE, which detects that no lchan is present and Clears. In fact only one<br>of those is necessary.<br><br>Checking for the presence of an lchan when entering ST_ACTIVE is an earlier<br>attempt to catch insane situations. Since then, osmo-bsc has acquired other<br>logic that will ensure sending a Clear Request in all cases, see<br>gscon_forget_lchan(). Sending another BSSMAP Clear Request in ST_ACTIVE's<br>onenter is simply not necessary. Drop gscon_fsm_active_onenter() entirely.<br><br>Note: the double Clear Request is currently hit by<br>TC_ho_out_fail_no_ho_detect(), which currently fails and will pass again after<br>this patch; however, osmo-bsc should actually not release the lchan at all<br>during this test, see OS#4093. In other words, osmo-bsc behavior for this<br>scenario as well as TC_ho_out_fail_no_ho_detect() need to be changed, and the<br>test will, once fixed, not be useful to trigger this issue anymore.<br><br>Related: OS#4078<br>Change-Id: Iac1519eb8b24e8523caec682f9ac8e6dcf1327ce<br>---<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>1 file changed, 0 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/bsc_subscr_conn_fsm.c b/src/osmo-bsc/bsc_subscr_conn_fsm.c</span><br><span>index bc5cb27..f8784f9 100644</span><br><span>--- a/src/osmo-bsc/bsc_subscr_conn_fsm.c</span><br><span>+++ b/src/osmo-bsc/bsc_subscr_conn_fsm.c</span><br><span>@@ -343,13 +343,6 @@</span><br><span>   }</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-static void gscon_fsm_active_onenter(struct osmo_fsm_inst *fi, uint32_t prev_state)</span><br><span style="color: hsl(0, 100%, 40%);">-{</span><br><span style="color: hsl(0, 100%, 40%);">-  struct gsm_subscriber_connection *conn = fi->priv;</span><br><span style="color: hsl(0, 100%, 40%);">-   if (!conn->lchan)</span><br><span style="color: hsl(0, 100%, 40%);">-            gscon_bssmap_clear(conn, GSM0808_CAUSE_EQUIPMENT_FAILURE);</span><br><span style="color: hsl(0, 100%, 40%);">-}</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span> /* We're on an active subscriber connection, passing DTAP back and forth */</span><br><span> static void gscon_fsm_active(struct osmo_fsm_inst *fi, uint32_t event, void *data)</span><br><span> {</span><br><span>@@ -605,7 +598,6 @@</span><br><span>                               S(GSCON_EV_HANDOVER_START),</span><br><span>                 .out_state_mask = S(ST_CLEARING) | S(ST_ASSIGNMENT) |</span><br><span>                                  S(ST_HANDOVER),</span><br><span style="color: hsl(0, 100%, 40%);">-               .onenter = gscon_fsm_active_onenter,</span><br><span>                 .action = gscon_fsm_active,</span><br><span>  },</span><br><span>   [ST_ASSIGNMENT] = {</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/14700">change 14700</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/+/14700"/><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: Iac1519eb8b24e8523caec682f9ac8e6dcf1327ce </div>
<div style="display:none"> Gerrit-Change-Number: 14700 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>