Project

General

Profile

Actions

Bug #3115

closed

libosmo-mgcp_client isn't able to properly encode SDP with multiple codec options

Added by laforge almost 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
libosmo-mgcp_client
Target version:
-
Start date:
03/26/2018
Due date:
% Done:

0%

Spec Reference:

Description

libosmo-mgcp_client doesn't seem capable of properly encoding SDP with multiple codec options

This is a problem, as both OsmoBSC as well as OsmoMSC need to be able to send the full list of available codecs to the MGW.


Related issues

Related to osmo-sip-connector - Bug #1683: osmo-sip-connector: Implement codec selection / move codec selection to osmo-mscStalledneels03/31/2016

Actions
Actions #1

Updated by laforge almost 6 years ago

  • Related to Bug #1683: osmo-sip-connector: Implement codec selection / move codec selection to osmo-msc added
Actions #2

Updated by laforge over 5 years ago

  • Assignee changed from 4368 to dexter
Actions #3

Updated by dexter over 5 years ago

  • Status changed from New to Feedback
  • Assignee changed from dexter to laforge

Probably I am overlooking/misunderstanding something here. Or this ticket is a duplicate of #3334. I am not sure.

From my understanding the client API has everything needed to negotiate multiple codec options. However, some parameters can not be altered, the sample rate (8000 in almost all cases) and the number of channels (always 1) can not be influenced. The ptime parameter (which is alsways 20 in our cases) can not be set individually for each codec. However I think it should, but SDP does not allow a per-codec ptime setting.

See: http://git.osmocom.org/osmo-mgw/tree/include/osmocom/mgcp_client/mgcp_client.h

In struct mgcp_msg

One findes two members:
enum mgcp_codecs codecs[MGCP_MAX_CODECS];
unsigned int codecs_len;

The member codecs can be populated with up to 10 codecs from enum mgcp_codecs. The payload types are automatically selected depending on IANA/3gpp recomendations. (If it is really necessary to map a codec to a different payload type one can do so by using the ptmap member.)

See also: https://osmocom.org/issues/3334

Actions #4

Updated by laforge over 5 years ago

  • Assignee changed from laforge to dexter
Actions #5

Updated by laforge over 5 years ago

If you think it is a duplicate and/or resolved, please resolve it. I just saw it,
and thought it relates to your current work and that's why I assigned it to you. There's no point in
assigning it back to me, so I have yet one more thing on my plate. Let's not play ticekt ping-pong. :/

Actions #6

Updated by dexter over 5 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)