Change in osmo-bsc[master]: power_control: add increase / reduce step size recommendations

[laforge]

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

Change subject: power_control: add increase / reduce step size recommendations
......................................................................

power_control: add increase / reduce step size recommendations

Change-Id: I82e762c0c2b5e0dd739850ee494ab0a798e353de
Related: SYS#4918
---
M doc/manuals/chapters/power_control.adoc
M src/osmo-bsc/bsc_vty.c
M tests/power_ctrl.vty
3 files changed, 30 insertions(+), 4 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/doc/manuals/chapters/power_control.adoc b/doc/manuals/chapters/power_control.adoc
index 0cf7d89..5de2b9c 100644
--- a/doc/manuals/chapters/power_control.adoc
+++ b/doc/manuals/chapters/power_control.adoc
@@ -201,6 +201,11 @@
 control loop to increase BS power, as well as RxLev=38 (-72 dBm) would not trigger
 power reduction.
 
+TIP: It's recommended to harmonize the increase step size with the RxLev threshold
+window in a way that the former is less or equal than/to the later.  For example,
+if the RxLev threshold is 32 .. 36 (-78 .. -74 dBm), then the window size is 4 dB,
+and thus the increase step should be less or equal (e.g. 2 or 4 dB).
+
 In 3GPP TS 45.008, lower and upper RxLev thresholds are referred as `L_RXLEV_XX_P`
 and `U_RXLEV_XX_P`, while the RxQual thresholds are referred as `L_RXQUAL_XX_P` and
 `U_RXQUAL_XX_P`, where the `XX` is either `DL` (Downlink) or `UL` (Uplink).
diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c
index 14be3fa..d61d379 100644
--- a/src/osmo-bsc/bsc_vty.c
+++ b/src/osmo-bsc/bsc_vty.c
@@ -4931,15 +4931,33 @@
 	      "Step size (2 or 4 dB)\n")
 {
 	struct gsm_power_ctrl_params *params = vty->index;
+	int inc_step_size_db = atoi(argv[0]);
+	int red_step_size_db = atoi(argv[1]);
 
-	if (atoi(argv[0]) % 2 || atoi(argv[1]) % 2) {
+	if (inc_step_size_db % 2 || red_step_size_db % 2) {
 		vty_out(vty, "%% Power change step size must be "
 			"an even number%s", VTY_NEWLINE);
 		return CMD_WARNING;
 	}
 
-	params->inc_step_size_db = atoi(argv[0]);
-	params->red_step_size_db = atoi(argv[1]);
+	/* Recommendation: POW_RED_STEP_SIZE <= POW_INCR_STEP_SIZE */
+	if (red_step_size_db > inc_step_size_db) {
+		vty_out(vty, "%% Increase step size (%d) should be greater "
+			"than reduce step size (%d), consider changing it%s",
+			inc_step_size_db, red_step_size_db, VTY_NEWLINE);
+	}
+
+	/* Recommendation: POW_INCR_STEP_SIZE <= (U_RXLEV_XX_P - L_RXLEV_XX_P) */
+	const struct gsm_power_ctrl_meas_params *mp = &params->rxlev_meas;
+	if (inc_step_size_db > (mp->upper_thresh - mp->lower_thresh)) {
+		vty_out(vty, "%% Increase step size (%d) should be less or equal "
+			"than/to the RxLev threshold window (%d, upper - lower), "
+			"consider changing it%s", inc_step_size_db,
+			mp->upper_thresh - mp->lower_thresh, VTY_NEWLINE);
+	}
+
+	params->inc_step_size_db = inc_step_size_db;
+	params->red_step_size_db = red_step_size_db;
 
 	return CMD_SUCCESS;
 }
diff --git a/tests/power_ctrl.vty b/tests/power_ctrl.vty
index 021f9df..7e4382a 100644
--- a/tests/power_ctrl.vty
+++ b/tests/power_ctrl.vty
@@ -124,6 +124,9 @@
 OsmoBSC(config-ms-power-ctrl)# step-size inc 2 red 3
 % Power change step size must be an even number
 OsmoBSC(config-ms-power-ctrl)# step-size inc 2 red 4
+% Increase step size (2) should be greater than reduce step size (4), consider changing it
+
+OsmoBSC(config-ms-power-ctrl)# step-size inc 6 red 4
 OsmoBSC(config-ms-power-ctrl)# show running-config
 ...
   bs-power-control
@@ -133,7 +136,7 @@
 ...
   ms-power-control
    mode dyn-bts
-   step-size inc 2 red 4
+   step-size inc 6 red 4
 ...
 
 OsmoBSC(config-ms-power-ctrl)# rxlev-thresh lower?

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I82e762c0c2b5e0dd739850ee494ab0a798e353de
Gerrit-Change-Number: 21873
Gerrit-PatchSet: 8
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
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/20210112/a9d6a63b/attachment.htm>