On Thu, 2016-11-10 at 16:53 -0800, Jason Ekstrand wrote:
> When Kristian reworked descriptor set allocation, somehow he forgot
> to
> actually store the offset in the free list.  Somehow, this completely
> missed CTS testing until now... This fixes all 2744 of the new
> 'dEQP-VK.texture.filtering.* tests in the latest CTS.
> 
> Cc: "12.0 13.0" <mesa-dev@lists.freedesktop.org>
> ---
>  src/intel/vulkan/anv_descriptor_set.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/intel/vulkan/anv_descriptor_set.c
> b/src/intel/vulkan/anv_descriptor_set.c
> index 7d5a78d..52f2df6 100644
> --- a/src/intel/vulkan/anv_descriptor_set.c
> +++ b/src/intel/vulkan/anv_descriptor_set.c
> @@ -489,6 +489,7 @@ anv_descriptor_set_destroy(struct anv_device
> *device,
>        struct surface_state_free_list_entry *entry =
>           set->buffer_views[b].surface_state.map;
>        entry->next = pool->surface_state_free_list;
> +      entry->offset = set->buffer_views[b].surface_state;

This will fail to compile, I think you meant:

entry->offset = set->buffer_views[b].surface_state.offset;

With that change, both patches are:
Reviewed-by: Iago Toral Quiroga <ito...@igalia.com>

>        pool->surface_state_free_list = entry;
>     }
>  
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to