<p>neels <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19864">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">CBSP: rewrite the CBSP link setup and 'cbc' VTY section<br><br>Firstly, make CBSP server and client mutually exclusive: Do not allow osmo-bsc<br>to be configured as CBC client *and* CBC server at the same time.<br>cbsp_link.c expects at most one CBSP link to be established, and, upon sending<br>CBSP messages, probes whether to send the message to a CBSP server or client<br>link. When both listen-port and remote-ip are configured (regardless of an<br>actual CBSP connection), osmo-bsc gets confused about where to send CBSP<br>messages.<br><br>One solution would be more accurate probing for an actual established TCP<br>connection. But the simpler and less confusing solution is to force the user to<br>configure only server or only client mode, never both.<br><br>Introduce 'cbc' / 'mode (server|client|disabled)'.<br><br>Secondly, clarify the 'cbc' config structure into distinct 'server' and<br>'client' subnodes. Refactor the 'cbc' VTY node in such a way that the IP<br>addresses for server and client mode can remain configured when the CBSP link<br>is switched between server/client/disabled modes.<br><br>To implement the above, switch the struct bsc_cbc_link to use osmo_sockaddr_str<br>for address configuration.<br><br>Related: OS#4702<br>Related: I7eea0dd39de50ed80af79e0f10c836b8685d8644 (osmo-ttcn3-hacks)<br>Related: I9e9760121265b3661f1c179610e975cf7a0873f1 (docker-playground)<br>Change-Id: Icaa2775cc20a99227dabe38a775ff808b374cf98<br>---<br>M include/osmocom/bsc/smscb.h<br>M include/osmocom/bsc/vty.h<br>M src/osmo-bsc/bsc_init.c<br>M src/osmo-bsc/cbsp_link.c<br>A tests/cbc.vty<br>5 files changed, 358 insertions(+), 110 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/64/19864/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19864">change 19864</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/+/19864"/><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: Icaa2775cc20a99227dabe38a775ff808b374cf98 </div>
<div style="display:none"> Gerrit-Change-Number: 19864 </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@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>