Re: [Mesa-dev] [PATCH] radv: don't assert on empty hash table

2017-08-29 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Sun, Aug 27, 2017, at 23:44, Grazvydas Ignotas wrote: > Currently if table_size is 0, it's falling through to: > > unreachable("hash table should never be full"); > > But table_size can be 0 when RADV_DEBUG=nocache is set, or when the > table allocation fails (

[Mesa-dev] [PATCH] radv: don't assert on empty hash table

2017-08-27 Thread Grazvydas Ignotas
Currently if table_size is 0, it's falling through to: unreachable("hash table should never be full"); But table_size can be 0 when RADV_DEBUG=nocache is set, or when the table allocation fails (which is not considered an error). Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan d