Re: [Piglit] [RFC] glean: remove pixel formats test

2017-06-29 Thread Timothy Arceri
On 30/06/17 14:49, Timothy Arceri wrote: In the bug for porting glean tests to piglit this is listed as "covered by Piglit" "Check & Remove". Since that list was made we also now have a public CTS. This is another of the extra slow glean tests. Can we assume this is all covered and remove? Fo

[Piglit] [RFC] glean: remove pixel formats test

2017-06-29 Thread Timothy Arceri
In the bug for porting glean tests to piglit this is listed as "covered by Piglit" "Check & Remove". Since that list was made we also now have a public CTS. This is another of the extra slow glean tests. Can we assume this is all covered and remove? https://bugs.freedesktop.org/show_bug.cgi?id=6

[Piglit] [PATCH] glsl-1.20: better test indirect indexing of literal array of structs

2017-06-29 Thread Timothy Arceri
This exposes some existing bugs in gallium and also provides more coverage for when we start packing uniforms. --- ...iteral-array-of-structs-vec4-member.shader_test | 39 +++ ...s-with-multiple-members-large-array.shader_test | 57 ++ ...ay-of-structs-with-multiple

Re: [Piglit] [PATCH] bugs: set KHR_no_error support for remaining bugs tests

2017-06-29 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset On 06/29/2017 07:39 AM, Timothy Arceri wrote: --- tests/bugs/crash-cubemap-order.c| 1 + tests/bugs/crash-texparameter-before-teximage.c | 1 + tests/bugs/drawbuffer-modes.c | 1 + tests/bugs/fdo10370.c

Re: [Piglit] [PATCH] gl-1.0: set KHR_no_error support for remaining gl-1.0 tests

2017-06-29 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset On 06/29/2017 08:07 AM, Timothy Arceri wrote: --- tests/spec/gl-1.0/blend.c | 1 + tests/spec/gl-1.0/dlist-bitmap.c| 1 + tests/spec/gl-1.0/dlist-shademodel.c| 1 + tests/spec/gl-1.0/drawpixels-co

Re: [Piglit] [PATCH] ARB_blend_func_extended: set KHR_no_error status

2017-06-29 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset On 06/29/2017 08:24 AM, Timothy Arceri wrote: --- .../api/bindfragdataindexed-invalid-parameters.c | 1 + tests/spec/arb_blend_func_extended/api/blend-api.c| 1 + tests/spec/arb_blend_func_extended/api/builtins.c | 1 + .../spec/arb_blend_func_e

Re: [Piglit] [PATCH] cmake: Check for linux/sync_file.h for egl_khr_fence_sync.

2017-06-29 Thread Rafael Antognolli
On Thu, Jun 29, 2017 at 05:29:06AM +, Vinson Lee wrote: > Fixes: 2e423dd3f4ca ("egl_khr_fence_sync: Add sw_sync lib.") > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101641 > Signed-off-by: Vinson Lee > --- > CMakeLists.txt |1 + > .../spec/eg

Re: [Piglit] [PATCH] gl30basic: add some extra suspected extension

2017-06-29 Thread Ilia Mirkin
On Thu, Jun 29, 2017 at 9:19 AM, Brian Paul wrote: > A couple more nit-picks below. > > On 06/29/2017 02:21 AM, Sandra Koroniewska wrote: >> >> WGL_EXT_SWAP_CONTROL is added for historical reasons. Is it WGL_EXT_SWAP_CONTROL or WGL_EXT_swap_control? I assume you tested this, so it's the latter (a

Re: [Piglit] [PATCH] gl30basic: add some extra suspected extension

2017-06-29 Thread Brian Paul
A couple more nit-picks below. On 06/29/2017 02:21 AM, Sandra Koroniewska wrote: WGL_EXT_SWAP_CONTROL is added for historical reasons. --- tests/general/gl30basic.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/general/gl30basic.c b/tests/general/gl30ba

Re: [Piglit] [PATCH] cmake: Check for linux/sync_file.h for egl_khr_fence_sync.

2017-06-29 Thread Brian Paul
On 06/28/2017 11:29 PM, Vinson Lee wrote: Fixes: 2e423dd3f4ca ("egl_khr_fence_sync: Add sw_sync lib.") Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101641 Signed-off-by: Vinson Lee --- CMakeLists.txt |1 + .../spec/egl_khr_fence_sync/CMakeList

[Piglit] [PATCH] gl30basic: add some extra suspected extension

2017-06-29 Thread Sandra Koroniewska
WGL_EXT_SWAP_CONTROL is added for historical reasons. --- tests/general/gl30basic.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/general/gl30basic.c b/tests/general/gl30basic.c index 2207787bb..92d21cd6b 100644 --- a/tests/general/gl30basic.c +++ b/tests/ge