Age | Commit message (Collapse) | Author | Files | Lines |
|
this just writes attribs and reads the current value back.
Signed-off-by: Dave Airlie <airlied@redhat.com>
Conflicts:
tests/all.py
tests/spec/CMakeLists.txt
|
|
While developing virgl I found this combo wasn't tested by any
of the current tests, emit, skip, next buffer, emit.
Just add a test to this to exercise the codepath.
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
This adds the tests so far to all.py.
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
All the extensions get fbo-colormask-formats called using the
add_fbo_formats_tests test helper, but core formats need to be added
manually.
v2: Rebase on massive refactor.
Reviewed-by: Dylan Baker <baker.dylan.c@gmail.com> (v1)
|
|
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
Signed-off-by: Martin Peres <martin.peres@linux.intel.com>
|
|
Brian's mixed buffer sizes tests broke all.py.
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
GL_ARB_framebuffer_object (unlike GL_EXT_framebuffer_object) allows
the FBO attachments to be off different sizes. AFAICT, we had no
tests to exercise rendering with mixed buffer sizes.
Signed-off-by: Brian Paul <brianp@vmware.com>
|
|
To exercise a Mesa failure after the format overhaul.
Note: this test fails on nvidia if the -auto option is given.
Signed-off-by: Brian Paul <brianp@vmware.com>
|
|
Signed-off-by: Brian Paul <brianp@vmware.com>
|
|
Tests the error paths of GetProgramResourceName
Tested on NVIDIA's proprietary driver version 346.35.
v2:
- Test that a NULL name does not crash (suggested by Tapani)
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
Signed-off-by: Martin Peres <martin.peres@linux.intel.com>
|
|
Tests tricky cases of resource naming along with some error cases of
getprogramresourceindex. It also replaces the resource-index test that
was incomplete and did not consistently report the same number of
subtests depending on errors.
Tested on NVIDIA's proprietary driver version 346.35.
v2:
- also test a NULL name (Tapani)
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
Signed-off-by: Martin Peres <martin.peres@linux.intel.com>
|
|
This test exercises the error codepaths of getprograminterfaceiv.
Tested on NVIDIA's proprietary driver version 346.35.
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
Signed-off-by: Martin Peres <martin.peres@linux.intel.com>
|
|
This tests primarily glGetProgramInterfaceiv in various (potentially
tricky) program pipelines. All the calls are supposed to succeed as
this test is purely functional.
This test requires a OpenGL 3.2 core context but also has optional
subtests that depend on the following extensions:
- GL_ARB_shader_atomic_counters
- GL_ARB_shader_storage_buffer_object
- GL_ARB_shader_subroutine
- GL_ARB_tessellation_shader
- GL_ARB_compute_shader
- GL_ARB_shader_image_load_store
Tested on NVIDIA's proprietary driver version 346.35.
v2: Martin Peres
- Fix the cs subtest to test PROGRAM_INPUT and not two times the OUTPUT
v3: Review from Tapani
- require GL_ARB_separate_shader_objects
- cosmetic changes
- add a comment in common.h to warn about the existence of strings in
resource-query.c
v4: Martin Peres
- add a second subroutine in vs_sub
v5: Martin Peres
- introduce a patch output in the TCS outputs
Signed-off-by: Martin Peres <martin.peres@linux.intel.com>
|
|
glClientWaitSync(fence, GL_SYNC_FLUSH_COMMANDS_BIT, GL_TIMEOUT_IGNORED)
may return GL_TIMEOUT_EXPIRED, even though GL_TIMEOUT_IGNORED is a huge
integer (18446744073709551615 on Intel Sandybridge with Mesa). The third
argument to glClientWaitSync should represent the number of nanoseconds the
implementation waits for the signal from fence before giving up and returning
TIMEOUT_EXPIRED.
This test, adapted from bufferstorage-persistent, demonstrates that
glClientWaitSync(fence, GL_SYNC_FLUSH_COMMANDS_BIT, GL_TIMEOUT_IGNORED)
must be called multiple times before the signal is actually recorded and
either GL_CONDITION_SATISFIED or GL_ALREADY_SIGNALED is returned.
This bug was successfully fixed by Mesa commit 10c82c6 by Kristian Hogsberg.
Reviewed-by: Martin Peres <martin.peres@linux.intel.com>
|
|
This test reproduces the cause of Mesa Bug 89526 in Piglit. While
investigating 89526, it was discovered that Piglit had no tests that created a
cube map texture without glTexStorage2D and then called glGenerateMipmap on
it. For this reason, the offending commit was upstreamed before the failure
was caught.
This test successfully fails when commit 1ee000a is present and passes when
1ee000a is reverted.
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Because gs is a 1.50 features, but is added in the 1.30 and 1.40 loops
it currently adds the same tests twice. This corrects that.
NOTE: Ilia and I aren't sure this is 100% correct, but it doesn't change
the current behavior.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Same as previous patch.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
This doesn't actually effect the nature or number of tests run.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
This test was added twice, remove the second copy.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
I traced these two tests back through git history, and through a series
of errors an option required to make them unique was dropped, this
corrects that and brings them back to their original state.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Fixes list for compressedtextureimage.
Trivial.
Reviewed-by: Dylan Baker <baker.dylan.c@gmail.com>
|
|
arb_direct_state_access-compressedtextureimage should use a more mainstream
compression format in all.py. This patch changes it to
GL_COMPRESSED_RGBA_S3TC_DXT5_EXT.
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|
|
Trivial.
|
|
This is a rip off of:
https://www.khronos.org/registry/webgl/conformance-suites/1.0.2/conformance/attribs/gl-vertex-attrib-render.html
Which presently fails with gallium drivers. The logic is identical, just
ported to GL and piglit infra.
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
|
|
This adds clip control to the set of piglit tests.
This time with an other bunch of tests covering
different depth range settings.
v2:
Check for ARB_depth_buffer_float where used.
Fix and extend comments.
Introduce helper for reading and checking the clip control state.
Split state and draw related parts into separate methods.
Test for Push/PopAttrib.
v3:
Add tests to tests/all.py.
v4:
Use piglit_draw_rect_z instead of a hand written function.
v5:
Check for different depth range values.
To implement the above, refactor the implementation.
Rebase to current upstream.
[airlied: fixed up all.py]
Signed-off-by: Mathias Froehlich <Mathias.Froehlich@web.de>
Tested-by Ilia Mirkin <imirkin@alum.mit.edu>
CC: Brian Paul <brianp@vmware.com>
CC: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
This was the test case that I wrote for Mesa commit b93dcb0.
Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
run_concurrent=True is the default in PiglitGLTest, so there is no need
to set it. The majority of the cases of it's use are rebase errors, or
the result of patches that were pushed at the time that the flag was
switched.
This should result in no functional change, but will hopefully result in
fewer copy and paste errors.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Dylan Baker <baker.dylan.c@gmail.com>
|
|
This fixes the following assertion on Windows:
AssertionError: Groups are not paths and cannot contain \. (glx\glx_ext_import_context)
Reviewed-by: Dylan Baker <baker.dylan.c@gmail.com>
|
|
On Intel hardware at least, SIMD16 dual source rendering requires handling
pixel data in two sets of 8 pixels each. Incorrect implementations may fail
to map correct colors for each pixel group (for example by using the color
for the first group as the color for the second group or viceversa). However,
tests that render using solid colors across the entire polygon won't catch
these cases (since in that case the color is the same for boths groups of
pixels).
This test blends using a checker board pattern where each cell is
10px wide and 10px tall. This makes it so that the two sets of 8 pixels
issued during SIMD16 operation pack different color data for the pixels
involved, enabling testing of correct behavior in that case.
v2 (Matt Turner)
- Make the test render to a window if -fbo or -auto are not present.
Reviewed-by: Matt Turner <mattst88@gmail.com>
|
|
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Dylan Baker <baker.dylan.c@gmail.com>
|
|
This fixes several tests on platforms that support
arb_transform_feedback3.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Mark Janes <mark.a.janes@intel.com>
|
|
This corrects a copy and paste error in all.py that matches the wrong
binary with a test name. This causes an artificial piglit regression on
mesa, at least for the Intel driver. This patch fixes that typo. Trivial
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This is a bug (https://bitbucket.org/logilab/pylint/issue/461/incorrect-hanging-indent-detection-with)
that misdetects the hanging indent of the context manager. This silences
roughly 300 warnings.
v2: - also turn this warning off in quick.py
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
The previous commit made True the default value. This removes ~400 LOC
from all.py (about 10%)
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This will flip the value of the flag for PiglitGLTest, ShaderTest, and
GLSLParserTest. It will not affect PiglitCLTest (That's OpenCL), because
of the previous patch, which causes PiglitCLTest to use a unique
concurrency setting method.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This will allow us to flip the default value of run_concurrent to True
in the next patch. Since the majority of tests are concurrent safe (and
honestly many of them that are marked not safe are), this change will
help to reduce the number of tests that are marked serial by default by
copy-n-paste.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This simplifies a egl specific tests.
|
|
This allows the simplification of a large number of tests, reducing them
to one argument, or at least reducing the number of lines that they
occupy (or should occupy with reasonable line wrapping).
|
|
This works much like the shader_test and glsl parser test options, the
difference being that rather than searching the whole tree only the
directories with asmparser tests are searched.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This patch is massive, with a huge amount of churn, because it rewrites almost
the entirety of all.py. It has to touch all of the helper functions as well. Of
these I modified them appropriately or removed them. Criteria for removal was
based on the complexity of the wrapper, and the frequency of use. Very simple
functions or those used only a couple of times were removed.
v2: - remove duplicate tests entries.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This makes it a lot easier to read and work with all.py, which is still
absolutely massive.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
These tests are already formated using grouptools, with no intermediate
tree structures. Moving them to the flat test list reduces the amount of
time spent migrating them to test_list before running tests.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
commit 36b63cbbe7 fixed the execution of two tests, but didn't fix the
fact that two tests are stored with the same name, thereby replacing the
first with the second. This corrects that.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Acked-by: Laura Ekstrand <laura@jlekstrand.net>
Reviewed-by: Martin Peres <martin.peres@linux.intel.com>
|
|
Test passes on Nvidia GTX 660 (binary driver version 331.38).
v2: separate atomic counter index subtest (Ilia Mirkin)
+ code and shader cleanups
+ added subroutine index query test
v3: fix indentation (argh!)
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
|
|
v2: review from Tapani
- add some documentation
- set the minimum GL version to 2.0
v3:
- set the minimum GL version to 3.1 core (Ilia)
- do some clean up at the end of the test
Signed-off-by: Martin Peres <martin.peres@linux.intel.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
|