Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net>

On Mon, Mar 11, 2019 at 12:34 PM Juan A. Suarez Romero <jasua...@igalia.com>
wrote:

> As stated in Vulkan spec:
>    "Resetting a descriptor pool recycles all of the resources from all
>     of the descriptor sets allocated from the descriptor pool back to
>     the descriptor pool, and the descriptor sets are implicitly freed."
>
> This fixes dEQP-VK.api.descriptor_pool.*
>
> Fixes: 14f6275c92f1 ("anv/descriptor_set: add reference counting for
> descriptor set layouts")
> CC: Tapani Pälli <tapani.pa...@intel.com>
> CC: Lionel Landwerlin <lionel.g.landwer...@intel.com>
> CC: Jason Ekstrand <ja...@jlekstrand.net>
> ---
>  src/intel/vulkan/anv_descriptor_set.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/src/intel/vulkan/anv_descriptor_set.c
> b/src/intel/vulkan/anv_descriptor_set.c
> index f293cf469ee..f34a44aefd7 100644
> --- a/src/intel/vulkan/anv_descriptor_set.c
> +++ b/src/intel/vulkan/anv_descriptor_set.c
> @@ -636,6 +636,12 @@ VkResult anv_ResetDescriptorPool(
>     }
>
>     anv_state_stream_finish(&pool->surface_state_stream);
> +
> +   list_for_each_entry_safe(struct anv_descriptor_set, set,
> +                            &pool->desc_sets, pool_link) {
> +      anv_descriptor_set_destroy(device, pool, set);
> +   }
> +
>     anv_state_stream_init(&pool->surface_state_stream,
>                           &device->surface_state_pool, 4096);
>     pool->surface_state_free_list = NULL;
> --
> 2.20.1
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to