Closed Bug 1374148 Opened 2 years ago Closed 2 years ago

Crash in nvwgf2um.dll | CDecodeContext::BeginFrame

Categories

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

55 Branch
x86
Windows 10
defect

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox-esr52 --- fixed
firefox54 + wontfix
firefox55 --- fixed
firefox56 --- fixed

People

(Reporter: gchang, Assigned: gerald)

Details

(Keywords: crash)

Crash Data

Attachments

(3 files)

This bug was filed from the Socorro interface and is 
report bp-55e3cf62-192c-41d6-9dad-5d4340170619.
=============================================================
Frame 	Module 	Signature 	Source
Ø 0 	nvwgf2um.dll 	nvwgf2um.dll@0xb53a7b 	
Ø 1 	nvwgf2um.dll 	nvwgf2um.dll@0xb5544a 	
Ø 2 	nvwgf2um.dll 	nvwgf2um.dll@0xb87118 	
Ø 3 	nvwgf2um.dll 	nvwgf2um.dll@0xba8512 	
Ø 4 	nvwgf2um.dll 	nvwgf2um.dll@0xbb7898 	
Ø 5 	nvwgf2um.dll 	nvwgf2um.dll@0xb578e7 	
Ø 6 	nvwgf2um.dll 	nvwgf2um.dll@0xb5d623 	
Ø 7 	nvwgf2um.dll 	nvwgf2um.dll@0xb46ee3 	
Ø 8 	nvwgf2um.dll 	nvwgf2um.dll@0x1b6a3d 	
9 	d3d11.dll 	CDecodeContext::BeginFrame(ID3D11VideoDecoderOutputView*, void const*, unsigned int) 	
10 	d3d11.dll 	CContext::DecoderBeginFrame(ID3D11VideoDecoder*, ID3D11VideoDecoderOutputView*, unsigned int, void const*) 	
Ø 11 	msmpeg2vdec.dll 	msmpeg2vdec.dll@0x77cab 	
Ø 12 	msmpeg2vdec.dll 	msmpeg2vdec.dll@0x7461f 	
Ø 13 	msmpeg2vdec.dll 	msmpeg2vdec.dll@0x75e50 	
Ø 14 	msmpeg2vdec.dll 	msmpeg2vdec.dll@0x747e6 	
Ø 15 	msmpeg2vdec.dll 	msmpeg2vdec.dll@0x78ebe 	
Ø 16 	msmpeg2vdec.dll 	msmpeg2vdec.dll@0x78c29 	
17 	kernel32.dll 	BaseThreadInitThunk 	
18 	ntdll.dll 	__RtlUserThreadStart 	
19 	ntdll.dll 	_RtlUserThreadStart

This is #7 topcrash of Firefox 54. 

Hi Blake,
Can you help shed some light here?
Flags: needinfo?(bwu)
Many of the comments mention crashing while using Facebook:

Crashed again while veiwing facebook. Why has firefox become so unstable? 
I always have issues with Facebook. Firefox usually freezes and I have to use Task Manager to kill it. 
seems that facebook as some memory leaks that causes crashes easily 
just looking on the facebook home wall and firefox crashed. no videos were running. 
One Tab open, Facebook. Crashed while scrolling.
Gerald, 
Can you check if we could blacklist it?
Flags: needinfo?(bwu) → needinfo?(gsquelart)
Priority: -- → P1
There were already a few versions for which D3D11 was blacklisted, and these don't appear in crash reports (meaning the blacklisting worked for them).
So I will add these new versions found from all the reports:
22.21.13.8253, 22.21.13.8233, 22.21.13.8205, 22.21.13.8189, 22.21.13.8178, 22.21.13.8165, 21.21.13.7892, 21.21.13.7878, 21.21.13.7866, 21.21.13.7849, 21.21.13.7654, 21.21.13.7653, 21.21.13.7633, 21.21.13.7619, 21.21.13.7563, 21.21.13.7306, 21.21.13.7290, 21.21.13.7270, 21.21.13.7254, 21.21.13.6939, 21.21.13.6926, 21.21.13.6909, 21.21.13.4201, 21.21.13.4200, 10.18.13.6881, 10.18.13.6839, 10.18.13.6472, 10.18.13.6143, 10.18.13.5946, 10.18.13.5923, 10.18.13.5921, 10.18.13.5445, 10.18.13.5362, 9.18.13.4788, 9.18.13.4752, 9.18.13.4725, 9.18.13.4709, 9.18.13.4192, 9.18.13.4144, 9.18.13.4052, 9.18.13.3788, 9.18.13.3523, 9.18.13.3235, 9.18.13.2723, 9.18.13.2702, 9.18.13.1422, 9.18.13.1407, 9.18.13.1106, 9.18.13.546
Assignee: nobody → gsquelart
Flags: needinfo?(gsquelart)
Comment on attachment 8879407 [details]
Bug 1374148 - Block D3D11 on more crashy nvwgf2um.dll versions -

https://reviewboard.mozilla.org/r/150732/#review155452
Attachment #8879407 - Flags: review?(bwu) → review+
Pushed by gsquelart@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/58e1d5cd63c0
Block D3D11 on more crashy nvwgf2um.dll versions - r=bwu
Comment on attachment 8879407 [details]
Bug 1374148 - Block D3D11 on more crashy nvwgf2um.dll versions -

Approval Request Comment
[Feature/Bug causing the regression]: D3D11 decoding with some Nvidia drivers
[User impact if declined]: Crashes during media playback (currently 8th top crasher on 54)
[Is this code covered by automated tests?]: Not possible as it's a pref change covering more driver versions, which we don't have on our try machines
[Has the fix been verified in Nightly?]: Not yet
[Needs manual test from QE? If yes, steps to reproduce]: Probably not possible, as it requires specific cards&drivers, and is still rare/intermittent enough and therefore hard to reproduce
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Only adding more versions of an already-blacklisted driver, in the worst case we are making video playback more CPU-intensive for a class of users who wouldn't have experienced crashes -- Note that we are considering removing D3D blacklists for e10s, so most users should be unblocked soon (see bug 1372070)
[String changes made/needed]: None
Attachment #8879407 - Flags: approval-mozilla-beta?
https://hg.mozilla.org/mozilla-central/rev/58e1d5cd63c0
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Comment on attachment 8879407 [details]
Bug 1374148 - Block D3D11 on more crashy nvwgf2um.dll versions -

d3d11 blocklist additions, beta55+
Attachment #8879407 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Should we nominate this for mozilla-release to get it on the radar for possible Fx54 dot release consideration? Also, how about ESR52?
Flags: needinfo?(gsquelart)
(Trivial rebase from the original patch, carrying r+ from comment 6)

Approval Request Comment
[Feature/Bug causing the regression]: Hardware video decoding
[User impact if declined]: ~2000 crashes per week on release
[Is this code covered by automated tests?]: No (it's only a block-list update)
[Has the fix been verified in Nightly?]: No (it's only a block-list update, and there are too few crashes on Nightly&beta to see a change)
[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?]: It's only a block-list update
[String changes made/needed]: None
Flags: needinfo?(gsquelart)
Attachment #8882218 - Flags: review+
Attachment #8882218 - Flags: approval-mozilla-release?
(Trivial rebase from the original patch, carrying r+ from comment 6)

[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration: Just an updated block-list to reduce crashes
User impact if declined: ~45 crashes per week
Fix Landed on Version: 56, 55b
Risk to taking this patch (and alternatives if risky): No risks (it's only a block-list update)
String or UUID changes made by this patch: None

See https://wiki.mozilla.org/Release_Management/ESR_Landing_Process for more info.
Attachment #8882222 - Flags: review+
Attachment #8882222 - Flags: approval-mozilla-esr52?
Comment on attachment 8882222 [details] [diff] [review]
1374148-esr52.patch

Block D3D11 to avoid crashes. Let's uplift to ESR52.3.
Attachment #8882222 - Flags: approval-mozilla-esr52? → approval-mozilla-esr52+
Comment on attachment 8882218 [details] [diff] [review]
1374148-release.patch

We are only one week from 55 RC build. Release54-.
Attachment #8882218 - Flags: approval-mozilla-release? → approval-mozilla-release-
You need to log in before you can comment on or make changes to this bug.