Bug #1801
closedAMR DTX: downlink logic flawed
100%
Description
- 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
Updated by laforge almost 8 years ago
- Related to Bug #1799: AMR DTX: last_sid should operate on codec frame intervals (20ms), not gsm frame number added
Updated by laforge almost 8 years ago
- Related to Bug #1800: AMR DTX: lchan->tch.last_sid is never populated with save_last_sid() added
Updated by laforge almost 8 years ago
- Related to Bug #1803: osmo-bts-litecell15: AMR FR: some RTP buffers are never released added
Updated by msuraev almost 8 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 20
Fix available as gerrit #858.
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.
Updated by msuraev over 7 years ago
- Related to Bug #1802: AMR DTX Downlink: We are not sending ONSET ?!? added
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.
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 david.bourget@nutaq.com. ************************************************* 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 david.bourget@nutaq.com.
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.
Updated by msuraev over 7 years ago
- Related to Bug #1892: AMR half rate crashes osmo-bts added
Updated by msuraev over 7 years ago
gerrit# 1508 should fix AMR HR problem with inhibition by voice frames. FACCH for AMR HR is WIP.
Updated by msuraev over 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.