<p>neels has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20626">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">BSSMAP RESET: move cancel-paging call to osmo_bsc_sigtran_reset()<br><br>So far we would cancel ongoing Paging for a given MSC only after receiving a<br>RESET message from that BSC. However, the typical operation would be that<br>OsmoBSC *sends* a RESET and receives a RESET-ACK.<br><br>Instead, move the call to within osmo_bsc_sigtran_reset(). This is also called<br>when OsmoBSC considers the A-interface link to be lost, from the a_reset.c<br>code, after multiple SCCP connection failures.<br><br>Change-Id: Ia14b916be56563d18632c69a833084e71799a468<br>---<br>M include/osmocom/bsc/osmo_bsc_sigtran.h<br>M src/osmo-bsc/osmo_bsc_bssap.c<br>M src/osmo-bsc/osmo_bsc_sigtran.c<br>3 files changed, 6 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/26/20626/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/bsc/osmo_bsc_sigtran.h b/include/osmocom/bsc/osmo_bsc_sigtran.h</span><br><span>index 79afa6b..df37cf4 100644</span><br><span>--- a/include/osmocom/bsc/osmo_bsc_sigtran.h</span><br><span>+++ b/include/osmocom/bsc/osmo_bsc_sigtran.h</span><br><span>@@ -38,7 +38,7 @@</span><br><span> int osmo_bsc_sigtran_init(struct llist_head *mscs);</span><br><span> </span><br><span> /* Close all open sigtran connections and channels */</span><br><span style="color: hsl(0, 100%, 40%);">-void osmo_bsc_sigtran_reset(const struct bsc_msc_data *msc);</span><br><span style="color: hsl(120, 100%, 40%);">+void osmo_bsc_sigtran_reset(struct bsc_msc_data *msc);</span><br><span> </span><br><span> void osmo_bsc_sigtran_tx_reset(const struct bsc_msc_data *msc);</span><br><span> </span><br><span>diff --git a/src/osmo-bsc/osmo_bsc_bssap.c b/src/osmo-bsc/osmo_bsc_bssap.c</span><br><span>index c2c05af..d37b3e0 100644</span><br><span>--- a/src/osmo-bsc/osmo_bsc_bssap.c</span><br><span>+++ b/src/osmo-bsc/osmo_bsc_bssap.c</span><br><span>@@ -100,9 +100,6 @@</span><br><span>        * close all active channels on the BTS side as well */</span><br><span>      osmo_bsc_sigtran_reset(msc);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        /* Drop all ongoing paging requests that this MSC has created on any BTS */</span><br><span style="color: hsl(0, 100%, 40%);">-     paging_flush_network(msc->network, msc);</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>  update_msc_osmux_support(msc, msg, length);</span><br><span> </span><br><span>      /* Inform the MSC that we have received the reset request and</span><br><span>diff --git a/src/osmo-bsc/osmo_bsc_sigtran.c b/src/osmo-bsc/osmo_bsc_sigtran.c</span><br><span>index 75b3b58..c2df17e 100644</span><br><span>--- a/src/osmo-bsc/osmo_bsc_sigtran.c</span><br><span>+++ b/src/osmo-bsc/osmo_bsc_sigtran.c</span><br><span>@@ -35,6 +35,7 @@</span><br><span> #include <osmocom/bsc/bsc_subscr_conn_fsm.h></span><br><span> #include <osmocom/bsc/gsm_data.h></span><br><span> #include <osmocom/bsc/bts.h></span><br><span style="color: hsl(120, 100%, 40%);">+#include <osmocom/bsc/paging.h></span><br><span> #include <osmocom/mgcp_client/mgcp_common.h></span><br><span> </span><br><span> /* A pointer to a list with all involved MSCs</span><br><span>@@ -415,11 +416,14 @@</span><br><span> }</span><br><span> </span><br><span> /* Close all open sigtran connections and channels */</span><br><span style="color: hsl(0, 100%, 40%);">-void osmo_bsc_sigtran_reset(const struct bsc_msc_data *msc)</span><br><span style="color: hsl(120, 100%, 40%);">+void osmo_bsc_sigtran_reset(struct bsc_msc_data *msc)</span><br><span> {</span><br><span>         struct gsm_subscriber_connection *conn, *conn_temp;</span><br><span>  OSMO_ASSERT(msc);</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+ /* Drop all ongoing paging requests that this MSC has created on any BTS */</span><br><span style="color: hsl(120, 100%, 40%);">+   paging_flush_network(msc->network, msc);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>        /* Close all open connections */</span><br><span>     llist_for_each_entry_safe(conn, conn_temp, &bsc_gsmnet->subscr_conns, entry) {</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20626">change 20626</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/+/20626"/><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: Ia14b916be56563d18632c69a833084e71799a468 </div>
<div style="display:none"> Gerrit-Change-Number: 20626 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>