Randall Barker [:rbarker] Reporter
	Description • 10 years ago

Running DMD on MacOS X will crash on an assert in when MediaStreamGraphImpl::CollectReports calls CurrentDriver()->WakeUp();

	Randall Barker [:rbarker] Reporter
	Comment 1 • 10 years ago

I have been unable to reproduce. I did not save the stack trace when I first encountered this and now that I'm trying to reproduce it, I am unable to.

	Eric Rahm [:erahm] Assignee
	Comment 2 • 10 years ago

Randall, do you have any sort of STR for this? Maybe a specific page or test?

	Randall Barker [:rbarker] Reporter
	Comment 3 • 10 years ago

(In reply to Eric Rahm [:erahm] from comment #2)
> Randall, do you have any sort of STR for this? Maybe a specific page or test?

I was debugging a memory leak while streaming a tab. I didn't save the stack trace and when I later tried to reproduce for the bug, I was unable to. I may have been the victim of a timing related bug that is difficult to reproduce? The media pipeline was busy encoding a tab while I was attempting to save a report so maybe that increased the chance of hitting the mGraphImpl->GetMonitor().AssertCurrentThreadOwns(); ?

	Randall Barker [:rbarker] Reporter
	Comment 4 • 10 years ago

Realized I forgot to uncomment the line causing problems yesterday when trying to reproduce. After uncommenting, reproduced on first try.

	Eric Rahm [:erahm] Assignee
	Comment 6 • 10 years ago

This appears to be some sort of fallout from bug 848954, previously we would grab a lock before calling |WakeUp|.

	Eric Rahm [:erahm] Assignee
	Comment 8 • 10 years ago

Randall, would you mind testing this patch to see if it fixes your issue? I've confirmed it builds, runs, and an about:memory report worked w/o crashing.

	Randall Barker [:rbarker] Reporter
	Comment 9 • 10 years ago

(In reply to Eric Rahm [:erahm] from comment #8)
> Randall, would you mind testing this patch to see if it fixes your issue?
> I've confirmed it builds, runs, and an about:memory report worked w/o
> crashing.

It appears to fix the crash I was seeing. I ran without the patch and crashed as before. After applying the patch was able to run multiple times without issue.

	Eric Rahm [:erahm] Assignee
	Comment 10 • 10 years ago

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/c37936058203

	Carsten Book [:Tomcat]
	Comment 12 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/c37936058203

	Kyle Huey (Exited; not receiving bugmail, old account, do not use)
	Comment 13 • 10 years ago

Comment on attachment 8495632 [details] [diff] [review]
Acquire monitor before calling WakeUp

This is blocking CAF stability testing. a=me for uplift.

	Randell Jesup [:jesup] (needinfo me)
	Comment 14 • 10 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/32f99eac065c