summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2015-11-11Use smaller buffer and different pixel values 1st and 2nd round91926_dirty_but_simpleTopi Pohjolainen1-8/+21
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
2015-10-26more dbgTopi Pohjolainen1-2/+2
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
2015-10-26dbgTopi Pohjolainen1-2/+2
2015-10-26dbg: texsubimage simplifiedTopi Pohjolainen1-1/+19
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
2015-10-26dbgTopi Pohjolainen2-3/+18
2015-10-26Revert "Reject invalid dimensions"Topi Pohjolainen1-13/+6
This reverts commit cd641f59bc5ffde9863b8dc1c684e152790f60d1.
2015-10-26Reject invalid dimensionsTopi Pohjolainen1-6/+13
Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
2015-10-19dbg: print all texsubimage failuresTopi Pohjolainen1-3/+9
2015-08-13gl-2.0: Add test for re-using shader objectsTopi Pohjolainen3-0/+107
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>
2015-08-13arb_program_interface_query: fix getprogramresourceiv errorsTapani Pälli1-5/+4
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>
2015-08-12xz: Use --help when detecting xz presenceJordan Justen1-77/+76
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>
2015-08-11framework tests: fix binary_check utility bugDylan Baker2-6/+20
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>
2015-08-10framework: remove _test_run_hook from TestDylan Baker2-8/+11
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>
2015-08-09arb_arrays_of_arrays: simple subroutine execution testTimothy Arceri1-0/+63
2015-08-09arb_arrays_of_arrays: add simple fs atomic counter execution testTimothy Arceri1-0/+108
V2: Initialise correct number of counters, remove incorrect testing of max buffer
2015-08-09arb_arrays_of_arrays: fix vs decrement atomic counter testTimothy Arceri1-7/+8
V2: Initialise correct number of counters, fix testing of required max buffers
2015-08-09arb_arrays_of_arrays: add some atomic counter indirect indexing testsTimothy Arceri2-0/+153
V2: Initialise correct number of counters, fix testing of required max buffers
2015-08-07tf3: remove broken and unused codeIlia Mirkin1-31/+1
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>
2015-08-06point-vertex-id: Fix memory leak.Vinson Lee1-0/+2
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>
2015-08-06arb_tessellation_shader-large-uniforms: Fix memory leaks.Vinson Lee1-0/+4
Fix resource leak defects reported by Coverity. Signed-off-by: Vinson Lee <vlee@freedesktop.org> Reviewed-by: Brian Paul <brianp@vmware.com>
2015-08-06arb_tessellation_shader-invalid-get-program-params: Remove unused variable.Vinson Lee1-4/+0
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>
2015-08-06active-sampler-conflict.c: remove unused variableDylan Baker1-1/+0
This silences a GCC warning. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com> Reviewed-by: Vinson Lee <vlee@freedesktop.org>
2015-08-05framework: when searching a directory ignore files ending in .oldDylan Baker2-1/+18
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>
2015-08-05occlusion_query_meta_save: also test scissored clearBrian Paul1-5/+24
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>
2015-08-05occlusion_query_meta_no_fragments: check more potential meta operationsBrian Paul1-2/+39
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>
2015-08-03core.py: use errno instead of int for error checkingDylan Baker1-2/+3
This makes the code a little cleaner and easier to read. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-08-02arb_shader_atomic_counters-semantics: test different binding pointsAlejandro Piñeiro1-43/+111
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
2015-08-01cl: add clLinkProgram testEdB3-0/+396
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>
2015-07-31Test interpolateAtSample with dynamically nonuniform valuesNeil Roberts3-0/+279
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.
2015-07-30tests: add missing requirements to two arb_arrays_of_arrays testsDylan Baker2-0/+4
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>
2015-07-29arb_tessellation_shader: Test large uniforms arrays. (v3)Fabian Bieler3-0/+335
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>
2015-07-29gen_builtin_uniform_tests.py: Add tessellation control shader tests.Fabian Bieler2-4/+142
v2: fixed by Marek Reviewed-by: Marek Olšák <marek.olsak@amd.com> Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
2015-07-29arb_tessellation_shader: add tests for partial output writesChris Forbes2-0/+128
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>
2015-07-29arb_tessellation_shader: Simplify tcs-input tests.Chris Forbes1-6/+5
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>
2015-07-29arb_tessellation_shader: add tests to exercise gl_PrimitiveIDChris Forbes2-0/+140
v2: fixed by Marek Reviewed-by: Marek Olšák <marek.olsak@amd.com> Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
2015-07-29arb_tessellation_shader: Add various simple cases for TCS input readsChris Forbes7-0/+479
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>
2015-07-29arb_tessellation_shader: Add yet another fairly minimal caseChris Forbes1-0/+50
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>
2015-07-29arb_tessellation_shader: add tests for correct set of invocationsChris Forbes2-0/+120
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>
2015-07-29arb_tessellation_shader: Add another minimal execution testChris Forbes1-0/+49
v2: fixed by Marek Reviewed-by: Marek Olšák <marek.olsak@amd.com> Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
2015-07-29all.py: don't add unsupported texwrap test combinationsMarek Olšák1-4/+6
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
2015-07-29arb_clear_texture: add tolerance to clear-max-level.c testYu Wang1-1/+1
Since we're querying a ubyte image as float values we can't expect exact matches. Reviewed-by: Brian Paul <brianp@vmware.com>
2015-07-29clearbuffer-invalid-drawbuffer: Fix test cases calling glClearBufferiv with ↵Eduardo Lima Mitev1-12/+18
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.
2015-07-29arb_shader_storage_buffer_object: modify tests to use gl_core_version = 32Samuel Iglesias Gonsalvez10-21/+33
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>
2015-07-29arb_arrays_of_arrays: fix simple atomic counters execution testTimothy Arceri1-13/+13
The test was always passing because the if statemants used AND rather than OR. Also some of the array indexes were wrong.
2015-07-29arb_arrays_of_arrays: basic image store execution testsTimothy Arceri2-0/+159
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
2015-07-29arb_arrays_of_arrays: add some sampler indexing execution testsTimothy Arceri18-0/+1117
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
2015-07-29arb_arrays_of_arrays: add simple atomic counters execution testTimothy Arceri1-0/+119
Test results: Nvidia GeForce 840M - NVIDIA 346.47: pass
2015-07-29arb_gpu_shader_fp64: test glGetUniformdv interfaceDave Airlie3-0/+201
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>
2015-07-28ext_image_dma_buf: Add tests for R8 and GR88 formats (v2)Chad Versace3-0/+305
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>
2015-07-28arb_get_texture_sub_image-errors: test getting zero-sized imagesBrian Paul1-0/+60
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>