<p>Neels Hofmeyr <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/11119">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">make SCCP timers configurable<br><br>The previous hardcoded SCCP timers may cause SCCP connection releases, if the<br>peer is configured with far lower timers than libosmo-sccp. Testing with a<br>specific SCCPlite MSC, I experienced an iar of just over three minutes, meaning<br>that calls would be cut off by the MSC, since the osmo-bsc failed to send an<br>Inactivity Timer message until seven minutes have passed.<br><br>With this patch, SCCP timers are configurable by the user, with millisecond<br>resolution (or by second or minute, for convenience).<br><br>Define constant global default timers, and variable user-configurable timers<br>with each osmo_sccp_instance.<br><br>Add VTY UI to configure the timers. Users must call osmo_sccp_vty_init() to get<br>the sccp-timer config nodes under the 'cs7' node. Show the new UI in<br>ss7_asp_test.vty.<br><br>Note that even though this function is not new at all, until recently, all of<br>our SCCP users (osmo-bsc, osmo-msc, osmo-sgsn, osmo-hnbgw) failed to call<br>osmo_sccp_vty_init(), and thus also missed out on the various 'show' commands<br>defined in sccp_vty.c. In other words, to benefit from the timer<br>configurability, the patches to call osmo_sccp_vty_init() must first be merged<br>to the corresponding master branches.<br><br>If a 'sccp-timer' config command occurs, the cs7 instance must allocate an SCCP<br>instance in order to store the timer config. Do that by calling the recently<br>added osmo_ss7_ensure_sccp() function.<br><br>Hence remove the limitation that the SCCP instance must not be populated from<br>the "simple" setup function. If we want to configure SCCP timers beforehand,<br>there must be an SCCP instance for that, and there is no hard reason to require<br>a NULL SCCP instance, besides the desire to prevent this function from being<br>invoked twice.<br><br>Change-Id: I28a7362aa838e648ecc9b26ee53dbcade81a9d65<br>---<br>M src/osmo_ss7_vty.c<br>M src/sccp_internal.h<br>M src/sccp_scoc.c<br>M src/sccp_user.c<br>M src/sccp_vty.c<br>M tests/vty/ss7_asp_test.vty<br>6 files changed, 327 insertions(+), 27 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/19/11119/4</pre><p>To view, visit <a href="https://gerrit.osmocom.org/11119">change 11119</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/11119"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I28a7362aa838e648ecc9b26ee53dbcade81a9d65 </div>
<div style="display:none"> Gerrit-Change-Number: 11119 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>