[Mesa-dev] [RFC 20/22] RFC: anv: Drop unneeded struct anv_image_create_info

Daniel Stone daniels at collabora.com
Thu Jun 8 18:44:18 UTC 2017


From: Chad Versace <chadversary at chromium.org>

Now that anv_image_create() is gone, there is no need for struct
anv_image_create_info. The struct just adds an unused layer of
indirection.

Signed-off-by: Daniel Stone <daniels at collabora.com>
---
 src/intel/vulkan/anv_image.c   | 38 +++++++++++---------------------------
 src/intel/vulkan/anv_private.h |  9 ---------
 2 files changed, 11 insertions(+), 36 deletions(-)

diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c
index 0002e4cd9c..708f233015 100644
--- a/src/intel/vulkan/anv_image.c
+++ b/src/intel/vulkan/anv_image.c
@@ -107,7 +107,7 @@ get_surface(struct anv_image *image, VkImageAspectFlags aspect)
 }
 
 static isl_tiling_flags_t
-choose_isl_tiling_flags(const struct anv_image_create_info *anv_info,
+choose_isl_tiling_flags(const VkImageCreateInfo *base_info,
                         const VkImportImageDmaBufInfoMESAX *import_dma_buf_info,
                         const VkExportImageDmaBufInfoMESAX *export_dma_buf_info)
 {
@@ -131,15 +131,12 @@ choose_isl_tiling_flags(const struct anv_image_create_info *anv_info,
             flags |= 1 << t;
          }
       }
-   } else if (anv_info->vk_info->tiling == VK_IMAGE_TILING_LINEAR) {
+   } else if (base_info->tiling == VK_IMAGE_TILING_LINEAR) {
       flags = ISL_TILING_LINEAR_BIT;
    } else {
       flags = ISL_TILING_ANY_MASK;
    }
 
-   if (anv_info->isl_tiling_flags)
-      flags &= anv_info->isl_tiling_flags;
-
    assert(flags != 0);
 
    return flags;
@@ -167,22 +164,6 @@ choose_isl_format(const struct anv_device *dev,
    }
 }
 
-static uint32_t
-choose_row_pitch(const struct anv_image_create_info *anv_info,
-                 const VkImportImageDmaBufPlaneInfoMESAX *plane_info)
-{
-   if (anv_info->stride != 0)
-      return anv_info->stride;
-
-   if (plane_info) {
-      assert(plane_info->rowPitch > 0);
-      return plane_info->rowPitch;
-   }
-
-   /* Let isl choose the pitch. */
-   return 0;
-}
-
 static void
 set_min_surface_offset(const struct anv_image *image, struct anv_surface *surf)
 {
@@ -330,14 +311,13 @@ make_aux_surface_maybe(const struct anv_device *dev,
  */
 static void
 make_main_surface(const struct anv_device *dev,
-                  const struct anv_image_create_info *anv_info,
+                  const VkImageCreateInfo *base_info,
                   const VkImportImageDmaBufInfoMESAX *import_dma_buf_info,
                   const VkExportImageDmaBufInfoMESAX *export_dma_buf_info,
                   VkImageAspectFlags aspect,
                   VkExternalMemoryHandleTypeFlagsKHX handle_types,
                   struct anv_image *image)
 {
-   const VkImageCreateInfo *base_info = anv_info->vk_info;
    bool ok UNUSED;
 
    static const enum isl_surf_dim vk_to_isl_surf_dim[] = {
@@ -351,8 +331,13 @@ make_main_surface(const struct anv_device *dev,
       plane_info = &import_dma_buf_info->pPlanes[0];
 
    const isl_tiling_flags_t tiling_flags =
-      choose_isl_tiling_flags(anv_info, import_dma_buf_info, export_dma_buf_info);
-   const uint32_t row_pitch = choose_row_pitch(anv_info, plane_info);
+      choose_isl_tiling_flags(base_info, import_dma_buf_info, export_dma_buf_info);
+
+   uint32_t row_pitch = 0;
+   if (plane_info) {
+      assert(plane_info->rowPitch > 0);
+      row_pitch = plane_info->rowPitch;
+   }
 
    struct anv_surface *anv_surf = get_surface(image, aspect);
 
@@ -398,7 +383,6 @@ anv_CreateImage(VkDevice _device,
                 VkImage *pImage)
 {
    ANV_FROM_HANDLE(anv_device, device, _device);
-   const struct anv_image_create_info anv_info = { .vk_info = base_info };
    const VkImportImageDmaBufInfoMESAX *import_dma_buf_info = NULL;
    const VkExportImageDmaBufInfoMESAX *export_dma_buf_info = NULL;
    VkExternalMemoryHandleTypeFlagsKHX handle_types = 0;
@@ -456,7 +440,7 @@ anv_CreateImage(VkDevice _device,
    uint32_t b;
    for_each_bit(b, image->aspects) {
       VkImageAspectFlagBits aspect = 1 << b;
-      make_main_surface(device, &anv_info, import_dma_buf_info,
+      make_main_surface(device, base_info, import_dma_buf_info,
                         export_dma_buf_info, aspect, handle_types, image);
       make_aux_surface_maybe(device, base_info, aspect, handle_types, image);
    }
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index 57f2cff197..ce97a970fd 100644
--- a/src/intel/vulkan/anv_private.h
+++ b/src/intel/vulkan/anv_private.h
@@ -2164,15 +2164,6 @@ struct anv_image_view {
    struct brw_image_param storage_image_param;
 };
 
-struct anv_image_create_info {
-   const VkImageCreateInfo *vk_info;
-
-   /** An opt-in bitmask which filters an ISL-mapping of the Vulkan tiling. */
-   isl_tiling_flags_t isl_tiling_flags;
-
-   uint32_t stride;
-};
-
 const struct anv_surface *
 anv_image_get_surface_for_aspect_mask(const struct anv_image *image,
                                       VkImageAspectFlags aspect_mask);
-- 
2.13.0



More information about the mesa-dev mailing list