[Piglit] [PATCH 5/9] shader_runner: Add 'program binary save restore' command

2018-06-08 Thread Jordan Justen
Signed-off-by: Jordan Justen --- tests/shaders/shader_runner.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c index 0b449c056..b954520f3 100644 --- a/tests/shaders/shader_runner.c +++

[Piglit] [PATCH 4/9] shader_runner: Support SHADER_RUNNER_GET_PROGRAM_BINARY env var

2018-06-08 Thread Jordan Justen
If set to 1/yes/true, SHADER_RUNNER_GET_PROGRAM_BINARY will run as if the -get-program-binary parameter were used. This allows comparing piglit runs with and without shader runner using GetProgramBinary. Signed-off-by: Jordan Justen --- tests/shaders/shader_runner.c | 5 - 1 file changed,

[Piglit] [PATCH 9/9] ARB_get_program_binary: Test that restoring active program takes effect

2018-06-08 Thread Jordan Justen
Signed-off-by: Jordan Justen Cc: Timothy Arceri --- .../arb_get_program_binary/CMakeLists.gl.txt | 2 + .../restore-implicit-use-program.c| 145 ++ 2 files changed, 147 insertions(+) create mode 100644

[Piglit] [PATCH 8/9] ARB_get_program_binary: Test that uniforms are reset on program reload (API)

2018-06-08 Thread Jordan Justen
Signed-off-by: Jordan Justen --- .../arb_get_program_binary/CMakeLists.gl.txt | 1 + .../arb_get_program_binary/reset-uniform.c| 121 ++ 2 files changed, 122 insertions(+) create mode 100644 tests/spec/arb_get_program_binary/reset-uniform.c diff --git

[Piglit] [PATCH 7/9] ARB_get_program_binary: Add some common functions

2018-06-08 Thread Jordan Justen
--- .../spec/arb_get_program_binary/gpb-common.c | 126 ++ .../spec/arb_get_program_binary/gpb-common.h | 37 + 2 files changed, 163 insertions(+) create mode 100644 tests/spec/arb_get_program_binary/gpb-common.c create mode 100644

[Piglit] [PATCH 1/9] shader_runner: Support testing GL_NUM_PROGRAM_BINARY_FORMATS

2018-06-08 Thread Jordan Justen
Signed-off-by: Jordan Justen --- tests/shaders/shader_runner.c | 16 1 file changed, 16 insertions(+) diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c index 794524e8f..c4e25a33d 100644 --- a/tests/shaders/shader_runner.c +++

[Piglit] [PATCH 3/9] util: Add piglit_env_var_as_boolean

2018-06-08 Thread Jordan Justen
This is copied from Mesa's env_var_as_boolean. Signed-off-by: Jordan Justen --- tests/util/piglit-util.c | 27 +++ tests/util/piglit-util.h | 9 + 2 files changed, 36 insertions(+) diff --git a/tests/util/piglit-util.c b/tests/util/piglit-util.c index

[Piglit] [PATCH 2/9] shader_runner: Add -get-program-binary parameter

2018-06-08 Thread Jordan Justen
This parameter will test ARB_get_program_binary and OES_get_program_binary with any shader runner test. If -get-program-binary, then shader_runner will check to see if the extension is supported, and if 1 or more binary formats are supported. If the extensions are not supported, or 0 formats are

[Piglit] [PATCH 6/9] ARB_get_program_binary: Test that uniforms are reset on program reload

2018-06-08 Thread Jordan Justen
Signed-off-by: Jordan Justen --- .../uniform-after-restore.shader_test | 35 +++ 1 file changed, 35 insertions(+) create mode 100644 tests/spec/arb_get_program_binary/execution/uniform-after-restore.shader_test diff --git

Re: [Piglit] [PATCH] framework: Add a PIGLIT_TEST_TIMEOUT environment variable

2018-06-08 Thread Eric Anholt
Jason Ekstrand writes: > This commit adds a new PIGLIT_TESTS_TIMEOUT environment variable which > overrides the pre-test timeout specified by the test runner. > --- > > This is almost certainly not the right solution but it gets the discussion > started. A couple of known issues: > > - No

Re: [Piglit] [PATCH 2/4] README.md: Remove not about .py on profiles

2018-06-08 Thread Eric Anholt
Dylan Baker writes: > It is still valid, but the behavior of profiles specified without an > extension is always preferable, since some profiles are available as > compressed xml and as python. The XML versions will start faster and > use less memory than the python version. Other profiles are

Re: [Piglit] [PATCH 3/4] README.md: Remove PIGLIT_FORCE_GLSLPASER_DESKTOP env variable

2018-06-08 Thread Eric Anholt
Dylan Baker writes: > This is no longer honored, instead use the glslparser_arb_compat > profile. I still see it in the tree here. signature.asc Description: PGP signature ___ Piglit mailing list Piglit@lists.freedesktop.org

[Piglit] [PATCH] framework: Add a PIGLIT_TEST_TIMEOUT environment variable

2018-06-08 Thread Jason Ekstrand
This commit adds a new PIGLIT_TESTS_TIMEOUT environment variable which overrides the pre-test timeout specified by the test runner. --- This is almost certainly not the right solution but it gets the discussion started. A couple of known issues: - No documentation - It overrides. Maybe it

Re: [Piglit] [PATCH 5/5] textureGather: Use piglit_probe_rect_rgba_varying

2018-06-08 Thread Eric Anholt
Jason Ekstrand writes: > Instead of doing 900 piglit_proble_pixel_rgba calls (one for each pixel) > which each call glReadPixels, use the new helper which only invokes > glReadPixels once. This series is: Reviewed-by: Eric Anholt signature.asc Description: PGP signature

[Piglit] [PATCH 4/4] README.md: Update profiles lists for xml changes

2018-06-08 Thread Dylan Baker
--- README.md | 66 ++- 1 file changed, 36 insertions(+), 30 deletions(-) diff --git a/README.md b/README.md index 6342051bb..5ccaaf077 100644 --- a/README.md +++ b/README.md @@ -302,51 +302,57 @@ The following test sets are currently

[Piglit] [PATCH 3/4] README.md: Remove PIGLIT_FORCE_GLSLPASER_DESKTOP env variable

2018-06-08 Thread Dylan Baker
This is no longer honored, instead use the glslparser_arb_compat profile. --- README.md | 6 -- 1 file changed, 6 deletions(-) diff --git a/README.md b/README.md index 2698bbef9..6342051bb 100644 --- a/README.md +++ b/README.md @@ -260,12 +260,6 @@ behaves. This values is honored by the

[Piglit] [PATCH 2/4] README.md: Remove not about .py on profiles

2018-06-08 Thread Dylan Baker
It is still valid, but the behavior of profiles specified without an extension is always preferable, since some profiles are available as compressed xml and as python. The XML versions will start faster and use less memory than the python version. Other profiles are only available as python, so

[Piglit] [PATCH 1/4] README.md: Note that python 3.3.x+ is supported

2018-06-08 Thread Dylan Baker
--- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 30d7a2b5f..2d7d95674 100644 --- a/README.md +++ b/README.md @@ -30,7 +30,7 @@ The original tests have been taken from First of all, you need to make sure that the following are

Re: [Piglit] [PATCH 0/3] glx-oml-sync-control-timing fixes

2018-06-08 Thread Michel Dänzer
On 2018-06-07 01:02 PM, Emil Velikov wrote: > On 7 June 2018 at 10:58, Michel Dänzer wrote: >> On 2017-04-14 09:06 AM, Michel Dänzer wrote: >>> From: Michel Dänzer >>> >>> Patch 1 fixes an error message typo I noticed along the way. >>> >>> Patches 2 & 3 allow the glx-oml-sync-control-timing

Re: [Piglit] [PATCH v1] cmake: Use Policy CMP0072 as OLD by default.

2018-06-08 Thread Dylan Baker
Quoting Sergii Romantsov (2018-06-08 02:19:01) > If system has installed both GL libraries such as > Legacy and GLVND, cmake will dump warning: > > "OpenGL_GL_PREFERENCE has not been set to "GLVND" or "LEGACY", > so for compatibility with CMake 3.10 and below the legacy GL > library will be

Re: [Piglit] [PATCH 1/8] README: Move README.md so it gets parsed by GitLab

2018-06-08 Thread Juan A. Suarez Romero
Hi! Sent a small suggestion to one of the patches. In either case, this is: Reviewed-by: Juan A. Suarez J.A. On Thu, 2018-06-07 at 09:03 -0700, Jason Ekstrand wrote: > --- > README => README.md | 0 > 1 file changed, 0 insertions(+), 0 deletions(-) > rename README => README.md

Re: [Piglit] [PATCH 3/8] README: Use proper markdown code quoting

2018-06-08 Thread Juan A. Suarez Romero
On Thu, 2018-06-07 at 09:03 -0700, Jason Ekstrand wrote: > --- > README.md | 107 +- > 1 file changed, 58 insertions(+), 49 deletions(-) > > diff --git a/README.md b/README.md > index 9f50d2ec9..eee613095 100644 > --- a/README.md > +++

[Piglit] [Bug 106857] Ubuntu 18.04. Fedora 27. Debian 9.4. Cmake 3.11. There is a warning during building piglit "Policy CMP0072 is not set"

2018-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106857 --- Comment #2 from Denis --- I tested the patch on Fedora 27. It works as expected. I don't see the warning anymore -- You are receiving this mail because: You are the QA Contact for the bug.___

[Piglit] [Bug 106857] Ubuntu 18.04. Fedora 27. Debian 9.4. Cmake 3.11. There is a warning during building piglit "Policy CMP0072 is not set"

2018-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106857 --- Comment #1 from Denis --- here is a patch for it https://lists.freedesktop.org/archives/piglit/2018-June/024521.html -- You are receiving this mail because: You are the QA Contact for the

[Piglit] [PATCH v1] cmake: Use Policy CMP0072 as OLD by default.

2018-06-08 Thread Sergii Romantsov
If system has installed both GL libraries such as Legacy and GLVND, cmake will dump warning: "OpenGL_GL_PREFERENCE has not been set to "GLVND" or "LEGACY", so for compatibility with CMake 3.10 and below the legacy GL library will be used." Added usage of Policy CMP0072 as OLD (that selects

[Piglit] [Bug 106857] New: Ubuntu 18.04. Fedora 27. Debian 9.4. Cmake 3.11. There is a warning during building piglit "Policy CMP0072 is not set"

2018-06-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106857 Bug ID: 106857 Summary: Ubuntu 18.04. Fedora 27. Debian 9.4. Cmake 3.11. There is a warning during building piglit "Policy CMP0072 is not set" Product: piglit