1316222 - Disable the crash guard in the GPU process

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Matt Woodrow (:mattwoodrow)]

Attachment: disable-crash-guard-gpu

This hasn't been made to work yet (it's disabled in Nightly builds so hasn't caused problems yet) and is basically obseleted when we have the GPU process.

Let's just disable it for now.

Comment 1

[Matt Woodrow (:mattwoodrow)]

Attachment: dont-sync-dispatch-gpu-process

If the crash guard isn't enabled, then we don't need to sync dispatch to the main thread to initialize DXVA.

This fixes a bug that tryserver found, where NS_DISPATCH_SYNC spins the event loop of the current thread while it waits, and we end up trying to handle Shutdown() (on VideoDecoderParent) while we're still in the middle of Init.

Comment 2

[Jean-Yves Avenard [:jya]]

Comment on attachment 8808899 [details] [diff] [review]
dont-sync-dispatch-gpu-process

Review of attachment 8808899 [details] [diff] [review]:
-----------------------------------------------------------------

I can understand that we don't need to do anything on the main thread (as that the crashguard is the only reason we need to run on the main thread anyway).

But why not use SyncRunnable instead? this doesn't start an event loop.

Otherwise r=me with the telemetry stuff put back

::: dom/media/platforms/wmf/WMFVideoMFTManager.cpp
@@ +125,5 @@
>                         mGotValidOutputAfterNullOutput ? 3 :
>                         4;
>  
> +  LOG(nsPrintfCString("Reporting telemetry VIDEO_MFT_OUTPUT_NULL_SAMPLES=%d", telemetry).get());
> +  Telemetry::Accumulate(Telemetry::ID::VIDEO_MFT_OUTPUT_NULL_SAMPLES, telemetry);

this seems unrelated... this isn't a sync dispatch.

And I believe telemetry needs to be run on the main thread regardless.

Comment 3

[Matt Woodrow (:mattwoodrow)]

Attachment: sync-runnable

Trying again using SyncRunnable.

Unfortunately the DXVA blacklist code uses ClearOnShutdown which is main thread only, so the previous patch had issues on try.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ccc2831433b6
Use SyncRunnable instead of NS_DISPATCH_SYNC in WMFVideoMFTManager since the latter can spin the event loop. r=jya
https://hg.mozilla.org/integration/mozilla-inbound/rev/f210f6b4c5cd
Disable the crash guard in the GPU process since it won't be useful. r=dvander

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ccc2831433b6
https://hg.mozilla.org/mozilla-central/rev/f210f6b4c5cd