From: Chad Versace <chad.versace@intel.com>
Date: Wed, 15 Jul 2015 21:14:04 +0000 (-0700)
Subject: vk: Refactor create/destroy code for anv_descriptor_set
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c6270e8044782080a0c38b5290eac96caa608168;p=mesa.git

vk: Refactor create/destroy code for anv_descriptor_set

Define two new functions:
    anv_descriptor_set_create
    anv_descriptor_set_destroy
---

diff --git a/src/vulkan/device.c b/src/vulkan/device.c
index c59814fb4e6..7e230249d07 100644
--- a/src/vulkan/device.c
+++ b/src/vulkan/device.c
@@ -1251,6 +1251,9 @@ VkResult anv_DestroyObject(
       return anv_DestroyDescriptorSetLayout(_device, (VkDescriptorSetLayout) _object);
 
    case VK_OBJECT_TYPE_DESCRIPTOR_SET:
+      anv_descriptor_set_destroy(device, anv_descriptor_set_from_handle(_object));
+      return VK_SUCCESS;
+
    case VK_OBJECT_TYPE_RENDER_PASS:
       /* These are trivially destroyable */
       anv_device_free(device, (void *) _object);
@@ -2070,6 +2073,35 @@ VkResult anv_ResetDescriptorPool(
    return VK_SUCCESS;
 }
 
+VkResult
+anv_descriptor_set_create(struct anv_device *device,
+                          const struct anv_descriptor_set_layout *layout,
+                          struct anv_descriptor_set **out_set)
+{
+   struct anv_descriptor_set *set;
+   size_t size = sizeof(*set) + layout->count * sizeof(set->descriptors[0]);
+
+   set = anv_device_alloc(device, size, 8, VK_SYSTEM_ALLOC_TYPE_API_OBJECT);
+   if (!set)
+      return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
+
+   /* A descriptor set may not be 100% filled. Clear the set so we can can
+    * later detect holes in it.
+    */
+   memset(set, 0, size);
+
+   *out_set = set;
+
+   return VK_SUCCESS;
+}
+
+void
+anv_descriptor_set_destroy(struct anv_device *device,
+                           struct anv_descriptor_set *set)
+{
+   anv_device_free(device, set);
+}
+
 VkResult anv_AllocDescriptorSets(
     VkDevice                                    _device,
     VkDescriptorPool                            descriptorPool,
@@ -2080,24 +2112,19 @@ VkResult anv_AllocDescriptorSets(
     uint32_t*                                   pCount)
 {
    ANV_FROM_HANDLE(anv_device, device, _device);
+
+   VkResult result;
    struct anv_descriptor_set *set;
-   size_t size;
 
    for (uint32_t i = 0; i < count; i++) {
       ANV_FROM_HANDLE(anv_descriptor_set_layout, layout, pSetLayouts[i]);
-      size = sizeof(*set) + layout->count * sizeof(set->descriptors[0]);
-      set = anv_device_alloc(device, size, 8,
-                             VK_SYSTEM_ALLOC_TYPE_API_OBJECT);
-      if (!set) {
+
+      result = anv_descriptor_set_create(device, layout, &set);
+      if (result != VK_SUCCESS) {
          *pCount = i;
-         return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
+         return result;
       }
 
-      /* Descriptor sets may not be 100% filled out so we need to memset to
-       * ensure that we can properly detect and handle holes.
-       */
-      memset(set, 0, size);
-
       pDescriptorSets[i] = anv_descriptor_set_to_handle(set);
    }
 
diff --git a/src/vulkan/private.h b/src/vulkan/private.h
index 5693a80af3d..949962b8c04 100644
--- a/src/vulkan/private.h
+++ b/src/vulkan/private.h
@@ -608,6 +608,15 @@ struct anv_descriptor_set {
    struct anv_descriptor descriptors[0];
 };
 
+VkResult
+anv_descriptor_set_create(struct anv_device *device,
+                          const struct anv_descriptor_set_layout *layout,
+                          struct anv_descriptor_set **out_set);
+
+void
+anv_descriptor_set_destroy(struct anv_device *device,
+                           struct anv_descriptor_set *set);
+
 #define MAX_VBS   32
 #define MAX_SETS   8
 #define MAX_RTS    8