<p style="white-space: pre-wrap; word-wrap: break-word;">I didn't know about abis_om2000.c before, I had a look now. I also thought about having a shared FSM for different objects, but in the end I decided to go to one different FSM per NM object since in the long term I think it makes sense to control them differently because different actions need to be taken (and different specific workarounds need to be applied on different objects depending on which type of peer are we talking too). I think this difference will become more evident as we start adding more fine control and features to each object.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This current implementation is not aiming to be the last ultimate implementation for these objects, but I think it's a pretty good one to start with and improve iteratively from. I'm open to changing stuff but I think it makes sense to have it merged without major changes and then start doing improvements on top of that, since I have spent already quite a lot of time on it and I also need to move on to other stuff which also requires my attention, and this implementation already solves some of the issues I spotted and/or provides means to fix them quickly, which means we avoid having currently broken setups (like sysmobts or previously some osmo-bts-trx ones).</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20400">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/+/20400">change 20400</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/+/20400"/><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: Ic001ce6ebeff6f51470ef58140b0235f4a30265e </div>
<div style="display:none"> Gerrit-Change-Number: 20400 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 05 Oct 2020 09:04:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>