<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review+1</p><p style="white-space: pre-wrap; word-wrap: break-word;">The patch looks fine, but I'd like laforge to ACK that we want to remove this.<br>IIUC this feature was used by a customer to redirect to different MSCs based on IMSI prefix, but iirc that customer is anyway using osmo-bsc-sccplite, so not affected by this.\</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes. The use case was as follows:</p><ul><li>BTS+BSC are on a remote site, with A/SCCPlite backhaul to the normal public network MSC</li><li>for some internal MSISDNs (think of a "PBX use case"), the connections would be transferred to a local MSC next to the BSC.  In fact, there actually was an (ISDN) PBX next to the BSC which would connect to local phones.  The reason was to reduce load on the backhaul and the public network, which also meant significant economic savings.</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">Can all be removed. I'm happy to reduce such non-standard complexity.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18496">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18496">change 18496</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/+/18496"/><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: Id54afe8ccf0e11b9121a733224054c9565eafb58 </div>
<div style="display:none"> Gerrit-Change-Number: 18496 </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-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 27 May 2020 20:50:57 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>