From 523e9603d3fba507be9fe9a70a24edfd0a41792b Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Tue, 28 Apr 2020 10:08:17 +0200 Subject: [PATCH] radv: enable FMASK for color attachments only The reason behind this is that FMASK requires CMASK and also that FMASK for non color attachments looks unnecessary. It's currently much easier to add this simple check because the driver tries to always enable DCC first and if we enable FMASK only if CMASK, we might loose some FMASK compressions. This helps fixing some new robustness2 tests which fails because only FMASK is enabled. Signed-off-by: Samuel Pitoiset Reviewed-by: Bas Nieuwenhuizen Part-of: --- src/amd/vulkan/radv_image.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c index f6b0ba0..7bfd340 100644 --- a/src/amd/vulkan/radv_image.c +++ b/src/amd/vulkan/radv_image.c @@ -1331,7 +1331,8 @@ radv_image_can_enable_cmask(struct radv_image *image) static inline bool radv_image_can_enable_fmask(struct radv_image *image) { - return image->info.samples > 1 && vk_format_is_color(image->vk_format); + return image->info.samples > 1 && + image->usage & VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT; } static inline bool -- 2.7.4