<p>neels has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21201">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">handover test: add test 30: de-congest TCH/F by moving to dyn TS TCH/H<br><br>In the test, show the undesired behavior of moving non-dynamic timeslots first,<br>because they are the first to be considered in the congestion resolution loop.<br>The behavior will be fixed in Ic221b8d2687cdec0bf94410c84a4da43853f0900.<br><br>Change-Id: I09ab9f2f79fa434c7279cb4d09899f69b047aa55<br>---<br>M tests/handover/handover_test.c<br>M tests/testsuite.at<br>2 files changed, 46 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/01/21201/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/handover/handover_test.c b/tests/handover/handover_test.c</span><br><span>index d442d99..46f7cb8 100644</span><br><span>--- a/tests/handover/handover_test.c</span><br><span>+++ b/tests/handover/handover_test.c</span><br><span>@@ -1655,6 +1655,45 @@</span><br><span>   NULL</span><br><span> };</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static char *test_case_30[] = {</span><br><span style="color: hsl(120, 100%, 40%);">+       "2",</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      "Congestion check: Balancing congestion by handover TCH/F -> TCH/H\n\n"</span><br><span style="color: hsl(120, 100%, 40%);">+  "With dynamic timeslots.\n"</span><br><span style="color: hsl(120, 100%, 40%);">+ "As soon as only one TCH/F is left, there should be HO to a dyn TS.\n"</span><br><span style="color: hsl(120, 100%, 40%);">+      ,</span><br><span style="color: hsl(120, 100%, 40%);">+     "create-bts", "1", "c+s4", "TCH/F", "TCH/F", "TCH/F", "dyn", "dyn", "dyn", "PDCH",</span><br><span style="color: hsl(120, 100%, 40%);">+        "set-min-free", "0", "TCH/F", "2",</span><br><span style="color: hsl(120, 100%, 40%);">+    "set-min-free", "0", "TCH/H", "0",</span><br><span style="color: hsl(120, 100%, 40%);">+    "as-enable", "0", "1",</span><br><span style="color: hsl(120, 100%, 40%);">+  "set-ts-use", "0", "0", "*", "TCH/F", "TCH/F", "TCH/F", "TCH/F", "PDCH", "PDCH", "PDCH",</span><br><span style="color: hsl(120, 100%, 40%);">+        "meas-rep", "0","0","1","0", "40","0", "1", "0","30",</span><br><span style="color: hsl(120, 100%, 40%);">+     "meas-rep", "0","0","2","0", "40","0", "1", "0","30",</span><br><span style="color: hsl(120, 100%, 40%);">+     "meas-rep", "0","0","3","0", "40","0", "1", "0","30",</span><br><span style="color: hsl(120, 100%, 40%);">+     "meas-rep", "0","0","4","0", "40","0", "1", "0","30",</span><br><span style="color: hsl(120, 100%, 40%);">+     "congestion-check",</span><br><span style="color: hsl(120, 100%, 40%);">+ "expect-no-chan",</span><br><span style="color: hsl(120, 100%, 40%);">+   "create-ms", "0", "TCH/F", "AMR",</span><br><span style="color: hsl(120, 100%, 40%);">+     "meas-rep", "0","0","5","0", "40","0", "1", "0","30",</span><br><span style="color: hsl(120, 100%, 40%);">+     "expect-ts-use", "0", "0", "*", "TCH/F", "TCH/F", "TCH/F", "TCH/F", "TCH/F", "PDCH", "PDCH",</span><br><span style="color: hsl(120, 100%, 40%);">+    "congestion-check",</span><br><span style="color: hsl(120, 100%, 40%);">+ "expect-chan", "0", "6",</span><br><span style="color: hsl(120, 100%, 40%);">+        "ack-chan",</span><br><span style="color: hsl(120, 100%, 40%);">+ /* Not so good: rather than moving static TCH/F, we should favor freeing dyn TS, for more PDCH */</span><br><span style="color: hsl(120, 100%, 40%);">+     "expect-ho", "0", "1",</span><br><span style="color: hsl(120, 100%, 40%);">+  "ho-complete",</span><br><span style="color: hsl(120, 100%, 40%);">+      "expect-ts-use", "0", "0", "*", "-", "TCH/F", "TCH/F", "TCH/F", "TCH/F", "TCH/H-", "PDCH",</span><br><span style="color: hsl(120, 100%, 40%);">+      "congestion-check",</span><br><span style="color: hsl(120, 100%, 40%);">+ "expect-chan", "0", "6",</span><br><span style="color: hsl(120, 100%, 40%);">+        "ack-chan",</span><br><span style="color: hsl(120, 100%, 40%);">+ "expect-ho", "0", "2",</span><br><span style="color: hsl(120, 100%, 40%);">+  "ho-complete",</span><br><span style="color: hsl(120, 100%, 40%);">+      "expect-ts-use", "0", "0", "*", "-", "-", "TCH/F", "TCH/F", "TCH/F", "TCH/HH", "PDCH",</span><br><span style="color: hsl(120, 100%, 40%);">+  "congestion-check",</span><br><span style="color: hsl(120, 100%, 40%);">+ "expect-no-chan",</span><br><span style="color: hsl(120, 100%, 40%);">+   NULL</span><br><span style="color: hsl(120, 100%, 40%);">+};</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> </span><br><span> static char **test_cases[] =  {</span><br><span>    test_case_0,</span><br><span>@@ -1687,6 +1726,7 @@</span><br><span>         test_case_27,</span><br><span>        test_case_28,</span><br><span>        test_case_29,</span><br><span style="color: hsl(120, 100%, 40%);">+ test_case_30,</span><br><span> };</span><br><span> </span><br><span> static const struct log_info_cat log_categories[] = {</span><br><span>diff --git a/tests/testsuite.at b/tests/testsuite.at</span><br><span>index a0d85ce..787f33d 100644</span><br><span>--- a/tests/testsuite.at</span><br><span>+++ b/tests/testsuite.at</span><br><span>@@ -230,3 +230,9 @@</span><br><span> cat $abs_srcdir/handover/handover_test.ok > expout</span><br><span> AT_CHECK([$abs_top_builddir/tests/handover/handover_test 29], [], [expout], [ignore])</span><br><span> AT_CLEANUP</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+AT_SETUP([handover test 30])</span><br><span style="color: hsl(120, 100%, 40%);">+AT_KEYWORDS([handover])</span><br><span style="color: hsl(120, 100%, 40%);">+cat $abs_srcdir/handover/handover_test.ok > expout</span><br><span style="color: hsl(120, 100%, 40%);">+AT_CHECK([$abs_top_builddir/tests/handover/handover_test 30], [], [expout], [ignore])</span><br><span style="color: hsl(120, 100%, 40%);">+AT_CLEANUP</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21201">change 21201</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-bsc/+/21201"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I09ab9f2f79fa434c7279cb4d09899f69b047aa55 </div>
<div style="display:none"> Gerrit-Change-Number: 21201 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>