<p>fixeria has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21873">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">power_control: add increase / reduce step size recommendations<br><br>Change-Id: I82e762c0c2b5e0dd739850ee494ab0a798e353de<br>Related: SYS#4918<br>---<br>M src/osmo-bsc/bsc_vty.c<br>M tests/power_ctrl.vty<br>2 files changed, 25 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/73/21873/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c</span><br><span>index 3cb479d..c106319 100644</span><br><span>--- a/src/osmo-bsc/bsc_vty.c</span><br><span>+++ b/src/osmo-bsc/bsc_vty.c</span><br><span>@@ -4953,15 +4953,33 @@</span><br><span>              "Step size (2 or 4 dB)\n")</span><br><span> {</span><br><span>      struct gsm_power_ctrl_params *params = vty->index;</span><br><span style="color: hsl(120, 100%, 40%);">+ int inc_step_size_db = atoi(argv[0]);</span><br><span style="color: hsl(120, 100%, 40%);">+ int red_step_size_db = atoi(argv[1]);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       if (atoi(argv[0]) % 2 || atoi(argv[1]) % 2) {</span><br><span style="color: hsl(120, 100%, 40%);">+ if (inc_step_size_db % 2 || red_step_size_db % 2) {</span><br><span>          vty_out(vty, "%% Power change step size must be "</span><br><span>                  "an even number%s", VTY_NEWLINE);</span><br><span>          return CMD_WARNING;</span><br><span>  }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   params->inc_step_size_db = atoi(argv[0]);</span><br><span style="color: hsl(0, 100%, 40%);">-    params->red_step_size_db = atoi(argv[1]);</span><br><span style="color: hsl(120, 100%, 40%);">+  /* Recommendation: POW_RED_STEP_SIZE <= POW_INCR_STEP_SIZE */</span><br><span style="color: hsl(120, 100%, 40%);">+      if (red_step_size_db > inc_step_size_db) {</span><br><span style="color: hsl(120, 100%, 40%);">+         vty_out(vty, "%% Increase step size (%d) should be greather "</span><br><span style="color: hsl(120, 100%, 40%);">+                       "than reduce step size (%d), consider changing it%s",</span><br><span style="color: hsl(120, 100%, 40%);">+                       inc_step_size_db, red_step_size_db, VTY_NEWLINE);</span><br><span style="color: hsl(120, 100%, 40%);">+     }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   /* Recommendation: POW_INCR_STEP_SIZE <= (U_RXLEV_XX_P - L_RXLEV_XX_P) */</span><br><span style="color: hsl(120, 100%, 40%);">+  const struct gsm_power_ctrl_meas_params *mp = &params->rxlev_meas;</span><br><span style="color: hsl(120, 100%, 40%);">+     if (inc_step_size_db > (mp->upper_thresh - mp->lower_thresh)) {</span><br><span style="color: hsl(120, 100%, 40%);">+              vty_out(vty, "%% Increase step size (%d) should be less or equal "</span><br><span style="color: hsl(120, 100%, 40%);">+                  "than/to the RxLev threshold window (%d, upper - lower), "</span><br><span style="color: hsl(120, 100%, 40%);">+                  "consider changing it%s", inc_step_size_db,</span><br><span style="color: hsl(120, 100%, 40%);">+                 mp->upper_thresh - mp->lower_thresh, VTY_NEWLINE);</span><br><span style="color: hsl(120, 100%, 40%);">+      }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   params->inc_step_size_db = inc_step_size_db;</span><br><span style="color: hsl(120, 100%, 40%);">+       params->red_step_size_db = red_step_size_db;</span><br><span> </span><br><span>  return CMD_SUCCESS;</span><br><span> }</span><br><span>diff --git a/tests/power_ctrl.vty b/tests/power_ctrl.vty</span><br><span>index d58c075..4fa416f 100644</span><br><span>--- a/tests/power_ctrl.vty</span><br><span>+++ b/tests/power_ctrl.vty</span><br><span>@@ -131,6 +131,9 @@</span><br><span> OsmoBSC(config-ms-power-ctrl)# step-size inc 2 red 3</span><br><span> % Power change step size must be an even number</span><br><span> OsmoBSC(config-ms-power-ctrl)# step-size inc 2 red 4</span><br><span style="color: hsl(120, 100%, 40%);">+% Increase step size (2) should be greather than reduce step size (4), consider changing it</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+OsmoBSC(config-ms-power-ctrl)# step-size inc 6 red 4</span><br><span> OsmoBSC(config-ms-power-ctrl)# show running-config</span><br><span> ...</span><br><span>   bs-power-control</span><br><span>@@ -140,7 +143,7 @@</span><br><span> ...</span><br><span>   ms-power-control</span><br><span>    mode dyn-bts</span><br><span style="color: hsl(0, 100%, 40%);">-   step-size inc 2 red 4</span><br><span style="color: hsl(120, 100%, 40%);">+   step-size inc 6 red 4</span><br><span> ...</span><br><span> </span><br><span> OsmoBSC(config-ms-power-ctrl)# rxlev-thresh lower?</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/21873">change 21873</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/21873"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I82e762c0c2b5e0dd739850ee494ab0a798e353de </div>
<div style="display:none"> Gerrit-Change-Number: 21873 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>