Closed Bug 778675 Opened 12 years ago Closed 11 years ago

Intermittent 481136-1.html, 466945-1.html, 468763-1.html, 459439-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.")

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Platform:
x86_64
macOS
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla19
Tracking Flags:
Tracking Status
firefox17 --- wontfix
firefox18 --- fixed
firefox19 --- fixed

People

(Reporter: emorley, Assigned: kinetik)

References

Details

(Keywords: assertion, intermittent-failure, Whiteboard: [qa-])

Attachments

(1 file)

patch v0
3.71 KB, patch
cajbir
: review+
lsblakk
: approval-mozilla-aurora+
lsblakk
: approval-mozilla-beta-
Details | Diff | Splinter Review 
Rev4 MacOSX Lion 10.7 mozilla-inbound opt test crashtest on 2012-07-27 18:30:57 PDT for push 73113b32afac

slave: talos-r4-lion-015

https://tbpl.mozilla.org/php/getParsedLog.php?id=13925059&tree=Mozilla-Inbound

{
REFTEST TEST-START | file:///Users/cltbld/talos-slave/test/build/reftest/tests/content/media/test/crashtests/474744-1.html | 417 / 2099 (19%)
REFTEST TEST-PASS | file:///Users/cltbld/talos-slave/test/build/reftest/tests/content/media/test/crashtests/474744-1.html | (LOAD ONLY)
REFTEST INFO | Loading a blank page
REFTEST TEST-START | http://localhost:4444/1343439475907/3/481136-1.html | 418 / 2099 (19%)
Assertion failed: (r == 0), function cubeb_stream_init, file ../../../../../media/libcubeb/src/cubeb_audiounit.c, line 184.
TEST-UNEXPECTED-FAIL | http://localhost:4444/1343439475907/3/481136-1.html | Exited with code -6 during test run
}
Blocks: 742160, 768265
OpenAComponent is failing (http://mxr.mozilla.org/mozilla-central/source/media/libcubeb/src/cubeb_audiounit.c#184).  I can make that non-fatal, but I'm curious about *why* it's failing...
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
https://tbpl.mozilla.org/php/getParsedLog.php?id=14044568&tree=Mozilla-Inbound
Summary: Intermittent 481136-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.") → Intermittent 481136-1.html, 466945-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.")
https://tbpl.mozilla.org/php/getParsedLog.php?id=14063250&tree=Mozilla-Inbound
Summary: Intermittent 481136-1.html, 466945-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.") → Intermittent 481136-1.html, 466945-1.html, 468763-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.")
Pushed some logging:

https://hg.mozilla.org/integration/mozilla-inbound/rev/c8f9da21bc88

Once this fails again, I'll look at the log and then make this non-fatal.
Whiteboard: [orange] → [orange][leave open]
Oh, cute.  This time it failed on the next assert(r == 0), which I hadn't instrumented.  I suspect this is going to turn out to be an OOM situation...
Summary: Intermittent 481136-1.html, 466945-1.html, 468763-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.") → [debug this] Intermittent 481136-1.html, 466945-1.html, 468763-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.")
(In reply to TinderboxPushlog Robot from comment #10)
> https://tbpl.mozilla.org/php/getParsedLog.php?id=14963650&tree=Try

cubeb_audiounit: FATAL: AudioUnitSetProperty(SetRenderCallback) returned -4
Assertion failed: (r == 0), function cubeb_stream_init, file ../../../../../media/libcubeb/src/cubeb_audiounit.c, line 197.

AudioUnitSetProperty returns an OSStatus, which (according to MacErrors.h and /usr/bin/macerror) is unimpErr (unimplemented core routine).  I have no idea what that means in this context.  I'll wait for another failure or so and see if it's the same error in each case.
https://tbpl.mozilla.org/php/getParsedLog.php?id=15151242&tree=Mozilla-Inbound
Summary: [debug this] Intermittent 481136-1.html, 466945-1.html, 468763-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.") → [debug this] Intermittent 481136-1.html, 466945-1.html, 468763-1.html, 459439-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.")
Same failure code from the same function that time.
Attached patch patch v0 — — Splinter Review 
I have no idea what to do about these, so let's just make them non-fatal.
Attachment #672678 - Flags: review?(chris.double)
Whiteboard: [orange][leave open] → [orange]
Summary: [debug this] Intermittent 481136-1.html, 466945-1.html, 468763-1.html, 459439-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.") → Intermittent 481136-1.html, 466945-1.html, 468763-1.html, 459439-1.html | Exited with code -6 during test run ("Assertion failed: (r == 0), function cubeb_stream_init, file media/libcubeb/src/cubeb_audiounit.c, line 184.")
Attachment #672678 - Flags: review?(chris.double) → review+
https://hg.mozilla.org/mozilla-central/rev/e0b2ebcbd56a
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Can we request approval to uplift this to aurora and beta please?
status-firefox17: --- → affected
status-firefox18: --- → affected
status-firefox19: --- → fixed
Comment on attachment 672678 [details] [diff] [review]
patch v0

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 768265
User impact if declined: fatal assertion in some rare cases
Testing completed (on m-c, etc.): yes, baked on m-c for approximately a month
Risk to taking this patch (and alternatives if risky): none
String or UUID changes made by this patch: none

We'll also want bug 805352's patch, which is the same but for cubeb_stream_shutdown.
Attachment #672678 - Flags: approval-mozilla-beta?
Attachment #672678 - Flags: approval-mozilla-aurora?
Comment on attachment 672678 [details] [diff] [review]
patch v0

Too late for Beta, but go ahead with uplift to Aurora before Monday 11/19 merge.
Attachment #672678 - Flags: approval-mozilla-beta?
Attachment #672678 - Flags: approval-mozilla-beta-
Attachment #672678 - Flags: approval-mozilla-aurora?
Attachment #672678 - Flags: approval-mozilla-aurora+
Whiteboard: [orange]
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.