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 <vyanits...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged