1178385 - DXVA doesn't work on G41/G45

Status: RESOLVED FIXED

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

Description

[Jeff Muizelaar [:jrmuizel]]

It is unclear why.

Comment 1

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Isn't it blacklisted?

Comment 2

[Chris Peterson [:cpeterson]]

Jeff, is this still an issue? Is DXVA blacklisted on G41/G45?

Comment 3

[Jeff Muizelaar [:jrmuizel]]

So it looks like G45 is blacklisted here: https://dxr.mozilla.org/mozilla-central/source/widget/windows/GfxInfo.cpp#1100. This comes from bug 1155608. It would be nice if there was a pref to override the blacklist so that we could do testing to see what if anything is actually wrong with DXVA on the G45. 
On the G45 playing http://people.mozilla.org/~jmuizelaar/video/hobbit-23.9fps.mp4 in DXVAChecker uses 5-11% CPU. Playing it in FF gives 56% CPU

The G41 does not appear to be blacklisted, but gives false in about:support. It's not clear what's happening on the G41.

Comment 4

[Ralph Giles (:rillian)]

So what do we need to do to resolve this?

Comment 5

[Jeff Muizelaar [:jrmuizel]]

Figure out if it's practical to use DXVA on the G45.

Comment 6

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

DXVA is disabled for the same reason as https://dxr.mozilla.org/mozilla-central/source/widget/windows/GfxInfo.cpp#1004 where the video reads back OK but doesn't composite correctly.

Comment 7

[Jeff Muizelaar [:jrmuizel]]

Attachment: Add a pref to force enable hw video decode

Comment 8

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8712330 [details] [diff] [review]
Add a pref to force enable hw video decode

This patch doesn't seem to work for me.

Comment 9

[tomy_homy]

(In reply to Jeff Muizelaar [:jrmuizel] from comment #8)
> Comment on attachment 8712330 [details] [diff] [review]
> Add a pref to force enable hw video decode
> 
> This patch doesn't seem to work for me.
Hmm, maybe disable this stupid dropped frames detector first?
https://bugzilla.mozilla.org/show_bug.cgi?id=1208415#c7
https://bugzilla.mozilla.org/show_bug.cgi?id=1198202#c6
https://bugzilla.mozilla.org/attachment.cgi?id=8656218&action=diff

Comment 10

[Jeff Muizelaar [:jrmuizel]]

Attachment: Add a pref to force enable hw video decode

This version actually works.

Comment 11

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

We also have gfx.blocklist.all preference, which, when set to -1, will turn off all blocklisting.  That may not be what we need in this scenario, but I thought I'd mention it.

Comment 12

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/52ad2fa6b79b

Comment 13

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/52ad2fa6b79b

Comment 14

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/52ad2fa6b79b

Comment 15

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Can you do hardware decoding on G41/G45 hardware now or are you still failing the self test and on the test page? 

http://people.mozilla.org/~ajones/blank-video/

Comment 16

[Jeff Muizelaar [:jrmuizel]]

(In reply to Anthony Jones (:kentuckyfriedtakahe, :k17e) from comment #15)
> Can you do hardware decoding on G41/G45 hardware now or are you still
> failing the self test and on the test page? 
> 
> http://people.mozilla.org/~ajones/blank-video/

DXVA works well on the G45 hardware. I've been running Youtube for most of the day without any issues. I've noticed a large drop in CPU usage on Vimeo (Youtube uses VP9 when without DXVA so isn't that comparable).

I can check out the G41 tomorrow.

Comment 17

[Jeff Muizelaar [:jrmuizel]]

The G41 doesn't support full H264 acceleration just IDCT+MoComp. I don't know if there's any easy way for us to use that support and it's probably not be worth the effort.

Comment 18

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

IIUC it isn't an intermittent issue. Can you reproduce the issue on the original offending version? It would be great to unblacklist it. You could also ask the original filer to test the latest nightly. Do you have reason to suspect that the issue has been fixed?

Comment 19

[Jeff Muizelaar [:jrmuizel]]

I can not reproduce the issue in FF38. Reading through the original bug 1155608 though it looks like the reporter could reproduce the problem even after the blacklist landed. This suggest that the blacklisting was didn't actually solve the problem and my suggestion is that we reduce it.

Comment 20

[Jeff Muizelaar [:jrmuizel]]

Attachment: Accept the latest driver on G45 for hardware video decoding.

Comment 21

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Comment on attachment 8715048 [details] [diff] [review]
Accept the latest driver on G45 for hardware video decoding.

Review of attachment 8715048 [details] [diff] [review]:
-----------------------------------------------------------------

I'm OK with this patch as is but I'd also accept a patch to remove this blacklist rule. It was blacklisted for a case that is now covered by our self test code.

Comment 22

[Pulsebot]

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

Comment 23

[Carsten Book [:Tomcat]]

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