<p>Harald Welte has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/14220">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bts: TC_pcu_{time,rts}_ind: Higher tolerances<br><br>When executing the test on our build slaves, we ran into the following:<br><br>> Number of TDMA Frames (1100) not matching (1073 .. 1093)<br>> Number of TDMA Frames (1096) not matching (1073 .. 1093)<br><br>So it seems the tolerance was a bit too tight.  If the timer runs<br>for a bit more than the requested amount of seconds (e.g. due to<br>high system load), the number of TDMA frames is likely a bit higher<br>and hence we need to permit a larger tolerance.<br><br>Change-Id: Ib00bb7e7f223fb0329d00e3835127b8c31299e27<br>---<br>M bts/BTS_Tests.ttcn<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/20/14220/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn</span><br><span>index 6786d68..968ed6c 100644</span><br><span>--- a/bts/BTS_Tests.ttcn</span><br><span>+++ b/bts/BTS_Tests.ttcn</span><br><span>@@ -3713,7 +3713,7 @@</span><br><span>    /* verify the number of frames expired matches our expectation */</span><br><span>    const float c_GSM_FN_DURATION_MS := 4.61538;</span><br><span>         var float fn_expected := test_duration * 1000.0 / c_GSM_FN_DURATION_MS;</span><br><span style="color: hsl(0, 100%, 40%);">- var template integer t_fn_expected := f_tolerance(float2int(fn_expected), 1, 100000, 10);</span><br><span style="color: hsl(120, 100%, 40%);">+     var template integer t_fn_expected := f_tolerance(float2int(fn_expected), 1, 100000, 20);</span><br><span>    if (not match(fn_expired, t_fn_expected)) {</span><br><span>          setverdict(fail, "Number of TDMA Frames (", fn_expired, ") not matching ", t_fn_expected);</span><br><span>       }</span><br><span>@@ -3778,7 +3778,7 @@</span><br><span>    /* verify the number of frames expired matches our expectation */</span><br><span>    const float c_GSM_FN_DURATION_MS := 4.61538;</span><br><span>         var float fn_expected := test_duration * 1000.0 / c_GSM_FN_DURATION_MS;</span><br><span style="color: hsl(0, 100%, 40%);">- var template integer t_fn_expected := f_tolerance(float2int(fn_expected), 1, 100000, 10);</span><br><span style="color: hsl(120, 100%, 40%);">+     var template integer t_fn_expected := f_tolerance(float2int(fn_expected), 1, 100000, 20);</span><br><span>    if (not match(fn_expired, t_fn_expected)) {</span><br><span>          setverdict(fail, "Number of TDMA Frames (", fn_expired, ") not matching ", t_fn_expected);</span><br><span>       }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/14220">change 14220</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/14220"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ib00bb7e7f223fb0329d00e3835127b8c31299e27 </div>
<div style="display:none"> Gerrit-Change-Number: 14220 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>