Re: [Piglit] [PATCH 2/2] Removed Glean read pixels sanity test.

2014-10-21 Thread Michel Dänzer
On 22.10.2014 06:27, Laura Ekstrand wrote: While we're at it, should I update r500.py and r300.py to use the new Piglit blendFunc test that I wrote? I'm not sure how to do that because r300.py and r500.py are setting the GLEAN_BLEND_*_TOLERANCE variables, and I haven't seen similar functionalit

[Piglit] [PATCH 1/1] cl: add log10 test

2014-10-21 Thread Jan Vesely
Signed-off-by: Jan Vesely --- passes on r600 (with posted libclc patch) and intel cpu ocl generated_tests/generate-cl-math-builtins.py | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/generated_tests/generate-cl-math-builtins.py b/generated_tests/generate-cl-mat

Re: [Piglit] [PATCH 2/2] Removed Glean read pixels sanity test.

2014-10-21 Thread Laura Ekstrand
While we're at it, should I update r500.py and r300.py to use the new Piglit blendFunc test that I wrote? I'm not sure how to do that because r300.py and r500.py are setting the GLEAN_BLEND_*_TOLERANCE variables, and I haven't seen similar functionality in Piglit yet. Maybe something involving p

[Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Laura Ekstrand
--- tests/sanity.py | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/sanity.py b/tests/sanity.py index db07447..4d21880 100644 --- a/tests/sanity.py +++ b/tests/sanity.py @@ -3,10 +3,11 @@ # from framework.profile import TestProfile -from framework.test import G

Re: [Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Dylan Baker
On Tuesday, October 21, 2014 03:19:41 PM Ilia Mirkin wrote: > On Tue, Oct 21, 2014 at 3:11 PM, Dylan Baker wrote: > > On Tuesday, October 21, 2014 02:39:06 PM Ilia Mirkin wrote: > >> On Tue, Oct 21, 2014 at 2:35 PM, Dylan Baker > >> wrote: > >> > Hi Laura, > >> > > >> > You should rebase your pi

Re: [Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Ilia Mirkin
On Tue, Oct 21, 2014 at 3:11 PM, Dylan Baker wrote: > On Tuesday, October 21, 2014 02:39:06 PM Ilia Mirkin wrote: >> On Tue, Oct 21, 2014 at 2:35 PM, Dylan Baker wrote: >> > Hi Laura, >> > >> > You should rebase your piglit work, this patch wont apply to master, as >> > I landed some changes to g

Re: [Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Dylan Baker
On Tuesday, October 21, 2014 02:39:06 PM Ilia Mirkin wrote: > On Tue, Oct 21, 2014 at 2:35 PM, Dylan Baker wrote: > > Hi Laura, > > > > You should rebase your piglit work, this patch wont apply to master, as > > I landed some changes to get rid of the need for a bunch of the test > > wrappers. The

Re: [Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Ilia Mirkin
On Tue, Oct 21, 2014 at 2:35 PM, Dylan Baker wrote: > Hi Laura, > > You should rebase your piglit work, this patch wont apply to master, as > I landed some changes to get rid of the need for a bunch of the test > wrappers. The big change is that the -auto and -fbo stuff is added > automatically by

Re: [Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Dylan Baker
Hi Laura, You should rebase your piglit work, this patch wont apply to master, as I landed some changes to get rid of the need for a bunch of the test wrappers. The big change is that the -auto and -fbo stuff is added automatically by the class now. It's also worth noting that the nested dictiona

[Piglit] [PATCH 3/6] README: Update result statuses

2014-10-21 Thread Dylan Baker
Adds the crash, timeout and the dmesg-* statuses. Signed-off-by: Dylan Baker --- README | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/README b/README index 055e8e4..e302ec3 100644 --- a/README +++ b/README @@ -217,21 +217,22 @@ Have a look at the r

[Piglit] [PATCH 1/6] README: Add simple-json and lxml as optional dependencies

2014-10-21 Thread Dylan Baker
These are accelerated libraries that the python framework can optionally use to run faster. Signed-off-by: Dylan Baker --- README | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/README b/README index 002ddd9..74bdc1d 100644 --- a/README +++ b/README @@ -36,8 +36,14

[Piglit] [PATCH 2/6] README: use the piglit command not piglit-* commands

2014-10-21 Thread Dylan Baker
These are old and maintained as compatibility options, modern development should be happening in the unified piglit command anyway. Signed-off-by: Dylan Baker --- README | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/README b/README index 74bdc1d..055e8e4 1006

[Piglit] [PATCH 6/6] README: Remove Todo section

2014-10-21 Thread Dylan Baker
This section has either been long since completed or is outside the scope of the piglit project. Signed-off-by: Dylan Baker --- README | 16 1 file changed, 16 deletions(-) diff --git a/README b/README index 62fbec0..619bfeb 100644 --- a/README +++ b/README @@ -346,19 +346,3 @@

[Piglit] [PATCH 4/6] README: Update the list of test profiles

2014-10-21 Thread Dylan Baker
Separate out the OpenCL and OpenGL profiles into different sections, and add a section for integration tests. This doesn't include es3conform and oglconform since I don't think that either of them actually work at this point. Signed-off-by: Dylan Baker --- README | 34 +++

[Piglit] [PATCH 5/6] README: update test classes information

2014-10-21 Thread Dylan Baker
Signed-off-by: Dylan Baker --- README | 57 ++--- 1 file changed, 42 insertions(+), 15 deletions(-) diff --git a/README b/README index 6d7b328..62fbec0 100644 --- a/README +++ b/README @@ -297,28 +297,55 @@ Every test is run as a separate proce

Re: [Piglit] [PATCH 2/2] Removed Glean read pixels sanity test.

2014-10-21 Thread Laura Ekstrand
Ok, I just sent a patch to update sanity.py. I'm not really experienced in Python, so let me know if you have any problems with that patch. It ran on my machine. Laura On Mon, Oct 20, 2014 at 8:48 PM, Michel Dänzer wrote: > On 21.10.2014 07:49, Laura Ekstrand wrote: > >> Thanks for the catch.

[Piglit] [PATCH] Updated sanity.py to reflect new Piglit version of the read pixels sanity test.

2014-10-21 Thread Laura Ekstrand
--- tests/sanity.py | 32 +--- 1 file changed, 29 insertions(+), 3 deletions(-) diff --git a/tests/sanity.py b/tests/sanity.py index 0e0e038..0a3c49c 100644 --- a/tests/sanity.py +++ b/tests/sanity.py @@ -3,10 +3,36 @@ # from framework.profile import TestProfile -f

[Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary() v3

2014-10-21 Thread Tom Stellard
v2: - Fix filename in doxygen comments - Remove redundant binary status check. v3: - Add test cases for passing binary programs to clBuildProgram() and clCompileProgram(). --- tests/cl.py | 1 + tests/cl/api/CMakeLists.cl.txt| 1 + tests/cl/

[Piglit] [PATCH] cl: Add test for clCreateProgramWithBinary() v2

2014-10-21 Thread Tom Stellard
v2: - Fix filename in doxygen comments - Remove redundant binary status check. --- tests/cl.py | 1 + tests/cl/api/CMakeLists.cl.txt| 1 + tests/cl/api/create-program-with-binary.c | 177 ++ 3 files changed, 179 insertio