Mesa (master): anv: intel: use anv_image' s computed size for importing a BO

Lionel Landwerlin llandwerlin at kemper.freedesktop.org
Wed Oct 11 21:30:06 UTC 2017


Module: Mesa
Branch: master
Commit: e568d2bd1f148821057a8bf071ce39f342112d83
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e568d2bd1f148821057a8bf071ce39f342112d83

Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date:   Wed Oct 11 17:24:37 2017 +0100

anv: intel: use anv_image's computed size for importing a BO

Rather than relying on size = stride * height, we can rely on
anv_image's total size.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Acked-by: Daniel Stone <daniels at collabora.com>

---

 src/intel/vulkan/anv_intel.c | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/src/intel/vulkan/anv_intel.c b/src/intel/vulkan/anv_intel.c
index d4ef3beb74..9f8cd92734 100644
--- a/src/intel/vulkan/anv_intel.c
+++ b/src/intel/vulkan/anv_intel.c
@@ -49,17 +49,7 @@ VkResult anv_CreateDmaBufImageINTEL(
    if (mem == NULL)
       return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
 
-   uint64_t size = (uint64_t)pCreateInfo->strideInBytes * pCreateInfo->extent.height;
-
-   result = anv_bo_cache_import(device, &device->bo_cache,
-                                pCreateInfo->fd, size, &mem->bo);
-   if (result != VK_SUCCESS)
-      goto fail;
-
-   if (device->instance->physicalDevice.supports_48bit_addresses)
-      mem->bo->flags |= EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
-
-   anv_image_create(_device,
+   result = anv_image_create(_device,
       &(struct anv_image_create_info) {
          .isl_tiling_flags = ISL_TILING_X_BIT,
          .stride = pCreateInfo->strideInBytes,
@@ -78,8 +68,19 @@ VkResult anv_CreateDmaBufImageINTEL(
          .flags = 0,
       }},
       pAllocator, &image_h);
+   if (result != VK_SUCCESS)
+      goto fail;
 
    image = anv_image_from_handle(image_h);
+
+   result = anv_bo_cache_import(device, &device->bo_cache,
+                                pCreateInfo->fd, image->size, &mem->bo);
+   if (result != VK_SUCCESS)
+      goto fail_import;
+
+   if (device->instance->physicalDevice.supports_48bit_addresses)
+      mem->bo->flags |= EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
+
    image->planes[0].bo = mem->bo;
    image->planes[0].bo_offset = 0;
 
@@ -92,6 +93,9 @@ VkResult anv_CreateDmaBufImageINTEL(
 
    return VK_SUCCESS;
 
+ fail_import:
+   vk_free2(&device->alloc, pAllocator, image);
+
  fail:
    vk_free2(&device->alloc, pAllocator, mem);
 




More information about the mesa-commit mailing list