<p style="white-space: pre-wrap; word-wrap: break-word;">looks good, but the actual timeout value should receive some more consideration.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/11307">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11307/1/src/libmsc/gsm_04_08_cc.c">File src/libmsc/gsm_04_08_cc.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11307/1/src/libmsc/gsm_04_08_cc.c@76">Patch Set #1, Line 76:</a> <code style="font-family:monospace,monospace">#define GUARD_TIMEOUT 65</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">should this be configurable?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">it should be configurable, and I am afraid 65 seconds is way too little.  According to TS 04.07, we have something like 180 seconds of default timeout when alerting the remote end alone.  This means that the phone of the called subscriber could be ringing for 3 minutes before an answer.  And if the called party picks up after 2:50 minutes after starting to alert (which is probably easily more than 3 minutes after attempting the call), it is still a successful call.  With 65 seconds global guard timer, we would kill the call too early.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11307">change 11307</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/11307"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I7375d1e17cd746aac4eadfe1e587e82cf1630d3d </div>
<div style="display:none"> Gerrit-Change-Number: 11307 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@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>
<div style="display:none"> Gerrit-Comment-Date: Fri, 12 Oct 2018 06:29:45 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>