Crash in mozilla::layers::SyncObjectD3D11::Init

RESOLVED FIXED in Firefox 53



Graphics: Layers
11 months ago
10 months ago


(Reporter: mccr8, Assigned: kechen)



Windows 8
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox52 unaffected, firefox53 fixed, firefox54 fixed)


(Whiteboard: gfx-noted, crash signature)


(1 attachment)



11 months ago
This bug was filed from the Socorro interface and is 
report bp-7e461dfa-d513-46f4-9e16-1c8bc2170210.

Bug 1322741 landed a patch for this issue, but I'm still seeing crashes with this signature on recent builds of Aurora and Nightly.

Comment 1

11 months ago
Can you please take a look Vincent? Thanks. Feel free to dupe this to the other bug and reopen that one if you want. I don't know if this is a separate issue or what.
status-firefox52: --- → unaffected
status-firefox53: --- → affected
status-firefox54: --- → affected
Flags: needinfo?(vliu)

Comment 2

11 months ago
There was a gfx critical note listed in the following, I believe we failed to create D3D11 texture and caused crash.

[C0][GFX1-]: Failed to OpenSharedResource for SyncObjectD3D11: 0x80070057 (t=14.7052) 

The 0x80070057(E_INVALIDARG) means one or more arguments are not valid'.
22 crashes with this signature in Aurora 20170210004018.

Comment 4

11 months ago
It seems this issue is unlikely the root cause on Bug 1322741, which was SyncObjectD3D11 missing the ID3D11Device. I saw some discussions on Bug 1160157 Comment 110 might similar to this one. In that discussion, there was link got the same error return number. Please see [1] I posted from Bug 1160157 Comment 110.


[C8][GFX1]: Failed to D3D11 OpenSharedResource for frame finalization: 0x80070057 (t=863.299)
Flags: needinfo?(vliu)
See Also: → bug 1160157


11 months ago
Priority: -- → P3
Whiteboard: gfx-noted

Comment 5

11 months ago
Created attachment 8837077 [details] [diff] [review]
Handle the OpenSharedResource failure when device rest happened.

Before Bug 1319557, we had an early return to handling failure of OpenSharedResource and preventing accessing null mD3D11Texture when device reset happened[1]. I think we should add it back.

David, could you help me to review it ? Thank you.

Attachment #8837077 - Flags: review?(dvander)
Assignee: nobody → kechen


11 months ago
Attachment #8837077 - Flags: review?(dvander) → review+

Comment 7

11 months ago
I will uplift this to Firefox 53 after the change is landed into Nightly.

Comment 8

11 months ago
Pushed by
Handle the OpenSharedResource failure in SyncObjectD3D11::Init when device rest happened. r=dvander
Keywords: checkin-needed

Comment 9

11 months ago
Last Resolved: 11 months ago
status-firefox54: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Please request Aurora approval on this when you get a chance.
Flags: needinfo?(kechen)

Comment 11

11 months ago
Comment on attachment 8837077 [details] [diff] [review]
Handle the OpenSharedResource failure when device rest happened.

Approval Request Comment
[Feature/Bug causing the regression]:
 Bug 1319557
[User impact if declined]:
 The crash will still happens in aurora
[Is this code covered by automated tests?]:
[Has the fix been verified in Nightly?]:
[Needs manual test from QE? If yes, steps to reproduce]: 
[List of other uplifts needed for the feature/fix]:
[Is the change risky?]:
 Low risk.
[Why is the change risky/not risky?]:
 A simple code flow change and the return condition exists in the versions before 53.
[String changes made/needed]:
Flags: needinfo?(kechen)
Attachment #8837077 - Flags: approval-mozilla-aurora?

Comment 12

11 months ago
Comment on attachment 8837077 [details] [diff] [review]
Handle the OpenSharedResource failure when device rest happened.

Fix a crash. Aurora53+.
Attachment #8837077 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 13

11 months ago
status-firefox53: affected → fixed
Setting qe-verify- based on Kevin's assessment on manual testing needs (see Comment 11).
Flags: qe-verify-


10 months ago
See Also: → bug 1345735
You need to log in before you can comment on or make changes to this bug.