Age | Commit message (Collapse) | Author | Files | Lines |
|
Tests try drawing into layer 1 of an framebuffer with no layers, so it's
still out of range and thus invalid.
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/913>
|
|
Intel compiler has a very old optimization for various combination
of u2f or i2f of an extract_* source. However, the variations that
do u2f of an extract_i* are incorrect.
Intel drivers with the known bug will produce blue.
v2: Use 0x000000ff in the int8 test to match the int16 test. Suggested
by Caio.
v3: Update header comment to explain why two possible incorrect values
are checked. Suggested by Caio.
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/883>
|
|
tests/spec/arb_draw_buffers_blend/state_set_get.c:91:12: warning: passing arguments to a function without a prototype is deprecated in all versions of C and is not supported in C23 [-Wdeprecated-non-prototype]
91 | glEndList(list);
| ^
tests/spec/arb_draw_buffers_blend/state_set_get.c:136:12: warning: passing arguments to a function without a prototype is deprecated in all versions of C and is not supported in C23 [-Wdeprecated-non-prototype]
136 | glEndList(list);
| ^
Fixes: 0fb057c06ddc ("Add new arb_draw_buffers_blend-state_set_get test")
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/906>
|
|
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/907>
|
|
On some architectures clearing depth and stencils can be optimized, but
tracking the state to know when the stencil has been written is necessary
and sometimes has bugs. This test finds a problem with state tracking of
stencil clears in an older version of the panfrost driver.
Signed-off-by: Eric R. Smith <eric.smith@collabora.com>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/905>
|
|
We have tests to validate depth offsets in general, but these
apply to the default depth buffer (usually 24 bit fixed point).
This test checks the details of the depth offset calculation
by using NV_read_pixel to read back the written values, and
verifies that e.g. the minimum resolvable difference used for
floating point buffers matches the spec. This revealed a bug
on panfrost.
Signed-off-by: Eric R. Smith <eric.smith@collabora.com>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/900>
|
|
When we build only a small subset of features, the bin directory
isn't prepared yet, so we need to be sure, it's there for us.
Reviewed-by: Vignesh Raman <vignesh.raman@collabora.com>
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/903>
|
|
For depth16 and depth24, let's reduce the precision of the float values
used as depth.
This makes the depth-clear-precision tests pass on rpi4 devices.
Reviewed-by: Jose Maria Casanova Crespo <jmcasanova@igalia.com>
Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/899>
|
|
Converts a 16-bit float to 32-bit float.
Reviewed-by: Jose Maria Casanova Crespo <jmcasanova@igalia.com>
Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/899>
|
|
Make sure that GBM doesn't dup() the FD from under us, as users rely on
this being the case.
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/865>
|
|
This test checks to see whether the Wayland compositor advertises the
expected device as its primary device through the dmabuf interface, so
can be used as a basic smoke test for checking that clients are able to
use the right renderer.
Signed-off-by: Daniel Stone <daniels@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/796>
|
|
this should yield correct results on all drivers
fixes mesa/mesa#10991
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/904>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
- Using make is slow. Use Ninja where possible.
- Use cmake --build, since depending on cmake settings it'll utilize
the right buildsystem.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Removes annoying warning.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
This is needed for following modern cmake versions.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
For ccmake is necessary to install cmake-curses-gui
which isn't a common package.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
- We don't need X11 libraries for EGL only build.
- User may want use Ninja for faster build.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Not very common for if,else,endif.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Make CMakeLists.txt cleaner.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Currently required cmake 3.2 defaults to NEW (e.g. ON).
Fixes: fdc5a4ca1112 ("cmake: Bump minimum version to 3.2.")
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/898>
|
|
On Panfrost, at least, a race condition could occur if a texture
is cleared and then initialized with texSubImage; the clear on
the GPU could happen after the memcpy in texSubImage. This test
checks for that and triggers the original Panfrost bug.
Signed-off-by: Eric R. Smith <eric.smith@collabora.com>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/901>
|
|
It's not the texture function that needs the prefix, it's the sampler.
Fixes: c55e22a5c ("arb_copy_image-formats: avoid undefined behavior")
Reviewed-by: Eric R. Smith <eric.smith@collabora.com>
Tested-by: Eric R. Smith <eric.smith@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/897>
|
|
Tests from desktop GL ARB_sync adapted to GLES extension APPLE_sync.
Signed-off-by: Lucas Stach <dev@lynxeye.de>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/844>
|
|
The somewhat twisted language in [1] seems to mean that words in the
ignore list must be passed as lowercase. Discussion in [2] supports
this understanding. Change all matches to lowercase and remove
uppercase entries that are already matched by lowercase ones.
[1] https://github.com/codespell-project/codespell/#ignoring-words
[2] https://github.com/codespell-project/codespell/issues/2375
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/894>
|
|
While c10a0731c940 ("fix the spelling in whole piglit") already fixed
a multitude of spelling issues, new versions of codespell find quite
a bit more worth fixing.
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/895>
|
|
Needed for 9405c6e80387 ("ci: teach codespell to accept my name")
to to take effect.
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/893>
|
|
tests/spec/arb_multisample/pushpop.c:144:9: warning: use of bitwise '&' with boolean operands [-Wbitwise-instead-of-logical]
144 | return test_enable_bits(enable_on) & test_coverage(coverage_mode);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| &&
tests/spec/arb_multisample/pushpop.c:144:9: note: cast one or both operands to int to silence this warning
Fixes: 1f58a74ca ("ARB_multisample/pushpop: New test for PushAttrib/PopAttrib.")
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/857>
|
|
Fix defects reported by Coverity Scan.
Resource leak (RESOURCE_LEAK)
leaked_storage: Variable name going out of scope leaks the storage it points to.
leaked_storage: Variable opt going out of scope leaks the storage it points to.
Fixes: cc1e4d6761a9 ("Add GL_EXT_texture_storage tests")
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Lucas Stach <l.stach@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/879>
|
|
The wrong index variable used leads to effectively random results
when the GPU doesn't support all of the tested internalformats.
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/889>
|
|
Don't test clearing float depth textures when the driver
doesn't advertise support for them.
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: David Heidelberg <david.heidelberg@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/890>
|
|
Some users are reporting that basename is not available when building on
Linux with musl libc. And since the POSIX spec[1] says that basename is
defined in libgen.h, we should include that when available.
So let's properly detect the header, and include it if it exists. This
should hopefully make things a bit more robust.
Since we're also including this from the CL program-tester, let's
rearrange the includes a bit so we know that config.h has been included.
Reviewed-by: David Heidelberg <david.heidelberg@collabora.com>
Tested-by: David Heidelberg <david.heidelberg@collabora.com>
Reviewed-by: Khem Raj <raj.khem@gmail.com>
Tested-by: Khem Raj <raj.khem@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/888>
|
|
Make piglit_multisample_texture() handle integer formats correctly.
From the GL spec: "If the values written by the fragment shader do not
match the format(s) of the corresponding color buffer(s), the result
is undefined."
Therefore we create the fragment shader with the appropriate output format
based on the internal format used.
Signed-off-by: Eric R. Smith <eric.smith@collabora.com>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/887>
|
|
Fix MinGW build error.
tests/general/ppgtt_memory_alignment.c: In function 'piglit_init':
tests/general/ppgtt_memory_alignment.c:99:40: error: implicit declaration of function 'getpagesize' [-Wimplicit-function-declaration]
99 | const unsigned int page_size = getpagesize();
| ^~~~~~~~~~~
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/885>
|
|
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Signed-off-by: David Heidelberg <david@ixit.cz>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/866>
|
|
this behavior is technically valid, and the test is mostly useless
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/884>
|
|
program@execute@builtin@builtin-float-degrees-1.0.generated issue about tolerance.
Initially, there is no tolerance for degrees operations set in gen_cl_common_builtins.py, which will cause the test to fail.
According to OpenCL related standards, the error value of the result calculated by the degrees function can be within the range of 2ulp.
Modified the tolerance of degrees function in the gen_cl_common_builtins.py file.
Signed-off-by: jiajia Qian <jiajia.qian@nxp.com>
Part-of: <https://gitlab.freedesktop.org/mesa/piglit/-/merge_requests/860>
|
|
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>
|