Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
|
|
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
|
|
|
|
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
|
|
|
|
This reverts commit cd641f59bc5ffde9863b8dc1c684e152790f60d1.
|
|
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
|
|
|
|
This is stimulating the shader binary re-use logic in i965 buffer
object uploading. Without relaxing the current constraints there
will be two identical copies in the cache. With the introduced
three patches the logic begins to share only one copy between
two entries in the cache.
v2 (Ilia, Ken): Add to tests/all.py
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
|
|
Move IS_PER_PATCH as part of tessellation queries, require
GL_ARB_compute_shader for compute shader enum queries.
Strictly speaking, spec would allow to query GL_REFERENCED_BY_COMPUTE_SHADER
without having compute shaders but as it disallows GL_COMPUTE_SUBROUTINE and
GL_COMPUTE_SUBROUTINE_UNIFORM if there is no compute shader support, it is
likely that this is a bug in the spec.
v2: add missing parenthesis! (spotted by Curro)
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91608
Tested-by: Mark Janes <mark.a.janes@intel.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
|
|
If xz is present, and the piglit command's output is redirected, then
xz will think its output is redirected. This will cause xz to try to
read data from stdin to compress.
Instead we can run xz with --help to cause it to print help
information if the xz executable is present, and prevent xz from
trying to compress data from stdin.
Since we don't want to see 'xz --help' output from piglit, we now need
to redirect both stdout and stderr to /dev/null.
This allows us to pipe the output of piglit summary when using the shell
xz compressor.
v2 (Dylan):
- send both stdout and stderr directly to /dev/null
- Fix xz detection in the xz shell path.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
|
|
This patch changes binary_check to work when a binary is present, but
doesn't work for some reason.
The example here is in the json content tests, where it looks for
glxinfo, but if X isn't running glxinfo provides no output, which will
cause a test to fail. This isn't useful, so instead of using the *nix
command 'which', call the command and allow an expected returncode to be
passed, if the returncode isn't that returncode, then skip.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
This was a design mistake from the start, subclassing the Test class and
overriding the run() method achieves the same result, without the need
for this hook
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
|
|
V2: Initialise correct number of counters, remove incorrect testing
of max buffer
|
|
V2: Initialise correct number of counters, fix testing of required
max buffers
|
|
V2: Initialise correct number of counters, fix testing of required
max buffers
|
|
draw_using_invalid_stream_index was failing due to a lacking input
qualifier in the GS program. However it never used the program. Remove
all of that unnecessary boilerplate and leave the TF draw. The test now
passes on nvc0.
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
|
|
Fix resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Neil Roberts <neil@linux.intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Fix resource leak defects reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Fix unused-const-variable warning.
invalid-get-program-params.c:47:26: warning: unused variable 'vs_source' [-Wunused-const-variable]
static const char *const vs_source =
^
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
This silences a GCC warning.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Vinson Lee <vlee@freedesktop.org>
|
|
Currently if piglit updates a result to a new version, then tries to
load from the same directory the first result will be
'results.json.old', and piglit will interpret the '.old' as a
compression extension, and die in a fire.
With this patch when searching a folder (but not when pointing directly
at a file) piglit will ignore any files ending in '.old', which will
correct the problem.
A unit test is also included.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Plain glClear is seldom a meta operation, but scissored clears are
implemented by drawing a quad in gallium. Now the test tries both
to check that glClear doesn't effect the occlusion query count.
Reviewed-by: Eric Anholt <eric@anholt.net>
|
|
Scissored or masked clears might be done with a meta operation that
draws a quad. Now check for those cases too.
These cases plus some of the original ones all fail with gallium.
Reviewed-by: Eric Anholt <eric@anholt.net>
|
|
This makes the code a little cleaner and easier to read.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Piglit was only checking the expected effects using binding point 0.
With this commit, all the values between 0 and
GL_MAX_ATOMIC_COUNTER_BUFFER_BINDINGS are checked.
It also checks that the binding point used on the shader and the one
returned by
glGetActiveAtomicCounterBufferiv(..., ...,
GL_ATOMIC_COUNTER_BUFFER_BINDING, ...) is the same.
This test detects mesa bug:
https://bugs.freedesktop.org/show_bug.cgi?id=90175
|
|
v2:
Use piglit_cl_get_program_build_info instead of piglit_cl_get_program_info,
I was expected it to fail so I didn't paid attention.
Remove "-invalid- --link-- options" on CL_INVALID_OPERATION test.
v3:
Add to cl.py
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
Adds a test which is similar to interpolateAtSample-different except
that it draws a triangle which covers more than one fragment and makes
sure to use a different sample ID for each fragment so that it won't
be dynamically uniform. The GLSL spec doesn't mention that the sample
ID has to be dynamically uniform which presumably means that it is
allowed. I think this is worth testing because it is not
straightforward to implement at least on Intel hardware.
|
|
These two tests currently fail on i965 because they don't check for the
proper requirements. This patch adds them.
cc: Timothy Arceri <t_arceri@yahoo.com.au>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
|
|
Test uniform arrays of maximal size as well as arrays of uniform blocks.
Also test for the required linking errors if too large uniform arrays are
requested.
v2: Marek - removed commented out code
v3: added missing TCS and TES
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
|
|
v2: fixed by Marek
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
These cases are interesting for i965 because writemasks for URB writes
require careful code generation.
The partial write is buried in uniform branch to defeat any coalescing a
clever implementation might otherwise do.
v2: fixed by Marek
Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
|
|
The purpose of these tests is not to exercise the compiler's
optimization of writes from the TCS -- we'll explore that in
other tests.
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
v2: fixed by Marek
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
v2: fixed by Marek
Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
This variant has a pair of (attributeless) output control points,
which is the very simplest case for i965's SIMD4x2 dispatch of the
control shader. This test passing but patch-single-vert failing
indicates a problem with the 'last unpaired invocation' logic.
v2: fixed by Marek
Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
These are interesting basic cases for i965, even without per-vertex
outputs.
v2: fixed by Marek
Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
v2: fixed by Marek
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
|
|
Since we're querying a ubyte image as float values we can't expect
exact matches.
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
depth buffers
Page 497 of the PDF, section '17.4.3.1 Clearing Individual Buffers' of the
OpenGL 4.5 spec states:
"An INVALID_ENUM error is generated by ClearBufferiv and
ClearNamedFramebufferiv if buffer is not COLOR or STENCIL."
The affected test is not complying with this wording, and is expecting a
GL_INVALID_VALUE when glClearBufferiv is called with a depth buffer but, due to
other constrain.
|
|
Recently, some of them were returning errors on glGetIntegerv() calls for
shader storage buffer constants in Mesa because they used GL compat 1.0
version.
Some shaders were modified to avoid "GL_INVALID_OPERATION in
glDrawArray(No VAO bound)" Mesa errors when calling piglit_draw_rect() with
this config.
Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
|
|
The test was always passing because the if statemants used AND
rather than OR. Also some of the array indexes were wrong.
|
|
Test results:
Nvidia GeForce 840M - NVIDIA 346.47
basic-imageStore-const-uniform-index.shader_test - pass
basic-imageStore-non-const-uniform-index.shader_test - pass
|
|
Test results:
Nvidia GeForce 840M - NVIDIA 346.47
fs-const-index-three-dimensions.shader_test - pass
fs-const-index.shader_test - pass
fs-initializer-const-index.shader_test - fail
fs-initializer-non-const-index.shader_test - fail
fs-mixed-const-and-non-const-index.shader_test - pass
fs-mixed-const-and-non-const-index2.shader_test - fail
fs-non-const-index.shader_test - pass
fs-struct-const-index-sampler-const-index.shader_test - pass
fs-struct-const-index.shader_test - pass
fs-struct-non-const-index-const-index.shader_test - pass
fs-struct-non-const-index-sampler-non-const-index.shader_test - pass
fs-struct-non-const-index.shader_test - pass
vs-const-index-three-dimensions.shader_test - pass
vs-const-index.shader_test - pass
vs-non-const-index.shader_test - pass
vs-struct-nonconst-sampler-const.shader_test - pass
vs-struct-nonconst-sampler-nonconst.shader_test - pass
vs-struct-nonconst.shader_test - pass
V3: add missing AoA extensions requirement to some of the tests
V2: add vertex shader tests, add most struct tests
|
|
Test results:
Nvidia GeForce 840M - NVIDIA 346.47: pass
|
|
I must have forgot this way back when, this just ports the
getuniform02.c test to glsl 1.50 and tests the fp64 interface.
Reviewed-by: Timothy Arceri <t_arceri@yahoo.com.au>
Signed-off-by: Dave Airlie <airlied@redhat.com>
|
|
Add test that transcodes NV12 to RGBA8 by importing the Y plane
as DRM_FORMAT_R8 and importing the UV plane as DRM_FORMAT_GR88.
v2: Delete commented-out code.
CC: Peter Frühberger <peter.fruehberger@gmail.com>
Cc: Rainer Hochecker <rainer.hochecker@onlinehome.de>
|
|
An additional set of checks for:
Getting 0x0 image from a 8x8 source.
Getting 0x0 image from a 0x0 source.
Getting offset+width > texwidth when width==0.
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
|