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)

Product:
Core
Component:
WebRTC
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Blocking Flags:
backlog webrtc/webaudio+

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.