Attention is currently required from: fixeria, dexter.

View Change

2 comments:

• Commit Message:

  • Patch Set #1, Line 7: gsm630: make osmo_hr_check_sid compatible with RFC 5993

    I assume you meant gsm620 here.

• Patchset:

  • Patch Set #1:

    I am not certain at all that the right solution to OS#5688 is to propagate the knowledge of two different RTP formats deep into the innards of our code. Wouldn't it be much cleaner to check the RTP encoding format at the very beginning of RTP input processing in OsmoBTS (or in OsmoMGW feeding E1 Abis), in the same place where calls to osmo_{fr,efr}_sid_preen() have been inserted recently, strip the extraneous RFC 5993 ToC octet if one is present, and then work only with the TS 101 318 format internally throughout our code base? And in the other direction, if the desire is to emit RFC 5993 format to the outside world (configurable per vty, ideally), likewise work in TS 101 318 format internally, but prepend the extra ToC octet just before RTP output.

To view, visit change 32740. To unsubscribe, or for help writing mail filters, visit settings.