Closed Bug 933198 Opened 6 years ago Closed 6 years ago
_test .html doesn't provide audio on Windows nightly builds
Running the audio tests on http://mozilla.github.io/webrtc-landing/gum_test.html with Windows nightly builds provides no audio feedback. I can swap to FF 24 release and it works fine. Bad versions: Nightly 28.0a1 2013-10-30, Aurora 27.0a2 2013-10-31 Known good: Aurora 26.0a2, 2013-10-28 So this affects 27 & later at the moment.
Confirmed on my win7 nightly, and 10/30 win nightly debug. (Lenovo W520 if it matters). Works fine on linux inbound. Appears to be an AudioStream/cubeb bug. I turned on medialatency:5,mediastreamgraph:5,getusermedia:5,mediamanager:5,audostream:5 and everything up to AudioStream seems to be working. however, the medialatency debugs tell me no DataCallbacks are occurring, so I suspect an earlier error or bug in cubeb. The only audiostream logs I get are: 4876[1390a570]: mozilla::BufferedAudioStream::Init channels: 2, rate: 16000 10100: Latency: AudioStream Create,342605744,131,340266112 (That just lets us connect the AudioStream to the MediaStream feeding it) Critical regression. Perhaps it's my changes in bug 920325 that landed over the weekend, but looking at the diffs I don't see how it could cause this (especially the windows-only aspect, doubly-so when the medialatency logging is off). Perhaps someone could try a backout of that bug and see. Since Paul is on PTO, can someone else have a look? Kinetik? Roc? I have to leave for IETF tomorrow so I don't know when I'll be able to attack this (I'll be traveling through Saturday, and have things planned for Sunday and need to do slides). This requires being able to build and debug windows. It might be as simple as relaxing the cubeb latency target (I hope). Also, regression window will help a lot.
The WASAPI backend's cubeb_stream_init is bailing here: http://mxr.mozilla.org/mozilla-central/source/media/libcubeb/src/cubeb_wasapi.cpp#681 because we're requesting 10ms (which was returned by cubeb_get_min_latency). Removing the 30ms sanity check and allowing the stream to be configured with 10ms latency seems to work fine on Windows 8.1 for me locally. It's possible that the 30ms check is too conservative now that we can query the OS capabilities, so let's remove it. If I'm wrong, Paul can explain why when he returns.
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Attachment #825811 - Flags: review?(rjesup)
In my experience shared mode WASAPI empties buffers in 10ms chunks. It may be that it does 15ms chunks on systems where thread time slice length defaults to 15ms - in that case you would need at least two, 15ms buffers to alternate between (one for the system to use, and one to write to). So in my experience, 10ms is the smallest sensible buffer length (outside exclusive mode, where you can request a different callback frequency). Alternatively, 30ms would be the smallest buffer length that is an integer multiple of both 10ms and 15ms. I haven't looked at the code enough to know which reasoning it uses, but maybe that helps to clear things up.
Oh, and of course it's possible that Windows 8.1 changes things so that the minimum buffer length is smaller (sorry about bugspam).
Comment on attachment 825811 [details] [diff] [review] bug933198_v0.patch Review of attachment 825811 [details] [diff] [review]: ----------------------------------------------------------------- Verified on Win7 Lenovo W520. If we end up finding that there are systems this *doesn't* work on, the alternative would be instead to put a lower limit on latency to 30ms. Try: https://tbpl.mozilla.org/?tree=Try&rev=b7a2f068ab8a
Attachment #825811 - Flags: review?(rjesup) → review+
checkin-needed because I have to get on an airplane and will be unavailable for a couple of days.
Comment on attachment 825811 [details] [diff] [review] bug933198_v0.patch [Approval Request Comment] Bug caused by (feature/regressing bug #): padenot's cubeb changes right before 27 uplift User impact if declined: Audio will not work on many/all windows machines Testing completed (on m-c, etc.): Tested locally, try pushed, should be considered for landing in Aurora as soon as it's on m-c. Risk to taking this patch (and alternatives if risky): Very low risk String or IDL/UUID changes made by this patch: none
Attachment #825811 - Flags: approval-mozilla-aurora?
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
I just verified this is working for me on the setup it was broken with when I reported the bug.
Status: RESOLVED → VERIFIED
Attachment #825811 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0 Mozilla/5.0 (Windows NT 6.1; rv:27.0) Gecko/20100101 Firefox/27.0 Reproduced with Nightly from 2013-10-30. Verified as fixed with Firefox 27 beta 6 (Build ID: 20140113161826): audio feedback is received on http://mozilla.github.io/webrtc-landing/gum_test.html (Audio and Audio/Video)
You need to log in before you can comment on or make changes to this bug.