<p style="white-space: pre-wrap; word-wrap: break-word;">I'm not quite sure why we would want to pass CTRL commands into the ConnHdlr.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The ConnHdlr exists for each "subscriber connection", so we can have different ConnHdlr components, one for each subscriber connection.  It makes sense to dispatch things like BSSAP, RSL dedicated channel/RLL, as well as MGCP to it, as all of those exist logically separate for each ConnHdlr.  Hoewver, the CTRL commands are BTS-global and they should IMHO not be handled in a per-connection handler component.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/14407">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/14407">change 14407</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-ttcn3-hacks/+/14407"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I3640f6889e3b6adc3b4601082f948543c64a2fe3 </div>
<div style="display:none"> Gerrit-Change-Number: 14407 </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-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 08 Jun 2019 09:56:42 +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>