Simulcast SSRCs aren't in encoding order when RTX is on
Categories
(Core :: WebRTC: Signaling, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox94 | --- | fixed |
People
(Reporter: lennox, Assigned: drno)
Details
Attachments
(2 files)
This is a bug exposed by the fix to #1727196.
When simulcast video is sent, prior to the fix to #1727196, the SSRCs are listed in the local SDP in the order corresponding to the configured encodings array.
After that fix, however (i.e. when RTX SSRCs are signaled), the primary (non-RTX) SSRCs are instead listed in numeric-order, irrespective of which encodings they correspond to.
This is a problem for Jitsi-Meet, because the Jitsi Videobridge needs to know which SSRC corresponds to which encoding, for proper encoding routing.
(The correct solution here would be to use rid header extensions, and we're working on that, but it'll be a while.)
It looks like the fix should be as simple as just printing the SSRCs (and RTX SSRCs) in mSsrcs order in JsepTrack::AddToMSection.
It'd be great if this could get fixed before FF 93 hits stable.
Assignee | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
Comment 3•3 years ago
•
|
||
P1 so we can get it worked in for uplift.
Comment 4•3 years ago
|
||
S2 as it effects Jitsi in a non-trivial way.
Assignee | ||
Comment 5•3 years ago
|
||
Depends on D126998
Updated•3 years ago
|
Assignee | ||
Comment 6•3 years ago
|
||
New try run to check if the modified tests are holding up https://treeherder.mozilla.org/jobs?repo=try&revision=753b2c6b819c5e4def73d7f0b5336835dfbee0a0
Pushed by na-g@nostrum.com: https://hg.mozilla.org/integration/autoland/rev/857e5d13da10 sort SSSRC with RTX the same as without. r=bwc https://hg.mozilla.org/integration/autoland/rev/22fd68c02d1f added unit test for verifying SSRC order r=bwc,ng
Comment 8•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/857e5d13da10
https://hg.mozilla.org/mozilla-central/rev/22fd68c02d1f
Description
•