Commit 523e9603 authored by Samuel Pitoiset's avatar Samuel Pitoiset

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's avatarSamuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen's avatarBas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Part-of: <mesa/mesa!4783>
parent 81ac741f
Pipeline #139061 waiting for manual action with stages
......@@ -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
......
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