[Piglit] [PATCH 7/7] glsl-1.20: Print generated outerProduct test names.

2013-11-04 Thread Matt Turner
--- tests/spec/glsl-1.20/execution/outerProduct-const.sh | 2 ++ tests/spec/glsl-1.20/execution/outerProduct.sh | 2 ++ 2 files changed, 4 insertions(+) diff --git a/tests/spec/glsl-1.20/execution/outerProduct-const.sh b/tests/spec/glsl-1.20/execution/outerProduct-const.sh index 508aa4a..9

[Piglit] [PATCH 5/7] glsl-1.20: Don't use ortho in outerProduct tests.

2013-11-04 Thread Matt Turner
--- tests/spec/glsl-1.20/execution/outerProduct-const.sh | 16 tests/spec/glsl-1.20/execution/outerProduct.sh | 11 ++- 2 files changed, 6 insertions(+), 21 deletions(-) diff --git a/tests/spec/glsl-1.20/execution/outerProduct-const.sh b/tests/spec/glsl-1.20/execut

[Piglit] [PATCH 4/7] glsl-1.10 / glsl-1.20: Don't use ortho in variable indexing tests.

2013-11-04 Thread Matt Turner
Using ortho makes shrinking the window size much more difficult, which makes running tests in simulation take much longer. --- tests/spec/glsl-1.10/variable-index-read.sh | 18 +++--- tests/spec/glsl-1.10/variable-index-write.sh | 17 +++-- 2 files changed, 6 insertions(+)

[Piglit] [PATCH 6/7] glsl-1.10 / glsl-1.20: Print generated variable-index test names.

2013-11-04 Thread Matt Turner
--- tests/spec/glsl-1.10/variable-index-read.sh | 64 +--- tests/spec/glsl-1.10/variable-index-write.sh | 64 +--- 2 files changed, 96 insertions(+), 32 deletions(-) diff --git a/tests/spec/glsl-1.10/variable-index-read.sh b/tests/spec/glsl-1.10/v

[Piglit] [PATCH 2/7] shaders: Remove #version from glsl-const tests.

2013-11-04 Thread Matt Turner
Also basically a revert of 6533d757 to make the cosh test match. --- tests/shaders/glsl-const-builtin-abs.shader_test | 1 - tests/shaders/glsl-const-builtin-acos.shader_test | 1 - tests/shaders/glsl-const-builtin-all.shader_test | 1 - tests/shaders/glsl-con

[Piglit] [PATCH 3/7] shaders: Stop using ortho in glsl-const tests.

2013-11-04 Thread Matt Turner
Scripted with sed with fix ups to - not - actually writes .w component, so probe rgba. - reflect - writes const 1.0 .w component, so probe rgb. - cosh - writes const 1.0 .w component, and used probe rgb with 4 components, so drop the .w from the probe. --- tests/shaders/glsl-const-builtin-ab

[Piglit] [PATCH 1/7] glsl-1.10: Delete bogus generated test.

2013-11-04 Thread Matt Turner
--- .../#fs-temp-array-mat2-index-col-wr.shader_test# | 97 -- 1 file changed, 97 deletions(-) delete mode 100644 tests/spec/glsl-1.10/execution/variable-indexing/#fs-temp-array-mat2-index-col-wr.shader_test# diff --git a/tests/spec/glsl-1.10/execution/variable-indexing/#f

[Piglit] [PATCH 0/7] Reducing ortho use in shader_runner

2013-11-04 Thread Matt Turner
We'd like to shrink the window sizes of shader runner when running in simulation, and ortho tests make it difficult to reason about reducing the window size. The following patches fix some tests and test generators to use non-ortho projections instead. This should allow us to shrink these tests' w

[Piglit] [PATCH 2/2] More comprehensive test for alpha blending with multisampling.

2013-11-04 Thread Paul Berry
This test validates a corner case of the fix for https://bugs.freedesktop.org/show_bug.cgi?id=53077 ([IVB] Output error with msaa when both of framebuffer and source color's alpha are not 1). Specifically, it verifies that the first time alpha blending is used on a multisampled framebuffer, the co

[Piglit] [PATCH 1/2] Add a "slow_cc" option to EXT_framebuffer_multisample/alpha-blending.

2013-11-04 Thread Paul Berry
When fast color clears are added to i965, there is a danger that they may cause this test to stop reproducing bug 53077. To avoid that, add a command-line option which causes the test to use a clear color that isn't fast color clearable. As a result of this patch, a new piglit test is added: "spe

[Piglit] [PATCH] GL_ARB_base_instance: Verify that the baseinstance setting does not affect the value of gl_InstanceID.

2013-11-04 Thread Nicholas Mack
--- tests/all.tests| 4 + tests/spec/CMakeLists.txt | 1 + tests/spec/arb_base_instance/CMakeLists.gl.txt | 9 ++ tests/spec/arb_base_instance/CMakeLists.txt| 1 + .../baseinstance-doesnt-affect-gl-instance-id.c|

Re: [Piglit] [PATCH V2 2/6] ARB_sample_shading: Add test to verify the functionality of gl_NumSamples

2013-11-04 Thread Chris Forbes
Ah, that's an interesting consideration. I don't think it's a factor for builtin-gl-sample-id though, if you follow Paul's suggestion to use a multisample texture instead of a blit resolve. On Tue, Nov 5, 2013 at 10:06 AM, Anuj Phogat wrote: > > > > On Sat, Oct 26, 2013 at 11:37 AM, Chris Forbes

Re: [Piglit] [PATCH V2 2/6] ARB_sample_shading: Add test to verify the functionality of gl_NumSamples

2013-11-04 Thread Anuj Phogat
On Sat, Oct 26, 2013 at 11:37 AM, Chris Forbes wrote: > On Sat, Oct 26, 2013 at 12:49 PM, Anuj Phogat > wrote: > > V2: Get rid of redundant projection matrix. > > Signed-off-by: Anuj Phogat > > --- > > tests/all.tests| 6 + > > .../arb_sample_shading/execu

Re: [Piglit] [PATCH 6/7] arb_vertex_attrib_binding: Test instance-divisor

2013-11-04 Thread Eric Anholt
Fredrik Höglund writes: > On Wednesday 30 October 2013, Eric Anholt wrote: >> Fredrik Höglund writes: >> > diff --git a/tests/spec/arb_vertex_attrib_binding/instance-divisor.c >> > b/tests/spec/arb_vertex_attrib_binding/instance-divisor.c >> > new file mode 100644 >> > index 000..88ae454 >>

Re: [Piglit] [PATCH] Test that the fs gl_PrimitiveID input works even when there's no gs.

2013-11-04 Thread Eric Anholt
Paul Berry writes: > For i965/gen7 hardware, the driver has to go to extra work to make > gl_PrimitiveID work in the case where there is no geometry shader. So > we need a separate test to make sure that works properly. Reviewed-by: Eric Anholt pgpoMbgEA32B4.pgp Description: PGP signature __

[Piglit] [PATCH 4/6] templates/index_body.mako: remove unused subtestResult

2013-11-04 Thread Dylan Baker
This code was left over from an early iteration of the code, and isn't used. Signed-off-by: Dylan Baker --- templates/index_body.mako | 4 1 file changed, 4 deletions(-) diff --git a/templates/index_body.mako b/templates/index_body.mako index 0056912..9e67311 100644 --- a/templates/index_b

[Piglit] [PATCH 1/6] Summary.py: use mako's TemplateLookup class

2013-11-04 Thread Dylan Baker
This change reduces code duplication by setting arguments to the templates to be set once, rather than once per template. Signed-off-by: Dylan Baker --- framework/summary.py | 56 ++-- 1 file changed, 24 insertions(+), 32 deletions(-) diff --git a

[Piglit] [PATCH 2/6] index.mako: Split index.mako into three files

2013-11-04 Thread Dylan Baker
This is groundwork for simplifying the generation of the HTML pages. Signed-off-by: Dylan Baker --- framework/summary.py | 25 -- templates/index.mako | 81 -- templates/index_body.mako | 41 +++ templates/i

[Piglit] [PATCH 6/6] Summary.py: Convert HTMLIndex class into a generator

2013-11-04 Thread Dylan Baker
This change does a couple of really nice things for us, first it cuts down on the memory consumption by using a generator (which is lazy). Further, as a part of the Summary.generate_html() method it doesn't need to have the summary instance passed to it, reducing complexity. Signed-off-by: Dylan B

[Piglit] [PATCH] tests/igt.tests: PEP8 compliance

2013-11-04 Thread Dylan Baker
formatting changes. - replace tabs with 8 spaces - fix long lines - remove trailing semicolons - fix hanging indent problems - add two spaces before and after top level classes and functions - use os.path.join instead of concat Signed-off-by: Dylan Baker --- tests/igt.tests | 73 +++

[Piglit] [PATCH 3/6] Summary.py: Move code out of HTMLIndex into a mako template

2013-11-04 Thread Dylan Baker
Moving the column headers out of the HTMLIndex class simplifies the class, since the code that existed for handling this was special case. Signed-off-by: Dylan Baker --- framework/summary.py | 17 + templates/index_columns.mako | 7 +++ 2 files changed, 12 insertions

[Piglit] [PATCH 5/6] summary.py: Simplify the way HTMLIndex works

2013-11-04 Thread Dylan Baker
This patch replaces home-rolled code with built-in python functions, this results in cleaner, simpler, and faster code. Signed-off-by: Dylan Baker --- framework/summary.py | 124 +++ 1 file changed, 36 insertions(+), 88 deletions(-) diff --git a/f

Re: [Piglit] [PATCH] update the HACKING file's Coding style section

2013-11-04 Thread Ken Phillis Jr
We Probably should also have two reminders for people that will be highly useful. 1) To please pay attention to the various FileSystem valid file names. This means to refer to the relevant wikipedia article ( http://en.wikipedia.org/wiki/Filename ) over this and pay attention to what each platform

[Piglit] [PATCH 9/9 v2] ARB_viewport_array: Rendering test with multiple scissor rectangles/enables

2013-11-04 Thread Jon Ashburn
Tests rendering into a single framebuffer surface with multiple viewports via a geometry shader. Scissoring is used to restrict quads to a smaller area on the surface. Confirm that each area of the surface delineated by a scissor rectangle for viewport index renders the correct color. Both indexe

Re: [Piglit] [PATCH 00/13] Signal Handler and GLFW framework

2013-11-04 Thread Ken Phillis Jr
I had to add a few more patches to the commit list due to bugs I found. * git: add install_manifest.txt to gitignore This is simple enough... When you type "Make Package" you will wind up having a file called install_manifest.txt appear. * cmake: Install templates on make package. Another simp

Re: [Piglit] [PATCH] update the HACKING file's Coding style section

2013-11-04 Thread Brian Paul
On 11/02/2013 09:02 PM, Dylan Baker wrote: Could you add a note about python code? Just something like: "All python code should conform to python's PEP8 standard, using only spaces and no tabs" would be great. Go for it! -Brian ___ Piglit mailing