<p>neels has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/22261">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">fix for test_dyn_ts_favor_moving_half_used_tch_h.ho_vty<br><br>So far, the test only works because osmo-bsc fails to notice that<br>occupying the dynamic TS 1 as TCH/F does, overall, not free a TCH/H, but<br>reduces available TCH/H from 5 to 4 (one TCH/H freed, but two TCH/H lost<br>from occupying a dynamic TS). An upcoming patch will make osmo-bsc<br>sensitive for these cross effects between TCH/F and TCH/H when dynamic<br>timeslots are involved, hence this test needs to be stabilized.<br><br>By using a static TCH/F for TS 1 instead of a dynamic timeslot, the dyn<br>TS cross effect does not occur and this test actually makes sense.<br><br>Change-Id: I492ea095cf3e3c3fd186c889166c4ed93ab3a007<br>---<br>M tests/handover/test_dyn_ts_favor_moving_half_used_tch_h.ho_vty<br>1 file changed, 6 insertions(+), 6 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/61/22261/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/handover/test_dyn_ts_favor_moving_half_used_tch_h.ho_vty b/tests/handover/test_dyn_ts_favor_moving_half_used_tch_h.ho_vty</span><br><span>index 3d1b009..b25cf96 100644</span><br><span>--- a/tests/handover/test_dyn_ts_favor_moving_half_used_tch_h.ho_vty</span><br><span>+++ b/tests/handover/test_dyn_ts_favor_moving_half_used_tch_h.ho_vty</span><br><span>@@ -2,23 +2,23 @@</span><br><span> # The algorithm should notice that this is about moving an lchan within the same cell, so all candidates will remain</span><br><span> # with unchanged rxlev after a re-assignment; hence the current rxlev for each candidate should not make a difference.</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-create-bts trx-count 1 timeslots c+s4 dyn   dyn    dyn    dyn    dyn  - -</span><br><span style="color: hsl(120, 100%, 40%);">+create-bts trx-count 1 timeslots c+s4 TCH/F dyn    dyn    dyn    dyn  - -</span><br><span> network</span><br><span>  handover2 min-free-slots tch/h 6</span><br><span> </span><br><span> # Test with identical rxlev across lchans (trivial and unrealistic)</span><br><span style="color: hsl(0, 100%, 40%);">-set-ts-use    trx 0 0     states *    pdch  TCH/HH TCH/H- TCH/HH pdch - -</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use    trx 0 0     states *    -     TCH/HH TCH/H- TCH/HH pdch - -</span><br><span> meas-rep lchan * * * * rxlev 30 rxqual 0 ta 0</span><br><span> congestion-check</span><br><span> expect-ho from lchan 0 0 3 0 to lchan 0 0 1 0</span><br><span> expect-ts-use trx 0 0     states *    TCH/F TCH/HH pdch TCH/HH pdch - -</span><br><span> </span><br><span> # clear measurements for the next run</span><br><span style="color: hsl(0, 100%, 40%);">-set-ts-use    trx 0 0     states *    pdch pdch pdch pdch pdch - -</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use    trx 0 0     states *    - pdch pdch pdch pdch - -</span><br><span> </span><br><span> # Check that a weaker rxlev coming up earlier in the congestion checking loop does not override the favored half-used</span><br><span> # TCH/H</span><br><span style="color: hsl(0, 100%, 40%);">-set-ts-use    trx 0 0     states *    pdch  TCH/HH TCH/H- TCH/HH pdch - -</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use    trx 0 0     states *    - TCH/HH TCH/H- TCH/HH pdch - -</span><br><span> meas-rep lchan 0 0 2 1 rxlev 30 rxqual 0 ta 0</span><br><span> meas-rep lchan 0 0 3 0 rxlev 31 rxqual 0 ta 0</span><br><span> meas-rep lchan 0 0 4 0 rxlev 32 rxqual 0 ta 0</span><br><span>@@ -28,11 +28,11 @@</span><br><span> expect-ts-use trx 0 0     states *    TCH/F TCH/HH pdch TCH/HH pdch - -</span><br><span> </span><br><span> # clear measurements for the next run</span><br><span style="color: hsl(0, 100%, 40%);">-set-ts-use    trx 0 0     states *    pdch pdch pdch pdch pdch - -</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use    trx 0 0     states *    - pdch pdch pdch pdch - -</span><br><span> </span><br><span> # Check that a weaker rxlev coming up later in the congestion checking loop does not override the favored half-used</span><br><span> # TCH/H</span><br><span style="color: hsl(0, 100%, 40%);">-set-ts-use    trx 0 0     states *    pdch  TCH/HH TCH/H- TCH/HH pdch - -</span><br><span style="color: hsl(120, 100%, 40%);">+set-ts-use    trx 0 0     states *    -  TCH/HH TCH/H- TCH/HH pdch - -</span><br><span> meas-rep lchan 0 0 2 1 rxlev 34 rxqual 0 ta 0</span><br><span> meas-rep lchan 0 0 3 0 rxlev 33 rxqual 0 ta 0</span><br><span> meas-rep lchan 0 0 4 0 rxlev 32 rxqual 0 ta 0</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/22261">change 22261</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/+/22261"/><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: I492ea095cf3e3c3fd186c889166c4ed93ab3a007 </div>
<div style="display:none"> Gerrit-Change-Number: 22261 </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>