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

2017-07-26 Thread sandra koroniewska
Hello, is this patch ok now? Because it hasn't been pushed. Regards, Sandra On Thu, Jul 20, 2017 at 12:12 PM, sandra koroniewska < sandra.koroniew...@gmail.com> wrote: > Hi, > if this patch is alright now, can someone push it please? > > Regards, > Sandra > > On Fri, Jun 30, 2017 at 4:45 PM,

[Piglit] [PATCH RFC] Handling missing tests

2017-07-26 Thread Arkadiusz Hiler
Hey all, This is my initial (and simple) attempt at adding a way to handle missing tests gracefully for an automated CI/bisecting/etc. scenarios. The change is okay(-ish) on it's own but we can make the missing tests case more of an first-class citizen, e.g.: 1) actually report the missing

[Piglit] [PATCH RFC] framework: Add --skip-missing option

2017-07-26 Thread Arkadiusz Hiler
Currently, if a test from provided testlist fails to be discovered by the framework, piglit blows up with an exception. This is both good - for consistency/early errors - and bad - for handling some CI/automation scenarios (e.g autobisecting the tests). So let's keep the current default, but

Re: [Piglit] [PATCH 1/1] cl: Use required format for sampler test

2017-07-26 Thread Jan Vesely
ping. Tom, Zoltan, you added the original test, does this look OK to you? thanks, Jan On Mon, 2017-07-17 at 17:47 -0400, Jan Vesely wrote: > CL_RGBA + CL_FLOAT is in minimal supported set. > Fixes sampler test on beignet (IVB and SKL). > > Signed-off-by: Jan Vesely >

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

2017-07-26 Thread Martin Peres
On 26/07/17 15:58, sandra koroniewska wrote: Hello, is this patch ok now? Because it hasn't been pushed. Regards, Sandra It still did not get any Reviewed-by. I guess the summer vacation is affecting the piglit team. ___ Piglit mailing list

Re: [Piglit] [PATCH RFC] framework: Add --skip-missing option

2017-07-26 Thread Dylan Baker
Quoting Arkadiusz Hiler (2017-07-26 00:46:21) > Currently, if a test from provided testlist fails to be discovered by > the framework, piglit blows up with an exception. Thank you for keeping the default behavior, it's important to have the "run these and exactly these or fail" option. > > This

[Piglit] [PATCH 3/3] README: Make initial smoke tests steps internally consistent

2017-07-26 Thread Rhys Kidd
No other stage of the suggested initial sanity.py test or other tests uses a folder structure with *.results. Use a consistent naming approach throughout the initial README to make initial setup clearer. Signed-off-by: Rhys Kidd --- README | 4 ++-- 1 file changed, 2

[Piglit] [PATCH 1/3] README: Update the list of test profiles

2017-07-26 Thread Rhys Kidd
Signed-off-by: Rhys Kidd --- README | 30 ++ 1 file changed, 30 insertions(+) diff --git a/README b/README index 43bdfcae7..cb49516b7 100644 --- a/README +++ b/README @@ -386,6 +386,9 @@ glslparser.py shader.py A subset of all.py which

[Piglit] [PATCH 2/3] README: Correct typo deQP -> dEQP

2017-07-26 Thread Rhys Kidd
Signed-off-by: Rhys Kidd --- README | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/README b/README index cb49516b7..56ea52b62 100644 --- a/README +++ b/README @@ -414,13 +414,13 @@ deqp_egl.py Support for running dEQP's EGL profile with

[Piglit] [PATCH 0/3] Small README clean ups

2017-07-26 Thread Rhys Kidd
Update or fix a few small items that I noticed when running through piglit setup on a new development environment. The first two should be fairly uncontroversial. The third patch is subjective. It came about when I was running through the initial suggested smoke tests as-is. It was apparent