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

2017-07-20 Thread sandra koroniewska
Hi, if this patch is alright now, can someone push it please? Regards, Sandra On Fri, Jun 30, 2017 at 4:45 PM, sandra koroniewska < sandra.koroniew...@gmail.com> wrote: > "Is it WGL_EXT_SWAP_CONTROL or WGL_EXT_swap_control? I assume you > tested this, so it's the latter (as that's what's in the

Re: [Piglit] [PATCH v2] khr_texture_compression_astc: change the subtest array declaration to prevent the crash coming from Visual Studio's optimisation

2017-07-20 Thread sandra koroniewska
Hi, is this code alright now, and if so, can someone push it? The purpose of these changes was to get rid of the compound literal which resulted in a crash caused by some Visual Studio optimalisations. Regards, Sandra On Wed, Jun 21, 2017 at 5:34 PM, sandra koroniewska < sandra.koroniew...@gmail.

[Piglit] [Bug 101850] [PIGLIT] Piglit does not recognize some tests

2017-07-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101850 arkadiusz.hi...@intel.com changed: What|Removed |Added Component|tests |IGT QA Contact|pig.

[Piglit] [PATCH] framework: remove duplicate import of framework.exceptions

2017-07-20 Thread Dylan Baker
Reported-by: Matt Turner Signed-off-by: Dylan Baker --- framework/programs/run.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/framework/programs/run.py b/framework/programs/run.py index ef268f722..6444cfedb 100644 --- a/framework/programs/run.py +++ b/framework/programs/r

Re: [Piglit] [PATCH] framework: Don't stop resume because no tests are left to be run

2017-07-20 Thread Dylan Baker
Quoting Matt Turner (2017-07-19 18:17:22) > On Tue, Jul 11, 2017 at 10:27 AM, Dylan Baker wrote: > > It's completely possible that if piglit resume is run with the -n flag > > (don't retry incomplete tests), that piglit resume could be invoked with > > no tests left to run. In this case it makes s

Re: [Piglit] [PATCH] framework: remove duplicate import of framework.exceptions

2017-07-20 Thread Martin Peres
On 20/07/17 19:20, Dylan Baker wrote: Reported-by: Matt Turner Splitting the line is indeed the right approach, as it is too easy to miss elements. Thanks! Reviewed-by: Martin Peres Signed-off-by: Dylan Baker --- framework/programs/run.py | 2 +- 1 file changed, 1 insertion(+), 1 del

[Piglit] [PATCH 2/2] arb_shader_group_vote: Add simple vertex shader tests

2017-07-20 Thread Matt Turner
--- .../arb_shader_group_vote/vs-all-const.shader_test | 33 .../vs-all-uniform.shader_test | 35 ++ .../arb_shader_group_vote/vs-any-const.shader_test | 33 .../vs-any-uniform.shader_test | 35 ++

[Piglit] [PATCH 1/2] arb_shader_group_vote: Prefix tests with "cs-"

2017-07-20 Thread Matt Turner
Going to add tests for other shader stages. --- tests/spec/arb_shader_group_vote/{all.shader_test => cs-all.shader_test} | 0 tests/spec/arb_shader_group_vote/{any.shader_test => cs-any.shader_test} | 0 tests/spec/arb_shader_group_vote/{eq.shader_test => cs-eq.shader_test} | 0 3 files changed,

Re: [Piglit] [PATCH 1/1] cl: Add support for OCL 2.0

2017-07-20 Thread Aaron Watry
Hey Jan, I've skimmed my way through this, and it's looking ok so far. I'll try to do a deeper dive over the changes needed for the 2.0 spec in the near future (I've been stuck in 1.1/1.2-land for too long). For now, you can have an: Acked-by: Aaron Watry I'll see if I can find some time tomor

Re: [Piglit] [PATCH 2/2] arb_shader_group_vote: Add simple vertex shader tests

2017-07-20 Thread Timothy Arceri
On 21/07/17 09:54, Matt Turner wrote: --- .../arb_shader_group_vote/vs-all-const.shader_test | 33 .../vs-all-uniform.shader_test | 35 ++ .../arb_shader_group_vote/vs-any-const.shader_test | 33 .../vs-any-uni

Re: [Piglit] [PATCH 1/2] arb_shader_group_vote: Prefix tests with "cs-"

2017-07-20 Thread Timothy Arceri
Reviewed-by: Timothy Arceri On 21/07/17 09:54, Matt Turner wrote: Going to add tests for other shader stages. --- tests/spec/arb_shader_group_vote/{all.shader_test => cs-all.shader_test} | 0 tests/spec/arb_shader_group_vote/{any.shader_test => cs-any.shader_test} | 0 tests/spec/arb_shader