Closed Bug 1274132 Opened 4 years ago Closed 4 years ago

crashes related to Intel devices

Categories

(Core :: Audio/Video: Playback, defect, P1, critical)

47 Branch
All
Windows
defect

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox47 --- fixed
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: gerald, Assigned: gerald)

References

Details

(Keywords: crash, regression)

Crash Data

Attachments

(1 file)

Spawned from bug 1269204, where crash signatures were added after a fix had already landed, which didn't account for DLLs in these new signatures:
[@ TppRaiseInvalidParameter ] ->
igd10iumd32.dll 10.18.10.3496,
igd10iumd32.dll 10.18.10.4358,
igd10iumd32.dll 10.18.10.4252
igd10iumd32.dll 10.18.14.4112 (ff46.0.1),
igd10iumd32.dll 10.18.10.4276 (ff46.0.1)
igdumdim32.dll 10.18.14.4170 (ff46.0b99),
igdumdim32.dll 10.18.10.4276 igd10iumd32.dll 10.18.10.4276 (ff45.0.2)
(igd10umd64.dll 9.17.10.4229 already moved to bug 1274127)

[@ CVideoProcessor::CLS::FinalConstruct ] ->
igd10iumd32.dll 20.19.15.4409,
igd10iumd32.dll 20.19.15.4380,
igd10iumd32.dll 10.18.15.4256,
igd10iumd32.dll 20.19.15.4360,
igd10iumd32.dll 20.19.15.4390,
igd10iumd32.dll 20.19.15.4331

[@ igd11dxva32.dll@0x3144f ] ->
igd11dxva32.dll 20.19.15.4444, igd10iumd32.dll 20.19.15.4444
igd11dxva32.dll 20.19.15.4424, igd10iumd32.dll 20.19.15.4424

Some of these look like quite recent versions of the igd* DLLs, so I'd like to be a bit more considerate before blindly blacklisting all of them.
Comments&insights welcome.
Anthony, you suggested disabling D3D11 for Toshiba DLLs in bug 1268905 comment 3.
Chris, you followed suit for Intel DLLs in bug 1269204 comment 4.

So far I've blacklisted some Intel DLL versions 9.x, I'll add a few more 9.x and 10.x in bug 1274127.

Now some of these reports point at versions 20.x.
Do you think I should just blacklist these as well, at least until we know more? (from Intel or our own investigations.)
Flags: needinfo?(cpearce)
Flags: needinfo?(ajones)
Depends on: 1274127
The review in comment 2 assumes we all agree to blacklist these DLL versions (see comment 1).

Note that it doesn't include igd11dxva32.dll because for those reports, igd10iumd32.dll was there as well, so no need to duplicate.

Also I didn't include igdumdim32.dll because it only seems to affect FF<46.
Let's blacklist.
Flags: needinfo?(cpearce)
Flags: needinfo?(ajones)
Comment on attachment 8754221 [details]
MozReview Request: Bug 1274132 - Disable D3D11 for some more recent igd10* DLLs - r?cpearce

https://reviewboard.mozilla.org/r/53848/#review50572
Attachment #8754221 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/mozilla-central/rev/830a12b195de
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Comment on attachment 8754221 [details]
MozReview Request: Bug 1274132 - Disable D3D11 for some more recent igd10* DLLs - r?cpearce

Approval Request Comment
[Feature/regressing bug #]: D3D11 crashes in 47+
[User impact if declined]: ~80 crashes per week in beta 47.
[Describe test coverage new/current, TreeHerder]: Media tests in try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3fa319dea6b6ae756fde468671e5590d77affa93
[Risks and why]: Only a pref change that adds some DLLs; Tiny risk in disabling too many DLLs, but we're falling back to proven D3D9, so it's not so bad.
[String/UUID change made/needed]: None.

Requires uplift of bug 1257028, bug 1273691, bug 1274115, and bug 1274127 first.
Attachment #8754221 - Flags: approval-mozilla-beta?
Attachment #8754221 - Flags: approval-mozilla-aurora?
Blocks: 1273504
Comment on attachment 8754221 [details]
MozReview Request: Bug 1274132 - Disable D3D11 for some more recent igd10* DLLs - r?cpearce

Let's uplift to Aurora48 and stabilize before uplifting to Beta47.
Attachment #8754221 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
This fix will also catch igd10iumd32.dll@0x216a88.
Crash Signature: [@ TppRaiseInvalidParameter ] [@ CVideoProcessor::CLS::FinalConstruct ] [@ igd11dxva32.dll@0x3144f ] → [@ TppRaiseInvalidParameter ] [@ CVideoProcessor::CLS::FinalConstruct ] [@ igd11dxva32.dll@0x3144f ] [@ igd10iumd32.dll@0x216a88 ]
This fix will also catch:
[@ igd10iumd32.dll@0x56154 ]
[@ igd10iumd32.dll@0x2e108 ]
[@ igd10iumd32.dll@0xae1c6 ]
[@ igd10umd32.dll@0x1ef819 ]
[@ igd10umd32.dll@0x1ef805 ]
[@ igd10iumd32.dll@0x30e47 ]
[@ igd10iumd32.dll@0x216a81 ]
Crash Signature: [@ TppRaiseInvalidParameter ] [@ CVideoProcessor::CLS::FinalConstruct ] [@ igd11dxva32.dll@0x3144f ] [@ igd10iumd32.dll@0x216a88 ] → [@ TppRaiseInvalidParameter ] [@ CVideoProcessor::CLS::FinalConstruct ] [@ igd11dxva32.dll@0x3144f ] [@ igd10iumd32.dll@0x216a88 ] [@ igd10iumd32.dll@0x56154 ] [@ igd10iumd32.dll@0x2e108 ] [@ igd10iumd32.dll@0xae1c6 ] [@ igd10umd32.dll@0x1ef819 ] …
Comment on attachment 8754221 [details]
MozReview Request: Bug 1274132 - Disable D3D11 for some more recent igd10* DLLs - r?cpearce

Disable D3D11 for some device drivers, this code stabilized on Aurora over the weekend, Beta47+
Attachment #8754221 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Depends on: 1275739
You need to log in before you can comment on or make changes to this bug.