Commit d5358ef3 authored by Miguel Casas's avatar Miguel Casas Committed by Commit Bot

Disable VP9.2 hw decoding on AMD CrOS

AMD ChromeOS has some issues with VP9 Profile 2 hw decoding. While
these are addressed, this CL adds a new
  disable_accelerated_vp9_profile2_decode
entry to gpu_workaround_list and uses it to prevent such decoding
from being tried.

This CL supersedes crrev.com/c/2518785.

Bug: 1147466, b/169591197
Change-Id: Id4bb69d6bf6ad02f4523247147c14f8d5e12938e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528582Reviewed-by: default avatarZhenyao Mo <zmo@chromium.org>
Reviewed-by: default avatarAndres Calderon Jaramillo <andrescj@chromium.org>
Auto-Submit: Miguel Casas <mcasas@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825913}
parent e7f0fe1d
...@@ -3658,6 +3658,19 @@ ...@@ -3658,6 +3658,19 @@
"features": [ "features": [
"max_msaa_sample_count_4" "max_msaa_sample_count_4"
] ]
},
{
"id": 361,
"description": "VP9 Profile 2 decoding doesn't work on ChromeOS AMDs",
"cr_bugs": [1147466],
"os": {
"type": "chromeos"
},
"vendor_id": "0x1002",
"driver_vendor": "Mesa",
"features": [
"disable_accelerated_vp9_profile2_decode"
]
} }
] ]
} }
...@@ -16,6 +16,7 @@ disable_accelerated_av1_decode ...@@ -16,6 +16,7 @@ disable_accelerated_av1_decode
disable_accelerated_vp8_decode disable_accelerated_vp8_decode
disable_accelerated_vp8_encode disable_accelerated_vp8_encode
disable_accelerated_vp9_decode disable_accelerated_vp9_decode
disable_accelerated_vp9_profile2_decode
disable_async_readpixels disable_async_readpixels
disable_av_sample_buffer_display_layer disable_av_sample_buffer_display_layer
disable_blend_equation_advanced disable_blend_equation_advanced
......
...@@ -1314,6 +1314,10 @@ VaapiWrapper::GetSupportedDecodeProfiles( ...@@ -1314,6 +1314,10 @@ VaapiWrapper::GetSupportedDecodeProfiles(
workarounds.disable_accelerated_vp8_decode) { workarounds.disable_accelerated_vp8_decode) {
continue; continue;
} }
if (media_profile == VP9PROFILE_PROFILE2 &&
workarounds.disable_accelerated_vp9_profile2_decode) {
continue;
}
const VASupportedProfiles::ProfileInfo* profile_info = const VASupportedProfiles::ProfileInfo* profile_info =
VASupportedProfiles::Get().IsProfileSupported(kDecode, va_profile); VASupportedProfiles::Get().IsProfileSupported(kDecode, va_profile);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment