Age | Commit message (Collapse) | Author | Files | Lines |
|
The tests for the integer/long minimums do not properly
check that the value is stored in the correct type.
E.g. (-2147483648) actually gets parsed as a long by the preprocessor,
and therefore INT_MIN is actually stored as long and can't be automatically
vectorized.
By subtracting a vector with value of 0 from the given defined *_MIN
and then grabbing the first element of the resulting vector, we can
make sure that the values are actually stored in the correct type.
v2: Remove char/short modifications.
Test *_MAX vectorization as well as _MIN for int/long
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
Reported-By: Moritz Pflanzer <moritz.pflanzer14@imperial.ac.uk>
Signed-off-by: Aaron Watry <awatry@gmail.com>
|
|
To silence warning: "/*" within comment.
|
|
To silence warning: implicit declaration of function ‘dsa_create_program’
|
|
Replaced by new piglit gl-2.0-two-sided-stencil test.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This test replaces the original glean tstencil2.cpp test.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Check that glCreateTextures rejects target=0, per the spec. Also, Mesa
internally uses target=0 for a special state (texture gen'd but not bound)
so it's good to test.
|
|
Previously the test was trying to convert a sample number to the range
[0,1] by dividing by 10.0. This doesn't work when testing 16x MSAA
because it ends up with values greater than 1.0 which get clamped.
This patch makes it divide by miplevels so that it will work whatever
the number of samples is.
Cc: Chris Forbes <chrisf@ijw.co.nz>
Cc: Marek Olšák <marek.olsak@amd.com>
|
|
V3: rebase and test double types
V2: renamed variables from 'inner' that should have been 'outer'
|
|
These tests check that 2D texture arrays work for the 5x5 and
12x12 block sizes.
v2. add to all.py (Ilia).
create test for gles2.
v3. upgrade required GLES version to 3.1 (for 2d array support).
v4. Move CONFIG block to the top of the file (Chad).
Acked-by: Chad Versace <chad.versace@intel.com>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
|
|
These tests check that the GL error returned by valid and invalid API calls
are as defined by the spec.
v2. add test to all.py (Ilia).
update copyright date (Dylan).
numerous changes (Chad).
v3. cube map array support is in OpenGL ES 3.2 (not 3.0).
Reviewed-by: Chad Versace <chad.versace@intel.com>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
|
|
These tests run through every ASTC configuration, comparing the
render of a compressed texture against a render of the decompressed
version of that compressed texture. The compressed and decompressed
texture was generated with a reference codec.
v2. numerous changes (Chad).
v3. upgrade required GLES version to 2.
Reviewed-by: Chad Versace <chad.versace@intel.com>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
|
|
ASTC formats are added to the list of formats that should not be returned by the
COMPRESSED_TEXTURE_FORMATS query.
v2. add astc test to all.py (Chad).
Reviewed-by: Chad Versace <chad.versace@intel.com>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
|
|
This function compares two rectangles for equality. This is useful
to compare the rendering of individual miplevels in a miptree while
avoiding too much resource consumption.
v2: insert spaces in addition operation (Chad).
Reviewed-by: Chad Versace <chad.versace@intel.com>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
|
|
Without glBindTexture(), the test was unintentionally modifying the default
texture object. Make it use the generated texture instead.
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
|
|
Use the piglit utility function for determining format block dimensions
instead of using a test-specific function. This requires adding some formats
not originally in the piglit-util function.
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
|
|
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>
|
|
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>
|
|
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>
|
|
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
|