diff options
author | Samuel Pitoiset <samuel.pitoiset@gmail.com> | 2021-11-08 08:50:14 +0100 |
---|---|---|
committer | Eric Engestrom <eric@engestrom.ch> | 2021-11-10 21:58:06 +0000 |
commit | b72699702972eca61171e205c19df0807ccb59fc (patch) | |
tree | a375425306bf13a2459018df56cd87fe41213f4e | |
parent | b5ebe0ec36b45105ab97523bb33285e3109a1f32 (diff) |
radv: do not expose buffer features for depth/stencil formats
The Vulkan spec got clarified recently and it's invalid (hw can support
it though). Fixes new CTS dEQP-VK.api.buffer.invalid_buffer_features.*.
Cc: 21.3 mesa-stable
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13701>
(cherry picked from commit ca7c748f45a9336fd22ca2c88b61ee4142821235)
-rw-r--r-- | .pick_status.json | 2 | ||||
-rw-r--r-- | src/amd/vulkan/radv_formats.c | 5 |
2 files changed, 6 insertions, 1 deletions
diff --git a/.pick_status.json b/.pick_status.json index 937670e3fe5..19e1589692a 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -382,7 +382,7 @@ "description": "radv: do not expose buffer features for depth/stencil formats", "nominated": true, "nomination_type": 0, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": null }, diff --git a/src/amd/vulkan/radv_formats.c b/src/amd/vulkan/radv_formats.c index 8d355d3a28d..6b5692d7690 100644 --- a/src/amd/vulkan/radv_formats.c +++ b/src/amd/vulkan/radv_formats.c @@ -868,6 +868,11 @@ radv_physical_device_get_format_properties(struct radv_physical_device *physical tiled |= VK_FORMAT_FEATURE_2_FRAGMENT_SHADING_RATE_ATTACHMENT_BIT_KHR; } + /* It's invalid to expose buffer features with depth/stencil formats. */ + if (vk_format_is_depth_or_stencil(format)) { + buffer = 0; + } + out_properties->linearTilingFeatures = linear; out_properties->optimalTilingFeatures = tiled; out_properties->bufferFeatures = buffer; |