1433790 - Crash in igd10umd64.dll

Status: RESOLVED FIXED

(Core :: Graphics, defect, P1)

Description

[[:philipp]]

This bug was filed from the Socorro interface and is
report bp-af1e6a31-292a-4f7c-aa44-d7edd0180124.
=============================================================

content crashes on windows 7 64bit with this intel graphics driver in the signature started taking off in with firefox 58:
https://crash-stats.mozilla.com/signature/?product=Firefox&process_type=content&submitted_from_infobar=%21__true__&signature=igd10umd64.dll&date=%3E%3D2017-12-24#graphs
so far in the 58 roll-out it's the #3 top content crasher with 1.8% of all tab crashes.

the spike seems to be mostly restricted to a particular driver version and one generation of intel gpus, so perhaps it would be possible to do something here with the help of a remote blocklist?

Adapter driver version facet
1 	8.15.10.2418 	583 	97.65 %

Adapter device id facet
1 	0x0116 	225 	37.69 %
2 	0x0126 	175 	29.31 %
3 	0x0102 	158 	26.47 %
4 	0x0106 	33 	5.53 %
5 	0x0112 	3 	0.50 %

Comment 1

[Milan Sreckovic [:milan] (needinfo for best results)]

Bas, can we make something out of these crashes, or is the blocklisting our only option?  Why the spike?  Related to OMTP?  I can't tell if it is OMTP, but it doesn't seem to be on the paint thread?

Comment 2

[Bas Schouten (:bas.schouten)]

(In reply to Milan Sreckovic [:milan] from comment #1)
> Bas, can we make something out of these crashes, or is the blocklisting our
> only option?  Why the spike?  Related to OMTP?  I can't tell if it is OMTP,
> but it doesn't seem to be on the paint thread?

This definitely seems to be OMTP related, note that in all these crashes the main thread is actually performing D2D operations and is waiting for what seems to be getting the D2D global lock. Considering the small size of this group, blacklisting for D2D (or OMTP) on this driver version and OS version is probably the right course of action. This is unlikely to be an actual 'bug' on our side, and probably a threading issue inside that driver version. Note that this is also a very old driver.

Comment 3

[Milan Sreckovic [:milan] (needinfo for best results)]

Bas, can you put together a patch to block this in code?  I can then put one together for downloadable blocklist.

Comment 4

[Bas Schouten (:bas.schouten)]

Attachment: Bug 1433790: Disable Direct2D on drivers/devices where OMTP causes crashes.

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

Comment 5

[Milan Sreckovic [:milan] (needinfo for best results)]

Comment on attachment 8947512 [details]
Bug 1433790: Disable Direct2D on drivers/devices where OMTP causes crashes.

https://reviewboard.mozilla.org/r/217218/#review223034

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fe003f33c158
Disable Direct2D on drivers/devices where OMTP causes crashes. r=milan

Comment 7

[Bas Schouten (:bas.schouten)]

Done.

Comment 8

[Raul Gurzau (:RaulG)]

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

Comment 9

[[:philipp]]

could you request uplift to beta and perhaps release as well? thnx

Comment 10

[Bas Schouten (:bas.schouten)]

Comment on attachment 8947512 [details]
Bug 1433790: Disable Direct2D on drivers/devices where OMTP causes crashes.

Approval Request Comment
[Feature/Bug causing the regression]: OMTP
[User impact if declined]: Crashes on some drivers.
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Simply blacklisting D2D on some devices where D2D benefits were modest at best.
[String changes made/needed]: None

Comment 11

[Bas Schouten (:bas.schouten)]

Let's do release through the downloadable blacklist unless that somehow doesn't work.

Comment 12

[[:philipp]]

i think there were plans for a 58.0.2 release this week for bug 1433591 anyway - the patch here could be a nice ridealong to that.

Comment 13

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8947512 [details]
Bug 1433790: Disable Direct2D on drivers/devices where OMTP causes crashes.

Let's make sure it gets into 59b7 today at least.

Comment 14

[Bas Schouten (:bas.schouten)]

Comment on attachment 8947512 [details]
Bug 1433790: Disable Direct2D on drivers/devices where OMTP causes crashes.

We might want to also ship this on release if there's a dot release scheduled, however doing it through the downloadable blacklist or a system add-on is probably a faster way to get this to users at this point.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/60dacc12cab1

Comment 16

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

Comment on attachment 8947512 [details]
Bug 1433790: Disable Direct2D on drivers/devices where OMTP causes crashes.

Checked with Milan, this is a low risk ride-along that we plan to include in 58.0.2.

Comment 17

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/5d909f473521