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/+/21812 )
Change subject: power_control: add VTY command to set static / maximum BS Power
......................................................................
power_control: add VTY command to set static / maximum BS Power
Change-Id: I11ca856aba46aaf84d94cbbdf4c39a01ee8289b9
Related: SYS#4918
---
M include/osmocom/bsc/gsm_data.h
M src/osmo-bsc/bsc_vty.c
M src/osmo-bsc/gsm_data.c
M src/osmo-bsc/lchan_fsm.c
M tests/power_ctrl.vty
5 files changed, 94 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/include/osmocom/bsc/gsm_data.h b/include/osmocom/bsc/gsm_data.h
index f9d5736..233c3b6 100644
--- a/include/osmocom/bsc/gsm_data.h
+++ b/include/osmocom/bsc/gsm_data.h
@@ -1328,6 +1328,10 @@
/* Power Control mode to be used by the BTS */
enum gsm_power_ctrl_mode mode;
+ /* BS Power reduction value / maximum (in dB) */
+ uint8_t bs_power_val_db; /* for static mode */
+ uint8_t bs_power_max_db; /* for dynamic mode */
+
/* Power change step size (dynamic mode only) */
uint8_t inc_step_size_db; /* increasing direction */
uint8_t red_step_size_db; /* reducing direction */
diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c
index c12fd12..fe46d45 100644
--- a/src/osmo-bsc/bsc_vty.c
+++ b/src/osmo-bsc/bsc_vty.c
@@ -950,10 +950,14 @@
case GSM_PWR_CTRL_MODE_STATIC:
cfg_out("%s%s", node_name, VTY_NEWLINE);
cfg_out(" mode static%s", VTY_NEWLINE);
+ if (cp->dir == GSM_PWR_CTRL_DIR_DL && cp->bs_power_val_db != 0)
+ cfg_out(" bs-power static %u%s", cp->bs_power_val_db, VTY_NEWLINE);
break;
case GSM_PWR_CTRL_MODE_DYN_BTS:
cfg_out("%s%s", node_name, VTY_NEWLINE);
cfg_out(" mode dyn-bts%s", VTY_NEWLINE);
+ if (cp->dir == GSM_PWR_CTRL_DIR_DL)
+ cfg_out(" bs-power dyn-max %u%s", cp->bs_power_max_db, VTY_NEWLINE);
cfg_out(" step-size inc %u red %u%s",
cp->inc_step_size_db, cp->red_step_size_db,
@@ -4895,6 +4899,40 @@
return CMD_SUCCESS;
}
+DEFUN_USRATTR(cfg_power_ctrl_bs_power,
+ cfg_power_ctrl_bs_power_cmd,
+ X(BSC_VTY_ATTR_VENDOR_SPECIFIC) |
+ X(BSC_VTY_ATTR_NEW_LCHAN),
+ "bs-power (static|dyn-max) <0-30>",
+ "BS Power IE value to be send to the BTS\n"
+ "Fixed BS Power reduction value (for static mode)\n"
+ "Maximum BS Power reduction value (for dynamic mode)\n"
+ "BS Power reduction value (in dB, even numbers only)\n")
+{
+ struct gsm_power_ctrl_params *params = vty->index;
+ bool dynamic = !strcmp(argv[0], "dyn-max");
+ int value = atoi(argv[1]);
+
+ if (params->dir != GSM_PWR_CTRL_DIR_DL) {
+ vty_out(vty, "%% This command is only valid for "
+ "'bs-power-control' node%s", VTY_NEWLINE);
+ return CMD_WARNING;
+ }
+
+ if (value % 2 != 0) {
+ vty_out(vty, "%% Incorrect BS Power reduction value, "
+ "an even number is expected%s", VTY_NEWLINE);
+ return CMD_WARNING;
+ }
+
+ if (dynamic) /* maximum value */
+ params->bs_power_max_db = value;
+ else /* static (fixed) value */
+ params->bs_power_val_db = value;
+
+ return CMD_SUCCESS;
+}
+
DEFUN_USRATTR(cfg_power_ctrl_step_size,
cfg_power_ctrl_step_size_cmd,
X(BSC_VTY_ATTR_VENDOR_SPECIFIC) |
@@ -7673,6 +7711,7 @@
install_element(BTS_NODE, &cfg_bts_no_power_ctrl_cmd);
install_node(&power_ctrl_node, dummy_config_write);
install_element(POWER_CTRL_NODE, &cfg_power_ctrl_mode_cmd);
+ install_element(POWER_CTRL_NODE, &cfg_power_ctrl_bs_power_cmd);
install_element(POWER_CTRL_NODE, &cfg_power_ctrl_step_size_cmd);
install_element(POWER_CTRL_NODE, &cfg_power_ctrl_rxlev_thresh_cmd);
install_element(POWER_CTRL_NODE, &cfg_power_ctrl_rxqual_thresh_cmd);
diff --git a/src/osmo-bsc/gsm_data.c b/src/osmo-bsc/gsm_data.c
index 0b5650f..4c87fd8 100644
--- a/src/osmo-bsc/gsm_data.c
+++ b/src/osmo-bsc/gsm_data.c
@@ -972,6 +972,10 @@
/* Static Power Control is the safe default */
.mode = GSM_PWR_CTRL_MODE_STATIC,
+ /* BS Power reduction value / maximum (in dB) */
+ .bs_power_val_db = 0, /* no attenuation in static mode */
+ .bs_power_max_db = 12, /* up to 12 dB in dynamic mode */
+
/* Power increasing/reducing step size */
.inc_step_size_db = 4, /* 2, 4, or 6 dB */
.red_step_size_db = 2, /* 2 or 4 dB */
diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index 1903645..5a42527 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -379,6 +379,9 @@
*/
static void lchan_reset(struct gsm_lchan *lchan)
{
+ const struct gsm_bts_trx *trx = lchan->ts->trx;
+ const struct gsm_bts *bts = trx->bts;
+
LOG_LCHAN(lchan, LOGL_DEBUG, "Clearing lchan state\n");
if (lchan->conn)
@@ -408,6 +411,12 @@
.release.rr_cause = GSM48_RR_CAUSE_NORMAL,
};
+
+ /* Default BS Power reduction value (in 2 dB steps) */
+ if (bts->bs_power_ctrl.mode == GSM_PWR_CTRL_MODE_DYN_BTS)
+ lchan->bs_power = bts->bs_power_ctrl.bs_power_max_db / 2;
+ else
+ lchan->bs_power = bts->bs_power_ctrl.bs_power_val_db / 2;
}
static void lchan_fsm_unused_onenter(struct osmo_fsm_inst *fi, uint32_t prev_state)
diff --git a/tests/power_ctrl.vty b/tests/power_ctrl.vty
index abd2768..7884c01 100644
--- a/tests/power_ctrl.vty
+++ b/tests/power_ctrl.vty
@@ -23,6 +23,7 @@
OsmoBSC(config-power-ctrl)# list with-flags
...
. lv mode (static|dyn-bts) [reset]
+ . lv bs-power (static|dyn-max) <0-30>
. lv step-size inc <2-6> red <2-4>
. lv rxlev-thresh lower <0-63> upper <0-63>
. lv rxqual-thresh lower <0-7> upper <0-7>
@@ -33,12 +34,30 @@
. lv (rxlev-avg|rxqual-avg) algo (unweighted|weighted|mod-median)
. lv (rxlev-avg|rxqual-avg) algo osmo-ewma beta <1-99>
+OsmoBSC(config-power-ctrl)# bs-power ?
+ static Fixed BS Power reduction value (for static mode)
+ dyn-max Maximum BS Power reduction value (for dynamic mode)
+OsmoBSC(config-power-ctrl)# bs-power static ?
+ <0-30> BS Power reduction value (in dB, even numbers only)
+OsmoBSC(config-power-ctrl)# bs-power static 3
+% Incorrect BS Power reduction value, an even number is expected
+OsmoBSC(config-power-ctrl)# bs-power static 6
+OsmoBSC(config-power-ctrl)# show running-config
+...
+ bs-power-control
+ mode static
+ bs-power static 6
+ ms-power-control
+ mode static
+... !bs-power
+
OsmoBSC(config-power-ctrl)# ### Check default BS Power Parameters
OsmoBSC(config-power-ctrl)# mode dyn-bts
OsmoBSC(config-power-ctrl)# show running-config
...
bs-power-control
mode dyn-bts
+ bs-power dyn-max 12
step-size inc 4 red 2
rxlev-thresh lower 32 upper 38
rxlev-thresh-comp lower 10 12 upper 19 20
@@ -50,11 +69,23 @@
mode static
...
+OsmoBSC(config-power-ctrl)# bs-power dyn-max 30
+OsmoBSC(config-power-ctrl)# show running-config
+...
+ bs-power-control
+ mode dyn-bts
+ bs-power dyn-max 30
+... !bs-power
+ ms-power-control
+ mode static
+... !bs-power
+
OsmoBSC(config-power-ctrl)# exit
OsmoBSC(config-net-bts)# ms-power-control
OsmoBSC(config-power-ctrl)# list with-flags
...
. lv mode (static|dyn-bts) [reset]
+ . lv bs-power (static|dyn-max) <0-30>
. lv step-size inc <2-6> red <2-4>
. lv rxlev-thresh lower <0-63> upper <0-63>
. lv rxqual-thresh lower <0-7> upper <0-7>
@@ -71,6 +102,7 @@
...
bs-power-control
mode dyn-bts
+ bs-power dyn-max 30
step-size inc 4 red 2
rxlev-thresh lower 32 upper 38
rxlev-thresh-comp lower 10 12 upper 19 20
@@ -89,6 +121,11 @@
no rxqual-avg
...
+OsmoBSC(config-power-ctrl)# bs-power static 30
+% This command is only valid for 'bs-power-control' node
+OsmoBSC(config-power-ctrl)# bs-power dyn-max 30
+% This command is only valid for 'bs-power-control' node
+
OsmoBSC(config-power-ctrl)# step-size inc 5 red 4
% Power change step size must be an even number
OsmoBSC(config-power-ctrl)# step-size inc 2 red 3
@@ -98,6 +135,7 @@
...
bs-power-control
mode dyn-bts
+ bs-power dyn-max 30
step-size inc 4 red 2
...
ms-power-control
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/21812
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I11ca856aba46aaf84d94cbbdf4c39a01ee8289b9
Gerrit-Change-Number: 21812
Gerrit-PatchSet: 2
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/20201222/ed40ee91/attachment.htm>