<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 3:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(3 comments)</p><p style="white-space: pre-wrap; word-wrap: break-word;">AS I said, I'd rather implement the roundrobin by keeping a pointer to last/next msc's llisthead. Since we have and use llists here, let's use their benefits.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Keeping an actual pointer has one fundamental flaw: if the MSC gets removed for any reason, the pointer points at a segmentation fault, and it is a potential source of hard to fix bugs.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Of course, I think we actually have no way of removing MSCs from osmo-bsc, do we? But consider that we could at some point introduce a 'no msc 2' command to change MSC pooling in a live running BSC instance without restart. It is actually likely to be requested by a customer at some point.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If I keep an msc_nr, no matter how many MSCs we add or remove, re-iterating existing MSCs will never ever end up with a segfault situation. That is why I would prefer to stay with storing MSC numbers instead of pointers.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, we won't really save any cycles, because for NRI matching, we need to iterate all MSCs anyway. Having a direct pointer will not be more optimal.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/18505">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/+/18505">change 18505</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/+/18505"/><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: Idf71f07ba5a17d5b870dc1a5a2875b6fedb61291 </div>
<div style="display:none"> Gerrit-Change-Number: 18505 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 29 May 2020 20:38:50 +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>