1274127 - crash in igd10umd32.dll@0x14b17c; igd10iumd32.dll@0x12d7bc, @0x12e1cc, @0x178a5f

Status: RESOLVED FIXED

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

Description

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

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:
[@ igd10iumd32.dll@0x12d7bc ] -> igd10iumd32.dll 10.18.15.4279
[@ igd10iumd32.dll@0x12e1cc ] -> igd10iumd32.dll 10.18.15.4281
[@ igd10umd32.dll@0x14b17c ],
[@ @0x0 | igd10umd32.dll@0x178a5f ] -> igd10umd32.dll 9.17.10.4229

Also one report in [@ TppRaiseInvalidParameter ] points at igd10umd64.dll 9.17.10.4229, I'll take that one here too.

Comment 1

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

Attachment: MozReview Request: Bug 1274127 - Disable D3D11 for some igd10* DLLs - r?cpearce

Review commit: https://reviewboard.mozilla.org/r/53844/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/53844/

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8754210 [details]
MozReview Request: Bug 1274127 - Disable D3D11 for some igd10* DLLs - r?cpearce

https://reviewboard.mozilla.org/r/53844/#review50570

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cfda20b697dd

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/cfda20b697dd

Comment 5

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

Comment on attachment 8754210 [details]
MozReview Request: Bug 1274127 - Disable D3D11 for some igd10* DLLs - r?cpearce

Approval Request Comment
[Feature/regressing bug #]: D3D11 crashes in 47+
[User impact if declined]: >600 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, and bug 1274115 first.

Comment 6

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8754210 [details]
MozReview Request: Bug 1274127 - Disable D3D11 for some igd10* DLLs - r?cpearce

Let's uplift to Aurora48 and stabilize before uplifting to Beta47.

Comment 7

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d514834efff9

Comment 8

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8754210 [details]
MozReview Request: Bug 1274127 - Disable D3D11 for some igd10* DLLs - r?cpearce

Disable D3D11 for some device drivers, this code stabilized on Aurora over the weekend, Beta47+

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-beta/rev/9b0821b8f85b