<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 15:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I've stumbled across this old patch.  As far as I can see, it makes LCLS disable-able.  The question is: Do we wnat that?  @keith, do you remember the context of this patch?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The context is possibly explained in https://osmocom.org/issues/2487</p><p style="white-space: pre-wrap; word-wrap: break-word;">There were 3 patches in Max's original patchset. This one, the one I voted down, (I don't think we need it) and https://gerrit.osmocom.org/c/osmo-msc/+/13422/18 which is marked WIP but should not be, as I removed the MNCC version change that @neels rightly objected to in the latest patchset.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The two proposed patches require further patchsets to become operable which depend on deciding how to encode the LCLS CGR (Global Call Reference) in SIP so that we transfer to/from the SIP connector.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Part of these further patch sets (prepared but not yet submitted) bump the MNCC version in both MSC and SIP connector. </p><p style="white-space: pre-wrap; word-wrap: break-word;">In forwarding porting this and the other above mentioned patch, I have tried to deviate as little as possible from the original functional changes and intention of Max's patch. </p><ul><li>see commit message "That's experimental patch to facilitate _TESTING_ ......"</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">so that they can be merged and then expanded on. As I commented on gerrit 13422, I guess the other option is to abandon this and write a new patchset, but for the moment I decided to leave these and go ahead with the follow up patches that complete the functionality.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>If we merge it, I would rather say it is "permit lcls" than "enable lcls", as whether or not LCLS can be used depends on a lot of other conditions, not just this newly introduced flag.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Renaming it works for me. As for whether we want the flag, I think that enable lcls 0 or disable or whatever would then not send the LCLS params on the A interface. Would we want that ever with a 3rd party BSC?</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br></p><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/13421">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/13421">change 13421</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-msc/+/13421"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ibb587e6ae47cff71f5bf2e2d22c1da86cd7e1762 </div>
<div style="display:none"> Gerrit-Change-Number: 13421 </div>
<div style="display:none"> Gerrit-PatchSet: 15 </div>
<div style="display:none"> Gerrit-Owner: Max <suraev@alumni.ntnu.no> </div>
<div style="display:none"> Gerrit-Assignee: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Max <suraev@alumni.ntnu.no> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: keith <keith@rhizomatica.org> </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-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 03 Feb 2021 21:16:25 +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>