diff options
| author | Neels Hofmeyr <neels@hofmeyr.de> | 2021-01-05 20:09:11 +0100 |
|---|---|---|
| committer | Neels Hofmeyr <neels@hofmeyr.de> | 2021-01-12 09:36:12 +0100 |
| commit | 0096f27b610d8d1dd3b9441e1b0c444f97af09bb (patch) | |
| tree | 8fa13edb439b89662103db739b937d9e5fb6da65 /tests/handover/test_congestion_no_oscillation2.ho_vty | |
| parent | db52370c2e89fc9a5bf4137be9717507fb83e09b (diff) | |
hodec2: fix congestion oscillation bug
When evenly distributing congestion across cells, count the number of
occupied lchans surpassing congestion, and not the overall number of
free lchans -- which disregards congestion thresholds.
Fix the bugs shown by
test_congestion_no_oscillation.ho_vty
test_balance_congestion_tchf_tchh.ho_vty
This implements a simple calculation for congestion load by counting
lchans in use above congestion. An improvement of this calculation
using percent follows in I55234c6c99eb02ceee52be0d7388bea14304930f.
Related: SYS#5259
Change-Id: Icb373dc6bfc9819446db5e96f71921781fe2026d
Diffstat (limited to 'tests/handover/test_congestion_no_oscillation2.ho_vty')
| -rw-r--r-- | tests/handover/test_congestion_no_oscillation2.ho_vty | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/tests/handover/test_congestion_no_oscillation2.ho_vty b/tests/handover/test_congestion_no_oscillation2.ho_vty index aee731d57..44c4176c2 100644 --- a/tests/handover/test_congestion_no_oscillation2.ho_vty +++ b/tests/handover/test_congestion_no_oscillation2.ho_vty @@ -1,8 +1,5 @@ -# Almost identical to test_amr_oscillation.ho_vty, this has just two more TCH/H slots in BTS 1, and does not trigger the -# oscillation bug. The number of free TCH/H in BTS 1 should be unrelated to the congestion status of BTS 0, which -# illustrates that the even distribution of congestion is fundamentally flawed. -# This test script shows the desired behavior, though by common sense there should be no reason why we see the bug in -# test_amr_oscillation.ho_vty and not here. +# Almost identical to test_amr_oscillation.ho_vty, this has just two more TCH/H slots in BTS 1, and did not trigger the +# oscillation bug (which has since been fixed, so that both tests behave identically now). create-bts trx-count 1 timeslots c+s4 TCH/F TCH/F TCH/F TCH/F TCH/F TCH/F PDCH network |