Re: [Mesa-dev] [PATCH] nvc0: enable 32 textures on kepler+

2016-05-24 Thread Ilia Mirkin
This wasn't from compilers yelling at me, it was just from a visual audit of the relevant code which all of a sudden got moved to using 32 objects from 16. I'd be happy to revert those hunks. On Tue, May 24, 2016 at 4:05 AM, Samuel Pitoiset wrote: > The idea sounds good to me, but this (1U << i)

Re: [Mesa-dev] [PATCH] nvc0: enable 32 textures on kepler+

2016-05-24 Thread Samuel Pitoiset
The idea sounds good to me, but this (1U << i) needs consistency in my opinion. It's correct and avoid some compiler warnings (if the corresponding flags are used) but we currently don't do that. I would suggest to keep the (1 << i) for now, and maybe make a separate patch which replaces 1 by

[Mesa-dev] [PATCH] nvc0: enable 32 textures on kepler+

2016-05-23 Thread Ilia Mirkin
For fermi, this likely will require use of linked tsc mode. However on bindless architectures, we can have as many as we want. As it stands, the AUX_TEX_INFO has 32 teture handles reserved. Signed-off-by: Ilia Mirkin --- src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 2 +- src/gallium/driver