Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-25 Thread Jan Vesely
On Tue, 2015-09-22 at 09:11 -0500, Aaron Watry wrote: > On Tue, Sep 22, 2015 at 12:02 AM, Jan Vesely > wrote: > > > On Mon, 2015-09-21 at 23:07 -0500, Aaron Watry wrote: > > > On Thu, Sep 17, 2015 at 7:06 PM, Jan Vesely < > > > jan.ves...@rutgers.edu> > > > wrote: > > >

Re: [Piglit] [PATCH 00/13] Add more arb_shader_storage_buffer_object tests

2015-09-25 Thread Mark Janes
With the push of SSBO support in mesa, one of these tests continues to fail, but only on broadwell: piglit.spec.arb_shader_storage_buffer_object.array-ssbo-binding Standard Output /tmp/build_root/m64/lib/piglit/bin/arb_shader_storage_buffer_object-array-ssbo-binding -auto -fbo

Re: [Piglit] [PATCH 00/13] Add more arb_shader_storage_buffer_object tests

2015-09-25 Thread Emil Velikov
On 25 September 2015 at 16:40, Mark Janes wrote: > With the push of SSBO support in mesa, one of these tests continues to > fail, but only on broadwell: > > piglit.spec.arb_shader_storage_buffer_object.array-ssbo-binding > > Standard Output > > >

Re: [Piglit] [PATCH 00/13] Add more arb_shader_storage_buffer_object tests

2015-09-25 Thread Mark Janes
Emil Velikov writes: > On 25 September 2015 at 16:40, Mark Janes wrote: >> With the push of SSBO support in mesa, one of these tests continues to >> fail, but only on broadwell: >> >>

[Piglit] [PATCH] SSO: new test to ensure correct deadcode optimization

2015-09-25 Thread Gregory Hainaut
Related to mesa issue: https://bugs.freedesktop.org/show_bug.cgi?id=79783 "Validated" on Nvidia driver --- .../arb_separate_shader_objects/CMakeLists.gl.txt | 1 + .../rendezvous_by_name.c | 219 + 2 files changed, 220 insertions(+) create mode

Re: [Piglit] [PATCH v2] Porting the two sided stencil extension tests from Glean to Piglit.

2015-09-25 Thread Juliet Fru
Hello, I am currently having this Compilation Error with this test. Can any one give me an idea on how to fix this? Thanks, Juliet On Fri, Sep 25, 2015 at 11:03 PM, Juliet Fru wrote: > This test replaces the original glean tstencil2.cpp test. >

[Piglit] [PATCH v2] Porting the two sided stencil extension tests from Glean to Piglit.

2015-09-25 Thread Juliet Fru
This test replaces the original glean tstencil2.cpp test. --- tests/spec/gl-2.0/ two-sided-stencil.c | 743 + tests/spec/gl-2.0/CMakeLists.gl.txt| 1 + 2 files changed, 744 insertions(+) create mode 100644 tests/spec/gl-2.0/ two-sided-stencil.c diff --git

[Piglit] [PATCH v1] Remove Occluqry Glean test.

2015-09-25 Thread Juliet Fru
--- tests/all.py | 1 - tests/glean/toccluqry.cpp | 708 -- tests/glean/toccluqry.h | 75 - 3 files changed, 784 deletions(-) delete mode 100644 tests/glean/toccluqry.cpp delete mode 100644 tests/glean/toccluqry.h diff --git

Re: [Piglit] [PATCH v1] Remove Occluqry Glean test.

2015-09-25 Thread Jordan Justen
Add aedc01a5979ec5e6a4498ed3054aa67927f70ca8 reference to commit message? Reviewed-by: Jordan Justen On 2015-09-25 15:19:51, Juliet Fru wrote: > --- > tests/all.py | 1 - > tests/glean/toccluqry.cpp | 708 >

Re: [Piglit] [PATCH v2] Porting the two sided stencil extension tests from Glean to Piglit.

2015-09-25 Thread Ilia Mirkin
On Fri, Sep 25, 2015 at 7:48 PM, Dylan Baker wrote: > On Fri, Sep 25, 2015 at 11:03:05PM +0100, Juliet Fru wrote: >> This test replaces the original glean tstencil2.cpp test. >> --- >> tests/spec/gl-2.0/ two-sided-stencil.c | 743 >> + >>

Re: [Piglit] [PATCH v2] Porting the two sided stencil extension tests from Glean to Piglit.

2015-09-25 Thread Dylan Baker
On Fri, Sep 25, 2015 at 11:03:05PM +0100, Juliet Fru wrote: > This test replaces the original glean tstencil2.cpp test. > --- > tests/spec/gl-2.0/ two-sided-stencil.c | 743 > + > tests/spec/gl-2.0/CMakeLists.gl.txt| 1 + > 2 files changed, 744 insertions(+)

Re: [Piglit] [PATCH v2] Porting the two sided stencil extension tests from Glean to Piglit.

2015-09-25 Thread Brian Paul
Hi Juliet, This looks pretty good overall. Assorted comments below. Also, for Piglit we use 8-column tabs, not spaces, for indentation. Can you fix that too? Thanks! -Brian On Fri, Sep 25, 2015 at 4:03 PM, Juliet Fru wrote: > This test replaces the original glean

[Piglit] [PATCH] arb_program_interface_query: add linker test for querying varyings

2015-09-25 Thread Tapani Pälli
This fails with current Mesa due to a bug in commit: 4639cea2921669527eb43dcb49724c05afb27e8e Signed-off-by: Tapani Pälli --- .../linker/query-varyings.shader_test | 33 ++ 1 file changed, 33 insertions(+) create mode 100644

Re: [Piglit] [PATCH] arb_program_interface_query: add linker test for querying varyings

2015-09-25 Thread Ilia Mirkin
On Fri, Sep 25, 2015 at 3:09 AM, Tapani Pälli wrote: > This fails with current Mesa due to a bug in commit: > 4639cea2921669527eb43dcb49724c05afb27e8e > > Signed-off-by: Tapani Pälli > --- > .../linker/query-varyings.shader_test