Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
The test sets the filter modes to GL_NEAREST, so levels other than zero
are never used.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Why was gluOrtho2D ever even a function?!?
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
I verified correctness by calling gluProject inside project and
comparing the results. My initial implementation forgot to do the
perspective divide, so this helped catch that bug.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
I think this was a holdover from having originated in glean.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
No change in output on Broadwell using my distro's Mesa 10.4.7 or Mesa
master 11.1-ish.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Move some of the rest to piglit_init.
Table 6.10 (page 275) in the OpenGL 2.1 spec says that GL_DEPTH_RANGE
(set by glDepthRange) is initially 0,1.
Table 6.10 (page 275) in the OpenGL 2.1 spec says that GL_NORMALIZE is
initially disabled.
Table 6.11 (page 276) in the OpenGL 2.1 spec says that GL_FOG is
initially disabled.
Table 6.12 (page 277) in the OpenGL 2.1 spec says that GL_COLOR_MATERIAL
is initially disabled.
Table 6.12 (page 277) in the OpenGL 2.1 spec says that GL_LIGHTING is
initially disabled.
Table 6.14 (page 279) in the OpenGL 2.1 spec says that
GL_POLYGON_STIPPLE is initially disabled.
Table 6.14 (page 279) in the OpenGL 2.1 spec says that GL_POLYGON_MODE
(set by glPolygonMode) is initially GL_FILL.
Table 6.14 (page 279) in the OpenGL 2.1 spec says that GL_FRONT_FACE
(set by glFrontFace) is initially GL_CCW.
Table 6.14 (page 279) in the OpenGL 2.1 spec says that GL_CULL_FACE_MODE
(set by glCullFace) is initially GL_BACK.
Table 6.16 (page 281) in the OpenGL 2.1 spec says that GL_TEXTURE_*D is
initially disabled.
Table 6.20 (page 285) in the OpenGL 2.1 spec says that GL_ALPHA_TEST is
initially disabled.
Table 6.20 (page 285) in the OpenGL 2.1 spec says that GL_SCISSOR_TEST
is initially disabled.
Table 6.20 (page 285) in the OpenGL 2.1 spec says that GL_STENCIL_TEST
is initially disabled.
Table 6.21 (page 286) in the OpenGL 2.1 spec says that GL_BLEND is
initially disabled.
Table 6.21 (page 286) in the OpenGL 2.1 spec says that GL_COLOR_LOGIC_OP
is initially disabled.
Table 6.22 (page 287) in the OpenGL 2.1 spec says that GL_COLOR_CLEAR_VALUE
(set by glClearColor) is initially 0,0,0,0.
Section 4.2.2 (Fine Control of Buffer Updates) of the OpenGL 2.1 spec
says (about glColorMask):
"In the initial state, all bits (in color index mode) and all color
values (in RGBA mode) are enabled for writing."
and (about glDepthMask):
"In the initial state, the depth buffer is enabled for writing."
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
It would be trivial it there weren't so many of them.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
This patch was created by:
sed -i -e '/[{]OPENGL_glu_LIBRARY[}]/d' $(grep -lr OPENGL_glu_LIBRARY)
git checkout tests/glean/CMakeLists.gl.txt
git checkout tests/spec/gl-1.4/CMakeLists.gl.txt
git checkout tests/fbo/CMakeLists.gl.txt
So, there are three places out of all of piglit that actually need
this library.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Thanked-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Reported-by: Francisco Jerez <currojerez@riseup.net>
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
|
|
This adds support for running Khronos' deqp-based conformance suite with
piglit. This gives access to all of the powerful features of pigilt,
per-process tests, the junit and json backends, the summary tools, and
the familiar interface, with minimal fuss.
This is a very small change, since it is deqp-based, and piglit has a
framework for handling deqp based suites already.
Reviewed-by: Mark Janes <mark.a.janes@intel.com>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Tests the second bug mentioned in the bug report.
Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=83508
|
|
This is the obvious thing to do. This eliminates the need to explicitly
link to libGL in every other CMakeLists.gl.txt.
Tested-by: Nanley Chery <nanley.g.chery@intel.com>
|
|
This fails with current Mesa due to a bug in commit:
4639cea2921669527eb43dcb49724c05afb27e8e
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
This test verifieds that rendering and compute operations can be
interleaved.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
These are a very simplified version of the similar image load/store
versions.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
|
|
v2:
- Modify command format.
Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
This fixes the test so it passes on the Nvidia binary driver.
|
|
Missed this change when committing "Port arb occlusion query
conformance tests from Glean to Piglit".
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
In compatiblity profiles, the GL_ARB_separate_shader_objects extension
allows passing data via built-in varyings such as gl_TexCoord[]. We
don't do compatibility profiles, but we do expose SSO in legacy GL
contexts and allow it with GLSL 1.30 and earlier.
This test actually tries to do that in a rendering test.
This is particularly interesting because Mesa's VARYING_SLOT_* enums
handle built-in varyings different than generic ones. I wanted to be
able to see how those came through; this provides a simple example.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
Having more than two stages makes SSO interface matching a lot more
interesting. However, the five-stage variant won't run on i965 for
a while. So, this patch adds a three-stage variant (VS/GS/FS, but
no tessellation).
Beyond that, this test is a little meaner: I made the VS have more
outputs than the GS has inputs, with the locations specified to have a
gap. An implementation that lays out VS outputs and GS inputs
contiguously would fail; they have to match up properly.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
This is a very invasive patch, because it replaces one of our core
data-structures with a completely different kind of object. This new
object is not a dict-like object (it doesn't use obj[key] = value
syntax), instead it's a standard object with the standard object
attributes. So result['time'] becomes result.time.
This approach has a couple of advantages. First, it allows us to use
properties, which allows us to encapsulate a lot of distributed logic
from the summary module in the results module, and in a way that is
easier to test for correctness. The second advantage of that
encapsulation is that correct behavior is used everywhere, instead of
just in most places. Finally, it allows us to use the flyweight pattern
on the results objects, limiting the amount of memory consumed.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Just like the non-array test. Always use layer=1.
On NVIDIA's closed-source driver the 4x test passes. The 2x, 6x, and 8x
test for both array and (previously existing) non-array textures fail.
NOTE: This test currently fails on the Mesa i965 driver.
v2: Fix dumb argument parsing that caused all the non-array tests to
fail.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
Tested-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
If the vertex shader inputs are named piglit_vertex and piglit_texcoord,
the utility function will handle everything for us.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Trivial.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Replaced by new piglit test.
Signed-off-by: Brian Paul <brianp@vmware.com>
|
|
This test replaces the original glean toccluqry.cpp test.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Replaced by new piglit test.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test replaces the original glean tpaths.cpp test.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This internal depth/stencil format was added in GL 3.0
We need to handle a few things specially for this format:
1. The random float data must be in the range [0,1].
2. When we compare pixels, we must skip the 3 unused bytes in the pixel.
This patch also simplifies the array indexing code in check_texture().
Note: This format fails with NVIDIA's 352.21 driver (at least).
It passes with Mesa softpipe with the proposed GL_ARB_copy_image patch
series.
v2: check for GL_ARB_depth_buffer_float instead of GL 3.0
|
|
Instead of printing potentially a thousand error messages when
there's a failure.
Also, some minor code reformatting.
v2: remove passrate code too, per Anuj.
|
|
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
When fixing some ES 3.1 conformance issues I noticed several subtests
started to fail. Pipeline has to be bound before validation, this makes
failing subtests to pass with planned Mesa changes.
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
|
|
This was never added to all.py
cc: Mike Stroyan <mike@lunarg.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This was never added to all.py
cc: Eric Anholt <eric@anholt.net>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Albert Freeman <albertwdfreeman@gmail.com>
|
|
This was never added to all.py.
cc: Eric Anholt <eric@anholt.net>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
I did this wrong when I converted the format of all.py
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This was never added to all.py, and was detected by my check.py script.
This fails on i965 against mesa master
cc: Jason Ekstrand <jason.ekstrand@intel.com>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Albert Freeman <albertwdfreeman@gmail.com>
|