<p>neels has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21974">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">handover_test: add AMR TCH/H->/F tests, showing a bug<br><br>Related: SYS#5032<br>Change-Id: Ib785ce8990beede603ba57171a60a531f0f396c9<br>---<br>M tests/handover/handover_tests.ok<br>A tests/handover/test_amr_tch_h_to_f_congestion.ho_vty<br>A tests/handover/test_amr_tch_h_to_f_congestion_two_cells.ho_vty<br>3 files changed, 39 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/74/21974/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/handover/handover_tests.ok b/tests/handover/handover_tests.ok</span><br><span>index 71f6255..8f53471 100644</span><br><span>--- a/tests/handover/handover_tests.ok</span><br><span>+++ b/tests/handover/handover_tests.ok</span><br><span>@@ -4,6 +4,8 @@</span><br><span> pass test_amr_tch_f_to_h_congestion_assignment.ho_vty</span><br><span> pass test_amr_tch_f_to_h_congestion_assignment_2.ho_vty</span><br><span> pass test_amr_tch_f_to_h_congestion_assignment_3.ho_vty</span><br><span style="color: hsl(120, 100%, 40%);">+pass test_amr_tch_h_to_f_congestion.ho_vty</span><br><span style="color: hsl(120, 100%, 40%);">+pass test_amr_tch_h_to_f_congestion_two_cells.ho_vty</span><br><span> pass test_balance_congestion.ho_vty</span><br><span> pass test_balance_congestion_2.ho_vty</span><br><span> pass test_congestion.ho_vty</span><br><span>diff --git a/tests/handover/test_amr_tch_h_to_f_congestion.ho_vty b/tests/handover/test_amr_tch_h_to_f_congestion.ho_vty</span><br><span>new file mode 100644</span><br><span>index 0000000..4d1194c</span><br><span>--- /dev/null</span><br><span>+++ b/tests/handover/test_amr_tch_h_to_f_congestion.ho_vty</span><br><span>@@ -0,0 +1,14 @@</span><br><span style="color: hsl(120, 100%, 40%);">+# Congestion check: Balancing congestion by handover TCH/H -> TCH/F</span><br><span style="color: hsl(120, 100%, 40%);">+# One BTS, TCH/H are congested and should move to TCH/F.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+network</span><br><span style="color: hsl(120, 100%, 40%);">+ handover2 min-free-slots tch/f 0</span><br><span style="color: hsl(120, 100%, 40%);">+ handover2 min-free-slots tch/h 6</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/H  TCH/H TCH/H PDCH</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use trx 0 0 states        *    -     -     -     TCH/H- -     -     *</span><br><span style="color: hsl(120, 100%, 40%);">+meas-rep lchan 0 0 4 0 rxlev 30 rxqual 0 ta 0</span><br><span style="color: hsl(120, 100%, 40%);">+expect-no-chan</span><br><span style="color: hsl(120, 100%, 40%);">+congestion-check</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ho from lchan 0 0 4 0 to lchan 0 0 1 0</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ts-use trx 0 0 states * TCH/F - - - - - *</span><br><span>diff --git a/tests/handover/test_amr_tch_h_to_f_congestion_two_cells.ho_vty b/tests/handover/test_amr_tch_h_to_f_congestion_two_cells.ho_vty</span><br><span>new file mode 100644</span><br><span>index 0000000..eaaeabd</span><br><span>--- /dev/null</span><br><span>+++ b/tests/handover/test_amr_tch_h_to_f_congestion_two_cells.ho_vty</span><br><span>@@ -0,0 +1,23 @@</span><br><span style="color: hsl(120, 100%, 40%);">+# Congestion check: Balancing congestion by handover TCH/H -> TCH/F</span><br><span style="color: hsl(120, 100%, 40%);">+# TCH/H are congested and should move to TCH/F</span><br><span style="color: hsl(120, 100%, 40%);">+# There are two cells, and the neighbor has weaker rxlev, so stay in the same cell.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+network</span><br><span style="color: hsl(120, 100%, 40%);">+ handover2 min-free-slots tch/f 0</span><br><span style="color: hsl(120, 100%, 40%);">+ handover2 min-free-slots tch/h 6</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/H  TCH/H TCH/H PDCH</span><br><span style="color: hsl(120, 100%, 40%);">+create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/H  TCH/H TCH/H PDCH</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use trx 1 0 states        *    -     -     -     TCH/H- -     -     *</span><br><span style="color: hsl(120, 100%, 40%);">+meas-rep repeat 10 lchan 1 0 4 0 rxlev 30 rxqual 0 ta 0 neighbors 20</span><br><span style="color: hsl(120, 100%, 40%);">+expect-no-chan</span><br><span style="color: hsl(120, 100%, 40%);">+congestion-check</span><br><span style="color: hsl(120, 100%, 40%);">+# FAIL: bts 1 has better rxlev, so the call should stay in bts 1. Instead, a handover to bts 0 happens.</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ho from lchan 1 0 4 0 to lchan 0 0 1 0</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ts-use trx 0 0 states * TCH/F - - - - - *</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ts-use trx 1 0 states * - - - - - - *</span><br><span style="color: hsl(120, 100%, 40%);">+# the fail continues: later the better rxqual does *another* ho back to the original cell</span><br><span style="color: hsl(120, 100%, 40%);">+meas-rep lchan 0 0 1 0 rxlev 20 rxqual 0 ta 0 neighbors 30</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ho from lchan 0 0 1 0 to lchan 1 0 1 0</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ts-use trx 0 0 states * - - - - - - *</span><br><span style="color: hsl(120, 100%, 40%);">+expect-ts-use trx 1 0 states * TCH/F - - - - - *</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21974">change 21974</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/+/21974"/><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: Ib785ce8990beede603ba57171a60a531f0f396c9 </div>
<div style="display:none"> Gerrit-Change-Number: 21974 </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>