Closed Bug 1150918 Opened 10 years ago Closed 10 years ago

[Crash] [@ mozilla::MediaStreamGraphImpl::UpdateStreamOrder]

Categories

(Firefox OS Graveyard :: Stability, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.2+)

RESOLVED WORKSFORME
blocking-b2g 2.2+

People

(Reporter: ntroast, Unassigned)

References

Details

(Keywords: crash, Whiteboard: [b2g-crash][caf-crash 530][caf priority: p3][CR 817667])

Attachments

(10 files)

We observed the following crash signature during testing. [@ mozilla::MediaStreamGraphImpl::UpdateStreamOrder | mozilla::MediaStreamGraphImpl::UpdateGraph | mozilla::MediaStreamGraphImpl::OneIteration | mozilla::AudioCallbackDriver::DataCallback ] Cafbot will upload the decoded minidump and extra file.
Attached file EXTRA file attachment - (deleted) —
Attached file decoded minidump - (deleted) —
roc/ehsan, any ideas here? See some changes from your side on m-c recently, but not sure if they are related. Can you please help investigate this crasher ?
Flags: needinfo?(roc)
Flags: needinfo?(ehsan)
Whiteboard: [CR 817667]
Whiteboard: [CR 817667] → [caf priority: p1][CR 817667]
Whiteboard: [caf priority: p1][CR 817667] → [b2g-crash][caf-crash 530][caf priority: p1][CR 817667]
Keywords: crash
Paul hacks on this stuff nowadays.
Flags: needinfo?(ehsan) → needinfo?(padenot)
blocking-b2g: 2.2? → 2.2+
Steve, could you help investigate as well?
Flags: needinfo?(slee)
this crash was produced during stability tests which involves monkey testing for several hours and there is no clear STR for this. If we are not able to identify the issue using provided logs then please feel free to provide us a debug patch with additional logging to identify the issue.
Flags: needinfo?(ntroast)
This seems like it could be related to bug 1150923. Both bugs suggest corruption of the graph, so that a destroyed stream is still present as the source or destination of a MediaInputPort. But I don't remember seeing any bugs like that for a long time... The MediaStream destructor logging patch I just attached to bug 1150923 might help here too. We should also add assertions that the mInputs and mConsumers arrays are empty when we destroy a MediaStream, to ensure that it's fully disconnected. Added here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4725d387fbbe https://hg.mozilla.org/try/rev/f15873e99ce5
Flags: needinfo?(roc)
(In reply to cafbot (PoC: ggrisco) from comment #17) > Created attachment 8589618 [details] > decoded minidump - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.123 From the log, I found that on frame 7 of Thread 20, it calls NS_ABORT_OOM. It looks like SetLength on [1] requests too many memory. roc, paul, Do you have any thoughts about this? [1] http://git.mozilla.org/?p=integration/gecko-dev.git;a=blob;f=dom/media/MediaStreamGraph.cpp;h=4a50ea3185be2f8f01ea44d2e52444e119c2e11a;hb=643f37b68305d90ecfa148f81db63891c30e4620#l373
Flags: needinfo?(slee)
Flags: needinfo?(roc)
Flags: needinfo?(padenot)
(In reply to StevenLee[:slee] from comment #18) > From the log, I found that on frame 7 of Thread 20, it calls NS_ABORT_OOM. > It looks like SetLength on [1] requests too many memory. > > roc, paul, > Do you have any thoughts about this? > > [1] > http://git.mozilla.org/?p=integration/gecko-dev.git;a=blob;f=dom/media/ > MediaStreamGraph.cpp;h=4a50ea3185be2f8f01ea44d2e52444e119c2e11a; > hb=643f37b68305d90ecfa148f81db63891c30e4620#l373 In this log, there are two MSGs running, one driven by a ThreadedDriver (system clock or offline, unfortunately I can't tell which), and another driven by the audio callback. That's unusual, but possible if there's an OfflineAudioContext (but should not happen for any other reason).
Flags: needinfo?(roc)
Inder, could you help to provide more information? more log or reproduce steps will be helpful. thanks.
Flags: needinfo?(ikumar)
Looking at the provided logs I see that Ringtones app may be the culprit. I am starting to see Ringtones as a recurring theme for these MSG issues. I do have additional logs for this issue, but please be specific as they are very large.
Flags: needinfo?(ikumar)
Flags: needinfo?(padenot)
Whiteboard: [b2g-crash][caf-crash 530][caf priority: p1][CR 817667] → [b2g-crash][caf-crash 530][caf priority: p3][CR 817667]
Not seen in past few builds, so closing this for now.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: