Re: [Mesa-dev] [PATCH 1/3] r600g: increase array size for shader inputs and outputs

2013-05-02 Thread Marek Olšák
glsl-max-varyings uses 33 vertex shader outputs, but you need to be lucky to see any difference. In my case, there was a weird assertion failure during command buffer generation for a vertex shader (valgrind didn't help). I found the cause by moving the input and output arrays to the end of r600_sh

Re: [Mesa-dev] [PATCH 1/3] r600g: increase array size for shader inputs and outputs

2013-05-02 Thread Michel Dänzer
On Don, 2013-05-02 at 05:45 +0200, Marek Olšák wrote: > and add assertions to prevent buffer overflow. This fixes corruption > of the r600_shader struct. Any pointers to tests exercising this? I noticed just yesterday that the corresponding code in radeonsi seems a bit wonky... -- Earthling Mi

[Mesa-dev] [PATCH 1/3] r600g: increase array size for shader inputs and outputs

2013-05-01 Thread Marek Olšák
and add assertions to prevent buffer overflow. This fixes corruption of the r600_shader struct. NOTE: This is a candidate for the stable branches. --- src/gallium/drivers/r600/r600_shader.c |2 ++ src/gallium/drivers/r600/r600_shader.h |4 ++-- 2 files changed, 4 insertions(+), 2 deletion