<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408/1/include/osmo-bts/bts.h">File include/osmo-bts/bts.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408/1/include/osmo-bts/bts.h@148">Patch Set #1, Line 148:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">/* NM BTS Site Manager FSM */<br>             struct osmo_fsm_inst *fi;<br>             bool opstart_success;<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">why don't we move this into the gsm_abis_mo?  Shouldn't every one of them have a fsm_inst and some state whether or not it was opstarted successfully?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408/1/src/common/bts.c">File src/common/bts.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408/1/src/common/bts.c@225">Patch Set #1, Line 225:</a> <code style="font-family:monospace,monospace">bts_sm%d</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">AFAIR, there can be only one site manager, it's not a per-BTS entity. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">fixeria is correct.  I made that mistake early on when we started with the BS-11, where there was only one BTS per physical box and hence a 1:1 relationship with the site manager.</p><p style="white-space: pre-wrap; word-wrap: break-word;">for osmo-bts, I currently don't forsee us having multiple logicla BTSs within one process, so it is not super critical.</p><p style="white-space: pre-wrap; word-wrap: break-word;">the biggest problem about this mistake is in osmo-bsc.  Particularly in context of RBS6000 support, it is quite sad that we currently can only support one BTS, while the hardware can do as many logical BTS within 24 TRX as we want.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408">change 20408</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-bts/+/20408"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iab2d17c45c9642860cd2d5d523c1baae24502243 </div>
<div style="display:none"> Gerrit-Change-Number: 20408 </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-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 02 Oct 2020 21:42:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>