Re: [Mesa-dev] [PATCH] radv: Don't expose heaps with 0 memory.

2017-11-07 Thread Bas Nieuwenhuizen
Hi Andres, It is a fix, but yeah the changes are pretty extensive. I don't have a strong opinion on this one. - Bas On Fri, Nov 3, 2017 at 4:10 PM, Andres Gomez wrote: > Bas, this commit landed without mentioning any specific stable queue. > > For 17.2, this is quite an extensive change and has

Re: [Mesa-dev] [PATCH] radv: Don't expose heaps with 0 memory.

2017-11-03 Thread Andres Gomez
Bas, this commit landed without mentioning any specific stable queue. For 17.2, this is quite an extensive change and has several trivial conflicts. I think it is OK to merge if you really meant it for 17.2 too, but just double checking. Thanks! On Wed, 2017-11-01 at 20:58 +0100, Bas Nieuwenhuiz

Re: [Mesa-dev] [PATCH] radv: Don't expose heaps with 0 memory.

2017-11-01 Thread Dave Airlie
On 2 November 2017 at 05:58, Bas Nieuwenhuizen wrote: > It confuses CTS. This pregenerates the heap info into the > physical device, so we can use it for translating contiguous > indices into our "standard" ones. > > This also makes the WSI a bit smarter in case the first preferred > heap does not

[Mesa-dev] [PATCH] radv: Don't expose heaps with 0 memory.

2017-11-01 Thread Bas Nieuwenhuizen
It confuses CTS. This pregenerates the heap info into the physical device, so we can use it for translating contiguous indices into our "standard" ones. This also makes the WSI a bit smarter in case the first preferred heap does not exist. CC: --- src/amd/vulkan/radv_device.c | 135 +++