Harald Welte has submitted this change and it was merged. (
https://gerrit.osmocom.org/14158 )
Change subject: load_indication: Fix missing re-set of RACH parameters
......................................................................
load_indication: Fix missing re-set of RACH parameters
While we re-set the PCH load counters after every report, we never
actually re-set the RACH load counters. This meant that the
period/window for RACH load averaging would always grow, rather than
being reset every load indication period.
Related: OS#3750
Change-Id: Icd9150ba56d77d031c3cf496c5936c2de52b364c
---
M src/common/load_indication.c
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
Vadim Yanitskiy: Looks good to me, but someone else must approve
Harald Welte: Looks good to me, approved
Pau Espin Pedrol: Looks good to me, but someone else must approve
diff --git a/src/common/load_indication.c b/src/common/load_indication.c
index e91f6d4..fa4745b 100644
--- a/src/common/load_indication.c
+++ b/src/common/load_indication.c
@@ -32,6 +32,7 @@
{
/* re-set the counters */
bts->load.ccch.pch_used = bts->load.ccch.pch_total = 0;
+ bts->load.rach.busy = bts->load.rach.access = bts->load.rach.total = 0;
}
static void load_timer_cb(void *data)
--
To view, visit https://gerrit.osmocom.org/14158
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Icd9150ba56d77d031c3cf496c5936c2de52b364c
Gerrit-Change-Number: 14158
Gerrit-PatchSet: 2
Gerrit-Owner: Harald Welte <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Vadim Yanitskiy <[email protected]>
