Closed Bug 1119854 Opened 6 years ago Closed 6 years ago

Ensure D3D11 sync texture never deadlocks

Categories

(Core :: Graphics: Layers, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: bas.schouten, Assigned: bas.schouten)

References

Details

Attachments

(1 file)

It would be nice to make 100% sure we're never deadlocking people on the D3D11 sync texture amongst our users. We can do this by setting a sufficiently long timeout and crashing when it expires.
Attached patch bug1119854Splinter Review
Attachment #8546728 - Flags: review?(jmuizelaar)
Comment on attachment 8546728 [details] [diff] [review]
bug1119854

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

::: gfx/layers/d3d11/TextureD3D11.cpp
@@ +785,5 @@
>  
>    if (mD3D11SyncedTextures.size()) {
>      RefPtr<IDXGIKeyedMutex> mutex;
>      hr = mD3D11Texture->QueryInterface((IDXGIKeyedMutex**)byRef(mutex));
> +    mutex->AcquireSync(0, 10000);

I fixed this...
Comment on attachment 8546728 [details] [diff] [review]
bug1119854

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

There's another AcquireSync in SharedSurfaceANGLE
Attachment #8546728 - Flags: review?(jmuizelaar) → review+
Bas: did you forget to change this AcquireSync(0, INFINITE) to AcquireSync(0, 10000) in the SharedSurfaceANGLE.cpp patch you landed?

https://hg.mozilla.org/integration/mozilla-inbound/diff/18bc11b32f6f/gfx/gl/SharedSurfaceANGLE.cpp#l1.15
Flags: needinfo?(bas)
(In reply to Chris Peterson [:cpeterson] from comment #4)
> Bas: did you forget to change this AcquireSync(0, INFINITE) to
> AcquireSync(0, 10000) in the SharedSurfaceANGLE.cpp patch you landed?
> 
> https://hg.mozilla.org/integration/mozilla-inbound/diff/18bc11b32f6f/gfx/gl/
> SharedSurfaceANGLE.cpp#l1.15

You're right, thank you.
Flags: needinfo?(bas)
https://hg.mozilla.org/mozilla-central/rev/18bc11b32f6f
https://hg.mozilla.org/mozilla-central/rev/ee29d84c968a
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Depends on: 1133623
You need to log in before you can comment on or make changes to this bug.