Change in osmo-bsc[master]: Disable C/I based MS Power Control Loop by default

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Thu Nov 25 14:45:13 UTC 2021


pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/26364 )

Change subject: Disable C/I based MS Power Control Loop by default
......................................................................

Disable C/I based MS Power Control Loop by default

osmo-trx-uhd with a B200 has proven to provide bad (lower than usually
considered good) C/I values due to high noise (even with band filters in
place). Hence, default thresholds (gathered from literature on the topic)
are too high and end up in bad algorithm output decisions.
Furthermore, most users of Osmocom don't use it in densely populated
areas, hence RXLEV based algorithm used when C/I based one is disabled
is good enough.
Let's disable C/I based one by default, and let advanced users which
specific needs to enable and confiure thresholds specifically for their
needs (hardware, cell surrounding conditions, etc.).

Related: SYS#4917
Change-Id: If1a73c60695379bcfcd0f44c6ec6dd659563e279
---
M src/osmo-bsc/gsm_data.c
M tests/power_ctrl.vty
2 files changed, 38 insertions(+), 13 deletions(-)

Approvals:
  Jenkins Builder: Verified
  Hoernchen: Looks good to me, but someone else must approve
  fixeria: Looks good to me, approved



diff --git a/src/osmo-bsc/gsm_data.c b/src/osmo-bsc/gsm_data.c
index 988d70e..38d8a7c 100644
--- a/src/osmo-bsc/gsm_data.c
+++ b/src/osmo-bsc/gsm_data.c
@@ -1200,7 +1200,7 @@
 	 * above the target.
 	 */
 	.ci_fr_meas = { /* FR: Target C/I = 15 dB, Soft blocking threshold = 10 dB */
-		.enabled = true,
+		.enabled = false,
 		.lower_thresh = 13,
 		.upper_thresh = 17,
 
@@ -1222,7 +1222,7 @@
 		.h_reqt = 6, /* TODO: investigate a reasonable default value */
 	},
 	.ci_hr_meas = { /* HR: Target C/I = 18 dB, Soft blocking threshold = 13 dB */
-		.enabled = true,
+		.enabled = false,
 		.lower_thresh = 16,
 		.upper_thresh = 21,
 
@@ -1244,7 +1244,7 @@
 		.h_reqt = 6, /* TODO: investigate a reasonable default value */
 	},
 	.ci_amr_fr_meas = { /* AMR-FR: Target C/I = 9 dB, Soft blocking threshold = 4 dB */
-		.enabled = true,
+		.enabled = false,
 		.lower_thresh = 7,
 		.upper_thresh = 11,
 
@@ -1266,7 +1266,7 @@
 		.h_reqt = 6, /* TODO: investigate a reasonable default value */
 	},
 	.ci_amr_hr_meas = { /* AMR-HR: Target C/I = 15 dB, Soft blocking threshold = 10 dB */
-		.enabled = true,
+		.enabled = false,
 		.lower_thresh = 13,
 		.upper_thresh = 17,
 
@@ -1288,7 +1288,7 @@
 		.h_reqt = 6, /* TODO: investigate a reasonable default value */
 	},
 	.ci_sdcch_meas = { /* SDCCH: Target C/I = 14 dB, Soft blocking threshold = 9 dB */
-		.enabled = true,
+		.enabled = false,
 		.lower_thresh = 12,
 		.upper_thresh = 16,
 
@@ -1310,7 +1310,7 @@
 		.h_reqt = 6, /* TODO: investigate a reasonable default value */
 	},
 	.ci_gprs_meas = { /* GPRS: Target C/I = 20 dB, Soft blocking threshold = 15 dB */
-		.enabled = true,
+		.enabled = false,
 		.lower_thresh = 18,
 		.upper_thresh = 24,
 
diff --git a/tests/power_ctrl.vty b/tests/power_ctrl.vty
index 374002d..f0ef46e 100644
--- a/tests/power_ctrl.vty
+++ b/tests/power_ctrl.vty
@@ -143,22 +143,22 @@
    rxlev-thresh-comp lower 10 12 upper 19 20
    rxqual-thresh lower 3 upper 0
    rxqual-thresh-comp lower 5 7 upper 15 18
-   ci-thresh fr-efr enable
+   ci-thresh fr-efr disable
    ci-thresh fr-efr lower 13 upper 17
    ci-thresh-comp fr-efr lower 5 7 upper 15 18
-   ci-thresh hr enable
+   ci-thresh hr disable
    ci-thresh hr lower 16 upper 21
    ci-thresh-comp hr lower 5 7 upper 15 18
-   ci-thresh amr-fr enable
+   ci-thresh amr-fr disable
    ci-thresh amr-fr lower 7 upper 11
    ci-thresh-comp amr-fr lower 5 7 upper 15 18
-   ci-thresh amr-hr enable
+   ci-thresh amr-hr disable
    ci-thresh amr-hr lower 13 upper 17
    ci-thresh-comp amr-hr lower 5 7 upper 15 18
-   ci-thresh sdcch enable
+   ci-thresh sdcch disable
    ci-thresh sdcch lower 12 upper 16
    ci-thresh-comp sdcch lower 5 7 upper 15 18
-   ci-thresh gprs enable
+   ci-thresh gprs disable
    ci-thresh gprs lower 18 upper 24
    ci-thresh-comp gprs lower 5 7 upper 15 18
 ...
@@ -304,12 +304,37 @@
    rxqual-thresh-comp lower 5 7 upper 15 18
 ...
 
-OsmoBSC(config-net-bts)# ### Check 'ci-thresh-comp disable all' works properly:
+OsmoBSC(config-net-bts)# ### Check 'ci-thresh-comp (enable|disable) all' works properly:
 OsmoBSC(config-net-bts)# ms-power-control
 OsmoBSC(config-ms-power-ctrl)# show running-config
 ...
   ms-power-control
 ...
+   ci-thresh fr-efr disable
+   ci-thresh fr-efr lower 13 upper 17
+   ci-thresh-comp fr-efr lower 5 7 upper 15 18
+   ci-thresh hr disable
+   ci-thresh hr lower 16 upper 21
+   ci-thresh-comp hr lower 5 7 upper 15 18
+   ci-thresh amr-fr disable
+   ci-thresh amr-fr lower 7 upper 11
+   ci-thresh-comp amr-fr lower 5 7 upper 15 18
+   ci-thresh amr-hr disable
+   ci-thresh amr-hr lower 13 upper 17
+   ci-thresh-comp amr-hr lower 5 7 upper 15 18
+   ci-thresh sdcch disable
+   ci-thresh sdcch lower 12 upper 16
+   ci-thresh-comp sdcch lower 5 7 upper 15 18
+   ci-thresh gprs disable
+   ci-thresh gprs lower 18 upper 24
+   ci-thresh-comp gprs lower 5 7 upper 15 18
+...
+
+OsmoBSC(config-ms-power-ctrl)# ci-thresh all enable
+OsmoBSC(config-ms-power-ctrl)# show running-config
+...
+  ms-power-control
+...
    ci-thresh fr-efr enable
    ci-thresh fr-efr lower 13 upper 17
    ci-thresh-comp fr-efr lower 5 7 upper 15 18

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/26364
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: If1a73c60695379bcfcd0f44c6ec6dd659563e279
Gerrit-Change-Number: 26364
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Hoernchen <ewild at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211125/7e17cf66/attachment.htm>


More information about the gerrit-log mailing list