aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeels Hofmeyr <nhofmeyr@sysmocom.de>2024-02-01 05:15:30 +0100
committerNeels Hofmeyr <nhofmeyr@sysmocom.de>2024-02-09 05:17:39 +0100
commita2bb6d5c4d4761078aa07796cbb0d9332f0d95ac (patch)
tree6ed414947ae37324a1343739074ed645560c1141
parent6266b3266b903992ced27c26084b317b566327c9 (diff)
pass dynamically configured AMR rates to BSS and RNC [3/3]
In ran_msg_iu.c for 3G: Compose a list of RAB subflows from the AMR codecs present in the codec list passed in from msc_a. These will hopefully result in the correct RFCIs configured in the IuUP Initialization that the MGW receives. Depends: osmo-iuh I61e0e9e75e3239662846fd797532acdefa9f73dc Change-Id: Ia9f4ad7f3646556aadc632bc5ffa477941626c5f
-rw-r--r--TODO-RELEASE1
-rw-r--r--src/libmsc/ran_msg_iu.c52
2 files changed, 52 insertions, 1 deletions
diff --git a/TODO-RELEASE b/TODO-RELEASE
index 142e073cd..81fae570b 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -9,3 +9,4 @@
#library what description / commit summary line
libosmogsm >1.9.0 ABI breakage in struct osmo_gsup_pdp_info, use new fields pdp_type_* and pdp_address.
libosmo-mgcp-client >= 1.13 uses new public API to parse SDP fmtp
+libosmo-ranap >= 1.5.2 uses new ranap_new_msg_rab_assign_voice_2() to pass SDU subflow selection
diff --git a/src/libmsc/ran_msg_iu.c b/src/libmsc/ran_msg_iu.c
index 13c43fe0a..a57a28481 100644
--- a/src/libmsc/ran_msg_iu.c
+++ b/src/libmsc/ran_msg_iu.c
@@ -35,6 +35,7 @@
#include <osmocom/msc/sccp_ran.h>
#include <osmocom/msc/ran_msg_iu.h>
#include <osmocom/msc/gsm_04_11.h>
+#include <osmocom/msc/codec_mapping.h>
#define LOG_RAN_IU_DEC(RAN_DEC, level, fmt, args...) \
LOG_RAN_DEC(RAN_DEC, DIUCS, level, "RANAP: " fmt, ## args)
@@ -344,6 +345,45 @@ static struct msgb *ran_iu_wrap_dtap(struct msgb *dtap)
return an_apdu;
}
+static uint32_t gsm0808_cfg_to_rab_modes(bool full_rate, uint16_t cfg)
+{
+ int bit;
+ uint32_t rab_modes = 0;
+ for (bit = 0; bit < 16; bit++) {
+ if (!(cfg & (1 << bit)))
+ continue;
+ rab_modes |= gsm0808_amr_modes_from_cfg[full_rate ? 1 : 0][bit];
+ }
+ return rab_modes;
+}
+
+/* Combine all the AMR cfg bits across the codecs */
+static uint32_t codecs_to_rab_modes(const struct sdp_audio_codecs *codecs)
+{
+ struct gsm0808_speech_codec_list scl = {};
+ int i;
+ uint32_t rab_modes = 0;
+ sdp_audio_codecs_to_speech_codec_list(&scl, codecs);
+ for (i = 0; i < scl.len; i++) {
+ bool amr = false;
+ bool fr = false;
+ switch (scl.codec[i].type) {
+ case GSM0808_SCT_FR3:
+ amr = true;
+ fr = true;
+ break;
+ case GSM0808_SCT_HR3:
+ amr = true;
+ break;
+ default:
+ break;
+ }
+ if (amr)
+ rab_modes |= gsm0808_cfg_to_rab_modes(fr, scl.codec[i].cfg);
+ }
+ return rab_modes;
+}
+
static struct msgb *ran_iu_make_rab_assignment(struct osmo_fsm_inst *caller_fi, const struct ran_assignment_command *ac)
{
struct msgb *msg;
@@ -351,6 +391,7 @@ static struct msgb *ran_iu_make_rab_assignment(struct osmo_fsm_inst *caller_fi,
uint32_t cn_rtp_ip;
static uint8_t next_rab_id = 1;
uint8_t rab_id = next_rab_id;
+ uint32_t rab_modes;
next_rab_id ++;
if (!next_rab_id)
@@ -371,7 +412,16 @@ static struct msgb *ran_iu_make_rab_assignment(struct osmo_fsm_inst *caller_fi,
LOG_RAN_IU_ENC(caller_fi, LOGL_DEBUG, "RAB Assignment: rab_id=%d, rtp=" OSMO_SOCKADDR_STR_FMT ", use_x213_nsap=%d\n",
rab_id, OSMO_SOCKADDR_STR_FMT_ARGS(ac->cn_rtp), use_x213_nsap);
- msg = ranap_new_msg_rab_assign_voice(rab_id, cn_rtp_ip, ac->cn_rtp->port, use_x213_nsap);
+ rab_modes = codecs_to_rab_modes(ac->codecs);
+ if (!rab_modes) {
+ LOG_RAN_IU_ENC(caller_fi, LOGL_ERROR, "Error during RAB Assignment:"
+ " no AMR codec available, or no matching AMR rates. AMR is required on 3G\n");
+ return NULL;
+ }
+ /* Always include SID */
+ rab_modes |= (1 << OSMO_RANAP_RAB_MODE_AMR_SID);
+
+ msg = ranap_new_msg_rab_assign_voice_2(rab_id, cn_rtp_ip, ac->cn_rtp->port, use_x213_nsap, rab_modes);
msg->l2h = msg->data;
return msg;