Change in openbsc[master]: rsl: Send IE MS Power Param during CHAN ACT and MS PWR CTRL messages

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/.

keith gerrit-no-reply at lists.osmocom.org
Fri Mar 27 19:02:08 UTC 2020


keith has submitted this change. ( https://gerrit.osmocom.org/c/openbsc/+/17635 )

Change subject: rsl: Send IE MS Power Param during CHAN ACT and MS PWR CTRL messages
......................................................................

rsl: Send IE MS Power Param during CHAN ACT and MS PWR CTRL messages

Backport of osmo-bsc 6b9e0e4e8834428f85f169106ed7b6141f5b185b (1)
and 60d6d530ac6883db4f5c0394541ad654ddfd526c (2)

(1) TS 48.058 sec 8.4.1 CHANNEL ACTIVATION and state:
"""
The BS and MS Power Parameters elements are included to indicate that BS
and/or MS power control is to be performed by BTS. The maximum power to
be used is indicated in the BS and MS Power elements respectively.
"""

Since we always want the BTS to do autonomous MS power control, let's
add it.

(2)Send IE MS Power Param to osmocom BTS models only

Since MS Power Param IE content is operator dependant, it's currently
not known which kind of content non-osmocom BTS support/allow, so let's
avod possibily breaking those BTS until each BTS has been checked
separately.

Change-Id: Ieb51d5f2202ebd2861d3c33f2b5598e6b29d78eb
---
M openbsc/src/libbsc/abis_rsl.c
1 file changed, 7 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  keith: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved



diff --git a/openbsc/src/libbsc/abis_rsl.c b/openbsc/src/libbsc/abis_rsl.c
index 5a508b2..b06f61d 100644
--- a/openbsc/src/libbsc/abis_rsl.c
+++ b/openbsc/src/libbsc/abis_rsl.c
@@ -686,6 +686,13 @@
 	msgb_tv_put(msg, RSL_IE_BS_POWER, lchan->bs_power);
 	msgb_tv_put(msg, RSL_IE_MS_POWER, lchan->ms_power);
 	msgb_tv_put(msg, RSL_IE_TIMING_ADVANCE, ta);
+	/* indicate MS power control to be performed by BTS: */
+	if (lchan->ts->trx->bts->type == GSM_BTS_TYPE_OSMOBTS)
+		msgb_tl_put(msg, RSL_IE_MS_POWER_PARAM);
+	/* else: Since IE MS_POWER_PARAM content is operator dependent, it's not
+	   known if non-osmocom BTS models will support an empty IE, so let's
+	   better skip sending it unless we know for sure what each expects. */
+
 	mr_config_for_bts(lchan, msg);
 
 	msg->dst = lchan->ts->trx->rsl_link;

-- 
To view, visit https://gerrit.osmocom.org/c/openbsc/+/17635
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Change-Id: Ieb51d5f2202ebd2861d3c33f2b5598e6b29d78eb
Gerrit-Change-Number: 17635
Gerrit-PatchSet: 1
Gerrit-Owner: keith <keith at rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <keith at rhizomatica.org>
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/20200327/c1230241/attachment.htm>


More information about the gerrit-log mailing list