<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20730">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">power_control: tolerate small deviations from 'rx-target'<br><br>Recently we've introduced EWMA based uplink power filtering, that<br>should reduce Uplink power oscillations.  However, the power loop<br>is still quite sensitive to small deviations from the target power<br>level: even such an insignificant deviation like 2-5 dBm triggers<br>the loop to increase or decrease the MS power level.  Even if the<br>EWMA based filtering is enabled with 80% smoothing (alpha = 0.2).<br><br>This change introduces a new configuration parameter - 'hysteresis':<br><br>  uplink-power-target <-110-0> hysteresis <1-25><br><br>that together with the 'uplink-power-target' defines a range:<br><br>  [target - hysteresis .. target + hysteresis]<br><br>in which the MS power loop would not trigger any power changes.<br><br>This feature is now *enabled* by default, so given that:<br><br>  - default 'uplink-power-target' is -75 dBm, and<br>  - default 'hysteresis' is 3 dBm,<br><br>the default target Uplink power range is: -78 dBm ... -72 dBm.<br><br>Change-Id: Iacedbd4d69d3d74e2499af5622a07a8af0423da0<br>Related: SYS#4916<br>---<br>M include/osmo-bts/bts.h<br>M src/common/bts.c<br>M src/common/power_control.c<br>M src/common/vty.c<br>M tests/power/power_test.c<br>M tests/power/power_test.err<br>M tests/power/power_test.ok<br>7 files changed, 90 insertions(+), 11 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmo-bts/bts.h b/include/osmo-bts/bts.h</span><br><span>index db8f741..a2eecc4 100644</span><br><span>--- a/include/osmo-bts/bts.h</span><br><span>+++ b/include/osmo-bts/bts.h</span><br><span>@@ -298,6 +298,7 @@</span><br><span> </span><br><span>   /* TODO: move it to bts->ul_power_ctrl struct */</span><br><span>  int ul_power_target;            /* Uplink Rx power target */</span><br><span style="color: hsl(120, 100%, 40%);">+  int ul_power_hysteresis;        /* Tolerable Uplink Rx power deviation */</span><br><span> </span><br><span>        /* Uplink power control */</span><br><span>   struct {</span><br><span>diff --git a/src/common/bts.c b/src/common/bts.c</span><br><span>index bb0795a..8a97b8f 100644</span><br><span>--- a/src/common/bts.c</span><br><span>+++ b/src/common/bts.c</span><br><span>@@ -303,6 +303,7 @@</span><br><span>  /* configurable via VTY */</span><br><span>   bts->paging_state = paging_init(bts, 200, 0);</span><br><span>     bts->ul_power_target = -75;  /* dBm default */</span><br><span style="color: hsl(120, 100%, 40%);">+     bts->ul_power_hysteresis = 3;        /* -78 .. -72 dBm */</span><br><span>         bts->ul_power_ctrl.pf_algo = MS_UL_PF_ALGO_EWMA;</span><br><span>  bts->ul_power_ctrl.pf.ewma.alpha = 20; /* 80% smoothing */</span><br><span>        bts->rtp_jitter_adaptive = false;</span><br><span>diff --git a/src/common/power_control.c b/src/common/power_control.c</span><br><span>index e2d1254..be6df48 100644</span><br><span>--- a/src/common/power_control.c</span><br><span>+++ b/src/common/power_control.c</span><br><span>@@ -144,12 +144,12 @@</span><br><span>    diff = bts->ul_power_target - avg_ul_rssi_dbm;</span><br><span> </span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       /* TODO: introduce 'hysteresis' defining a range of tolerable deviation */</span><br><span style="color: hsl(0, 100%, 40%);">-      if (diff == 0) {</span><br><span style="color: hsl(120, 100%, 40%);">+      /* Tolerate small deviations from 'rx-target' */</span><br><span style="color: hsl(120, 100%, 40%);">+      if (abs(diff) <= bts->ul_power_hysteresis) {</span><br><span>           LOGPLCHAN(lchan, DLOOP, LOGL_INFO,</span><br><span>                     "Keeping MS power at control level %d (%d dBm) because diff %d dBm "</span><br><span style="color: hsl(0, 100%, 40%);">-                          "from 'rx-target' %d dBm is not significant\n",</span><br><span style="color: hsl(0, 100%, 40%);">-                       ms_power_lvl, ms_dbm, diff, bts->ul_power_target);</span><br><span style="color: hsl(120, 100%, 40%);">+                         "from 'rx-target' %d dBm is not significant (hysteresis %d dBm)\n",</span><br><span style="color: hsl(120, 100%, 40%);">+                         ms_power_lvl, ms_dbm, diff, bts->ul_power_target, bts->ul_power_hysteresis);</span><br><span>                 /* Keep the current power level in sync (just to be sure) */</span><br><span>                 lchan->ms_power_ctrl.current = ms_power_lvl;</span><br><span>              bts_model_adjst_ms_pwr(lchan);</span><br><span>diff --git a/src/common/vty.c b/src/common/vty.c</span><br><span>index 16ffe72..d20b143 100644</span><br><span>--- a/src/common/vty.c</span><br><span>+++ b/src/common/vty.c</span><br><span>@@ -255,7 +255,10 @@</span><br><span>           VTY_NEWLINE);</span><br><span>        vty_out(vty, " paging lifetime %u%s", paging_get_lifetime(bts->paging_state),</span><br><span>           VTY_NEWLINE);</span><br><span style="color: hsl(0, 100%, 40%);">-   vty_out(vty, " uplink-power-target %d%s", bts->ul_power_target, VTY_NEWLINE);</span><br><span style="color: hsl(120, 100%, 40%);">+    vty_out(vty, " uplink-power-target %d", bts->ul_power_target);</span><br><span style="color: hsl(120, 100%, 40%);">+   if (bts->ul_power_hysteresis > 0)</span><br><span style="color: hsl(120, 100%, 40%);">+               vty_out(vty, " hysteresis %d", bts->ul_power_hysteresis);</span><br><span style="color: hsl(120, 100%, 40%);">+        vty_out(vty, "%s", VTY_NEWLINE);</span><br><span> </span><br><span>       /* MS Tx power filtering algorithm and parameters */</span><br><span>         switch (bts->ul_power_ctrl.pf_algo) {</span><br><span>@@ -615,15 +618,37 @@</span><br><span>     return CMD_SUCCESS;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+#define UL_POWER_TARGET_CMD \</span><br><span style="color: hsl(120, 100%, 40%);">+   "uplink-power-target <-110-0>"</span><br><span style="color: hsl(120, 100%, 40%);">+#define UL_POWER_TARGET_CMD_DESC \</span><br><span style="color: hsl(120, 100%, 40%);">+    "Set the nominal target Rx Level for uplink power control loop\n" \</span><br><span style="color: hsl(120, 100%, 40%);">+ "Target uplink Rx level in dBm\n"</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> DEFUN_ATTR(cfg_bts_ul_power_target, cfg_bts_ul_power_target_cmd,</span><br><span style="color: hsl(0, 100%, 40%);">-          "uplink-power-target <-110-0>",</span><br><span style="color: hsl(0, 100%, 40%);">-         "Set the nominal target Rx Level for uplink power control loop\n"</span><br><span style="color: hsl(0, 100%, 40%);">-     "Target uplink Rx level in dBm\n",</span><br><span style="color: hsl(120, 100%, 40%);">+          UL_POWER_TARGET_CMD, UL_POWER_TARGET_CMD_DESC,</span><br><span>       CMD_ATTR_IMMEDIATE)</span><br><span> {</span><br><span>  struct gsm_bts *bts = vty->index;</span><br><span> </span><br><span>     bts->ul_power_target = atoi(argv[0]);</span><br><span style="color: hsl(120, 100%, 40%);">+      bts->ul_power_hysteresis = 0;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+    return CMD_SUCCESS;</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%);">+/* FIXME: libosmovty is unable to handle 'foo <-110-0> [bar <1-25>]' */</span><br><span style="color: hsl(120, 100%, 40%);">+DEFUN_ATTR(cfg_bts_ul_power_target_hysteresis,</span><br><span style="color: hsl(120, 100%, 40%);">+    cfg_bts_ul_power_target_hysteresis_cmd,</span><br><span style="color: hsl(120, 100%, 40%);">+       UL_POWER_TARGET_CMD " hysteresis <1-25>",</span><br><span style="color: hsl(120, 100%, 40%);">+     UL_POWER_TARGET_CMD_DESC</span><br><span style="color: hsl(120, 100%, 40%);">+      "Target Rx Level hysteresis\n"</span><br><span style="color: hsl(120, 100%, 40%);">+      "Tolerable deviation in dBm\n",</span><br><span style="color: hsl(120, 100%, 40%);">+     CMD_ATTR_IMMEDIATE)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+     struct gsm_bts *bts = vty->index;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+        bts->ul_power_target = atoi(argv[0]);</span><br><span style="color: hsl(120, 100%, 40%);">+      bts->ul_power_hysteresis = atoi(argv[1]);</span><br><span> </span><br><span>     return CMD_SUCCESS;</span><br><span> }</span><br><span>@@ -1849,6 +1874,7 @@</span><br><span>     install_element(BTS_NODE, &cfg_bts_agch_queue_mgmt_default_cmd);</span><br><span>         install_element(BTS_NODE, &cfg_bts_agch_queue_mgmt_params_cmd);</span><br><span>  install_element(BTS_NODE, &cfg_bts_ul_power_target_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_element(BTS_NODE, &cfg_bts_ul_power_target_hysteresis_cmd);</span><br><span>      install_element(BTS_NODE, &cfg_bts_no_ul_power_filter_cmd);</span><br><span>      install_element(BTS_NODE, &cfg_bts_ul_power_filter_ewma_cmd);</span><br><span>    install_element(BTS_NODE, &cfg_bts_min_qual_rach_cmd);</span><br><span>diff --git a/tests/power/power_test.c b/tests/power/power_test.c</span><br><span>index bf474ac..c5c4db3 100644</span><br><span>--- a/tests/power/power_test.c</span><br><span>+++ b/tests/power/power_test.c</span><br><span>@@ -1,5 +1,6 @@</span><br><span> /*</span><br><span>  * (C) 2013,2014 by Holger Hans Peter Freyther</span><br><span style="color: hsl(120, 100%, 40%);">+ * Contributions by sysmocom - s.m.f.c. GmbH <info@sysmocom.de></span><br><span>  *</span><br><span>  * All Rights Reserved</span><br><span>  *</span><br><span>@@ -205,6 +206,32 @@</span><br><span>        CHECK_UL_RSSI_AVG100(-92.0);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static void test_power_hysteresis(void)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+     struct gsm_lchan *lchan;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+    init_test(__func__);</span><br><span style="color: hsl(120, 100%, 40%);">+  lchan = &g_trx->ts[0].lchan[0];</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      /* Tolerate power deviations in range -80 .. -70 */</span><br><span style="color: hsl(120, 100%, 40%);">+   g_bts->ul_power_hysteresis = 5;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+  lchan->ms_power_ctrl.current = ms_pwr_ctl_lvl(GSM_BAND_1800, 0);</span><br><span style="color: hsl(120, 100%, 40%);">+   OSMO_ASSERT(lchan->ms_power_ctrl.current == 15);</span><br><span style="color: hsl(120, 100%, 40%);">+   lchan->ms_power_ctrl.max = ms_pwr_ctl_lvl(GSM_BAND_1800, 26);</span><br><span style="color: hsl(120, 100%, 40%);">+      OSMO_ASSERT(lchan->ms_power_ctrl.max == 2);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      apply_power_test(lchan, g_bts->ul_power_target, 0, 15);</span><br><span style="color: hsl(120, 100%, 40%);">+    apply_power_test(lchan, g_bts->ul_power_target + 3, 0, 15);</span><br><span style="color: hsl(120, 100%, 40%);">+        apply_power_test(lchan, g_bts->ul_power_target - 3, 0, 15);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      apply_power_test(lchan, g_bts->ul_power_target, 0, 15);</span><br><span style="color: hsl(120, 100%, 40%);">+    apply_power_test(lchan, g_bts->ul_power_target + 5, 0, 15);</span><br><span style="color: hsl(120, 100%, 40%);">+        apply_power_test(lchan, g_bts->ul_power_target - 5, 0, 15);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      apply_power_test(lchan, g_bts->ul_power_target - 10, 1, 13);</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> int main(int argc, char **argv)</span><br><span> {</span><br><span>         printf("Testing power loop...\n");</span><br><span>@@ -220,6 +247,7 @@</span><br><span> </span><br><span>       test_power_loop();</span><br><span>   test_pf_algo_ewma();</span><br><span style="color: hsl(120, 100%, 40%);">+  test_power_hysteresis();</span><br><span> </span><br><span>         printf("Power loop test OK\n");</span><br><span> </span><br><span>diff --git a/tests/power/power_test.err b/tests/power/power_test.err</span><br><span>index 1aac5e5..988b3ba 100644</span><br><span>--- a/tests/power/power_test.err</span><br><span>+++ b/tests/power/power_test.err</span><br><span>@@ -4,7 +4,7 @@</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 11 (8 dBm) to 9, 12 dBm (rx-ms-pwr-lvl 11, max-ms-pwr-lvl 2, rx-current -90 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 9 (12 dBm) to 7, 16 dBm (rx-ms-pwr-lvl 9, max-ms-pwr-lvl 2, rx-current -90 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 7 (16 dBm) to 5, 20 dBm (rx-ms-pwr-lvl 7, max-ms-pwr-lvl 2, rx-current -90 dBm, rx-target -75 dBm)</span><br><span style="color: hsl(0, 100%, 40%);">-(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 5 (20 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 5 (20 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant (hysteresis 0 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 5 (20 dBm) to 3, 24 dBm (rx-ms-pwr-lvl 5, max-ms-pwr-lvl 2, rx-current -90 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 3 (24 dBm) to 2, 26 dBm (rx-ms-pwr-lvl 3, max-ms-pwr-lvl 2, rx-current -90 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 2, 26 dBm (rx-ms-pwr-lvl 2, max-ms-pwr-lvl 2, rx-current -90 dBm, rx-target -75 dBm)</span><br><span>@@ -13,7 +13,7 @@</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 0 (30 dBm) to 30, 34 dBm (rx-ms-pwr-lvl 0, max-ms-pwr-lvl 29, rx-current -90 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 30 (34 dBm) to 29, 36 dBm (rx-ms-pwr-lvl 30, max-ms-pwr-lvl 29, rx-current -90 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 29, 36 dBm (rx-ms-pwr-lvl 29, max-ms-pwr-lvl 29, rx-current -90 dBm, rx-target -75 dBm)</span><br><span style="color: hsl(0, 100%, 40%);">-(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 29 (36 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 29 (36 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant (hysteresis 0 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Lowering MS power from control level 29 (36 dBm) to 1, 28 dBm (rx-ms-pwr-lvl 29, max-ms-pwr-lvl 29, rx-current -45 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Lowering MS power from control level 1 (28 dBm) to 5, 20 dBm (rx-ms-pwr-lvl 1, max-ms-pwr-lvl 29, rx-current -45 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Lowering MS power from control level 5 (20 dBm) to 9, 12 dBm (rx-ms-pwr-lvl 5, max-ms-pwr-lvl 29, rx-current -45 dBm, rx-target -75 dBm)</span><br><span>@@ -21,10 +21,17 @@</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 10 (10 dBm) to 9, 12 dBm (rx-ms-pwr-lvl 10, max-ms-pwr-lvl 29, rx-current -77 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Lowering MS power from control level 9 (12 dBm) to 14, 2 dBm (rx-ms-pwr-lvl 9, max-ms-pwr-lvl 14, rx-current -73 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Lowering MS power from control level 14 (2 dBm) to 15, 0 dBm (rx-ms-pwr-lvl 14, max-ms-pwr-lvl 0, rx-current -40 dBm, rx-target -75 dBm)</span><br><span style="color: hsl(0, 100%, 40%);">-(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant (hysteresis 0 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 15 (0 dBm) to 13, 3 dBm (rx-ms-pwr-lvl 15, max-ms-pwr-lvl 2, rx-current -78 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 13 (4 dBm) to 11, 8 dBm (rx-ms-pwr-lvl 13, max-ms-pwr-lvl 2, rx-current -80 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 11 (8 dBm) to 9, 11 dBm (rx-ms-pwr-lvl 11, max-ms-pwr-lvl 2, rx-current -78 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15, 0 dBm (rx-ms-pwr-lvl 15, max-ms-pwr-lvl 2, rx-current -50 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15, 0 dBm (rx-ms-pwr-lvl 15, max-ms-pwr-lvl 2, rx-current -50 dBm, rx-target -75 dBm)</span><br><span> (bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 15 (0 dBm) to 13, 4 dBm (rx-ms-pwr-lvl 15, max-ms-pwr-lvl 2, rx-current -92 dBm, rx-target -75 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant (hysteresis 5 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff -3 dBm from 'rx-target' -75 dBm is not significant (hysteresis 5 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff 3 dBm from 'rx-target' -75 dBm is not significant (hysteresis 5 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff 0 dBm from 'rx-target' -75 dBm is not significant (hysteresis 5 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff -5 dBm from 'rx-target' -75 dBm is not significant (hysteresis 5 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Keeping MS power at control level 15 (0 dBm) because diff 5 dBm from 'rx-target' -75 dBm is not significant (hysteresis 5 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+(bts=0,trx=0,ts=0,ss=0) Raising MS power from control level 15 (0 dBm) to 13, 4 dBm (rx-ms-pwr-lvl 15, max-ms-pwr-lvl 2, rx-current -85 dBm, rx-target -75 dBm)</span><br><span>diff --git a/tests/power/power_test.ok b/tests/power/power_test.ok</span><br><span>index 25551d8..d463d87 100644</span><br><span>--- a/tests/power/power_test.ok</span><br><span>+++ b/tests/power/power_test.ok</span><br><span>@@ -72,4 +72,20 @@</span><br><span> lchan_ms_pwr_ctrl(RxLvl=-110 dBm) returns 1 (expected 1)</span><br><span>      MS current power 15 -> 13 (expected 13)</span><br><span>   Avg[t] is -92.00 dBm (expected -92.00 dBm)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Starting test case 'test_power_hysteresis'</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-75 dBm) returns 0 (expected 0)</span><br><span style="color: hsl(120, 100%, 40%);">+     MS current power 15 -> 15 (expected 15)</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-72 dBm) returns 0 (expected 0)</span><br><span style="color: hsl(120, 100%, 40%);">+   MS current power 15 -> 15 (expected 15)</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-78 dBm) returns 0 (expected 0)</span><br><span style="color: hsl(120, 100%, 40%);">+   MS current power 15 -> 15 (expected 15)</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-75 dBm) returns 0 (expected 0)</span><br><span style="color: hsl(120, 100%, 40%);">+   MS current power 15 -> 15 (expected 15)</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-70 dBm) returns 0 (expected 0)</span><br><span style="color: hsl(120, 100%, 40%);">+   MS current power 15 -> 15 (expected 15)</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-80 dBm) returns 0 (expected 0)</span><br><span style="color: hsl(120, 100%, 40%);">+   MS current power 15 -> 15 (expected 15)</span><br><span style="color: hsl(120, 100%, 40%);">+lchan_ms_pwr_ctrl(RxLvl=-85 dBm) returns 1 (expected 1)</span><br><span style="color: hsl(120, 100%, 40%);">+   MS current power 15 -> 13 (expected 13)</span><br><span> Power loop test OK</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20730">change 20730</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-bts/+/20730"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iacedbd4d69d3d74e2499af5622a07a8af0423da0 </div>
<div style="display:none"> Gerrit-Change-Number: 20730 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>