Closed Bug 797084 Opened 12 years ago Closed 6 years ago

DataChannel needs to proxy IO (usrsctp_sendv()) to off-main-thread

Categories

(Core :: WebRTC, defect, P4)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jesup, Assigned: jesup)

References

Details

SendControlMessage and SendMsgInternal both call usrsctp_sendv directly.  We should proxy all IO to another thread (off mainthread), as WebSockets does with the OutboundEnqueue stuff.

Note that DataChannels works in non-blocking mode currently, and internally queues using the SendDeferredMessages() function, but this requires extra memory buffering for large binary transfers.

SendDeferredMessages() is also non-optimal since it uses a timer instead of waiting, so it may not be 100% efficient at saturating your link ;-)

Probably does NOT block preffing on webrtc
Whiteboard: [WebRTC], [blocking-webrtc-]
backlog: --- → webRTC+
Rank: 38
Priority: -- → P3
QA Contact: jsmith
Whiteboard: [WebRTC], [blocking-webrtc-]
Mass change P3->P4 to align with new Mozilla triage process.
Priority: P3 → P4
Fixed long ago.  Happens on Socket Thread
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.