<p style="white-space: pre-wrap; word-wrap: break-word;">Hi Neels,</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Possible problem here is changing the meaning of the T3212 value.<br>If we use T3212 as name, I guess we should also stick to the 3GPP definition of its value.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think after the split of OsmoNiTB it has already lost its original meaning. I personally find the current description of T3212 misleading as it may seem like we're broadcasting this value in System Information Type 3, while we do not. We could of course rename it to X3212, but do we really want to do that?</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">In 3GPP specs, AFAIU, this timer number is defined to be of that weird format.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That weird format only applies to the coding of its value in SI3 message, which is quite limited in terms of its maximum size (23 octets). We already accept the values in minutes, and here we never deal with the contents of SI3, so I don't see a point in using that 'compression' for the internal state...</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">b) techies setting T3212 according to spec and getting a different timeout;</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Fortunately, we do not accept the value of T3212 in deci-hours, as the specs. define it ;)</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">a) users moving the timer to the new vty config and not understanding the different number value (would the config change?)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Well, yeah. This magic multiplication (2 * T3212 + 1) comes from the NiTB times. However, as a user when I am setting T3212 to 30 minutes, I basically expect a subscriber to expire in 30 minutes and not in 61 (currently this transformation is opaque). That's why I am printing a warning that we're emulating the old behaviour.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(-1 just for the typo, the rest is +0)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Nice catch! Thanks!</p><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16934">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/+/16934">change 16934</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/+/16934"/><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: I9b12066599a7c834a53a93acf5902d91273bc74f </div>
<div style="display:none"> Gerrit-Change-Number: 16934 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 23 Jan 2020 23:08:36 +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>