Re: [Mesa-dev] [PATCH] radeonsi: Always reserve space for tessellation factors.

2016-05-27 Thread Dave Airlie
On 27 May 2016 at 22:40, Bas Nieuwenhuizen wrote: > Reviewed-by: Bas Nieuwenhuizen Tested-by: Dave Airlie Passes the CTS test now. Dave. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] radeonsi: Always reserve space for tessellation factors.

2016-05-27 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Fri, May 27, 2016 at 12:40 PM, Marek Olšák wrote: > And how about the attached patch? > > Marek > > On Fri, May 27, 2016 at 10:08 AM, Bas Nieuwenhuizen > wrote: >> Those are always read for writing to the TF ring. >> >> Should fix CTS test >> GL45-CTS.shader_im

Re: [Mesa-dev] [PATCH] radeonsi: Always reserve space for tessellation factors.

2016-05-27 Thread Marek Olšák
And how about the attached patch? Marek On Fri, May 27, 2016 at 10:08 AM, Bas Nieuwenhuizen wrote: > Those are always read for writing to the TF ring. > > Should fix CTS test > GL45-CTS.shader_image_load_store.multiple-uniforms > after a regression due to the new tessellation code. > > Signed-of

[Mesa-dev] [PATCH] radeonsi: Always reserve space for tessellation factors.

2016-05-27 Thread Bas Nieuwenhuizen
Those are always read for writing to the TF ring. Should fix CTS test GL45-CTS.shader_image_load_store.multiple-uniforms after a regression due to the new tessellation code. Signed-off-by: Bas Nieuwenhuizen --- I have no CTS, so it actually is not tested whether it fixes this test. src/galli