Module: Mesa Branch: main Commit: e50af52e3d9ad7a0a2bdeef5cfea19c973b25aa4 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=e50af52e3d9ad7a0a2bdeef5cfea19c973b25aa4
Author: Nanley Chery <[email protected]> Date: Thu Jul 20 09:40:04 2023 -0400 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 <[email protected]> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24120> --- 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 430a7196d98..4ad6c0963a5 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;
