Project

General

Profile

Bug #4002

AMR defaults to bandwidth-efficient mode

Added by tnt over 2 years ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
05/15/2019
Due date:
% Done:

100%

Spec Reference:

Description

0d9a1a7583842203cbbd60517a2c4d63247db954 introduced a new vty option to select between bandwidth efficient and octet-aligned mode.

Previously it would basically always be octet align since that's what all the BTS use natively, but now, with this option the default has changed which can break existing setup that are not aware of this.


Related issues

Related to OsmoBTS - Bug #3807: short HRv1 RTP payload on PHY based BTS implementationsResolved02/19/2019

Related to OsmoBTS - Bug #3746: Sending RTP stream with unconfigured AMR mode crashes OSClosed01/04/2019

Associated revisions

Revision bad82d31 (diff)
Added by laforge over 2 years ago

Make sure to default to AMR octet-aligned mode

In Change-Id If6d40b2407b87aad2227ea7f15533ef01a3771b3 we introduced
the support for both octet-aligned and bandwidth-efficient mode, and
added a related VTY option about it.

However, we accidentially also changed the default behavior. Without
any explicit configuration, we should behave just like we behaved all
those years before: Continue with octet-aligned mode.

Change-Id: I8b6fd8bec05b35d84ad16275d84f9d363e238d2b
Closes: OS#4002

History

#1 Updated by laforge over 2 years ago

  • Related to Bug #3807: short HRv1 RTP payload on PHY based BTS implementations added

#2 Updated by laforge over 2 years ago

gerrit link to original patch: https://gerrit.osmocom.org/#/c/osmo-bsc/+/13199/

I think we should ensure that the default behavior without any explicit config change is to remain in octet-aligned.

#3 Updated by laforge over 2 years ago

  • Status changed from New to In Progress
  • Assignee set to laforge
  • % Done changed from 0 to 70

#4 Updated by pespin over 2 years ago

  • Related to Bug #3746: Sending RTP stream with unconfigured AMR mode crashes OS added

#5 Updated by laforge over 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 70 to 100

patch has been merged one year ago.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)