From: Ian Romanick ian.d.roman...@intel.com
It was just impractical to generate these tests using scripts, so they
are hand-written.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
.../uniform-initializer/fs-structure.shader_test | 44
.../uniform-initializer
(Apologies to people on CC who received this twice. I mistyped the
address for the piglit list the first time.)
This is an initial set of tests for GLX_ARB_create_context_profile and
GLX_EXT_create_context_es2_profile. These tests just verify that the
GLX API generates the correct errors and
From: Ian Romanick ian.d.roman...@intel.com
NVIDIA's closed-source driver passes this test. AMD's closed-source
driver has not been tested.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/all.tests|4 +
.../spec/glx_arb_create_context
From: Ian Romanick ian.d.roman...@intel.com
NVIDIA's closed-source driver passes this test. AMD's closed-source
driver has not been tested. NVIDIA's closed-source driver supports
both core and compatibility profiles.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/all.tests
From: Ian Romanick ian.d.roman...@intel.com
The spec doesn't forbid indirect rendering with OpenGL ES 2.0.
There's no protocol defined, so it seems impossible that this could
ever work.
NVIDIA's closed-source driver fails this test. An indirect-rendering
ES2 context is created. I have
On 06/12/2012 02:56 PM, James Jones wrote:
On 6/12/12 2:47 PM, Ian Romanick wrote:
On 06/12/2012 02:35 PM, James Jones wrote:
On 6/12/12 2:25 PM, Ian Romanick wrote:
From: Ian Romanickian.d.roman...@intel.com
The spec doesn't forbid indirect rendering with OpenGL ES 2.0.
There's no protocol
On 06/12/2012 02:40 PM, James Jones wrote:
On 6/12/12 2:25 PM, Ian Romanick wrote:
From: Ian Romanickian.d.roman...@intel.com
NVIDIA's closed-source driver fails this test. GLXBadProfileARB is
generated instead of BadMatch. The specification isn't specific about
what error should
On 06/12/2012 03:02 PM, Paul Berry wrote:
On 21 May 2012 14:10, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
On 05/21/2012 10:34 AM, Paul Berry wrote:
On 16 May 2012 14:23, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org
stops using the wrapper functions; the next patch will
remove them.
Death to the wrappers!
Acked-by: Ian Romanick ian.d.roman...@intel.com
---
tests/spec/ext_transform_feedback/alignment.c | 13 ++---
tests/spec/ext_transform_feedback/api-errors.c | 16
This is an initial set of tests for GLX_ARB_create_context_profile,
GLX_EXT_create_context_es2_profile, and
GLX_ARB_create_context_robustness. These tests just verify that the
GLX API generates the correct errors and allows certain required
things to work.
Since these tests share a lot of
From: Ian Romanick ian.d.roman...@intel.com
Make sure that all the functions are available when they should be.
This is currently broken in Mesa.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=52003
---
tests/all.tests
From: Ian Romanick ian.d.roman...@intel.com
If the extension is not supported, applications should not be able to
bind to the texture target or the buffer object target.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
This currently fails on Mesa, and I'm working on a fix. It's
On 07/31/2012 06:38 PM, Matt Turner wrote:
---
tests/all_egl.tests|3 +
tests/egl/CMakeLists.gl.txt|2 +
tests/egl/spec/CMakeLists.txt |1 +
.../spec/egl_khr_create_context/CMakeLists.gl.txt | 14 ++
On 07/31/2012 06:38 PM, Matt Turner wrote:
---
tests/all_egl.tests|1 +
.../spec/egl_khr_create_context/CMakeLists.gl.txt |2 +
.../egl_khr_create_context/default-major-version.c | 84
3 files changed, 87 insertions(+), 0
the coordinate transformations involved in cube map
texture lookup.
Sorry for the lag...
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
tests/all.tests |1 +
tests/texturing/CMakeLists.gl.txt |1 +
tests/texturing/array-depth-roundtrip.c | 203
On 10/09/2012 09:01 AM, Kenneth Graunke wrote:
On 10/09/2012 07:57 AM, Chad Versace wrote:
On 10/08/2012 03:40 PM, Kenneth Graunke wrote:
On 10/08/2012 03:21 PM, Chad Versace wrote:
The mesa commit 7dc0be8 changed the EGLConfig ID's for the Intel
driver,
and piglit's GTF integration hardcodes
The series is:
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
On 10/08/2012 04:57 PM, Matt Turner wrote:
---
tests/util/piglit-util-egl.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tests/util/piglit-util-egl.c b/tests/util/piglit-util-egl.c
index 5de290a
On 10/05/2012 03:27 AM, Neil Roberts wrote:
Eric Anholt e...@anholt.net writes:
If this is testing GL_EXT_unpack_subimage, it should go in
tests/spec/gl_ext_unpack_subimage/
It's not really testing GL_EXT_unpack_subimage but is testing a specific
bug in the Intel driver when uploading a
On 10/16/2012 03:37 PM, Marek Olšák wrote:
There is one test fdo25614-genmipmap which reproduces a bug only
occuring with a framebuffer without alpha. If we enforce RGBA
everywhere, not only would the test be useless, but also we'd end up
with the RGBX framebuffer config mostly untested.
While
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=56057
---
tests/all.tests | 2 +
tests/glx/CMakeLists.gl.txt | 2 +
tests/glx/glx-string-sanity.c | 187
On 10/17/2012 09:42 AM, Chad Versace wrote:
On 10/16/2012 05:12 PM, Ian Romanick wrote:
On 10/16/2012 03:37 PM, Marek Olšák wrote:
There is one test fdo25614-genmipmap which reproduces a bug only
occuring with a framebuffer without alpha. If we enforce RGBA
everywhere, not only would the test
On 10/19/2012 11:13 AM, Eric Anholt wrote:
Apparently I added Mesa support for it without adding tests, and
shockingly the Mesa support was broken.
Of course. :) Other than a small nit below, this patch is
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
tests/all.tests
On 10/19/2012 11:13 AM, Eric Anholt wrote:
I was going to go touch this code in Mesa, then I realised that we
didn't have a single test for it.
v2: Rebase on Chad's BUILD_* sedjob, and explicitly test various swap intervals.
A couple comments below...
---
tests/all.tests
On 10/12/2012 01:42 PM, Matt Turner wrote:
---
tests/spec/glx_arb_create_context/core-profile.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/tests/spec/glx_arb_create_context/core-profile.c
b/tests/spec/glx_arb_create_context/core-profile.c
index
On 10/23/2012 06:25 AM, Marek Olšák wrote:
---
tests/all.tests | 265 +++
1 files changed, 149 insertions(+), 116 deletions(-)
diff --git a/tests/all.tests b/tests/all.tests
index 341d021..a260073 100644
--- a/tests/all.tests
+++
On 10/28/2012 09:55 AM, Marek Olšák wrote:
There's one other case that's important to test: GL_WRITE_ONLY mapping
of a buffer in-use by the GPU. On at least some implementations, this
would cause a temporary buffer to be allocated using malloc and returned
to the application. See also my
On 10/29/2012 08:04 AM, Brian Paul wrote:
I think any implementation that can do ARB_fragment_program can also do
OpenGL 1.4. Does this remove all of the GLU dependencies?
The series is:
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
To remove dependency on glu.h
On 11/05/2012 02:53 PM, Chad Versace wrote:
In test egl-create-context-invalid-gl-version, try to create OpenGL ES2
contexts with additional invalid versions: 3.2, 3.9, 4.7.
Fails with mesa-84b437 on Intel Sandybridge.
CC: Matt Turner matts...@gmail.com
---
On 11/06/2012 12:51 PM, Chad Versace wrote:
On 11/06/2012 12:19 PM, Ian Romanick wrote:
On 11/05/2012 02:53 PM, Chad Versace wrote:
In test egl-create-context-invalid-gl-version, try to create OpenGL ES2
contexts with additional invalid versions: 3.2, 3.9, 4.7.
Fails with mesa-84b437 on Intel
On 11/08/2012 11:00 AM, Chad Versace wrote:
Fails on mesa-a196f43 with Intel Sandybridge with:
Created OpenGL ES1 context with invalid version 3.9
CC: Ian Romanick i...@freedesktop.org
CC: Matt Turner matts...@gmail.com
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
tests/egl
On 11/08/2012 01:20 PM, Ian Romanick wrote:
On 11/08/2012 10:19 AM, Chad Versace wrote:
This test requests various flavors (that is, api and version and
profile) of
contexts and verifies that the context's actual flavor is compatible with
the requested flavor.
The following subcases fail
equivalent.
CC: Ian Romanick i...@freedesktop.org
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
tests/egl/egl-query-surface.c | 15 +++
tests/egl/spec/egl_khr_create_context/core-profile.c
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
tests/general/windowoverlap.c | 2 +-
tests/glean/tbase.h | 2 +-
tests/glx/glx-copy-sub-buffer.c
On 11/09/2012 11:58 AM, Chad Versace wrote:
On 11/08/2012 01:29 PM, Ian Romanick wrote:
On 11/08/2012 11:00 AM, Chad Versace wrote:
Fails on mesa-a196f43 with Intel Sandybridge with:
Created OpenGL ES1 context with invalid version 3.9
CC: Ian Romanick i...@freedesktop.org
CC: Matt Turner
Right now Mesa and Apple only expose GL versions 3.0 without the
GL_ARB_compatilibity extension. This means that applications that
want, say, OpenGL 3.1 so that they can test GLSL 1.40 have to
specifically ask for OpenGL 3.1. This series modifies shader_runner
and glslparser test to request a
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/shaders/shader_runner.c | 19 +++
1 file changed, 11 insertions(+), 8 deletions(-)
diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c
index f0037c1
From: Ian Romanick ian.d.roman...@intel.com
There's no default VAO (without GL_ARB_compatibility), so we have to
create and bind one.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/shaders/shader_runner.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff
From: Ian Romanick ian.d.roman...@intel.com
A test script can require a specific GLSL version or a specific GL
version. To satisfy this requirement, the piglit framework code needs
to know about the requirement before creating the context. However,
the requirements section can contain other
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/glslparsertest/glslparsertest.c | 37 +--
1 file changed, 35 insertions(+), 2 deletions(-)
diff --git a/tests/glslparsertest/glslparsertest.c
b/tests
On 11/05/2012 11:39 AM, Frank Henigman wrote:
This test shows the problem fixed by my add bounds checking for
uniform array access patch, sent to mesa-dev on Nov. 2.
Check behavior of glGetUniformLocation, glGetUniform and glUniform(Matrix)
when attempting to access non-existent array elements.
On 11/14/2012 12:11 PM, Tom Gall wrote:
Hi All,
I've an interest in using piglit for GLES testing on ARM for both linux
and android. I've noticed that things are a bit light in the GLES test
department.
That is something of an understatement. :) Which version of GLES do you
want to test?
On 11/14/2012 01:18 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
In floating point, 3.1 is actually 3.09, and that's just
awsome for comparing GL versions.
+ sscanf(line, %u.%u, major, minor
On 11/14/2012 08:29 AM, Chad Versace wrote:
On 11/13/2012 04:36 PM, Ian Romanick wrote:
On 11/09/2012 11:45 AM, Chad Versace wrote:
This test requests various flavors (that is, api and version and profile) of
contexts and verifies that the context's actual flavor is compatible
On 11/16/2012 11:08 AM, Chad Versace wrote:
On 11/13/2012 05:00 PM, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
A test script can require a specific GLSL version or a specific GL
version. To satisfy this requirement, the piglit framework code needs
to know about
From: Ian Romanick ian.d.roman...@intel.com
I haven't observed any failures with this yet. Since we just did the
same thing with shader_runner, it seems like a good idea.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/glslparsertest/glslparsertest.c | 54
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
tests/glslparsertest/glslparsertest.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/glslparsertest/glslparsertest.c
b/tests/glslparsertest/glslparsertest.c
index 4620b4e..14a7a13
From: Ian Romanick ian.d.roman...@intel.com
Mesa currently fails array-sized-by-initializer.vert.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
.../compiler/array-sized-by-initializer.vert | 31 ++
.../compiler/no-unsized-arrays-01.vert | 25
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
.../glsl-es-3.00/compiler/line-continuation.vert | 27 ++
1 file changed, 27 insertions(+)
create mode 100644 tests/spec/glsl-es-3.00/compiler/line-continuation.vert
diff
The series is
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
.../egl/spec/egl_khr_create_context/verify-gl-flavor.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/tests/egl/spec/egl_khr_create_context/verify-gl-flavor.c
b/tests/egl/spec/egl_khr_create_context
On 11/15/2012 11:10 AM, Chad Versace wrote:
The series adds support for OpenGL ES3 to the piglit_gl_framework.
The series doesn't add any GLES3 tests, just support for the API.
Do you have a test case in hand to be sure your changes actually work?
Chad Versace (3):
util/gl: Add config
From: Ian Romanick ian.d.roman...@intel.com
This is related to the WebGL shader-with-non-reserved-words tests.
Several of these tests use name that are reserved words in other GLSL
versions as variable names, structure names, structure field names,
and function names. Some of the cases use
From: Ian Romanick ian.d.roman...@intel.com
Currently on Mesa's gles3 branch this leads to an assertion failure.
It works correctly on Mesa master and known Mesa releases.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
I'm not sure this is the right place for this test script to live
On 12/03/2012 09:59 PM, Chad Versace wrote:
all.tests already auto-detects all files named *.shader_test, and runs
them with shader_runner. This patch does the same for files
*.shader_test_gles3 and runs them with shader_runner_gles3.
Yuck. :( It seems like the python script could grep for
On 12/03/2012 09:59 PM, Chad Versace wrote:
all.tests already auto-detects all files named *.shader_test, and runs
them with shader_runner. This patch does the same for files
*.shader_test_gles3 and runs them with shader_runner_gles3.
Yuck. :( It seems like the python script could grep for
Gak! Sorry for the multiple sends. My mailer kept telling me that
delivery failed, and asked if I wanted to resend. Apparently it was
full of crap. :(
On 12/04/2012 05:43 PM, Ian Romanick wrote:
On 12/03/2012 09:59 PM, Chad Versace wrote:
Since piglit-dispatch does not yet support GLES
On 12/04/2012 06:19 PM, Stuart Abercrombie wrote:
With this change they run on, for instance the last 6 years of Ubuntu.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
tests/asmparsertest/make_test_list.sh |2 +-
tests/asmparsertest/vp-tex.sh
On 12/04/2012 06:24 PM, Stuart Abercrombie wrote:
It was incorrectly hardwired to 1.10, leading to contrdictory output.
Oops. That was my mistake. I got the other place correct. :)
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
tests/spec/glsl-1.10/variable-index-write.sh |2
From: Ian Romanick ian.d.roman...@intel.com
The spec says that the name of a uniform block is in the same scope as
other symbols. Verify that.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Kenneth Graunke kenn...@whitecape.org
---
.../compiler/block-name-clashes-with-function.vert
On 12/11/2012 10:12 AM, Matt Turner wrote:
On Tue, Dec 11, 2012 at 9:57 AM, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
+void
+piglit_init(int argc, char **argv)
+{
+ /* ES 3 adds many new queries over ES 2. This test confirms that ES 2
+ * correctly
From: Ian Romanick ian.d.roman...@intel.com
This test passes on NVIDIA's closed-source driver (version 304.64 on
GTX 260).
This test fails on AMD's closed-source driver (Catalyst 12.6 on HD
3650). Queries of GL_NUM_SAMPLE_COUNTS merrily write data even when
bufSize is zero.
Signed-off-by: Ian
From: Ian Romanick ian.d.roman...@intel.com
This test passes on NVIDIA's closed-source driver (version 304.64 on
GTX260). This driver supports GL_ARB_texture_multisample.
This test fails on AMD's closed-source driver (Catalyst 12.6 on HD
3650). Querying GL_SRGB8_ALPHA8 (for all targets
From: Ian Romanick ian.d.roman...@intel.com
This test fails on NVIDIA's closed-source driver (version 304.64). It
fails because the value of GL_PROGRAM_BINARY_RETRIEVABLE_HINT is not
latched by glLinkProgram. The spec is a little unclear on this
topic, and I have filed a bug in the Khronos
From: Ian Romanick ian.d.roman...@intel.com
This test fails on NVIDIA's closed-source driver (version 304.64):
- GL_INVALID_VALUE is not generated for specifying values other than
GL_TRUE or GL_FALSE for GL_PROGRAM_BINARY_RETRIEVABLE_HINT.
- GL_INVALID_VALUE is not generated
A lot of shader_runner tests are failing with the message Failed to
find [require] block. Could the default of requiring desktop GL + GLSL
1.10 please be restored?
I haven't bisected, but I'm guessing that this happened when support for
GLSL ES 3.00 was added...
On 12/11/2012 12:23 PM, Tom Gall wrote:
Hi,
I've been looking at how to take advantage of the glslparsertest code
so that it could be used for OpenGL ES 2 testing as well.
Thought I'd ask for an opinion on versions for glsl on gles2.
For gles2, should the lower bound of valid versions be
On 12/11/2012 04:13 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
+/**
+ * \file overrun.c
+ * Verify that queries don't over-run the size of the supplied buffer.
+ */
+
+#include piglit-util-gl-common.h
+
+PIGLIT_GL_TEST_CONFIG_BEGIN
On 12/11/2012 04:03 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
This test passes on NVIDIA's closed-source driver (version 304.64 on
GTX 260). This driver supports GL_ARB_texture_multisample, but it
does not support
On 12/13/2012 08:33 AM, Tom Gall wrote:
Within many of the frags in tests/glslparsertest/shaders there is the
following in the config section:
// # NOTE: Config section was auto-generated from file
// # NOTE: 'glslparser.tests' at git revision
// # NOTE: 6cc17ae70b70d150aa1751f8e28db7b2a9bd50f0
On 12/13/2012 10:25 AM, Matt Turner wrote:
Reviewed-by: Eric Anholt e...@anholt.net (v2)
Two minor nits below. With those fixed (no need to resubmit),
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
tests/spec/gles-2.0/CMakeLists.gles2.txt |2 +
tests/spec/gles-2.0/invalid
From: Ian Romanick ian.d.roman...@intel.com
Both the block name and the names of the individual variables in the
block live at global scope. Therefore, they can't both have the same
names. The GLSL 1.40 spec says:
Uniform block names and variable names declared within uniform
blocks
On 12/13/2012 10:00 AM, Stuart Abercrombie wrote:
The idea is to ultimately remove explicit #version directives from all
.shader_test files, so we can just rely on GLSL =, and then the ES
analog, allowing these tests to run on GL or GLES -- what Paul Berry
outlined a while back, basically.
This
On 12/13/2012 12:58 PM, Tom Gall wrote:
This is a small change in two parts to repair support for
gles2 by glslparsertest.
To glslparsertest.c, add 100 into the list of versions the switch
handles when passed from the command line instead of the default
path. Next in piglit_init when version
From: Ian Romanick ian.d.roman...@intel.com
This is required by OpenGL ES 3.0 and desktop OpenGL 4.2. Previous
version were ambiguous.
Previously these tests failed on NVIDIA's closed-source driver (version
304.64) with the error messages:
0: f1 loc=0 size=1 type=0x1406
1: f2 loc=1
From: Ian Romanick ian.d.roman...@intel.com
This test passes on NVIDIA's closed-source driver (version 304.64 on
GTX 260). This driver supports GL_ARB_texture_multisample, but it
does not support GL_ARB_internalformat_query2.
This test fails on AMD's closed-source driver (Catalyst 12.6 on HD
From: Ian Romanick ian.d.roman...@intel.com
This test passes on NVIDIA's closed-source driver (version 304.64 on
GTX 260).
This test fails on AMD's closed-source driver (Catalyst 12.6 on HD
3650). Queries of GL_NUM_SAMPLE_COUNTS merrily write data even when
bufSize is zero.
v2: Add some
On 12/18/2012 10:50 AM, Eric Anholt wrote:
---
tests/all.tests|4 +
.../arb_uniform_buffer_object/CMakeLists.gl.txt|1 +
.../maxuniformblocksize.c | 237
3 files changed, 242 insertions(+)
On 12/19/2012 08:30 AM, Paul Berry wrote:
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Now for the bad news...
In GLSL ES 3.00 and GLSL 1.50 this language changes to:
They can only be float, floating-point vectors, matrices,
signed or unsigned integers or integer vectors
On 12/19/2012 08:30 AM, Paul Berry wrote:
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
.../varying-packing-mixed-types.shader_test| 60 ++
1 file changed, 60 insertions(+)
create mode 100644
tests/spec/glsl-1.30/execution/varying-packing-mixed
On 12/31/2012 11:14 AM, Paul Berry wrote:
This patch adds compressed representations of uniform grayscale images
to piglit, in the ETC1 and FXT1 compression formats. Although they
don't exercise the full capabilities of these compression formats,
they should be sufficient to create basic tests
On 01/03/2013 02:39 PM, Eric Anholt wrote:
Paul Berry stereotype...@gmail.com writes:
+#ifdef PIGLIT_USE_OPENGL
+#define GRAYSCALE_IMAGES piglit_fxt1_grayscale_images
+#define COMPRESSED_FORMAT GL_COMPRESSED_RGB_FXT1_3DFX
FXT1 does mean that non-intel drivers won't run this test. Not
, even though I had to rewrite some GLSL
code from ints to floats. (texelFetch - texture2DRect; % - mod)
The series is
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Marek
___
Piglit mailing list
Piglit@lists.freedesktop.org
http
On 01/22/2013 02:29 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
OpenGL ES 3.0 (and presumably other versions of OpenGL) have two
requirements that these tests expect to be violated:
1. Members of a uniform block
From: Ian Romanick ian.d.roman...@intel.com
OpenGL ES 3.0 requires that the row_major and column_major layout
qualifiers only be accepted on matrix types. Previously this test would
put the layout qualifier on any type.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
From: Ian Romanick ian.d.roman...@intel.com
OpenGL ES 3.0 (and presumably other versions of OpenGL) requires that
UBO members that are not accessed by the shader not be marked as
active. Previously these tests assumed that all members would be active
regardless of use.
Signed-off-by: Ian
These changes seem innocuous enough.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
On 01/28/2013 10:30 AM, Tom Gall wrote:
Android is a bit more strict when it builds. Missing was ctype.h.
It doesn't break Linux.
Signed-off-by: Tom Gall tom.g...@linaro.org
---
tests/util/piglit-vbo.cpp
From: Ian Romanick ian.d.roman...@intel.com
The tests would previously write 0.0 to the alpha and expect to read 1.0
back. If the test got a visual without alpha (and it doesn't request
alpha), this would happen to work. However, drivers are not required to
expose visuals without alpha, so
From: Ian Romanick ian.d.roman...@intel.com
This is required by OpenGL ES 3.0 and desktop OpenGL 4.2. Previous
version were ambiguous.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Frank Henigman fjhenig...@google.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59688
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59833
---
tests/glx/glx-swap-pixmap.c | 16 ++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/tests/glx/glx-swap
On 01/28/2013 04:44 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
The tests would previously write 0.0 to the alpha and expect to read 1.0
back. If the test got a visual without alpha (and it doesn't request
alpha), this would
On 01/28/2013 05:18 PM, Frank Henigman wrote:
Ignore this. Ian sent a better patch at the same time.
Should I count that as a Reviewed-by for my patch? :)
On Mon, Jan 28, 2013 at 4:18 PM, Frank Henigman fjhenig...@google.com
mailto:fjhenig...@google.com wrote:
Make
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Chris Wolfe cwo...@chromium.org
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=53256
---
.../shaders/glsl-vs-channel-overwrite.shader_test | 42 ++
1 file changed, 42
From: Ian Romanick ian.d.roman...@intel.com
v2: Add several other, similar test cases that tickle other related bugs
in the same optimization passes.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Chris Wolfe cwo...@chromium.org
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id
On 02/02/2013 10:40 AM, Jordan Justen wrote:
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
The short long and the longer description should describe what the test
is testing. I've looked at the test, and I'm still not really sure what
it's doing. Just add a link in the commit
From: Ian Romanick ian.d.roman...@intel.com
This somehow went missing from commit 10628a0.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=60298
Cc: Vinson Lee v...@freedesktop.org
---
tests/spec/arb_uniform_buffer_object/row-major.c
On 02/19/2013 12:00 PM, Eric Anholt wrote:
The shaders compile failed as of
10628a004e18190b015bacf7640b638c51db82d0
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59648
---
.../layout-std140-base-size-and-alignment.c |2 +-
1 file changed, 1
On 02/19/2013 11:40 AM, Eric Anholt wrote:
Makes the test pass on Mesa master.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59957
With the spurious hunk removed,
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Since I wrote the exact same patch at about the exact same time
From: Ian Romanick ian.d.roman...@intel.com
Appendix E.1 (Profiles and Deprecated Features of OpenGL 3.0) of the
OpenGL 3.1 spec says:
• Depth texture mode - DEPTH TEXTURE MODE. Section 3.9.14 is to be
changed so that r is returned to texture samplers directly, and
the OpenGL
From: Ian Romanick ian.d.roman...@intel.com
There are several variations of
void foo(inout int i, inout float f);
...
foo(i, f[i]);
that Mesa's GLSL compiler gets wrong. NVIDIA's closed-source driver
(version 304.64) fails vs-inout-index-inout-mat2-col.shader_test and
vs-inout-index-inout-mat2
On 03/22/2013 03:28 PM, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
Similar to glsl-fs-main-return (and glsl-vs-main-return), this is testing
using return in main. Contrary to the these other tests, this hits both
the cases where the return path is and is NOT taken
On 03/25/2013 09:04 AM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
There are several variations of
void foo(inout int i, inout float f);
...
foo(i, f[i]);
that Mesa's GLSL compiler gets wrong. NVIDIA's closed-source driver
1 - 100 of 921 matches
Mail list logo