From e50af52e3d9ad7a0a2bdeef5cfea19c973b25aa4 Mon Sep 17 00:00:00 2001 From: Nanley Chery Date: Thu, 20 Jul 2023 09:40:04 -0400 Subject: [PATCH] anv: Don't support ASTC images with modifiers Before this change, anv_get_image_format_features2 reported support for ASTC formats with any modifier (even those not supported by anv). But, we didn't intend to support that compressed image format with modifiers. With this change, the format feature function reports no support for modifiers on ASTC-formatted images. This prevents the next patch from causing assertion failures due to unsupported modifiers. Fixes: 355f3188438 ("anv: Allow transfer-only linear ASTC images") Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/vulkan/anv_formats.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/intel/vulkan/anv_formats.c b/src/intel/vulkan/anv_formats.c index 430a719..4ad6c09 100644 --- a/src/intel/vulkan/anv_formats.c +++ b/src/intel/vulkan/anv_formats.c @@ -577,11 +577,16 @@ anv_get_image_format_features2(const struct anv_physical_device *physical_device enum isl_format base_isl_format = base_plane_format.isl_format; if (isl_format_supports_sampling(devinfo, plane_format.isl_format)) { - /* ASTC textures must be in Y-tiled memory, and we reject compressed - * formats with modifiers. We do however interpret ASTC textures with - * uncompressed formats during data transfers. + + /* Unlike other surface formats, our sampler requires that the ASTC + * format only be used on surfaces in non-linearly-tiled memory. + * Thankfully, we can make an exception for linearly-tiled images that + * are only used for transfers. blorp_copy will reinterpret any + * compressed format to an uncompressed one. + * + * We handle modifier tilings further down in this function. */ - if (vk_tiling != VK_IMAGE_TILING_OPTIMAL && + if (vk_tiling == VK_IMAGE_TILING_LINEAR && isl_format_get_layout(plane_format.isl_format)->txc == ISL_TXC_ASTC) return VK_FORMAT_FEATURE_2_TRANSFER_SRC_BIT | VK_FORMAT_FEATURE_2_TRANSFER_DST_BIT; -- 2.7.4