neels has uploaded this change for review. (
https://gerrit.osmocom.org/c/osmo-bsc/+/21372 )
Change subject: fix AMR start-mode auto: reflect proper ICMI in MultiRate
Config IE
......................................................................
fix AMR start-mode auto: reflect proper ICMI in MultiRate Config IE
Send the proper ICMI value, instead of always sending ICMI=1 regardless of the
AMR start-mode setting.
To avoid test fallout from the fix, we should first merge
osmo-ttcn3-hacks I4cff01c37d5c7e301e9a01f773b7e009a789519b
Change-Id: I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1
---
M src/osmo-bsc/lchan_fsm.c
1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/72/21372/1
diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index e97c1ba..0ca7f2c 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -467,6 +467,7 @@
mr = &bts->mr_full;
else
mr = &bts->mr_half;
+ mr_conf_bts = (struct gsm48_multi_rate_conf *)mr->gsm48_ie;
if (lchan->activate.info.activ_for == FOR_VTY)
/* If the channel is activated manually from VTY, then there is
no
@@ -503,7 +504,6 @@
* if the channel activation was triggerd by the VTY, do not
* filter anything (see also comment above) */
if (lchan->activate.info.activ_for != FOR_VTY) {
- mr_conf_bts = (struct gsm48_multi_rate_conf *)mr->gsm48_ie;
rc_rate = calc_amr_rate_intersection(&mr_conf_filtered,
mr_conf_bts, &mr_conf_filtered);
if (rc_rate < 0) {
LOG_LCHAN(lchan, LOGL_ERROR,
@@ -512,6 +512,10 @@
}
}
+ /* Set the ICMI according to the BTS. Above
gsm48_mr_cfg_from_gsm0808_sc_cfg() always sets ICMI = 1, which
+ * carried through all of the above rate intersections. */
+ mr_conf_filtered.icmi = mr_conf_bts->icmi;
+
/* Proceed with the generation of the multirate configuration IE
* (MS and BTS) */
rc = gsm48_multirate_config(lchan->mr_ms_lv, &mr_conf_filtered,
mr->ms_mode, mr->num_modes);
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/21372
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1
Gerrit-Change-Number: 21372
Gerrit-PatchSet: 1
Gerrit-Owner: neels <[email protected]>
Gerrit-MessageType: newchange
