[Piglit] [PATCH v2 12/16] squash! shader_runner: add ability to run multiple tests per process

linking - Cleanup textures after each test - Simplify the logic for calling piglit_present_results Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- This patch is unchanged since v1, and was acked in v1. All further changes are in the next patch. tests/shaders/shader_runner.c

[Piglit] [PATCH v2 15/16] framework: Plug in fast-skipping to MultiShaderTest

This adds fast-skipping support to the MultiShaderTest class. Both patches seemed significant enough to warrant splitting them in two. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/test/base.py| 8 +- framework/test/shader_test.p

[Piglit] [PATCH v2 16/16] framework: add boilerplate to turn off process isolation

This adds a switch to to turn off process isolation (it is normally on), for now this only affects shader_runner, but in the future could affect more tests, like glslparsertest or dEQP. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/options.py | 1 + framework/pr

[Piglit] [PATCH v2 13/16] squash! shader_runner: add ability to run multiple tests per process

These are changes since version 1, again split out for easier review. - use glClearDepthf for GL ES instead of glClearDepth, which doesn't exist in GLES - Never attempt to disable GL_PROGRAM_POINT_SIZE in OpenGL ES (which causes failures, and I can't seem to find any evidence of it's

[Piglit] [PATCH v2 2/16] framework: Split the file parsing code out of ShaderTest

This is both nicer since it provides better code quality (a separation of roles), and because it's going to make writing a shader_test class that can do N:1 tests much easier. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/test/shader_test.py

[Piglit] [PATCH v2 8/16] util: always set piglit_is_core_profile

From: Marek Olk --- tests/util/piglit-framework-gl/piglit_glut_framework.c | 7 --- tests/util/piglit-framework-gl/piglit_wfl_framework.c | 2 ++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/util/piglit-framework-gl/piglit_glut_framework.c

[Piglit] [PATCH v2 7/16] util: guard against double context destruction

From: Marek Olk this no longer crashes: - create context (calls atexit) - teardown context - create context 2 (calls atexit) - exit() --- tests/util/piglit-framework-gl.c | 4 1 file changed, 4 insertions(+), 0 deletions(-) diff --git

[Piglit] [PATCH v2 1/16] framework/results: Lower subtest names

This makes the behavior of the Subtests class more like that of the TestDict class, and fixes a bug in MultiShaderRunner, which will be added in a subsequent patch in this series. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/results.py | 6 +++--- 1 file chan

[Piglit] [PATCH v2 0/16] Add support for shader_runner to run more than one test per process

ces, doing all of our guarding at the front of the function, and doing a better job cleaning up between tests. Dylan Baker (10): framework/results: Lower subtest names framework: Split the file parsing code out of ShaderTest framework: Split FastSkipMixin. framework: Bump JSON on disk for

[Piglit] [PATCH v2 9/16] util: rename piglit_gl_reinitialize_extensions->piglit_gl_invalidate_extensions

From: Marek Olk --- tests/util/piglit-framework-gl/piglit_wfl_framework.c | 3 ++- tests/util/piglit-util-gl.c | 2 +- tests/util/piglit-util-gl.h | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git

[Piglit] [PATCH v2 4/16] framework: Bump JSON on disk format to version 9

This change makes the TestResult.pid field into a list of ints instead of a list. This will be used by the multiple test per process tests to ensure that all PIDs are recorded when the cherry features is invoked. This is groundwork for later patches in the series. Signed-off-by: Dylan Baker

[Piglit] [PATCH v2 11/16] shader_runner: add ability to run multiple tests per process

From: Marek Olk The performance is measured by running these: spec/glsl-1.30/execution/built-in-functions/ (1420 tests) Running piglit-run.py using the old way: 98 s Running .../shader_runner *.shader_test: 8 s It's more than 12x faster!! The tests being run must be

[Piglit] [PATCH v2 10/16] util: move PIGLIT_PATH_SEP to piglit-util.h

From: Marek Olk --- tests/util/piglit-util.c | 6 -- tests/util/piglit-util.h | 6 ++ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/util/piglit-util.c b/tests/util/piglit-util.c index 0a66a85..6ed9c9c 100644 --- a/tests/util/piglit-util.c

[Piglit] [PATCH v2 6/16] shader_runner: propagate all errors to piglit_init

From: Marek Olk piglit_report_result prevents running multiple tests per process. --- tests/shaders/shader_runner.c | 229 +-- 1 file changed, 139 insertions(+), 90 deletions(-) diff --git a/tests/shaders/shader_runner.c

[Piglit] [PATCH v2 3/16] framework: Split FastSkipMixin.

This splits the FastSkipMixin into two classes. One that actually does the fast skipping, and one that provides a mixin for Test. This split will allow the class to be used in the traditional way, but also in shader_test.MultiShaderTest. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.

[Piglit] [PATCH 1/2] tests: remove unused variable from ext_image_dma_buf_import/refcount

tests/spec/ext_image_dma_buf_import/refcount.c:59:6: warning: unused variable ‘cpp’ int cpp = 4; ^~~ Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/spec/ext_image_dma_buf_import/refcount.c | 1 - 1 file changed, 1 deletion(-) diff --git a/test

[Piglit] [PATCH 2/2] tests: fix unused variable in gles versions of arb_viewport_array/queries

tests/spec/arb_viewport_array/queries.c: In function ‘piglit_init’: tests/spec/arb_viewport_array/queries.c:58:11: warning: unused variable ‘vald’ GLdouble vald[4] = {0.0, 0.0, 0.0, 0.0}; Which is only used if compiled with OpenGL support, but not when compiled for GLES. Signed-off-by: Dylan

[Piglit] [PATCH 2/2] framework/log: Add new test to running in start not constructor

Otherwise there will be a spinner in the final call to summary that's printed when piglit finishes. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/log.py | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/framework/log.py b/framework/log.py

[Piglit] [PATCH 1/2] framework: If exiting a run, ensure that the status is readable

This patch catches C-c, and any Exception while tests are running, prints the status again with a newline, and then raises the exception. This makes the output visible even when a stacktrace is printed. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/profile.p

Re: [Piglit] [PATCH v2] utils: define DRM_FORMATS needed by dma_buf tests

Quoting Dylan Baker (2016-09-28 12:09:09) > If they're not defined by the installed libdrm already. > > This moves code already defined in transcode-nv12-as-r8-gr88 to the > utils level, so that all tests that require these formats get them, and also > adds on additional def

[Piglit] [PATCH v2] utils: define DRM_FORMATS needed by dma_buf tests

://bugs.freedesktop.org/show_bug.cgi?id=97951 cc: Vinson Lee <v...@freedesktop.org> cc: Rob Clark <robdcl...@gmail.com> Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- This is a completely different approach than the last time, but I think this might be better than the previous approach

Re: [Piglit] [PATCH] CMake: Don't turn on dma-buf tests for intel without libdrm >= 2.4.68

Quoting Vinson Lee (2016-09-28 11:21:12) > On Tue, Sep 27, 2016 at 3:17 PM, Dylan Baker <dy...@pnwbakers.com> wrote: > > Quoting Dylan Baker (2016-09-27 15:14:18) > >> This version of libdrm was the first to privde DRM_FORMAT_R8, > >> DRM_FORMAT_GR88 and DRM_FORM

Re: [Piglit] [PATCH] CMake: Don't turn on dma-buf tests for intel without libdrm >= 2.4.68

Quoting Vinson Lee (2016-09-28 11:21:12) > On Tue, Sep 27, 2016 at 3:17 PM, Dylan Baker <dy...@pnwbakers.com> wrote: > > Quoting Dylan Baker (2016-09-27 15:14:18) > >> This version of libdrm was the first to privde DRM_FORMAT_R8, > >> DRM_FORMAT_GR88 and DRM_FORM

Re: [Piglit] [PATCH] CMake: Don't turn on dma-buf tests for intel without libdrm >= 2.4.68

Quoting Dylan Baker (2016-09-27 15:14:18) > This version of libdrm was the first to privde DRM_FORMAT_R8, > DRM_FORMAT_GR88 and DRM_FORMAT_RG88, which are needed for dma-buf tests > since commit 0222f5db00fd47fb745402514a6d8cf2747f8434. > > bugzilla: https://bugs.freedesktop.org/

[Piglit] [PATCH] CMake: Don't turn on dma-buf tests for intel without libdrm >= 2.4.68

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- CMakeLists.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 4002fe9..084ccf0 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -178,11 +178,11 @@ ENDIF() # The presen

Re: [Piglit] [PATCH] Add profile cts_gl45

TRA_ARGS if not x.startswith('--deqp-case')] > + > +profile = deqp.make_profile( # pylint: disable=invalid-name > +itertools.chain( > +deqp.iter_deqp_test_cases( > + deqp.gen_caselist_txt(_CTS_BIN, 'GL45-CTS-cases.txt', > _EXT

Re: [Piglit] [PATCH 1/2] unittests: Add test for DriverClassifier.get_glxinfo

piglit is broken on python3 without these, I'm planning to push soon. Quoting Dylan Baker (2016-09-16 13:46:04) > Ensure that it converts bytes to str and also collects correctly. > > Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> > --- > unittests/framework/test

[Piglit] [PATCH 2/2] framework: Make driver_classifier convert to str (unicode)

it again. CC: Eric Anholt <e...@anholt.net> Reported-by: Dan Kegel <d...@kegel.com> Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/driver_classifier.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/framework/driver_classif

[Piglit] [PATCH 1/2] unittests: Add test for DriverClassifier.get_glxinfo

Ensure that it converts bytes to str and also collects correctly. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- unittests/framework/test_driver_classifier.py | 25 + 1 file changed, 25 insertions(+) diff --git a/unittests/framework/test_driver_classif

Re: [Piglit] Fresh driver_classifier.py type error fix

Quoting Dan Kegel (2016-09-16 09:12:21) > piglit stopped working for me this morning with a TypeError on ubuntu 16.04. > That's what I get for running off of tip, the file that broke was only > created six hours ago :-) > > Here's a tiny change that got me running again. > > From

[Piglit] [PATCH v2] framework: Fix FastSkipMixin version comparisons

fixes that, and also generalizes the fast-skip mechanism a bit. It does this by allowing the caller to set an operation as well as a value. This also updates glslparsertest and shadertest to use the newly updated interfaces. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- v2:

Re: [Piglit] [PATCH 0/3] apitrace test runner for piglit

@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/piglit I haven't look too closely at them this time, but everything seemed good. For the python bits at least, Reviewed-by: Dylan Baker <dy...@pnwbakers.com> signature.asc Description: signature

Re: [Piglit] [PATCH 0/15] Add support for shader_runner to run more than one test per process

Quoting Dylan Baker (2016-09-13 13:46:11) > Quoting Marek Olšák (2016-09-13 13:27:21) > > On Tue, Sep 13, 2016 at 8:30 PM, Dylan Baker <dy...@pnwbakers.com> wrote: > > > So, there were a few comments I've addressed, and Marek acked one patch. > > > Do y'all want t

Re: [Piglit] [PATCH 0/15] Add support for shader_runner to run more than one test per process

Quoting Marek Olšák (2016-09-13 13:27:21) > On Tue, Sep 13, 2016 at 8:30 PM, Dylan Baker <dy...@pnwbakers.com> wrote: > > So, there were a few comments I've addressed, and Marek acked one patch. > > Do y'all want to see a v2, or should I just push this? > > There are at

Re: [Piglit] [PATCH 0/15] Add support for shader_runner to run more than one test per process

alse, yes, no; with any capitalization). I forgot to update the cover letter. Dylan > > Marek > > On Fri, Sep 9, 2016 at 9:18 PM, Dylan Baker <dy...@pnwbakers.com> wrote: > > This allows shader_runner to consume more than one test at a time. When > > it does this it repo

[Piglit] [PATCH 3/4] generators: add ARB_shader_viewport_layer_array to gen_extensions_defined

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- generated_tests/gen_extensions_defined.py | 1 + 1 file changed, 1 insertion(+) diff --git a/generated_tests/gen_extensions_defined.py b/generated_tests/gen_extensions_defined.py index cbacc12..745eb4a 100644 --- a/generated

[Piglit] [PATCH 4/4] tests: port tests for arb_shader_viewport_layer_array

This adds a couple of original compiler tests, and ports the tests for amd_vertex_shader_layer and amd_vertex_shader_viewport_index to the ARB extension. These tests touch the VS, but not the tes eval. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/

[Piglit] [PATCH 1/4] generators: Extend gen_extensions_defined for stage requirements

This allows extensions to specify which stages they support, allowing such extensions as ARB_shader_viewport_layer_array to be added. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- generated_tests/gen_extensions_defined.py | 176 -- 1 file chang

[Piglit] [PATCH 2/4] generated_tests: sort gen_extensions_defined list

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- generated_tests/gen_extensions_defined.py | 101 +++--- 1 file changed, 50 insertions(+), 51 deletions(-) diff --git a/generated_tests/gen_extensions_defined.py b/generated_tests/gen_extensions_defined.py

[Piglit] [PATCH 9/15] util: rename piglit_gl_reinitialize_extensions->piglit_gl_invalidate_extensions

From: Marek Olk --- tests/util/piglit-framework-gl/piglit_wfl_framework.c | 3 ++- tests/util/piglit-util-gl.c | 2 +- tests/util/piglit-util-gl.h | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git

[Piglit] [PATCH 15/15] framework: add boilerplate to turn off process isolation

This adds a switch to to turn off process isolation (it is normally on), for now this only affects shader_runner, but in the future could affect more tests, like glslparsertest or dEQP. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/options.py | 1 + framework/pr

[Piglit] [PATCH 11/15] shader_runner: add ability to run multiple tests per process

From: Marek Olk The performance is measured by running these: spec/glsl-1.30/execution/built-in-functions/ (1420 tests) Running piglit-run.py using the old way: 98 s Running .../shader_runner *.shader_test: 8 s It's more than 12x faster!! The tests being run must be

[Piglit] [PATCH 14/15] framework: Plug in fast-skipping to MultiShaderTest

This adds fast-skipping support to the MultiShaderTest class. Both patches seemed significant enough to warrant splitting them in two. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/test/base.py| 8 +- framework/test/shader_test.p

[Piglit] [PATCH 10/15] util: move PIGLIT_PATH_SEP to piglit-util.h

From: Marek Olk --- tests/util/piglit-util.c | 6 -- tests/util/piglit-util.h | 6 ++ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/util/piglit-util.c b/tests/util/piglit-util.c index 0a66a85..6ed9c9c 100644 --- a/tests/util/piglit-util.c

[Piglit] [PATCH 12/15] squash! shader_runner: add ability to run multiple tests per process

linking - call glDisable with GL_TEXTURE_2D - Cleanup textures after each test - Simplify the logic for calling piglit_present_results Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/shaders/shader_runner.c | 165 +++ 1 file changed, 110 inse

[Piglit] [PATCH 8/15] util: always set piglit_is_core_profile

From: Marek Olk --- tests/util/piglit-framework-gl/piglit_glut_framework.c | 7 --- tests/util/piglit-framework-gl/piglit_wfl_framework.c | 2 ++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/util/piglit-framework-gl/piglit_glut_framework.c

[Piglit] [PATCH 4/15] framework: Bump JSON on disk format to version 9

This change makes the TestResult.pid field into a list of ints instead of a list. This will be used by the multiple test per process tests to ensure that all PIDs are recorded when the cherry features is invoked. This is groundwork for later patches in the series. Signed-off-by: Dylan Baker

[Piglit] [PATCH 5/15] framework: Add a Mixin class for running multiple tests in a single process

a cherry-like resume feature that starts again after a test crashes, without rerunning the crashed test. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/test/base.py| 147 ++- framework/test/deqp.py| 4 +- unittests/fra

[Piglit] [PATCH 7/15] util: guard against double context destruction

From: Marek Olk this no longer crashes: - create context (calls atexit) - teardown context - create context 2 (calls atexit) - exit() --- tests/util/piglit-framework-gl.c | 4 1 file changed, 4 insertions(+), 0 deletions(-) diff --git

[Piglit] [PATCH 6/15] shader_runner: propagate all errors to piglit_init

From: Marek Olk piglit_report_result prevents running multiple tests per process. --- tests/shaders/shader_runner.c | 231 +-- 1 file changed, 140 insertions(+), 91 deletions(-) diff --git a/tests/shaders/shader_runner.c

[Piglit] [PATCH 3/15] framework: Split FastSkipMixin.

This splits the FastSkipMixin into two classes. One that actually does the fast skipping, and one that provides a mixin for Test. This split will allow the class to be used in the traditional way, but also in shader_test.MultiShaderTest. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.

[Piglit] [PATCH 0/15] Add support for shader_runner to run more than one test per process

tal For the general use case (With fast skipping) this is a reduction of 5 minutes of runtime. With fast-skipping seems pretty useful, as it reduces runtime by 3 minutes even with the removal of process isolation. Dylan Baker (9): framework/results: Lower subtest names framework: Split the file parsing c

[Piglit] [PATCH 2/15] framework: Split the file parsing code out of ShaderTest

This is both nicer since it provides better code quality (a separation of roles), and because it's going to make writing a shader_test class that can do N:1 tests much easier. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- framework/test/shader_test.py

Re: [Piglit] [PATCH] cts_gl.py: only generate tests for GL45-CTS

Quoting Marek Olšák (2016-09-09 06:10:56) > From: Marek Olšák > > Older versions don't work and there is no interest in fixing the official > gl45release CTS branch, so it's useless to add those forever-broken tests. > > Using '-t gl45-cts' is annoying, because adding '-t

Re: [Piglit] [PATCH] egl-khr-debug: Start adding tests for EGL_KHR_debug

Quoting Adam Jackson (2016-09-08 10:41:30) > First test does some basic API error checking, then provokes an error > against the (implicit) thread object. There is clearly much more to > cover here, but this at least touches all of the new entrypoints and > verifies that it errors reasonably when

Re: [Piglit] Newbie question: command to run tests that passed before?

Quoting Dylan Baker (2016-09-07 16:23:43) > Quoting Dan Kegel (2016-09-07 15:59:21) > > Hi! > > I'd like to use piglit as a regression test for my opengl test rigs. > > My plan is to run piglit a few times on each OS/card combo I need to > > support, > > get a

Re: [Piglit] Newbie question: command to run tests that passed before?

Quoting Mark Janes (2016-09-07 16:24:11) > Piglit converts "known failing" tests to a "skip" status. This is only true of the JUnit backend. I've never bothered to get that working with the default JSON backend. Its on my list of things to do, just not very high up on that list. Dylan

Re: [Piglit] Newbie question: command to run tests that passed before?

Quoting Dan Kegel (2016-09-07 16:10:46) > Hrm. A solution that involved Jenkins would be way overkill. > I'm just looking for the little core idiom for running just the > test of tests that passed in the, um, past. > > Maybe that's not the way people do it? Does everyone just compare > logs

Re: [Piglit] Newbie question: command to run tests that passed before?

Quoting Dan Kegel (2016-09-07 15:59:21) > Hi! > I'd like to use piglit as a regression test for my opengl test rigs. > My plan is to run piglit a few times on each OS/card combo I need to support, > get a list of tests that reliably pass on each platform, and then > forevermore run just those

[Piglit] [PATCH v2 33/37] glean/tfragprog1: port exp-fog tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 2 +- tests/glean/tfragprog1.cpp | 43 +--- tests/spec/arb_fragment_program/fog/calculated-exp.shader_test | 35 ++- test

[Piglit] [PATCH v2 35/37] glean/tfragprog1: port z-write test to shader_runner

Thanks to Ken for helping me work out a couple of bugs in this test. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py| 1 +- tests/glean/tfragprog1.cpp | 15 +-- tests/spec/arb_fragment_pro

[Piglit] [PATCH v2 31/37] shader_runner: add command for glFog

I've added this with the intention of using it to ease porting the numerous fog tests from glean to the piglit framework, many of which can easily be ported to shader_runner, except for the need for glFog setup. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/s

[Piglit] [PATCH v2 36/37] glean/tfragprog1: remove stress tests

These test divide by zero and divide by HUGE_NUMBER and -HUGE_NUMBER, which don't seem particularly useful, especially since there are already tests for that in GLSL, and if it's going to be a problem it'll be in the compiler. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> ---

[Piglit] [PATCH v2 34/37] glean/tfragprog1: port exp2 fog tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py| 3 +- tests/glean/tfragprog1.cpp | 47 +--- tests/spec/arb_fragment_program/fog/calculated-exp2.shader_test | 36 +- test

[Piglit] [PATCH v2 30/37] glean/tfragprog1: port XPD test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 14 -- tests/spec/arb_fragment_program/built-in-functions/xpd.shader_tes

[Piglit] [PATCH v2 8/37] glean/tfragprog1: port DPH test to shader_runner

This is a simplified version of the same test, but without some extra operations. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp

[Piglit] [PATCH v2 32/37] glean/tfragprog1: port linear fog tests to shader_runner

These make use of the new fog feature for shader_runner. The ported tests differ slightly in that they use the non EXT version of fog_coord, and thus require GL >= 1.4 Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tes

[Piglit] [PATCH v2 26/37] glean/tfragprog1: port SIN tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 2 -- tests/glean/tfragprog1.cpp | 31 --- tests/spec/arb_fragment_program

[Piglit] [PATCH v2 23/37] glean/tfragprog1: port RSQ tests to shader_runner

This single test combines the two tests in glean into one, adding a negative number. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 2 -- tests/glean/tfragprog1.cpp

[Piglit] [PATCH v2 15/37] glean/tfragprog1: port LRP test to shader_runner

It's worth noting that this test wasn't included in all.py, and therefore hasn't been running. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/glean/tfragprog1.cpp | 14 -- tests/spec/arb_fragment_program/built-in-fun

[Piglit] [PATCH v2 3/37] glean/tfragprog1: port ADD tests to shader_runner

an be used with a swizzle. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> v2: - Cover more cases in the ADD_SAT test --- tests/all.py | 5 +- tests/glea

[Piglit] [PATCH v2 13/37] glean/tfragprog1: port LG2 test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 19 --- tests/spec/arb_fragment_program/built-in-functions/lg2.shade

[Piglit] [PATCH v2 25/37] glean/tfragprog1: port SGE test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 14 -- tests/spec/arb_fragment_program/built-in-functions/sge.shader_tes

[Piglit] [PATCH v2 29/37] glean/tfragprog1: port swizzle tests to shader_runner

This includes the SWZ tests and the swizzled ADD and MOV tests. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 3 --- tests/glean/tfragprog1.cpp

[Piglit] [PATCH v2 24/37] glean/tfragprog1: port SCS test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 14 -- tests/spec/arb_fragment_program/built-in-functions/scs.shader_tes

[Piglit] [PATCH v2 7/37] glean/tfragprog1: port DP4 test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 13 - tests/spec/arb_fragment_program/built-in-functions/dp4.shader_tes

[Piglit] [PATCH v2 20/37] glean/tfragprog1: port MUL tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py| 2 -- tests/glean/tfragprog1.cpp | 28 tests/spec/arb_fragment_program/built-in-fun

[Piglit] [PATCH v2 21/37] glean/tfragprog1: port POW test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 15 --- tests/spec/arb_fragment_program/built-in-functions/pow.shader_tes

[Piglit] [PATCH v2 16/37] glean/tfragprog1: port MAD test to shader_runner

This isn't an exact port, some of the values were changed to be simpler, easier for humans to calculate values, but it shouldn't affect the viability of the test. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py

[Piglit] [PATCH v2 22/37] glean/tfragprog1: port RCP tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 2 -- tests/glean/tfragprog1.cpp | 23 --- tests/spec/arb_fragment_program

[Piglit] [PATCH v2 2/37] glean/tfragprog1: port ABS test to shader_runner

Possible duplicate tests: shaders/fp-abs-01.c shaders/fp-abs-02.c Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 2 +- tests/glean/tfragprog1.cpp | 13 -

[Piglit] [PATCH v2 0/37] Port Glean Frag-prog tests to shader_runner

mess. Dylan Baker (37): editorconfig: update the editorconfig file glean/tfragprog1: port ABS test to shader_runner glean/tfragprog1: port ADD tests to shader_runner glean/tfragprog1: Port CMP test to shader_runner glean/tfragprog1: port COS tests to shader runner glean/tfragprog1: port

[Piglit] [PATCH v2 19/37] glean/tfragprog1: port MOV tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 8 tests/spec/arb_fragment_program/built-in-functions/mov.shader_tes

[Piglit] [PATCH v2 17/37] glean/tfragprog1: port MAX test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 14 -- tests/spec/arb_fragment_program/built-in-functions/max.shader_tes

[Piglit] [PATCH v2 5/37] glean/tfragprog1: port COS tests to shader runner

This ports both of the COS tests to shader runner. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 2 -- tests/glean/tfragprog1.cpp

[Piglit] [PATCH v2 14/37] glean/tfragprog1: port LIT tests to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 3 --- tests/glean/tfragprog1.cpp| 39 --- tests/spec/arb_fragment_p

[Piglit] [PATCH v2 12/37] glean/tfragprog1: port FRC test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 9 - tests/spec/arb_fragment_program/built-in-functions/frc.shader_tes

[Piglit] [PATCH v2 4/37] glean/tfragprog1: Port CMP test to shader_runner

This also adds a similar CMP_SAT test. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 11 --- test

[Piglit] [PATCH v2 9/37] glean/tfragprog1: port DST test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 15 --- tests/spec/arb_fragment_program/built-in-functions/dst.shader_tes

[Piglit] [PATCH v2 1/37] editorconfig: update the editorconfig file

- set python to utf-8 - remove "remove trailing whitespace" from python (like the emacs config) - add glsl shaders to C style Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- .editorconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

[Piglit] [PATCH v2 6/37] glean/tfragprog1: port DP3 tests to shader_runner

This ports the first test and just removes the second test, which like some of the other dropped tests doesn't really focus on DP3, or on a reported bug for DP3, it just adds complexity. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/

[Piglit] [PATCH v2 11/37] glean/tfragprog1: port FLR test to shader_runner

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- tests/all.py | 1 - tests/glean/tfragprog1.cpp | 16 tests/spec/arb_fragment_program/built-in-functions/flr.shader_tes

Re: [Piglit] Building current piglit on ubuntu 12.04?

Quoting Ilia Mirkin (2016-09-03 20:59:23) > piglit does not require python3, it is meant to be able to run with > either python2.7 or python3.3+. If you find that something *requires* > python3, please report that separately. Indeed. The chbangs for all of the scripts are python3 because we have

[Piglit] [PATCH] framework: Fix FastSkipMixin version comparisons

Currently the FastSkipMixin assumes that versions will always be a minimum required version, and nothing else. This is mostly true of piglit tests, but isn't always true. Sometimes tests specify that they need a version greater-or-equal, and a few have less-than or less-than-equal. This patch

[Piglit] [PATCH] tests: fix ext_shader_samples_identical ES 3.1 vert test

This test is currently completely broken. It uses names defined in GL_OES_texture_storage_multisample_2d_array without adding a #extension directive or a config requirement. It also doesn't declare precision. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> cc: i...@freedeskt

Re: [Piglit] [PATCH 1/5] gl-1.0: Move scissor tests here.

Quoting Dylan Baker (2016-09-01 12:03:35) > > Maybe it's also a good time to evaluate whether these actually need to > be run serially? > Or maybe I should read all the patches before I send comments. signature.asc Description: signature __

Re: [Piglit] [PATCH 1/5] gl-1.0: Move scissor tests here.

Quoting Matt Turner (2016-09-01 11:44:48) > --- > tests/all.py | 16 > tests/general/CMakeLists.gl.txt| 8 > tests/spec/gl-1.0/CMakeLists.gl.txt| 8 > tests/{general =>

[Piglit] [PATCH v2 5/5] unittests: don't require totals section in schema

We generate this on the fly if it's not there and add it. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- unittests/framework/backends/schema/piglit-8.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/unittests/framework/backends/schema/piglit-8.json b/uni

[Piglit] [PATCH v2 4/5] unittests: Fix JSON schema test to pass time_elapsed

Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- unittests/framework/backends/test_json.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/unittests/framework/backends/test_json.py b/unittests/framework/backends/test_json.py index e6574a6..0c49aff

[Piglit] [PATCH v2 3/5] tox: add a streams target.

Also enables the target in travis and appveyor. Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- .travis.yml | 8 appveyor.yml | 8 tox.ini | 3 ++- 3 files changed, 10 insertions(+), 9 deletions(-) diff --git a/.travis.yml b/.travis.yml index 5c3bf08..9

<    2   3   4   5   6   7   8   9   10   11   >