Closed Bug 1416614 Opened 2 years ago Closed 2 years ago

Crash in mozilla::layers::HostIPCAllocator::SendPendingAsyncMessages

Categories

(Core :: Graphics: WebRender, defect, P1, critical)

x86_64
All
defect

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox-esr52 --- unaffected
firefox57 --- wontfix
firefox58 --- fixed
firefox59 --- fixed

People

(Reporter: darkspirit, Assigned: sotaro)

References

(Blocks 1 open bug)

Details

(Keywords: crash, nightly-community, Whiteboard: [wr-mvp])

Crash Data

Attachments

(1 file)

Nearly all crashes on Windows from the last 2 weeks and some from Mac have WebRender enabled.
> 3 	xul.dll 	mozilla::wr::NotifyDidRender 	gfx/webrender_bindings/RenderThread.cpp:201
They seem to affect WebRender in some way.

Windows example: bp-aec114af-d64c-473e-affb-ae2b20171112 (build 2017-11-09_220104)
> 0 	xul.dll 	mozilla::layers::HostIPCAllocator::SendPendingAsyncMessages() 	gfx/layers/ipc/ISurfaceAllocator.cpp:41
> 1 	xul.dll 	mozilla::layers::ImageBridgeParent::NotifyImageComposites(nsTArray<mozilla::layers::ImageCompositeNotificationInfo>&) 	gfx/layers/ipc/ImageBridgeParent.cpp:392
> 2 	xul.dll 	mozilla::layers::CompositorBridgeParent::NotifyDidCompositeToPipeline(mozilla::wr::PipelineId const&, mozilla::wr::Epoch const&, mozilla::TimeStamp&, mozilla::TimeStamp&) 	gfx/layers/ipc/CompositorBridgeParent.cpp:1983
> 3 	xul.dll 	mozilla::wr::NotifyDidRender 	gfx/webrender_bindings/RenderThread.cpp:201
> 4 	xul.dll 	RunnableFunction<void (*)(mozilla::layers::CompositorBridgeParentBase*, mozilla::wr::WrRenderedEpochs*, mozilla::TimeStamp, mozilla::TimeStamp), mozilla::Tuple<mozilla::layers::CompositorBridgeParentBase*, mozilla::wr::WrRenderedEpochs*, mozilla::TimeStamp, mozilla::TimeStamp> >::Run() 	ipc/chromium/src/base/task.h:350
> 5 	xul.dll 	MessageLoop::DoWork() 	ipc/chromium/src/base/message_loop.cc:535
> 6 	xul.dll 	base::MessagePumpForUI::DoRunLoop() 	ipc/chromium/src/base/message_pump_win.cc:210
> 7 	xul.dll 	base::MessagePumpWin::Run(base::MessagePump::Delegate*) 	ipc/chromium/src/base/message_pump_win.h:80
> 8 	xul.dll 	MessageLoop::RunHandler() 	ipc/chromium/src/base/message_loop.cc:319
> 9 	xul.dll 	MessageLoop::Run() 	ipc/chromium/src/base/message_loop.cc:299
> 10 	xul.dll 	base::Thread::ThreadMain() 	ipc/chromium/src/base/thread.cc:181
> 11 	xul.dll 	`anonymous namespace'::ThreadFunc 	ipc/chromium/src/base/platform_thread_win.cc:28
> 12 	kernel32.dll 	BaseThreadInitThunk 	
> 13 	ntdll.dll 	RtlUserThreadStart

Two weeks old Mac example: bp-85130e50-9f46-4763-ae65-1157f0171031 (build 2017-10-30_103605)
> 3 	XUL 	mozilla::wr::NotifyDidRender(mozilla::layers::CompositorBridgeParentBase*, mozilla::wr::WrRenderedEpochs*, mozilla::TimeStamp, mozilla::TimeStamp) 	gfx/webrender_bindings/RenderThread.cpp:200
Blocks: wr-stability
Severity: normal → critical
From the crash reports, the crash seemed to happen with/without WebRender.
Assignee: nobody → sotaro.ikeda.g
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [wr-mvp] [triage] → [wr-mvp]
Attachment #8927698 - Flags: review?(nical.bugzilla)
Attachment #8927698 - Flags: review?(nical.bugzilla) → review+
Pushed by sikeda@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/db2a8ddbdb48
Add checks to ImageBridgeParent::NotifyImageComposites r=nical
https://hg.mozilla.org/mozilla-central/rev/db2a8ddbdb48
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Please request Beta approval on this.
Flags: needinfo?(sotaro.ikeda.g)
Comment on attachment 8927698 [details] [diff] [review]
patch - Add checks to ImageBridgeParent::NotifyImageComposites

Approval Request Comment
[Feature/Bug causing the regression]: Bug 1143575
[User impact if declined]: Could cause crash in some cases with video playback.
[Is this code covered by automated tests?]: Yes.
[Has the fix been verified in Nightly?]: Yes.
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No.
[Why is the change risky/not risky?]: It just added simple checks.
[String changes made/needed]: No
Flags: needinfo?(sotaro.ikeda.g)
Attachment #8927698 - Flags: approval-mozilla-beta?
Comment on attachment 8927698 [details] [diff] [review]
patch - Add checks to ImageBridgeParent::NotifyImageComposites

Fix a crash. Beta58+.
Attachment #8927698 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Depends on: 1462885
You need to log in before you can comment on or make changes to this bug.