radv: Fix assert that is too strict.
[mesa.git] / src / amd / vulkan / radv_image.c
index 6fea8f911d3f05df3ee354eaa48f4a040328826d..fc0fb81175e50a19a8faf78bb4de973ecde47375 100644 (file)
 #include "vk_util.h"
 #include "radv_radeon_winsys.h"
 #include "sid.h"
-#include "gfx9d.h"
 #include "util/debug.h"
 #include "util/u_atomic.h"
+#include "vulkan/util/vk_format.h"
+
+#include "gfx10_format_table.h"
+
 static unsigned
 radv_choose_tiling(struct radv_device *device,
-                  const struct radv_image_create_info *create_info)
+                  const VkImageCreateInfo *pCreateInfo,
+                  VkFormat format)
 {
-       const VkImageCreateInfo *pCreateInfo = create_info->vk_info;
-
        if (pCreateInfo->tiling == VK_IMAGE_TILING_LINEAR) {
                assert(pCreateInfo->samples <= 1);
                return RADEON_SURF_MODE_LINEAR_ALIGNED;
        }
 
-       if (!vk_format_is_compressed(pCreateInfo->format) &&
-           !vk_format_is_depth_or_stencil(pCreateInfo->format)
-           && device->physical_device->rad_info.chip_class <= VI) {
+       if (!vk_format_is_compressed(format) &&
+           !vk_format_is_depth_or_stencil(format)
+           && device->physical_device->rad_info.chip_class <= GFX8) {
                /* this causes hangs in some VK CTS tests on GFX9. */
                /* Textures with a very small height are recommended to be linear. */
                if (pCreateInfo->imageType == VK_IMAGE_TYPE_1D ||
@@ -66,14 +68,14 @@ radv_choose_tiling(struct radv_device *device,
 
 static bool
 radv_use_tc_compat_htile_for_image(struct radv_device *device,
-                                  const VkImageCreateInfo *pCreateInfo)
+                                  const VkImageCreateInfo *pCreateInfo,
+                                  VkFormat format)
 {
        /* TC-compat HTILE is only available for GFX8+. */
-       if (device->physical_device->rad_info.chip_class < VI)
+       if (device->physical_device->rad_info.chip_class < GFX8)
                return false;
 
-       if ((pCreateInfo->usage & VK_IMAGE_USAGE_STORAGE_BIT) ||
-           (pCreateInfo->flags & VK_IMAGE_CREATE_EXTENDED_USAGE_BIT))
+       if ((pCreateInfo->usage & VK_IMAGE_USAGE_STORAGE_BIT))
                return false;
 
        if (pCreateInfo->tiling == VK_IMAGE_TILING_LINEAR)
@@ -82,26 +84,37 @@ radv_use_tc_compat_htile_for_image(struct radv_device *device,
        if (pCreateInfo->mipLevels > 1)
                return false;
 
+       /* Do not enable TC-compatible HTILE if the image isn't readable by a
+        * shader because no texture fetches will happen.
+        */
+       if (!(pCreateInfo->usage & (VK_IMAGE_USAGE_SAMPLED_BIT |
+                                   VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT |
+                                   VK_IMAGE_USAGE_TRANSFER_SRC_BIT)))
+               return false;
+
        /* FIXME: for some reason TC compat with 2/4/8 samples breaks some cts
-        * tests - disable for now */
+        * tests - disable for now. On GFX10 D32_SFLOAT is affected as well.
+        */
        if (pCreateInfo->samples >= 2 &&
-           pCreateInfo->format == VK_FORMAT_D32_SFLOAT_S8_UINT)
+           (format == VK_FORMAT_D32_SFLOAT_S8_UINT ||
+            (format == VK_FORMAT_D32_SFLOAT &&
+             device->physical_device->rad_info.chip_class >= GFX10)))
                return false;
 
        /* GFX9 supports both 32-bit and 16-bit depth surfaces, while GFX8 only
         * supports 32-bit. Though, it's possible to enable TC-compat for
         * 16-bit depth surfaces if no Z planes are compressed.
         */
-       if (pCreateInfo->format != VK_FORMAT_D32_SFLOAT_S8_UINT &&
-           pCreateInfo->format != VK_FORMAT_D32_SFLOAT &&
-           pCreateInfo->format != VK_FORMAT_D16_UNORM)
+       if (format != VK_FORMAT_D32_SFLOAT_S8_UINT &&
+           format != VK_FORMAT_D32_SFLOAT &&
+           format != VK_FORMAT_D16_UNORM)
                return false;
 
        if (pCreateInfo->flags & VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT) {
-               const struct VkImageFormatListCreateInfoKHR *format_list =
-                       (const struct  VkImageFormatListCreateInfoKHR *)
+               const struct VkImageFormatListCreateInfo *format_list =
+                       (const struct  VkImageFormatListCreateInfo *)
                                vk_find_struct_const(pCreateInfo->pNext,
-                                                    IMAGE_FORMAT_LIST_CREATE_INFO_KHR);
+                                                    IMAGE_FORMAT_LIST_CREATE_INFO);
 
                /* We have to ignore the existence of the list if viewFormatCount = 0 */
                if (format_list && format_list->viewFormatCount) {
@@ -109,7 +122,10 @@ radv_use_tc_compat_htile_for_image(struct radv_device *device,
                         * one format with everything else.
                         */
                        for (unsigned i = 0; i < format_list->viewFormatCount; ++i) {
-                               if (pCreateInfo->format != format_list->pViewFormats[i])
+                               if (format_list->pViewFormats[i] == VK_FORMAT_UNDEFINED)
+                                       continue;
+
+                               if (format != format_list->pViewFormats[i])
                                        return false;
                        }
                } else {
@@ -120,43 +136,76 @@ radv_use_tc_compat_htile_for_image(struct radv_device *device,
        return true;
 }
 
+static bool
+radv_surface_has_scanout(struct radv_device *device, const struct radv_image_create_info *info)
+{
+       if (info->bo_metadata) {
+               if (device->physical_device->rad_info.chip_class >= GFX9)
+                       return info->bo_metadata->u.gfx9.scanout;
+               else
+                       return info->bo_metadata->u.legacy.scanout;
+       }
+
+       return info->scanout;
+}
+
+static bool
+radv_image_use_fast_clear_for_image(const struct radv_image *image)
+{
+       if (image->info.samples <= 1 &&
+           image->info.width * image->info.height <= 512 * 512) {
+               /* Do not enable CMASK or DCC for small surfaces where the cost
+                * of the eliminate pass can be higher than the benefit of fast
+                * clear. RadeonSI does this, but the image threshold is
+                * different.
+                */
+               return false;
+       }
+
+       return image->usage & VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT &&
+              (image->exclusive || image->queue_family_mask == 1);
+}
+
 static bool
 radv_use_dcc_for_image(struct radv_device *device,
                       const struct radv_image *image,
-                      const struct radv_image_create_info *create_info,
-                      const VkImageCreateInfo *pCreateInfo)
+                      const VkImageCreateInfo *pCreateInfo,
+                      VkFormat format)
 {
        bool dcc_compatible_formats;
        bool blendable;
 
        /* DCC (Delta Color Compression) is only available for GFX8+. */
-       if (device->physical_device->rad_info.chip_class < VI)
+       if (device->physical_device->rad_info.chip_class < GFX8)
                return false;
 
        if (device->instance->debug_flags & RADV_DEBUG_NO_DCC)
                return false;
 
-       /* FIXME: DCC is broken for shareable images starting with GFX9 */
-       if (device->physical_device->rad_info.chip_class >= GFX9 &&
-           image->shareable)
+       if (image->shareable)
                return false;
 
        /* TODO: Enable DCC for storage images. */
-       if ((pCreateInfo->usage & VK_IMAGE_USAGE_STORAGE_BIT) ||
-           (pCreateInfo->flags & VK_IMAGE_CREATE_EXTENDED_USAGE_BIT))
+       if ((pCreateInfo->usage & VK_IMAGE_USAGE_STORAGE_BIT))
                return false;
 
        if (pCreateInfo->tiling == VK_IMAGE_TILING_LINEAR)
                return false;
 
-       if (vk_format_is_subsampled(pCreateInfo->format))
+       if (vk_format_is_subsampled(format) ||
+           vk_format_get_plane_count(format) > 1)
                return false;
 
-       /* TODO: Enable DCC for mipmaps and array layers. */
-       if (pCreateInfo->mipLevels > 1 || pCreateInfo->arrayLayers > 1)
+       if (!radv_image_use_fast_clear_for_image(image))
                return false;
 
-       if (create_info->scanout)
+       /* TODO: Enable DCC for mipmaps on GFX9+. */
+       if ((pCreateInfo->arrayLayers > 1 || pCreateInfo->mipLevels > 1) &&
+           device->physical_device->rad_info.chip_class >= GFX9)
+               return false;
+
+       /* Do not enable DCC for mipmapped arrays because performance is worse. */
+       if (pCreateInfo->arrayLayers > 1 && pCreateInfo->mipLevels > 1)
                return false;
 
        /* FIXME: DCC for MSAA with 4x and 8x samples doesn't work yet, while
@@ -169,21 +218,24 @@ radv_use_dcc_for_image(struct radv_device *device,
 
        /* Determine if the formats are DCC compatible. */
        dcc_compatible_formats =
-               radv_is_colorbuffer_format_supported(pCreateInfo->format,
+               radv_is_colorbuffer_format_supported(format,
                                                     &blendable);
 
        if (pCreateInfo->flags & VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT) {
-               const struct VkImageFormatListCreateInfoKHR *format_list =
-                       (const struct  VkImageFormatListCreateInfoKHR *)
+               const struct VkImageFormatListCreateInfo *format_list =
+                       (const struct  VkImageFormatListCreateInfo *)
                                vk_find_struct_const(pCreateInfo->pNext,
-                                                    IMAGE_FORMAT_LIST_CREATE_INFO_KHR);
+                                                    IMAGE_FORMAT_LIST_CREATE_INFO);
 
                /* We have to ignore the existence of the list if viewFormatCount = 0 */
                if (format_list && format_list->viewFormatCount) {
                        /* compatibility is transitive, so we only need to check
                         * one format with everything else. */
                        for (unsigned i = 0; i < format_list->viewFormatCount; ++i) {
-                               if (!radv_dcc_formats_compatible(pCreateInfo->format,
+                               if (format_list->pViewFormats[i] == VK_FORMAT_UNDEFINED)
+                                       continue;
+
+                               if (!radv_dcc_formats_compatible(format,
                                                                 format_list->pViewFormats[i]))
                                        dcc_compatible_formats = false;
                        }
@@ -198,29 +250,208 @@ radv_use_dcc_for_image(struct radv_device *device,
        return true;
 }
 
+static inline bool
+radv_use_fmask_for_image(const struct radv_image *image)
+{
+       return image->info.samples > 1 &&
+              image->usage & VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
+}
+
+static inline bool
+radv_use_htile_for_image(const struct radv_image *image)
+{
+       return image->info.levels == 1 &&
+              image->info.width * image->info.height >= 8 * 8;
+}
+
+static bool
+radv_use_tc_compat_cmask_for_image(struct radv_device *device,
+                                  struct radv_image *image)
+{
+       if (!(device->instance->perftest_flags & RADV_PERFTEST_TC_COMPAT_CMASK))
+               return false;
+
+       /* TC-compat CMASK is only available for GFX8+. */
+       if (device->physical_device->rad_info.chip_class < GFX8)
+               return false;
+
+       if (image->usage & VK_IMAGE_USAGE_STORAGE_BIT)
+               return false;
+
+       if (radv_image_has_dcc(image))
+               return false;
+
+       if (!radv_image_has_cmask(image))
+               return false;
+
+       return true;
+}
+
+static uint32_t si_get_bo_metadata_word1(const struct radv_device *device)
+{
+       return (ATI_VENDOR_ID << 16) | device->physical_device->rad_info.pci_id;
+}
+
+static bool
+radv_is_valid_opaque_metadata(const struct radv_device *device,
+                              const struct radeon_bo_metadata *md)
+{
+       if (md->metadata[0] != 1 ||
+           md->metadata[1] != si_get_bo_metadata_word1(device))
+               return false;
+
+       if (md->size_metadata < 40)
+               return false;
+
+       return true;
+}
+
+static void
+radv_patch_surface_from_metadata(struct radv_device *device,
+                                 struct radeon_surf *surface,
+                                 const struct radeon_bo_metadata *md)
+{
+       surface->flags = RADEON_SURF_CLR(surface->flags, MODE);
+
+       if (device->physical_device->rad_info.chip_class >= GFX9) {
+               if (md->u.gfx9.swizzle_mode > 0)
+                       surface->flags |= RADEON_SURF_SET(RADEON_SURF_MODE_2D, MODE);
+               else
+                       surface->flags |= RADEON_SURF_SET(RADEON_SURF_MODE_LINEAR_ALIGNED, MODE);
+
+               surface->u.gfx9.surf.swizzle_mode = md->u.gfx9.swizzle_mode;
+       } else {
+               surface->u.legacy.pipe_config = md->u.legacy.pipe_config;
+               surface->u.legacy.bankw = md->u.legacy.bankw;
+               surface->u.legacy.bankh = md->u.legacy.bankh;
+               surface->u.legacy.tile_split = md->u.legacy.tile_split;
+               surface->u.legacy.mtilea = md->u.legacy.mtilea;
+               surface->u.legacy.num_banks = md->u.legacy.num_banks;
+
+               if (md->u.legacy.macrotile == RADEON_LAYOUT_TILED)
+                       surface->flags |= RADEON_SURF_SET(RADEON_SURF_MODE_2D, MODE);
+               else if (md->u.legacy.microtile == RADEON_LAYOUT_TILED)
+                       surface->flags |= RADEON_SURF_SET(RADEON_SURF_MODE_1D, MODE);
+               else
+                       surface->flags |= RADEON_SURF_SET(RADEON_SURF_MODE_LINEAR_ALIGNED, MODE);
+
+       }
+}
+
+static VkResult
+radv_patch_image_dimensions(struct radv_device *device,
+                            struct radv_image *image,
+                            const struct radv_image_create_info *create_info,
+                            struct ac_surf_info *image_info)
+{
+       unsigned width = image->info.width;
+       unsigned height = image->info.height;
+
+       /*
+        * minigbm sometimes allocates bigger images which is going to result in
+        * weird strides and other properties. Lets be lenient where possible and
+        * fail it on GFX10 (as we cannot cope there).
+        *
+        * Example hack: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/1457777/
+        */
+       if (create_info->bo_metadata &&
+           radv_is_valid_opaque_metadata(device, create_info->bo_metadata)) {
+               const struct radeon_bo_metadata *md = create_info->bo_metadata;
+
+               if (device->physical_device->rad_info.chip_class >= GFX10) {
+                       width = G_00A004_WIDTH_LO(md->metadata[3]) +
+                               (G_00A008_WIDTH_HI(md->metadata[4]) << 2) + 1;
+                       height = S_00A008_HEIGHT(md->metadata[4]) + 1;
+               } else {
+                       width = G_008F18_WIDTH(md->metadata[4]) + 1;
+                       height = G_008F18_HEIGHT(md->metadata[4]) + 1;
+               }
+       }
+
+       if (image->info.width == width && image->info.height == height)
+               return VK_SUCCESS;
+
+       if (width < image->info.width || height < image->info.height) {
+               fprintf(stderr,
+                       "The imported image has smaller dimensions than the internal\n"
+                       "dimensions. Using it is going to fail badly, so we reject\n"
+                       "this import.\n"
+                       "(internal dimensions: %d x %d, external dimensions: %d x %d)\n",
+                       image->info.width, image->info.height, width, height);
+               return VK_ERROR_INVALID_EXTERNAL_HANDLE;
+       } else if (device->physical_device->rad_info.chip_class >= GFX10) {
+               fprintf(stderr,
+                       "Tried to import an image with inconsistent width on GFX10.\n"
+                       "As GFX10 has no separate stride fields we cannot cope with\n"
+                       "an inconsistency in width and will fail this import.\n"
+                       "(internal dimensions: %d x %d, external dimensions: %d x %d)\n",
+                       image->info.width, image->info.height, width, height);
+               return VK_ERROR_INVALID_EXTERNAL_HANDLE;
+       } else {
+               fprintf(stderr,
+                       "Tried to import an image with inconsistent width on pre-GFX10.\n"
+                       "As GFX10 has no separate stride fields we cannot cope with\n"
+                       "an inconsistency and would fail on GFX10.\n"
+                       "(internal dimensions: %d x %d, external dimensions: %d x %d)\n",
+                       image->info.width, image->info.height, width, height);
+       }
+       image_info->width = width;
+       image_info->height = height;
+
+       return VK_SUCCESS;
+}
+
+static VkResult
+radv_patch_image_from_extra_info(struct radv_device *device,
+                                 struct radv_image *image,
+                                 const struct radv_image_create_info *create_info,
+                                 struct ac_surf_info *image_info)
+{
+       VkResult result = radv_patch_image_dimensions(device, image, create_info, image_info);
+       if (result != VK_SUCCESS)
+               return result;
+
+       for (unsigned plane = 0; plane < image->plane_count; ++plane) {
+               if (create_info->bo_metadata) {
+                       radv_patch_surface_from_metadata(device, &image->planes[plane].surface,
+                                                        create_info->bo_metadata);
+               }
+
+               if (radv_surface_has_scanout(device, create_info)) {
+                       image->planes[plane].surface.flags |= RADEON_SURF_SCANOUT;
+                       image->planes[plane].surface.flags |= RADEON_SURF_DISABLE_DCC;
+
+                       image->info.surf_index = NULL;
+               }
+       }
+       return VK_SUCCESS;
+}
+
 static int
 radv_init_surface(struct radv_device *device,
                  const struct radv_image *image,
                  struct radeon_surf *surface,
-                 const struct radv_image_create_info *create_info)
+                 unsigned plane_id,
+                 const VkImageCreateInfo *pCreateInfo,
+                 VkFormat image_format)
 {
-       const VkImageCreateInfo *pCreateInfo = create_info->vk_info;
-       unsigned array_mode = radv_choose_tiling(device, create_info);
-       const struct vk_format_description *desc =
-               vk_format_description(pCreateInfo->format);
+       unsigned array_mode = radv_choose_tiling(device, pCreateInfo, image_format);
+       VkFormat format = vk_format_get_plane_format(image_format, plane_id);
+       const struct vk_format_description *desc = vk_format_description(format);
        bool is_depth, is_stencil;
 
        is_depth = vk_format_has_depth(desc);
        is_stencil = vk_format_has_stencil(desc);
 
-       surface->blk_w = vk_format_get_blockwidth(pCreateInfo->format);
-       surface->blk_h = vk_format_get_blockheight(pCreateInfo->format);
+       surface->blk_w = vk_format_get_blockwidth(format);
+       surface->blk_h = vk_format_get_blockheight(format);
 
-       surface->bpe = vk_format_get_blocksize(vk_format_depth_only(pCreateInfo->format));
+       surface->bpe = vk_format_get_blocksize(vk_format_depth_only(format));
        /* align byte per element on dword */
        if (surface->bpe == 3) {
                surface->bpe = 4;
        }
+
        surface->flags = RADEON_SURF_SET(array_mode, MODE);
 
        switch (pCreateInfo->imageType){
@@ -243,9 +474,15 @@ radv_init_surface(struct radv_device *device,
                unreachable("unhandled image type");
        }
 
+       /* Required for clearing/initializing a specific layer on GFX8. */
+       surface->flags |= RADEON_SURF_CONTIGUOUS_DCC_LAYERS;
+
        if (is_depth) {
                surface->flags |= RADEON_SURF_ZBUFFER;
-               if (radv_use_tc_compat_htile_for_image(device, pCreateInfo))
+               if (!radv_use_htile_for_image(image) ||
+                   (device->instance->debug_flags & RADV_DEBUG_NO_HIZ))
+                       surface->flags |= RADEON_SURF_NO_HTILE;
+               if (radv_use_tc_compat_htile_for_image(device, pCreateInfo, image_format))
                        surface->flags |= RADEON_SURF_TC_COMPATIBLE_HTILE;
        }
 
@@ -254,32 +491,26 @@ radv_init_surface(struct radv_device *device,
 
        if (device->physical_device->rad_info.chip_class >= GFX9 &&
            pCreateInfo->imageType == VK_IMAGE_TYPE_3D &&
-           vk_format_get_blocksizebits(pCreateInfo->format) == 128 &&
-           vk_format_is_compressed(pCreateInfo->format))
+           vk_format_get_blocksizebits(image_format) == 128 &&
+           vk_format_is_compressed(image_format))
                surface->flags |= RADEON_SURF_NO_RENDER_TARGET;
 
-       surface->flags |= RADEON_SURF_OPTIMIZE_FOR_SPACE;
-
-       if (!radv_use_dcc_for_image(device, image, create_info, pCreateInfo))
+       if (!radv_use_dcc_for_image(device, image, pCreateInfo, image_format))
                surface->flags |= RADEON_SURF_DISABLE_DCC;
 
-       if (create_info->scanout)
-               surface->flags |= RADEON_SURF_SCANOUT;
-       return 0;
-}
+       if (!radv_use_fmask_for_image(image))
+               surface->flags |= RADEON_SURF_NO_FMASK;
 
-static uint32_t si_get_bo_metadata_word1(struct radv_device *device)
-{
-       return (ATI_VENDOR_ID << 16) | device->physical_device->rad_info.pci_id;
+       return 0;
 }
 
 static inline unsigned
-si_tile_mode_index(const struct radv_image *image, unsigned level, bool stencil)
+si_tile_mode_index(const struct radv_image_plane *plane, unsigned level, bool stencil)
 {
        if (stencil)
-               return image->surface.u.legacy.stencil_tiling_index[level];
+               return plane->surface.u.legacy.stencil_tiling_index[level];
        else
-               return image->surface.u.legacy.tiling_index[level];
+               return plane->surface.u.legacy.tiling_index[level];
 }
 
 static unsigned radv_map_swizzle(unsigned swizzle)
@@ -318,15 +549,12 @@ radv_make_buffer_descriptor(struct radv_device *device,
        first_non_void = vk_format_get_first_non_void_channel(vk_format);
        stride = desc->block.bits / 8;
 
-       num_format = radv_translate_buffer_numformat(desc, first_non_void);
-       data_format = radv_translate_buffer_dataformat(desc, first_non_void);
-
        va += offset;
        state[0] = va;
        state[1] = S_008F04_BASE_ADDRESS_HI(va >> 32) |
                S_008F04_STRIDE(stride);
 
-       if (device->physical_device->rad_info.chip_class != VI && stride) {
+       if (device->physical_device->rad_info.chip_class != GFX8 && stride) {
                range /= stride;
        }
 
@@ -334,93 +562,149 @@ radv_make_buffer_descriptor(struct radv_device *device,
        state[3] = S_008F0C_DST_SEL_X(radv_map_swizzle(desc->swizzle[0])) |
                   S_008F0C_DST_SEL_Y(radv_map_swizzle(desc->swizzle[1])) |
                   S_008F0C_DST_SEL_Z(radv_map_swizzle(desc->swizzle[2])) |
-                  S_008F0C_DST_SEL_W(radv_map_swizzle(desc->swizzle[3])) |
-                  S_008F0C_NUM_FORMAT(num_format) |
-                  S_008F0C_DATA_FORMAT(data_format);
+                  S_008F0C_DST_SEL_W(radv_map_swizzle(desc->swizzle[3]));
+
+       if (device->physical_device->rad_info.chip_class >= GFX10) {
+               const struct gfx10_format *fmt = &gfx10_format_table[vk_format_to_pipe_format(vk_format)];
+
+               /* OOB_SELECT chooses the out-of-bounds check:
+                *  - 0: (index >= NUM_RECORDS) || (offset >= STRIDE)
+                *  - 1: index >= NUM_RECORDS
+                *  - 2: NUM_RECORDS == 0
+                *  - 3: if SWIZZLE_ENABLE == 0: offset >= NUM_RECORDS
+                *       else: swizzle_address >= NUM_RECORDS
+                */
+               state[3] |= S_008F0C_FORMAT(fmt->img_format) |
+                           S_008F0C_OOB_SELECT(V_008F0C_OOB_SELECT_STRUCTURED_WITH_OFFSET) |
+                           S_008F0C_RESOURCE_LEVEL(1);
+       } else {
+               num_format = radv_translate_buffer_numformat(desc, first_non_void);
+               data_format = radv_translate_buffer_dataformat(desc, first_non_void);
+
+               assert(data_format != V_008F0C_BUF_DATA_FORMAT_INVALID);
+               assert(num_format != ~0);
+
+               state[3] |= S_008F0C_NUM_FORMAT(num_format) |
+                           S_008F0C_DATA_FORMAT(data_format);
+       }
 }
 
 static void
 si_set_mutable_tex_desc_fields(struct radv_device *device,
                               struct radv_image *image,
                               const struct legacy_surf_level *base_level_info,
+                              unsigned plane_id,
                               unsigned base_level, unsigned first_level,
                               unsigned block_width, bool is_stencil,
-                              bool is_storage_image, uint32_t *state)
+                              bool is_storage_image, bool disable_compression,
+                              uint32_t *state)
 {
+       struct radv_image_plane *plane = &image->planes[plane_id];
        uint64_t gpu_address = image->bo ? radv_buffer_get_va(image->bo) + image->offset : 0;
-       uint64_t va = gpu_address;
+       uint64_t va = gpu_address + plane->offset;
        enum chip_class chip_class = device->physical_device->rad_info.chip_class;
        uint64_t meta_va = 0;
        if (chip_class >= GFX9) {
                if (is_stencil)
-                       va += image->surface.u.gfx9.stencil_offset;
+                       va += plane->surface.u.gfx9.stencil_offset;
                else
-                       va += image->surface.u.gfx9.surf_offset;
+                       va += plane->surface.u.gfx9.surf_offset;
        } else
                va += base_level_info->offset;
 
        state[0] = va >> 8;
        if (chip_class >= GFX9 ||
            base_level_info->mode == RADEON_SURF_MODE_2D)
-               state[0] |= image->surface.tile_swizzle;
+               state[0] |= plane->surface.tile_swizzle;
        state[1] &= C_008F14_BASE_ADDRESS_HI;
        state[1] |= S_008F14_BASE_ADDRESS_HI(va >> 40);
 
-       if (chip_class >= VI) {
+       if (chip_class >= GFX8) {
                state[6] &= C_008F28_COMPRESSION_EN;
                state[7] = 0;
-               if (!is_storage_image && radv_dcc_enabled(image, first_level)) {
-                       meta_va = gpu_address + image->dcc_offset;
-                       if (chip_class <= VI)
+               if (!disable_compression && radv_dcc_enabled(image, first_level)) {
+                       meta_va = gpu_address + plane->surface.dcc_offset;
+                       if (chip_class <= GFX8)
                                meta_va += base_level_info->dcc_offset;
-               } else if (!is_storage_image &&
+
+                       unsigned dcc_tile_swizzle = plane->surface.tile_swizzle << 8;
+                       dcc_tile_swizzle &= plane->surface.dcc_alignment - 1;
+                       meta_va |= dcc_tile_swizzle;
+               } else if (!disable_compression &&
                           radv_image_is_tc_compat_htile(image)) {
-                       meta_va = gpu_address + image->htile_offset;
+                       meta_va = gpu_address +  plane->surface.htile_offset;
                }
 
                if (meta_va) {
                        state[6] |= S_008F28_COMPRESSION_EN(1);
-                       state[7] = meta_va >> 8;
-                       state[7] |= image->surface.tile_swizzle;
+                       if (chip_class <= GFX9)
+                               state[7] = meta_va >> 8;
                }
        }
 
-       if (chip_class >= GFX9) {
+       if (chip_class >= GFX10) {
+               state[3] &= C_00A00C_SW_MODE;
+
+               if (is_stencil) {
+                       state[3] |= S_00A00C_SW_MODE(plane->surface.u.gfx9.stencil.swizzle_mode);
+               } else {
+                       state[3] |= S_00A00C_SW_MODE(plane->surface.u.gfx9.surf.swizzle_mode);
+               }
+
+               state[6] &= C_00A018_META_DATA_ADDRESS_LO &
+                           C_00A018_META_PIPE_ALIGNED;
+
+               if (meta_va) {
+                       struct gfx9_surf_meta_flags meta = {
+                               .rb_aligned = 1,
+                               .pipe_aligned = 1,
+                       };
+
+                       if (plane->surface.dcc_offset)
+                               meta = plane->surface.u.gfx9.dcc;
+
+                       state[6] |= S_00A018_META_PIPE_ALIGNED(meta.pipe_aligned) |
+                                   S_00A018_META_DATA_ADDRESS_LO(meta_va >> 8);
+               }
+
+               state[7] = meta_va >> 16;
+       } else if (chip_class == GFX9) {
                state[3] &= C_008F1C_SW_MODE;
-               state[4] &= C_008F20_PITCH_GFX9;
+               state[4] &= C_008F20_PITCH;
 
                if (is_stencil) {
-                       state[3] |= S_008F1C_SW_MODE(image->surface.u.gfx9.stencil.swizzle_mode);
-                       state[4] |= S_008F20_PITCH_GFX9(image->surface.u.gfx9.stencil.epitch);
+                       state[3] |= S_008F1C_SW_MODE(plane->surface.u.gfx9.stencil.swizzle_mode);
+                       state[4] |= S_008F20_PITCH(plane->surface.u.gfx9.stencil.epitch);
                } else {
-                       state[3] |= S_008F1C_SW_MODE(image->surface.u.gfx9.surf.swizzle_mode);
-                       state[4] |= S_008F20_PITCH_GFX9(image->surface.u.gfx9.surf.epitch);
+                       state[3] |= S_008F1C_SW_MODE(plane->surface.u.gfx9.surf.swizzle_mode);
+                       state[4] |= S_008F20_PITCH(plane->surface.u.gfx9.surf.epitch);
                }
 
                state[5] &= C_008F24_META_DATA_ADDRESS &
                            C_008F24_META_PIPE_ALIGNED &
                            C_008F24_META_RB_ALIGNED;
                if (meta_va) {
-                       struct gfx9_surf_meta_flags meta;
+                       struct gfx9_surf_meta_flags meta = {
+                               .rb_aligned = 1,
+                               .pipe_aligned = 1,
+                       };
 
-                       if (image->dcc_offset)
-                               meta = image->surface.u.gfx9.dcc;
-                       else
-                               meta = image->surface.u.gfx9.htile;
+                       if (plane->surface.dcc_offset)
+                               meta = plane->surface.u.gfx9.dcc;
 
                        state[5] |= S_008F24_META_DATA_ADDRESS(meta_va >> 40) |
                                    S_008F24_META_PIPE_ALIGNED(meta.pipe_aligned) |
                                    S_008F24_META_RB_ALIGNED(meta.rb_aligned);
                }
        } else {
-               /* SI-CI-VI */
+               /* GFX6-GFX8 */
                unsigned pitch = base_level_info->nblk_x * block_width;
-               unsigned index = si_tile_mode_index(image, base_level, is_stencil);
+               unsigned index = si_tile_mode_index(plane, base_level, is_stencil);
 
                state[3] &= C_008F1C_TILING_INDEX;
                state[3] |= S_008F1C_TILING_INDEX(index);
-               state[4] &= C_008F20_PITCH_GFX6;
-               state[4] |= S_008F20_PITCH_GFX6(pitch - 1);
+               state[4] &= C_008F20_PITCH;
+               state[4] |= S_008F20_PITCH(pitch - 1);
        }
 }
 
@@ -480,8 +764,142 @@ static unsigned gfx9_border_color_swizzle(const enum vk_swizzle swizzle[4])
        return bc_swizzle;
 }
 
+bool vi_alpha_is_on_msb(struct radv_device *device, VkFormat format)
+{
+       const struct vk_format_description *desc = vk_format_description(format);
+
+       if (device->physical_device->rad_info.chip_class >= GFX10 && desc->nr_channels == 1)
+               return desc->swizzle[3] == VK_SWIZZLE_X;
+
+       return radv_translate_colorswap(format, false) <= 1;
+}
+/**
+ * Build the sampler view descriptor for a texture (GFX10).
+ */
+static void
+gfx10_make_texture_descriptor(struct radv_device *device,
+                          struct radv_image *image,
+                          bool is_storage_image,
+                          VkImageViewType view_type,
+                          VkFormat vk_format,
+                          const VkComponentMapping *mapping,
+                          unsigned first_level, unsigned last_level,
+                          unsigned first_layer, unsigned last_layer,
+                          unsigned width, unsigned height, unsigned depth,
+                          uint32_t *state,
+                          uint32_t *fmask_state)
+{
+       const struct vk_format_description *desc;
+       enum vk_swizzle swizzle[4];
+       unsigned img_format;
+       unsigned type;
+
+       desc = vk_format_description(vk_format);
+       img_format = gfx10_format_table[vk_format_to_pipe_format(vk_format)].img_format;
+
+       if (desc->colorspace == VK_FORMAT_COLORSPACE_ZS) {
+               const unsigned char swizzle_xxxx[4] = {0, 0, 0, 0};
+               vk_format_compose_swizzles(mapping, swizzle_xxxx, swizzle);
+       } else {
+               vk_format_compose_swizzles(mapping, desc->swizzle, swizzle);
+       }
+
+       type = radv_tex_dim(image->type, view_type, image->info.array_size, image->info.samples,
+                           is_storage_image, device->physical_device->rad_info.chip_class == GFX9);
+       if (type == V_008F1C_SQ_RSRC_IMG_1D_ARRAY) {
+               height = 1;
+               depth = image->info.array_size;
+       } else if (type == V_008F1C_SQ_RSRC_IMG_2D_ARRAY ||
+                  type == V_008F1C_SQ_RSRC_IMG_2D_MSAA_ARRAY) {
+               if (view_type != VK_IMAGE_VIEW_TYPE_3D)
+                       depth = image->info.array_size;
+       } else if (type == V_008F1C_SQ_RSRC_IMG_CUBE)
+               depth = image->info.array_size / 6;
+
+       state[0] = 0;
+       state[1] = S_00A004_FORMAT(img_format) |
+                  S_00A004_WIDTH_LO(width - 1);
+       state[2] = S_00A008_WIDTH_HI((width - 1) >> 2) |
+                  S_00A008_HEIGHT(height - 1) |
+                  S_00A008_RESOURCE_LEVEL(1);
+       state[3] = S_00A00C_DST_SEL_X(radv_map_swizzle(swizzle[0])) |
+                  S_00A00C_DST_SEL_Y(radv_map_swizzle(swizzle[1])) |
+                  S_00A00C_DST_SEL_Z(radv_map_swizzle(swizzle[2])) |
+                  S_00A00C_DST_SEL_W(radv_map_swizzle(swizzle[3])) |
+                  S_00A00C_BASE_LEVEL(image->info.samples > 1 ?
+                                       0 : first_level) |
+                  S_00A00C_LAST_LEVEL(image->info.samples > 1 ?
+                                       util_logbase2(image->info.samples) :
+                                       last_level) |
+                  S_00A00C_BC_SWIZZLE(gfx9_border_color_swizzle(swizzle)) |
+                  S_00A00C_TYPE(type);
+       /* Depth is the the last accessible layer on gfx9+. The hw doesn't need
+        * to know the total number of layers.
+        */
+       state[4] = S_00A010_DEPTH(type == V_008F1C_SQ_RSRC_IMG_3D ? depth - 1 : last_layer) |
+                  S_00A010_BASE_ARRAY(first_layer);
+       state[5] = S_00A014_ARRAY_PITCH(0) |
+                  S_00A014_MAX_MIP(image->info.samples > 1 ?
+                                   util_logbase2(image->info.samples) :
+                                   image->info.levels - 1) |
+                  S_00A014_PERF_MOD(4);
+       state[6] = 0;
+       state[7] = 0;
+
+       if (radv_dcc_enabled(image, first_level)) {
+               state[6] |= S_00A018_MAX_UNCOMPRESSED_BLOCK_SIZE(V_028C78_MAX_BLOCK_SIZE_256B) |
+                           S_00A018_MAX_COMPRESSED_BLOCK_SIZE(V_028C78_MAX_BLOCK_SIZE_128B) |
+                           S_00A018_ALPHA_IS_ON_MSB(vi_alpha_is_on_msb(device, vk_format));
+       }
+
+       /* Initialize the sampler view for FMASK. */
+       if (radv_image_has_fmask(image)) {
+               uint64_t gpu_address = radv_buffer_get_va(image->bo);
+               uint32_t format;
+               uint64_t va;
+
+               assert(image->plane_count == 1);
+
+               va = gpu_address + image->offset + image->planes[0].surface.fmask_offset;
+
+               switch (image->info.samples) {
+               case 2:
+                       format = V_008F0C_IMG_FORMAT_FMASK8_S2_F2;
+                       break;
+               case 4:
+                       format = V_008F0C_IMG_FORMAT_FMASK8_S4_F4;
+                       break;
+               case 8:
+                       format = V_008F0C_IMG_FORMAT_FMASK32_S8_F8;
+                       break;
+               default:
+                       unreachable("invalid nr_samples");
+               }
+
+               fmask_state[0] = (va >> 8) | image->planes[0].surface.fmask_tile_swizzle;
+               fmask_state[1] = S_00A004_BASE_ADDRESS_HI(va >> 40) |
+                                S_00A004_FORMAT(format) |
+                                S_00A004_WIDTH_LO(width - 1);
+               fmask_state[2] = S_00A008_WIDTH_HI((width - 1) >> 2) |
+                                S_00A008_HEIGHT(height - 1) |
+                                S_00A008_RESOURCE_LEVEL(1);
+               fmask_state[3] = S_00A00C_DST_SEL_X(V_008F1C_SQ_SEL_X) |
+                                S_00A00C_DST_SEL_Y(V_008F1C_SQ_SEL_X) |
+                                S_00A00C_DST_SEL_Z(V_008F1C_SQ_SEL_X) |
+                                S_00A00C_DST_SEL_W(V_008F1C_SQ_SEL_X) |
+                                S_00A00C_SW_MODE(image->planes[0].surface.u.gfx9.fmask.swizzle_mode) |
+                                S_00A00C_TYPE(radv_tex_dim(image->type, view_type, image->info.array_size, 0, false, false));
+               fmask_state[4] = S_00A010_DEPTH(last_layer) |
+                                S_00A010_BASE_ARRAY(first_layer);
+               fmask_state[5] = 0;
+               fmask_state[6] = S_00A018_META_PIPE_ALIGNED(1);
+               fmask_state[7] = 0;
+       } else if (fmask_state)
+               memset(fmask_state, 0, 8 * 4);
+}
+
 /**
- * Build the sampler view descriptor for a texture.
+ * Build the sampler view descriptor for a texture (SI-GFX9)
  */
 static void
 si_make_texture_descriptor(struct radv_device *device,
@@ -523,7 +941,7 @@ si_make_texture_descriptor(struct radv_device *device,
        }
 
        /* S8 with either Z16 or Z32 HTILE need a special format. */
-       if (device->physical_device->rad_info.chip_class >= GFX9 &&
+       if (device->physical_device->rad_info.chip_class == GFX9 &&
            vk_format == VK_FORMAT_S8_UINT &&
            radv_image_is_tc_compat_htile(image)) {
                if (image->vk_format == VK_FORMAT_D32_SFLOAT_S8_UINT)
@@ -532,7 +950,7 @@ si_make_texture_descriptor(struct radv_device *device,
                        data_format = V_008F14_IMG_DATA_FORMAT_S8_16;
        }
        type = radv_tex_dim(image->type, view_type, image->info.array_size, image->info.samples,
-                           is_storage_image, device->physical_device->rad_info.chip_class >= GFX9);
+                           is_storage_image, device->physical_device->rad_info.chip_class == GFX9);
        if (type == V_008F1C_SQ_RSRC_IMG_1D_ARRAY) {
                height = 1;
                depth = image->info.array_size;
@@ -544,8 +962,8 @@ si_make_texture_descriptor(struct radv_device *device,
                depth = image->info.array_size / 6;
 
        state[0] = 0;
-       state[1] = (S_008F14_DATA_FORMAT_GFX6(data_format) |
-                   S_008F14_NUM_FORMAT_GFX6(num_format));
+       state[1] = (S_008F14_DATA_FORMAT(data_format) |
+                   S_008F14_NUM_FORMAT(num_format));
        state[2] = (S_008F18_WIDTH(width - 1) |
                    S_008F18_HEIGHT(height - 1) |
                    S_008F18_PERF_MOD(4));
@@ -564,7 +982,7 @@ si_make_texture_descriptor(struct radv_device *device,
        state[6] = 0;
        state[7] = 0;
 
-       if (device->physical_device->rad_info.chip_class >= GFX9) {
+       if (device->physical_device->rad_info.chip_class == GFX9) {
                unsigned bc_swizzle = gfx9_border_color_swizzle(swizzle);
 
                /* Depth is the last accessible layer on Gfx9.
@@ -584,15 +1002,13 @@ si_make_texture_descriptor(struct radv_device *device,
                state[4] |= S_008F20_DEPTH(depth - 1);
                state[5] |= S_008F24_LAST_ARRAY(last_layer);
        }
-       if (image->dcc_offset) {
-               unsigned swap = radv_translate_colorswap(vk_format, FALSE);
-
-               state[6] = S_008F28_ALPHA_IS_ON_MSB(swap <= 1);
+       if (image->planes[0].surface.dcc_offset) {
+               state[6] = S_008F28_ALPHA_IS_ON_MSB(vi_alpha_is_on_msb(device, vk_format));
        } else {
                /* The last dword is unused by hw. The shader uses it to clear
                 * bits in the first dword of sampler state.
                 */
-               if (device->physical_device->rad_info.chip_class <= CIK && image->info.samples <= 1) {
+               if (device->physical_device->rad_info.chip_class <= GFX7 && image->info.samples <= 1) {
                        if (first_level == last_level)
                                state[7] = C_008F30_MAX_ANISO_RATIO;
                        else
@@ -606,9 +1022,11 @@ si_make_texture_descriptor(struct radv_device *device,
                uint64_t gpu_address = radv_buffer_get_va(image->bo);
                uint64_t va;
 
-               va = gpu_address + image->offset + image->fmask.offset;
+               assert(image->plane_count == 1);
+
+               va = gpu_address + image->offset + image->planes[0].surface.fmask_offset;
 
-               if (device->physical_device->rad_info.chip_class >= GFX9) {
+               if (device->physical_device->rad_info.chip_class == GFX9) {
                        fmask_format = V_008F14_IMG_DATA_FORMAT_FMASK;
                        switch (image->info.samples) {
                        case 2:
@@ -642,10 +1060,10 @@ si_make_texture_descriptor(struct radv_device *device,
                }
 
                fmask_state[0] = va >> 8;
-               fmask_state[0] |= image->fmask.tile_swizzle;
+               fmask_state[0] |= image->planes[0].surface.fmask_tile_swizzle;
                fmask_state[1] = S_008F14_BASE_ADDRESS_HI(va >> 40) |
-                       S_008F14_DATA_FORMAT_GFX6(fmask_format) |
-                       S_008F14_NUM_FORMAT_GFX6(num_format);
+                       S_008F14_DATA_FORMAT(fmask_format) |
+                       S_008F14_NUM_FORMAT(num_format);
                fmask_state[2] = S_008F18_WIDTH(width - 1) |
                        S_008F18_HEIGHT(height - 1);
                fmask_state[3] = S_008F1C_DST_SEL_X(V_008F1C_SQ_SEL_X) |
@@ -658,22 +1076,67 @@ si_make_texture_descriptor(struct radv_device *device,
                fmask_state[6] = 0;
                fmask_state[7] = 0;
 
-               if (device->physical_device->rad_info.chip_class >= GFX9) {
-                       fmask_state[3] |= S_008F1C_SW_MODE(image->surface.u.gfx9.fmask.swizzle_mode);
+               if (device->physical_device->rad_info.chip_class == GFX9) {
+                       fmask_state[3] |= S_008F1C_SW_MODE(image->planes[0].surface.u.gfx9.fmask.swizzle_mode);
                        fmask_state[4] |= S_008F20_DEPTH(last_layer) |
-                                         S_008F20_PITCH_GFX9(image->surface.u.gfx9.fmask.epitch);
-                       fmask_state[5] |= S_008F24_META_PIPE_ALIGNED(image->surface.u.gfx9.cmask.pipe_aligned) |
-                                         S_008F24_META_RB_ALIGNED(image->surface.u.gfx9.cmask.rb_aligned);
+                                         S_008F20_PITCH(image->planes[0].surface.u.gfx9.fmask.epitch);
+                       fmask_state[5] |= S_008F24_META_PIPE_ALIGNED(1) |
+                                         S_008F24_META_RB_ALIGNED(1);
+
+                       if (radv_image_is_tc_compat_cmask(image)) {
+                               va = gpu_address + image->offset + image->planes[0].surface.cmask_offset;
+
+                               fmask_state[5] |= S_008F24_META_DATA_ADDRESS(va >> 40);
+                               fmask_state[6] |= S_008F28_COMPRESSION_EN(1);
+                               fmask_state[7] |= va >> 8;
+                       }
                } else {
-                       fmask_state[3] |= S_008F1C_TILING_INDEX(image->fmask.tile_mode_index);
+                       fmask_state[3] |= S_008F1C_TILING_INDEX(image->planes[0].surface.u.legacy.fmask.tiling_index);
                        fmask_state[4] |= S_008F20_DEPTH(depth - 1) |
-                               S_008F20_PITCH_GFX6(image->fmask.pitch_in_pixels - 1);
+                               S_008F20_PITCH(image->planes[0].surface.u.legacy.fmask.pitch_in_pixels - 1);
                        fmask_state[5] |= S_008F24_LAST_ARRAY(last_layer);
+
+                       if (radv_image_is_tc_compat_cmask(image)) {
+                               va = gpu_address + image->offset + image->planes[0].surface.cmask_offset;
+
+                               fmask_state[6] |= S_008F28_COMPRESSION_EN(1);
+                               fmask_state[7] |= va >> 8;
+                       }
                }
        } else if (fmask_state)
                memset(fmask_state, 0, 8 * 4);
 }
 
+static void
+radv_make_texture_descriptor(struct radv_device *device,
+                            struct radv_image *image,
+                            bool is_storage_image,
+                            VkImageViewType view_type,
+                            VkFormat vk_format,
+                            const VkComponentMapping *mapping,
+                            unsigned first_level, unsigned last_level,
+                            unsigned first_layer, unsigned last_layer,
+                            unsigned width, unsigned height, unsigned depth,
+                            uint32_t *state,
+                            uint32_t *fmask_state)
+{
+       if (device->physical_device->rad_info.chip_class >= GFX10) {
+               gfx10_make_texture_descriptor(device, image, is_storage_image,
+                                             view_type, vk_format, mapping,
+                                             first_level, last_level,
+                                             first_layer, last_layer,
+                                             width, height, depth,
+                                             state, fmask_state);
+       } else {
+               si_make_texture_descriptor(device, image, is_storage_image,
+                                          view_type, vk_format, mapping,
+                                          first_level, last_level,
+                                          first_layer, last_layer,
+                                          width, height, depth,
+                                          state, fmask_state);
+       }
+}
+
 static void
 radv_query_opaque_metadata(struct radv_device *device,
                           struct radv_image *image,
@@ -682,6 +1145,8 @@ radv_query_opaque_metadata(struct radv_device *device,
        static const VkComponentMapping fixedmapping;
        uint32_t desc[8], i;
 
+       assert(image->plane_count == 1);
+
        /* Metadata image format format version 1:
         * [0] = 1 (metadata format identifier)
         * [1] = (VENDOR_ID << 16) | PCI_ID
@@ -697,31 +1162,32 @@ radv_query_opaque_metadata(struct radv_device *device,
        md->metadata[1] = si_get_bo_metadata_word1(device);
 
 
-       si_make_texture_descriptor(device, image, false,
-                                  (VkImageViewType)image->type, image->vk_format,
-                                  &fixedmapping, 0, image->info.levels - 1, 0,
-                                  image->info.array_size - 1,
-                                  image->info.width, image->info.height,
-                                  image->info.depth,
-                                  desc, NULL);
+       radv_make_texture_descriptor(device, image, false,
+                                    (VkImageViewType)image->type, image->vk_format,
+                                    &fixedmapping, 0, image->info.levels - 1, 0,
+                                    image->info.array_size - 1,
+                                    image->info.width, image->info.height,
+                                    image->info.depth,
+                                    desc, NULL);
 
-       si_set_mutable_tex_desc_fields(device, image, &image->surface.u.legacy.level[0], 0, 0,
-                                      image->surface.blk_w, false, false, desc);
+       si_set_mutable_tex_desc_fields(device, image, &image->planes[0].surface.u.legacy.level[0], 0, 0, 0,
+                                      image->planes[0].surface.blk_w, false, false, false, desc);
 
        /* Clear the base address and set the relative DCC offset. */
        desc[0] = 0;
        desc[1] &= C_008F14_BASE_ADDRESS_HI;
-       desc[7] = image->dcc_offset >> 8;
+       desc[7] = image->planes[0].surface.dcc_offset >> 8;
 
        /* Dwords [2:9] contain the image descriptor. */
        memcpy(&md->metadata[2], desc, sizeof(desc));
 
        /* Dwords [10:..] contain the mipmap level offsets. */
-       if (device->physical_device->rad_info.chip_class <= VI) {
+       if (device->physical_device->rad_info.chip_class <= GFX8) {
                for (i = 0; i <= image->info.levels - 1; i++)
-                       md->metadata[10+i] = image->surface.u.legacy.level[i].offset >> 8;
+                       md->metadata[10+i] = image->planes[0].surface.u.legacy.level[i].offset >> 8;
                md->size_metadata = (11 + image->info.levels - 1) * 4;
-       }
+       } else
+               md->size_metadata = 10 * 4;
 }
 
 void
@@ -729,12 +1195,13 @@ radv_init_metadata(struct radv_device *device,
                   struct radv_image *image,
                   struct radeon_bo_metadata *metadata)
 {
-       struct radeon_surf *surface = &image->surface;
+       struct radeon_surf *surface = &image->planes[0].surface;
 
        memset(metadata, 0, sizeof(*metadata));
 
        if (device->physical_device->rad_info.chip_class >= GFX9) {
                metadata->u.gfx9.swizzle_mode = surface->u.gfx9.surf.swizzle_mode;
+               metadata->u.gfx9.scanout = (surface->flags & RADEON_SURF_SCANOUT) != 0;
        } else {
                metadata->u.legacy.microtile = surface->u.legacy.level[0].mode >= RADEON_SURF_MODE_1D ?
                        RADEON_LAYOUT_TILED : RADEON_LAYOUT_LINEAR;
@@ -752,198 +1219,136 @@ radv_init_metadata(struct radv_device *device,
        radv_query_opaque_metadata(device, image, metadata);
 }
 
-/* The number of samples can be specified independently of the texture. */
-static void
-radv_image_get_fmask_info(struct radv_device *device,
-                         struct radv_image *image,
-                         unsigned nr_samples,
-                         struct radv_fmask_info *out)
-{
-       if (device->physical_device->rad_info.chip_class >= GFX9) {
-               out->alignment = image->surface.fmask_alignment;
-               out->size = image->surface.fmask_size;
-               out->tile_swizzle = image->surface.fmask_tile_swizzle;
-               return;
-       }
-
-       out->slice_tile_max = image->surface.u.legacy.fmask.slice_tile_max;
-       out->tile_mode_index = image->surface.u.legacy.fmask.tiling_index;
-       out->pitch_in_pixels = image->surface.u.legacy.fmask.pitch_in_pixels;
-       out->bank_height = image->surface.u.legacy.fmask.bankh;
-       out->tile_swizzle = image->surface.fmask_tile_swizzle;
-       out->alignment = image->surface.fmask_alignment;
-       out->size = image->surface.fmask_size;
-
-       assert(!out->tile_swizzle || !image->shareable);
-}
-
-static void
-radv_image_alloc_fmask(struct radv_device *device,
-                      struct radv_image *image)
+void
+radv_image_override_offset_stride(struct radv_device *device,
+                                  struct radv_image *image,
+                                  uint64_t offset, uint32_t stride)
 {
-       radv_image_get_fmask_info(device, image, image->info.samples, &image->fmask);
-
-       image->fmask.offset = align64(image->size, image->fmask.alignment);
-       image->size = image->fmask.offset + image->fmask.size;
-       image->alignment = MAX2(image->alignment, image->fmask.alignment);
+       ac_surface_override_offset_stride(&device->physical_device->rad_info,
+                                         &image->planes[0].surface,
+                                         image->info.levels, offset, stride);
 }
 
 static void
-radv_image_get_cmask_info(struct radv_device *device,
-                         struct radv_image *image,
-                         struct radv_cmask_info *out)
+radv_image_alloc_single_sample_cmask(const struct radv_image *image,
+                                     struct radeon_surf *surf)
 {
-       unsigned pipe_interleave_bytes = device->physical_device->rad_info.pipe_interleave_bytes;
-       unsigned num_pipes = device->physical_device->rad_info.num_tile_pipes;
-       unsigned cl_width, cl_height;
-
-       if (device->physical_device->rad_info.chip_class >= GFX9) {
-               out->alignment = image->surface.cmask_alignment;
-               out->size = image->surface.cmask_size;
+       if (!surf->cmask_size || surf->cmask_offset || surf->bpe > 8 ||
+           image->info.levels > 1 || image->info.depth > 1 ||
+           radv_image_has_dcc(image) || !radv_image_use_fast_clear_for_image(image))
                return;
-       }
-
-       switch (num_pipes) {
-       case 2:
-               cl_width = 32;
-               cl_height = 16;
-               break;
-       case 4:
-               cl_width = 32;
-               cl_height = 32;
-               break;
-       case 8:
-               cl_width = 64;
-               cl_height = 32;
-               break;
-       case 16: /* Hawaii */
-               cl_width = 64;
-               cl_height = 64;
-               break;
-       default:
-               assert(0);
-               return;
-       }
 
-       unsigned base_align = num_pipes * pipe_interleave_bytes;
+       assert(image->info.storage_samples == 1);
 
-       unsigned width = align(image->surface.u.legacy.level[0].nblk_x, cl_width*8);
-       unsigned height = align(image->surface.u.legacy.level[0].nblk_y, cl_height*8);
-       unsigned slice_elements = (width * height) / (8*8);
-
-       /* Each element of CMASK is a nibble. */
-       unsigned slice_bytes = slice_elements / 2;
-
-       out->slice_tile_max = (width * height) / (128*128);
-       if (out->slice_tile_max)
-               out->slice_tile_max -= 1;
-
-       out->alignment = MAX2(256, base_align);
-       out->size = (image->type == VK_IMAGE_TYPE_3D ? image->info.depth : image->info.array_size) *
-                   align(slice_bytes, base_align);
+       surf->cmask_offset = align64(surf->total_size, surf->cmask_alignment);
+       surf->total_size = surf->cmask_offset + surf->cmask_size;
+       surf->alignment = MAX2(surf->alignment, surf->cmask_alignment);
 }
 
 static void
-radv_image_alloc_cmask(struct radv_device *device,
-                      struct radv_image *image)
+radv_image_alloc_values(const struct radv_device *device, struct radv_image *image)
 {
-       uint32_t clear_value_size = 0;
-       radv_image_get_cmask_info(device, image, &image->cmask);
-
-       image->cmask.offset = align64(image->size, image->cmask.alignment);
-       /* + 8 for storing the clear values */
-       if (!image->clear_value_offset) {
-               image->clear_value_offset = image->cmask.offset + image->cmask.size;
-               clear_value_size = 8;
-       }
-       image->size = image->cmask.offset + image->cmask.size + clear_value_size;
-       image->alignment = MAX2(image->alignment, image->cmask.alignment);
-}
+       if (radv_image_has_dcc(image)) {
+               image->fce_pred_offset = image->size;
+               image->size += 8 * image->info.levels;
 
-static void
-radv_image_alloc_dcc(struct radv_image *image)
-{
-       image->dcc_offset = align64(image->size, image->surface.dcc_alignment);
-       /* + 16 for storing the clear values + dcc pred */
-       image->clear_value_offset = image->dcc_offset + image->surface.dcc_size;
-       image->fce_pred_offset = image->clear_value_offset + 8;
-       image->dcc_pred_offset = image->clear_value_offset + 16;
-       image->size = image->dcc_offset + image->surface.dcc_size + 24;
-       image->alignment = MAX2(image->alignment, image->surface.dcc_alignment);
-}
+               image->dcc_pred_offset = image->size;
+               image->size += 8 * image->info.levels;
+       }
 
-static void
-radv_image_alloc_htile(struct radv_image *image)
-{
-       image->htile_offset = align64(image->size, image->surface.htile_alignment);
+       if (radv_image_has_dcc(image) || radv_image_has_cmask(image) ||
+           radv_image_has_htile(image)) {
+               image->clear_value_offset = image->size;
+               image->size += 8 * image->info.levels;
+       }
 
-       /* + 8 for storing the clear values */
-       image->clear_value_offset = image->htile_offset + image->surface.htile_size;
-       image->size = image->clear_value_offset + 8;
-       if (radv_image_is_tc_compat_htile(image)) {
+       if (radv_image_is_tc_compat_htile(image) &&
+           device->physical_device->rad_info.has_tc_compat_zrange_bug) {
                /* Metadata for the TC-compatible HTILE hardware bug which
                 * have to be fixed by updating ZRANGE_PRECISION when doing
                 * fast depth clears to 0.0f.
                 */
-               image->tc_compat_zrange_offset = image->clear_value_offset + 8;
-               image->size = image->clear_value_offset + 16;
+               image->tc_compat_zrange_offset = image->size;
+               image->size += image->info.levels * 4;
        }
-       image->alignment = align64(image->alignment, image->surface.htile_alignment);
 }
 
-static inline bool
-radv_image_can_enable_dcc_or_cmask(struct radv_image *image)
+VkResult
+radv_image_create_layout(struct radv_device *device,
+                         struct radv_image_create_info create_info,
+                         struct radv_image *image)
 {
-       if (image->info.samples <= 1 &&
-           image->info.width * image->info.height <= 512 * 512) {
-               /* Do not enable CMASK or DCC for small surfaces where the cost
-                * of the eliminate pass can be higher than the benefit of fast
-                * clear. RadeonSI does this, but the image threshold is
-                * different.
-                */
-               return false;
-       }
+       /* Check that we did not initialize things earlier */
+       assert(!image->planes[0].surface.surf_size);
+
+       /* Clear the pCreateInfo pointer so we catch issues in the delayed case when we test in the
+        * common internal case. */
+       create_info.vk_info = NULL;
+
+       struct ac_surf_info image_info = image->info;
+       VkResult result = radv_patch_image_from_extra_info(device, image, &create_info, &image_info);
+       if (result != VK_SUCCESS)
+               return result;
+
+       image->size = 0;
+       image->alignment = 1;
+       for (unsigned plane = 0; plane < image->plane_count; ++plane) {
+               struct ac_surf_info info = image_info;
+
+               if (plane) {
+                       const struct vk_format_description *desc = vk_format_description(image->vk_format);
+                       assert(info.width % desc->width_divisor == 0);
+                       assert(info.height % desc->height_divisor == 0);
+
+                       info.width /= desc->width_divisor;
+                       info.height /= desc->height_divisor;
+               }
 
-       return image->usage & VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT &&
-              (image->exclusive || image->queue_family_mask == 1);
-}
+               if (create_info.no_metadata_planes || image->plane_count > 1) {
+                       image->planes[plane].surface.flags |= RADEON_SURF_DISABLE_DCC |
+                                                             RADEON_SURF_NO_FMASK |
+                                                             RADEON_SURF_NO_HTILE;
+               }
 
-static inline bool
-radv_image_can_enable_dcc(struct radv_image *image)
-{
-       return radv_image_can_enable_dcc_or_cmask(image) &&
-              radv_image_has_dcc(image);
-}
+               device->ws->surface_init(device->ws, &info, &image->planes[plane].surface);
 
-static inline bool
-radv_image_can_enable_cmask(struct radv_image *image)
-{
-       if (image->surface.bpe > 8 && image->info.samples == 1) {
-               /* Do not enable CMASK for non-MSAA images (fast color clear)
-                * because 128 bit formats are not supported, but FMASK might
-                * still be used.
-                */
-               return false;
+               if (!create_info.no_metadata_planes && image->plane_count == 1)
+                       radv_image_alloc_single_sample_cmask(image, &image->planes[plane].surface);
+
+               image->planes[plane].offset = align(image->size, image->planes[plane].surface.alignment);
+               image->size = image->planes[plane].offset + image->planes[plane].surface.total_size;
+               image->alignment = MAX2(image->alignment, image->planes[plane].surface.alignment);
+
+               image->planes[plane].format = vk_format_get_plane_format(image->vk_format, plane);
        }
 
-       return radv_image_can_enable_dcc_or_cmask(image) &&
-              image->info.levels == 1 &&
-              image->info.depth == 1 &&
-              !image->surface.is_linear;
-}
+       image->tc_compatible_cmask = radv_image_has_cmask(image) &&
+                                    radv_use_tc_compat_cmask_for_image(device, 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);
+       image->tc_compatible_htile = radv_image_has_htile(image) &&
+                                    image->planes[0].surface.flags & RADEON_SURF_TC_COMPATIBLE_HTILE;
+
+       radv_image_alloc_values(device, image);
+
+       assert(image->planes[0].surface.surf_size);
+       return VK_SUCCESS;
 }
 
-static inline bool
-radv_image_can_enable_htile(struct radv_image *image)
+static void
+radv_destroy_image(struct radv_device *device,
+                  const VkAllocationCallbacks *pAllocator,
+                  struct radv_image *image)
 {
-       return radv_image_has_htile(image) &&
-              image->info.levels == 1 &&
-              image->info.width * image->info.height >= 8 * 8;
+       if ((image->flags & VK_IMAGE_CREATE_SPARSE_BINDING_BIT) && image->bo)
+               device->ws->buffer_destroy(image->bo);
+
+       if (image->owned_memory != VK_NULL_HANDLE) {
+               RADV_FROM_HANDLE(radv_device_memory, mem, image->owned_memory);
+               radv_free_memory(device, pAllocator, mem);
+       }
+
+       vk_object_base_finish(&image->base);
+       vk_free2(&device->vk.alloc, pAllocator, image);
 }
 
 VkResult
@@ -955,8 +1360,13 @@ radv_image_create(VkDevice _device,
        RADV_FROM_HANDLE(radv_device, device, _device);
        const VkImageCreateInfo *pCreateInfo = create_info->vk_info;
        struct radv_image *image = NULL;
+       VkFormat format = radv_select_android_external_format(pCreateInfo->pNext,
+                                                             pCreateInfo->format);
        assert(pCreateInfo->sType == VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO);
 
+       const unsigned plane_count = vk_format_get_plane_count(format);
+       const size_t image_struct_size = sizeof(*image) + sizeof(struct radv_image_plane) * plane_count;
+
        radv_assert(pCreateInfo->mipLevels > 0);
        radv_assert(pCreateInfo->arrayLayers > 0);
        radv_assert(pCreateInfo->samples > 0);
@@ -964,11 +1374,13 @@ radv_image_create(VkDevice _device,
        radv_assert(pCreateInfo->extent.height > 0);
        radv_assert(pCreateInfo->extent.depth > 0);
 
-       image = vk_zalloc2(&device->alloc, alloc, sizeof(*image), 8,
+       image = vk_zalloc2(&device->vk.alloc, alloc, image_struct_size, 8,
                           VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
        if (!image)
                return vk_error(device->instance, VK_ERROR_OUT_OF_HOST_MEMORY);
 
+       vk_object_base_init(&device->vk, &image->base, VK_OBJECT_TYPE_IMAGE);
+
        image->type = pCreateInfo->imageType;
        image->info.width = pCreateInfo->extent.width;
        image->info.height = pCreateInfo->extent.height;
@@ -977,73 +1389,50 @@ radv_image_create(VkDevice _device,
        image->info.storage_samples = pCreateInfo->samples;
        image->info.array_size = pCreateInfo->arrayLayers;
        image->info.levels = pCreateInfo->mipLevels;
-       image->info.num_channels = vk_format_get_nr_components(pCreateInfo->format);
+       image->info.num_channels = vk_format_get_nr_components(format);
 
-       image->vk_format = pCreateInfo->format;
+       image->vk_format = format;
        image->tiling = pCreateInfo->tiling;
        image->usage = pCreateInfo->usage;
        image->flags = pCreateInfo->flags;
+       image->plane_count = plane_count;
 
        image->exclusive = pCreateInfo->sharingMode == VK_SHARING_MODE_EXCLUSIVE;
        if (pCreateInfo->sharingMode == VK_SHARING_MODE_CONCURRENT) {
                for (uint32_t i = 0; i < pCreateInfo->queueFamilyIndexCount; ++i)
-                       if (pCreateInfo->pQueueFamilyIndices[i] == VK_QUEUE_FAMILY_EXTERNAL)
+                       if (pCreateInfo->pQueueFamilyIndices[i] == VK_QUEUE_FAMILY_EXTERNAL ||
+                           pCreateInfo->pQueueFamilyIndices[i] == VK_QUEUE_FAMILY_FOREIGN_EXT)
                                image->queue_family_mask |= (1u << RADV_MAX_QUEUE_FAMILIES) - 1u;
                        else
                                image->queue_family_mask |= 1u << pCreateInfo->pQueueFamilyIndices[i];
        }
 
-       image->shareable = vk_find_struct_const(pCreateInfo->pNext,
-                                               EXTERNAL_MEMORY_IMAGE_CREATE_INFO) != NULL;
-       if (!vk_format_is_depth_or_stencil(pCreateInfo->format) && !create_info->scanout && !image->shareable) {
+       const VkExternalMemoryImageCreateInfo *external_info =
+               vk_find_struct_const(pCreateInfo->pNext,
+                                    EXTERNAL_MEMORY_IMAGE_CREATE_INFO) ;
+
+       image->shareable = external_info;
+       if (!vk_format_is_depth_or_stencil(format) && !image->shareable) {
                image->info.surf_index = &device->image_mrt_offset_counter;
        }
 
-       radv_init_surface(device, image, &image->surface, create_info);
-
-       device->ws->surface_init(device->ws, &image->info, &image->surface);
+       for (unsigned plane = 0; plane < image->plane_count; ++plane) {
+               radv_init_surface(device, image, &image->planes[plane].surface, plane, pCreateInfo, format);
+       }
 
-       image->size = image->surface.surf_size;
-       image->alignment = image->surface.surf_alignment;
+       bool delay_layout = external_info &&
+               (external_info->handleTypes & VK_EXTERNAL_MEMORY_HANDLE_TYPE_ANDROID_HARDWARE_BUFFER_BIT_ANDROID);
 
-       if (!create_info->no_metadata_planes) {
-               /* Try to enable DCC first. */
-               if (radv_image_can_enable_dcc(image)) {
-                       radv_image_alloc_dcc(image);
-                       if (image->info.samples > 1) {
-                               /* CMASK should be enabled because DCC fast
-                                * clear with MSAA needs it.
-                                */
-                               assert(radv_image_can_enable_cmask(image));
-                               radv_image_alloc_cmask(device, image);
-                       }
-               } else {
-                       /* When DCC cannot be enabled, try CMASK. */
-                       image->surface.dcc_size = 0;
-                       if (radv_image_can_enable_cmask(image)) {
-                               radv_image_alloc_cmask(device, image);
-                       }
-               }
-
-               /* Try to enable FMASK for multisampled images. */
-               if (radv_image_can_enable_fmask(image)) {
-                       radv_image_alloc_fmask(device, image);
-               } else {
-                       /* Otherwise, try to enable HTILE for depth surfaces. */
-                       if (radv_image_can_enable_htile(image) &&
-                           !(device->instance->debug_flags & RADV_DEBUG_NO_HIZ)) {
-                               image->tc_compatible_htile = image->surface.flags & RADEON_SURF_TC_COMPATIBLE_HTILE;
-                               radv_image_alloc_htile(image);
-                       } else {
-                               image->surface.htile_size = 0;
-                       }
-               }
-       } else {
-               image->surface.dcc_size = 0;
-               image->surface.htile_size = 0;
+       if (delay_layout) {
+               *pImage = radv_image_to_handle(image);
+               assert (!(image->flags & VK_IMAGE_CREATE_SPARSE_BINDING_BIT));
+               return VK_SUCCESS;
        }
 
-       if (pCreateInfo->flags & VK_IMAGE_CREATE_SPARSE_BINDING_BIT) {
+       ASSERTED VkResult result = radv_image_create_layout(device, *create_info, image);
+       assert(result == VK_SUCCESS);
+
+       if (image->flags & VK_IMAGE_CREATE_SPARSE_BINDING_BIT) {
                image->alignment = MAX2(image->alignment, 4096);
                image->size = align64(image->size, image->alignment);
                image->offset = 0;
@@ -1051,7 +1440,7 @@ radv_image_create(VkDevice _device,
                image->bo = device->ws->buffer_create(device->ws, image->size, image->alignment,
                                                      0, RADEON_FLAG_VIRTUAL, RADV_BO_PRIORITY_VIRTUAL);
                if (!image->bo) {
-                       vk_free2(&device->alloc, alloc, image);
+                       radv_destroy_image(device, alloc, image);
                        return vk_error(device->instance, VK_ERROR_OUT_OF_DEVICE_MEMORY);
                }
        }
@@ -1064,57 +1453,100 @@ radv_image_create(VkDevice _device,
 static void
 radv_image_view_make_descriptor(struct radv_image_view *iview,
                                struct radv_device *device,
+                               VkFormat vk_format,
                                const VkComponentMapping *components,
-                               bool is_storage_image)
+                               bool is_storage_image, bool disable_compression,
+                               unsigned plane_id, unsigned descriptor_plane_id)
 {
        struct radv_image *image = iview->image;
+       struct radv_image_plane *plane = &image->planes[plane_id];
+       const struct vk_format_description *format_desc = vk_format_description(image->vk_format);
        bool is_stencil = iview->aspect_mask == VK_IMAGE_ASPECT_STENCIL_BIT;
        uint32_t blk_w;
-       uint32_t *descriptor;
+       union radv_descriptor *descriptor;
        uint32_t hw_level = 0;
 
        if (is_storage_image) {
-               descriptor = iview->storage_descriptor;
+               descriptor = &iview->storage_descriptor;
        } else {
-               descriptor = iview->descriptor;
+               descriptor = &iview->descriptor;
        }
 
-       assert(image->surface.blk_w % vk_format_get_blockwidth(image->vk_format) == 0);
-       blk_w = image->surface.blk_w / vk_format_get_blockwidth(image->vk_format) * vk_format_get_blockwidth(iview->vk_format);
+       assert(vk_format_get_plane_count(vk_format) == 1);
+       assert(plane->surface.blk_w % vk_format_get_blockwidth(plane->format) == 0);
+       blk_w = plane->surface.blk_w / vk_format_get_blockwidth(plane->format) * vk_format_get_blockwidth(vk_format);
 
        if (device->physical_device->rad_info.chip_class >= GFX9)
                hw_level = iview->base_mip;
-       si_make_texture_descriptor(device, image, is_storage_image,
-                                  iview->type,
-                                  iview->vk_format,
-                                  components,
-                                  hw_level, hw_level + iview->level_count - 1,
-                                  iview->base_layer,
-                                  iview->base_layer + iview->layer_count - 1,
-                                  iview->extent.width,
-                                  iview->extent.height,
-                                  iview->extent.depth,
-                                  descriptor,
-                                  descriptor + 8);
+       radv_make_texture_descriptor(device, image, is_storage_image,
+                                    iview->type,
+                                    vk_format,
+                                    components,
+                                    hw_level, hw_level + iview->level_count - 1,
+                                    iview->base_layer,
+                                    iview->base_layer + iview->layer_count - 1,
+                                    iview->extent.width  / (plane_id ? format_desc->width_divisor : 1),
+                                    iview->extent.height  / (plane_id ? format_desc->height_divisor : 1),
+                                    iview->extent.depth,
+                                    descriptor->plane_descriptors[descriptor_plane_id],
+                                    descriptor_plane_id ? NULL : descriptor->fmask_descriptor);
 
        const struct legacy_surf_level *base_level_info = NULL;
        if (device->physical_device->rad_info.chip_class <= GFX9) {
                if (is_stencil)
-                       base_level_info = &image->surface.u.legacy.stencil_level[iview->base_mip];
+                       base_level_info = &plane->surface.u.legacy.stencil_level[iview->base_mip];
                else
-                       base_level_info = &image->surface.u.legacy.level[iview->base_mip];
+                       base_level_info = &plane->surface.u.legacy.level[iview->base_mip];
        }
        si_set_mutable_tex_desc_fields(device, image,
                                       base_level_info,
+                                      plane_id,
                                       iview->base_mip,
                                       iview->base_mip,
-                                      blk_w, is_stencil, is_storage_image, descriptor);
+                                      blk_w, is_stencil, is_storage_image,
+                                      is_storage_image || disable_compression,
+                                      descriptor->plane_descriptors[descriptor_plane_id]);
+}
+
+static unsigned
+radv_plane_from_aspect(VkImageAspectFlags mask)
+{
+       switch(mask) {
+       case VK_IMAGE_ASPECT_PLANE_1_BIT:
+               return 1;
+       case VK_IMAGE_ASPECT_PLANE_2_BIT:
+               return 2;
+       default:
+               return 0;
+       }
+}
+
+VkFormat
+radv_get_aspect_format(struct radv_image *image, VkImageAspectFlags mask)
+{
+       switch(mask) {
+       case VK_IMAGE_ASPECT_PLANE_0_BIT:
+               return image->planes[0].format;
+       case VK_IMAGE_ASPECT_PLANE_1_BIT:
+               return image->planes[1].format;
+       case VK_IMAGE_ASPECT_PLANE_2_BIT:
+               return image->planes[2].format;
+       case VK_IMAGE_ASPECT_STENCIL_BIT:
+               return vk_format_stencil_only(image->vk_format);
+       case VK_IMAGE_ASPECT_DEPTH_BIT:
+               return vk_format_depth_only(image->vk_format);
+       case VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_STENCIL_BIT:
+               return vk_format_depth_only(image->vk_format);
+       default:
+               return image->vk_format;
+       }
 }
 
 void
 radv_image_view_init(struct radv_image_view *iview,
                     struct radv_device *device,
-                    const VkImageViewCreateInfo* pCreateInfo)
+                    const VkImageViewCreateInfo* pCreateInfo,
+                    const struct radv_image_view_extra_create_info* extra_create_info)
 {
        RADV_FROM_HANDLE(radv_image, image, pCreateInfo->image);
        const VkImageSubresourceRange *range = &pCreateInfo->subresourceRange;
@@ -1134,8 +1566,16 @@ radv_image_view_init(struct radv_image_view *iview,
        iview->image = image;
        iview->bo = image->bo;
        iview->type = pCreateInfo->viewType;
-       iview->vk_format = pCreateInfo->format;
+       iview->plane_id = radv_plane_from_aspect(pCreateInfo->subresourceRange.aspectMask);
        iview->aspect_mask = pCreateInfo->subresourceRange.aspectMask;
+       iview->multiple_planes = vk_format_get_plane_count(image->vk_format) > 1 && iview->aspect_mask == VK_IMAGE_ASPECT_COLOR_BIT;
+
+       iview->vk_format = pCreateInfo->format;
+
+       /* If the image has an Android external format, pCreateInfo->format will be
+        * VK_FORMAT_UNDEFINED. */
+       if (iview->vk_format == VK_FORMAT_UNDEFINED)
+               iview->vk_format = image->vk_format;
 
        if (iview->aspect_mask == VK_IMAGE_ASPECT_STENCIL_BIT) {
                iview->vk_format = vk_format_stencil_only(iview->vk_format);
@@ -1157,7 +1597,7 @@ radv_image_view_init(struct radv_image_view *iview,
                };
        }
 
-       if (iview->vk_format != image->vk_format) {
+       if (iview->vk_format != image->planes[iview->plane_id].format) {
                unsigned view_bw = vk_format_get_blockwidth(iview->vk_format);
                unsigned view_bh = vk_format_get_blockheight(iview->vk_format);
                unsigned img_bw = vk_format_get_blockwidth(image->vk_format);
@@ -1202,8 +1642,8 @@ radv_image_view_init(struct radv_image_view *iview,
                         lvl_width <<= range->baseMipLevel;
                         lvl_height <<= range->baseMipLevel;
 
-                        iview->extent.width = CLAMP(lvl_width, iview->extent.width, iview->image->surface.u.gfx9.surf_pitch);
-                        iview->extent.height = CLAMP(lvl_height, iview->extent.height, iview->image->surface.u.gfx9.surf_height);
+                        iview->extent.width = CLAMP(lvl_width, iview->extent.width, iview->image->planes[0].surface.u.gfx9.surf_pitch);
+                        iview->extent.height = CLAMP(lvl_height, iview->extent.height, iview->image->planes[0].surface.u.gfx9.surf_height);
                 }
        }
 
@@ -1212,45 +1652,63 @@ radv_image_view_init(struct radv_image_view *iview,
        iview->base_mip = range->baseMipLevel;
        iview->level_count = radv_get_levelCount(image, range);
 
-       radv_image_view_make_descriptor(iview, device, &pCreateInfo->components, false);
-       radv_image_view_make_descriptor(iview, device, &pCreateInfo->components, true);
-}
-
-bool radv_layout_has_htile(const struct radv_image *image,
-                           VkImageLayout layout,
-                           unsigned queue_mask)
-{
-       if (radv_image_is_tc_compat_htile(image))
-               return layout != VK_IMAGE_LAYOUT_GENERAL;
-
-       return radv_image_has_htile(image) &&
-              (layout == VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL ||
-               (layout == VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL &&
-                queue_mask == (1u << RADV_QUEUE_GENERAL)));
+       bool disable_compression = extra_create_info ? extra_create_info->disable_compression: false;
+       for (unsigned i = 0; i < (iview->multiple_planes ? vk_format_get_plane_count(image->vk_format) : 1); ++i) {
+               VkFormat format = vk_format_get_plane_format(iview->vk_format, i);
+               radv_image_view_make_descriptor(iview, device, format,
+                                               &pCreateInfo->components,
+                                               false, disable_compression,
+                                               iview->plane_id + i, i);
+               radv_image_view_make_descriptor(iview, device,
+                                               format, &pCreateInfo->components,
+                                               true, disable_compression,
+                                               iview->plane_id + i, i);
+       }
 }
 
 bool radv_layout_is_htile_compressed(const struct radv_image *image,
                                      VkImageLayout layout,
+                                    bool in_render_loop,
                                      unsigned queue_mask)
 {
-       if (radv_image_is_tc_compat_htile(image))
+       if (radv_image_is_tc_compat_htile(image)) {
+               if (layout == VK_IMAGE_LAYOUT_GENERAL &&
+                   !in_render_loop &&
+                   !(image->usage & VK_IMAGE_USAGE_STORAGE_BIT)) {
+                       /* It should be safe to enable TC-compat HTILE with
+                        * VK_IMAGE_LAYOUT_GENERAL if we are not in a render
+                        * loop and if the image doesn't have the storage bit
+                        * set. This improves performance for apps that use
+                        * GENERAL for the main depth pass because this allows
+                        * compression and this reduces the number of
+                        * decompressions from/to GENERAL.
+                        */
+                       return true;
+               }
+
                return layout != VK_IMAGE_LAYOUT_GENERAL;
+       }
 
        return radv_image_has_htile(image) &&
               (layout == VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL ||
+               layout == VK_IMAGE_LAYOUT_DEPTH_ATTACHMENT_OPTIMAL_KHR ||
+               layout == VK_IMAGE_LAYOUT_STENCIL_ATTACHMENT_OPTIMAL_KHR ||
                (layout == VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL &&
                 queue_mask == (1u << RADV_QUEUE_GENERAL)));
 }
 
 bool radv_layout_can_fast_clear(const struct radv_image *image,
                                VkImageLayout layout,
+                               bool in_render_loop,
                                unsigned queue_mask)
 {
        return layout == VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
 }
 
-bool radv_layout_dcc_compressed(const struct radv_image *image,
+bool radv_layout_dcc_compressed(const struct radv_device *device,
+                               const struct radv_image *image,
                                VkImageLayout layout,
+                               bool in_render_loop,
                                unsigned queue_mask)
 {
        /* Don't compress compute transfer dst, as image stores are not supported. */
@@ -1266,7 +1724,8 @@ unsigned radv_image_queue_family_mask(const struct radv_image *image, uint32_t f
 {
        if (!image->exclusive)
                return image->queue_family_mask;
-       if (family == VK_QUEUE_FAMILY_EXTERNAL)
+       if (family == VK_QUEUE_FAMILY_EXTERNAL ||
+           family == VK_QUEUE_FAMILY_FOREIGN_EXT)
                return (1u << RADV_MAX_QUEUE_FAMILIES) - 1u;
        if (family == VK_QUEUE_FAMILY_IGNORED)
                return 1u << queue_family;
@@ -1311,13 +1770,7 @@ radv_DestroyImage(VkDevice _device, VkImage _image,
        if (!image)
                return;
 
-       if (image->flags & VK_IMAGE_CREATE_SPARSE_BINDING_BIT)
-               device->ws->buffer_destroy(image->bo);
-
-       if (image->owned_memory != VK_NULL_HANDLE)
-               radv_FreeMemory(_device, image->owned_memory, pAllocator);
-
-       vk_free2(&device->alloc, pAllocator, image);
+       radv_destroy_image(device, pAllocator, image);
 }
 
 void radv_GetImageSubresourceLayout(
@@ -1330,18 +1783,38 @@ void radv_GetImageSubresourceLayout(
        RADV_FROM_HANDLE(radv_device, device, _device);
        int level = pSubresource->mipLevel;
        int layer = pSubresource->arrayLayer;
-       struct radeon_surf *surface = &image->surface;
+
+       unsigned plane_id = radv_plane_from_aspect(pSubresource->aspectMask);
+
+       struct radv_image_plane *plane = &image->planes[plane_id];
+       struct radeon_surf *surface = &plane->surface;
 
        if (device->physical_device->rad_info.chip_class >= GFX9) {
-               pLayout->offset = surface->u.gfx9.offset[level] + surface->u.gfx9.surf_slice_size * layer;
-               pLayout->rowPitch = surface->u.gfx9.surf_pitch * surface->bpe;
+               uint64_t level_offset = surface->is_linear ? surface->u.gfx9.offset[level] : 0;
+               
+               pLayout->offset = plane->offset + level_offset + surface->u.gfx9.surf_slice_size * layer;
+               if (image->vk_format == VK_FORMAT_R32G32B32_UINT ||
+                   image->vk_format == VK_FORMAT_R32G32B32_SINT ||
+                   image->vk_format == VK_FORMAT_R32G32B32_SFLOAT) {
+                       /* Adjust the number of bytes between each row because
+                        * the pitch is actually the number of components per
+                        * row.
+                        */
+                       pLayout->rowPitch = surface->u.gfx9.surf_pitch * surface->bpe / 3;
+               } else {
+                       uint32_t pitch = surface->is_linear ? surface->u.gfx9.pitch[level] : surface->u.gfx9.surf_pitch;
+
+                       assert(util_is_power_of_two_nonzero(surface->bpe));
+                       pLayout->rowPitch = pitch * surface->bpe;
+               }
+
                pLayout->arrayPitch = surface->u.gfx9.surf_slice_size;
                pLayout->depthPitch = surface->u.gfx9.surf_slice_size;
                pLayout->size = surface->u.gfx9.surf_slice_size;
                if (image->type == VK_IMAGE_TYPE_3D)
                        pLayout->size *= u_minify(image->info.depth, level);
        } else {
-               pLayout->offset = surface->u.legacy.level[level].offset + (uint64_t)surface->u.legacy.level[level].slice_size_dw * 4 * layer;
+               pLayout->offset = plane->offset + surface->u.legacy.level[level].offset + (uint64_t)surface->u.legacy.level[level].slice_size_dw * 4 * layer;
                pLayout->rowPitch = surface->u.legacy.level[level].nblk_x * surface->bpe;
                pLayout->arrayPitch = (uint64_t)surface->u.legacy.level[level].slice_size_dw * 4;
                pLayout->depthPitch = (uint64_t)surface->u.legacy.level[level].slice_size_dw * 4;
@@ -1361,12 +1834,15 @@ radv_CreateImageView(VkDevice _device,
        RADV_FROM_HANDLE(radv_device, device, _device);
        struct radv_image_view *view;
 
-       view = vk_alloc2(&device->alloc, pAllocator, sizeof(*view), 8,
+       view = vk_alloc2(&device->vk.alloc, pAllocator, sizeof(*view), 8,
                           VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
        if (view == NULL)
                return vk_error(device->instance, VK_ERROR_OUT_OF_HOST_MEMORY);
 
-       radv_image_view_init(view, device, pCreateInfo);
+       vk_object_base_init(&device->vk, &view->base,
+                           VK_OBJECT_TYPE_IMAGE_VIEW);
+
+       radv_image_view_init(view, device, pCreateInfo, NULL);
 
        *pView = radv_image_view_to_handle(view);
 
@@ -1382,7 +1858,9 @@ radv_DestroyImageView(VkDevice _device, VkImageView _iview,
 
        if (!iview)
                return;
-       vk_free2(&device->alloc, pAllocator, iview);
+
+       vk_object_base_finish(&iview->base);
+       vk_free2(&device->vk.alloc, pAllocator, iview);
 }
 
 void radv_buffer_view_init(struct radv_buffer_view *view,
@@ -1409,11 +1887,14 @@ radv_CreateBufferView(VkDevice _device,
        RADV_FROM_HANDLE(radv_device, device, _device);
        struct radv_buffer_view *view;
 
-       view = vk_alloc2(&device->alloc, pAllocator, sizeof(*view), 8,
+       view = vk_alloc2(&device->vk.alloc, pAllocator, sizeof(*view), 8,
                           VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
        if (!view)
                return vk_error(device->instance, VK_ERROR_OUT_OF_HOST_MEMORY);
 
+       vk_object_base_init(&device->vk, &view->base,
+                           VK_OBJECT_TYPE_BUFFER_VIEW);
+
        radv_buffer_view_init(view, device, pCreateInfo);
 
        *pView = radv_buffer_view_to_handle(view);
@@ -1431,5 +1912,6 @@ radv_DestroyBufferView(VkDevice _device, VkBufferView bufferView,
        if (!view)
                return;
 
-       vk_free2(&device->alloc, pAllocator, view);
+       vk_object_base_finish(&view->base);
+       vk_free2(&device->vk.alloc, pAllocator, view);
 }