Age | Commit message (Collapse) | Author | Files | Lines |
|
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
|
|
in more cases.
The subtest with at most the maximum number of atomic counter buffers
can be run whenever the combined limit is at least one more than the
limit for the FS stage. The subtest exceeding the maximum number of
combined atomic counter buffers can be run whenever the sum of the
limits for the VS and FS stages is greater than the combined limit.
Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
|
|
offsets in the VS of the combined test.
It is illegal for different atomic counter uniform declarations to
overlap, even across stages, use different offsets in the VS to avoid
that.
Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
|
|
literal.
This subtest was probably only tested on nVidia and this happened to
work by luck.
Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
|
|
max-counters.
This isn't used anywhere except for the max-counters test so we can
move the definition into the C file of that test and make it static.
Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
|
|
Check for GL_INVALID_VALUE, not GL_INVALID_OPERATION. The extension spec
isn't clear on this, but GL_INVALID_VALUE is typically generated for
out-of-range int/uint values.
That's what NVIDIA does. Plus, that's what the other bind-texture-unit
piglit test checks for at line 91. Let's at least be consistent.
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
|
|
After looking at CTS I realised we have to accept all types
for get uniform for double conversion, and we should do double
conversion on the non-double types.
I assume this is due to it being the only interface to check
if the location has a value or something.
This adds getting non-double values with GetUniformdv, and
getting double values with float/int/uint interfaces.
notes: currently mesa fails this.
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
|
|
Like command, this somehow didn't get moved to the new TestResult class.
Generally this isn't a problem since this is a very uncommon path, but
in some cases this does get hit, and it would cause an exception in the
runner threads. Python has very bad exception handling in threads,
namely it doesn't have any; this results in an exception being raised
and remaining uncaught, terminating the thread, which in turn results in
the test result being incomplete, even though it should have been crash.
This patch fixes that situation.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Tested-by: Mark Janes <mark.a.janes@intel.com>
Reviewed-by: Mark Janes <mark.a.janes@intel.com>
|
|
CTS tests this a lot better than piglit, but this is a good place
to start, I've got a bunch of fixes queued for mesa to address
some of these problems.
This adds 3 tests:
simple double vs->tcs->tes->fs
simple dvec2 vs->tcs->tes->fs
simple double[2] vs->tcs->tes->fs
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
This tests vertex shaders work with shader subroutine.
Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
It's like glxinfo or wglinfo, but for OpenCL.
Reviewed-by: Serge Martin <edb+piglit@sigluy.net>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
|
|
The command is:
ssbo <size>
where <size> is the size in bytes to allocate for the SSBO buffer.
This command only sets of a buffer for SSBO index 0.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
|
|
|
|
|
|
v2: (Jan Vesely comments)
Remove extra whitespace
Single flag test cases
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
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>
|
|
v2: Fix typo in comment noticed by Matt. When interpolating from the
north pole to the south pole of the sphere, correctly interpolate from 0
to pi instead of 0 to 2pi. See the "NOTE" in the code.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Acked-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>
|
|
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 fixes the lists of regressions and fixes to not include skip or
notrun statuses.
This fixes a regression introduced in the recent summary rework of
piglit.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
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>
|
|
This was an oversight, it should have always been written to the json,
but was missed in the to_json method.
cc: Dave Airlie <airlied@gmail.com>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This patch fixes the handling of subtests when more than one result is
presented. Currently this will raise a KeyError and die horribly.
cc: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Sub-classing built-in types in python is an error-prone leap, and many
corner cases came out to bit us in this instance. By wrapping a dict
inside of another class we can get the desired results, it means more
code, but it also means less corner cases.
Also change the equality test of two unit tests, since Subtests isn't a
dictionary anymore assert_dict_equal wont work anymore.
cc: Mark Janes <mark.a.janes@intel.com>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
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>
|