Project

General

Profile

Actions

Bug #1801

closed

AMR DTX: downlink logic flawed

Added by laforge over 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
08/20/2016
Due date:
% Done:

100%

Spec Reference:

Description

The AMR DTX logic operates as follows:
  • if there is an incoming downlink RTP frame in the queue for this lchan, send it
  • else, generate PH-EMPTY-FRAME.req or PH-DATA.req with SID payload, depending on time (also broken, see #1799)

This is flawed, as it assumes that the incoming RTP stream is always well-formed and -timed. It would for example also mean that a RTP stream with lots of extra SID frames (more than every 160ms) would result in all of those SID frames transmitted over the radio interface, where the spec says we should only transmit one SID_UPDATE every 160ms.

To quote a rule from IETF: we should be conservative in what we send and tolerant in what we receive.


Related issues

Related to OsmoBTS - Bug #1799: AMR DTX: last_sid should operate on codec frame intervals (20ms), not gsm frame numberClosedlaforge08/20/2016

Actions
Related to OsmoBTS - Bug #1800: AMR DTX: lchan->tch.last_sid is never populated with save_last_sid()Closedlaforge08/20/2016

Actions
Related to OsmoBTS - Bug #1803: osmo-bts-litecell15: AMR FR: some RTP buffers are never releasedClosedlaforge08/20/2016

Actions
Related to OsmoBTS - Bug #1802: AMR DTX Downlink: We are not sending ONSET ?!?Closedlaforge08/20/2016

Actions
Related to OsmoBTS - Bug #1892: AMR half rate crashes osmo-btsClosedlaforge12/20/2016

Actions
Actions #1

Updated by laforge over 7 years ago

  • Description updated (diff)
Actions #2

Updated by laforge over 7 years ago

  • Related to Bug #1799: AMR DTX: last_sid should operate on codec frame intervals (20ms), not gsm frame number added
Actions #3

Updated by laforge over 7 years ago

  • Related to Bug #1800: AMR DTX: lchan->tch.last_sid is never populated with save_last_sid() added
Actions #4

Updated by laforge over 7 years ago

Actions #5

Updated by laforge over 7 years ago

  • Related to Bug #1803: osmo-bts-litecell15: AMR FR: some RTP buffers are never released added
Actions #6

Updated by laforge over 7 years ago

  • Assignee changed from 4368 to msuraev
Actions #7

Updated by msuraev over 7 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 20

Fix available as gerrit #858.

Actions #8

Updated by msuraev over 7 years ago

  • % Done changed from 20 to 80

Should be fixed by gerrit # 1029 and corresponding fixes from related bugs.

Actions #9

Updated by msuraev over 7 years ago

  • Related to Bug #1802: AMR DTX Downlink: We are not sending ONSET ?!? added
Actions #10

Updated by msuraev over 7 years ago

  • File dtx.dot added

Not sure what's the right place for .dot file so attaching current version here for the time being.

Actions #11

Updated by msuraev over 7 years ago

AMR FR is working fine with #rtp jitter-buffer 0'.

Actions #12

Updated by ylebel over 7 years ago

Je serai absent pour la période du 19 décembre 2016 au 3 février 2017 inclusivement. Pour toute question, contactez M. David Bourget à l'adresse . ************************************************* I'll be out of the office from December 19th 2016 to February 3rd 2017 inclusively. If you need immediate assistance, please contact Mr David Bourget at .

Actions #13

Updated by msuraev over 7 years ago

Remaining AMR HR issues:
- FACCH insertion: proper use of SID_FIRST_INH (if SID_FIRST_P1 was used before FACCH) and SID_UPDATE_INH (if SID_UPDATE was used before FACCH)
- SID_UPDATE_INH is not sent to L1 in downlink
- SID_FIRST_INH is not sent to L1 in downlink
- Undesired speech frame during DTX seems to mess up the FSM. SID_UPDATE becomes SID_FIRST_P1

Note: this has not been re-tested with latest master yet.

Actions #14

Updated by msuraev over 7 years ago

  • Related to Bug #1892: AMR half rate crashes osmo-bts added
Actions #15

Updated by msuraev over 7 years ago

  • File deleted (dtx.dot)
Actions #16

Updated by msuraev about 7 years ago

gerrit# 1508 should fix AMR HR problem with inhibition by voice frames. FACCH for AMR HR is WIP.

Actions #17

Updated by msuraev about 7 years ago

Gerrit # 1611 should fix the issue with FACCH for AMR HR.

Actions #18

Updated by msuraev about 7 years ago

  • Status changed from In Progress to Stalled
Actions #19

Updated by msuraev about 7 years ago

  • Status changed from Stalled to Resolved
  • % Done changed from 80 to 100

All fixes has been merged, docs update will follow up shortly.

Actions #20

Updated by laforge about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)