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/.
laforge gerrit-no-reply at lists.osmocom.orglaforge 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 = ¶ms->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>