summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2015-10-04cl: add popcount builtinSerge Martin1-4/+14
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
2015-10-04arb_shader_atomic_counters: add error output if uniform not foundTimothy Arceri1-2/+5
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
2015-10-03arb_shader_atomic_counters/max-counters: Run the combined atomic buffer test ↵Francisco Jerez1-10/+12
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>
2015-10-03arb_shader_atomic_counters/max-counters: Use different atomic counter ↵Francisco Jerez1-1/+1
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>
2015-10-03arb_shader_atomic_counters/max-counters: Init uint variable with unsigned ↵Francisco Jerez1-2/+2
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>
2015-10-03arb_shader_atomic_counters: Make atomic_counters_generate_source local to ↵Francisco Jerez3-60/+57
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>
2015-10-03arb_direct_state_access-texunits: fix incorrect error testBrian Paul1-4/+6
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>
2015-10-03arb_gpu_shader_fp64: expand getuniform double coverageDave Airlie1-6/+86
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>
2015-10-02arb_shader_storage_buffer_object: invalid binding qualifier testTapani Pälli1-0/+22
Signed-off-by: Tapani Pälli <tapani.palli@intel.com> Reviewed-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>
2015-10-01framework: add exception back to TestResult classDylan Baker2-2/+15
Like command, this somehow didn't get moved to the new TestResult class. Generally this isn't a problem since this is a very uncommon path, but in some cases this does get hit, and it would cause an exception in the runner threads. Python has very bad exception handling in threads, namely it doesn't have any; this results in an exception being raised and remaining uncaught, terminating the thread, which in turn results in the test result being incomplete, even though it should have been crash. This patch fixes that situation. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com> Tested-by: Mark Janes <mark.a.janes@intel.com> Reviewed-by: Mark Janes <mark.a.janes@intel.com>
2015-10-01tess/fp64: add some basic cross over testsDave Airlie3-0/+258
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>
2015-10-01arb_shader_subroutine: add a vertex shader testDave Airlie1-0/+52
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>
2015-09-30framework: add clinfo to system informationDylan Baker5-2/+17
It's like glxinfo or wglinfo, but for OpenCL. Reviewed-by: Serge Martin <edb+piglit@sigluy.net> Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-09-30arb_compute_shader: A simple CS test with SSBOJordan Justen1-0/+74
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
2015-09-30shader_runner: Add SSBO buffer initialization commandJordan Justen1-0/+8
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>
2015-09-30cl: fix get-kernel-arg-info indentationSerge Martin1-208/+208
2015-09-30cl: add clEnqueueFillImageSerge Martin3-0/+336
2015-09-30cl: add clEnqueueMigrateMemObjectsSerge Martin3-0/+246
v2: (Jan Vesely comments) Remove extra whitespace Single flag test cases Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
2015-09-29Remove dangling references to GLUIan Romanick2-2/+1
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>
2015-09-29fbo-depth-sample-compare: Eliminate dependency on gluSphereIan Romanick2-17/+129
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>
2015-09-29fbo-depth-sample-compare: Use piglit_draw_rect_texIan Romanick1-10/+1
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>
2015-09-29fbo-depth-sample-compare: Use piglit_build_simple_programIan Romanick1-9/+3
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>
2015-09-29glean: Remove dependency on libGLUIan Romanick2-4/+0
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>
2015-09-29glean: Just use glTexImage2D in ttexgenIan Romanick1-1/+3
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>
2015-09-29glean: Remove unused method Image::makeMipmapsIan Romanick2-16/+0
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>
2015-09-29glean: Replace gluOrtho2D with plain glOrthoIan Romanick1-1/+1
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>
2015-09-29glean: Replace gluErrorString with piglit_get_gl_error_nameIan Romanick5-12/+16
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>
2015-09-29polygon-offset: Replace gluProject to remove the dependency on libGLUIan Romanick2-16/+62
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>
2015-09-29polygon-offset: Small bit of output refactoringIan Romanick1-11/+6
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>
2015-09-29polygon-offset: Remove a layer of tabsIan Romanick1-16/+16
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>
2015-09-29polygon-offset: Replace draw_2x2_quad with piglit_draw_rectIan Romanick1-14/+3
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>
2015-09-29polygon-offset: Remove a lot of redundant state setupIan Romanick1-31/+4
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>
2015-09-29polygon-offset: Assortment of coding standards fixesIan Romanick1-176/+142
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>
2015-09-29Remove unnecessary linking with libGLUIan Romanick145-145/+0
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>
2015-09-29arb_compute_shader: Fix missed varargs termination in cs-ids-common.cJordan Justen1-1/+2
Reported-by: Francisco Jerez <currojerez@riseup.net> Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
2015-09-29tests: Add integration for khronos CTS runner.Dylan Baker2-0/+86
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>
2015-09-26glsl-1.50: test shader compiles with unused uniform blockTimothy Arceri1-0/+19
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
2015-09-25framework/summary: Fix regressions and fixes list to not include skipDylan Baker1-2/+6
This fixes the lists of regressions and fixes to not include skip or notrun statuses. This fixes a regression introduced in the recent summary rework of piglit. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-09-25cmake: Link libpiglitutil_gl to libGLChad Versace1-0/+1
This is the obvious thing to do. This eliminates the need to explicitly link to libGL in every other CMakeLists.gl.txt. Tested-by: Nanley Chery <nanley.g.chery@intel.com>
2015-09-25arb_program_interface_query: add linker test for querying varyingsTapani Pälli1-0/+33
This fails with current Mesa due to a bug in commit: 4639cea2921669527eb43dcb49724c05afb27e8e Signed-off-by: Tapani Pälli <tapani.palli@intel.com> Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
2015-09-24framework/results.py: write command to jsonDylan Baker1-4/+5
This was an oversight, it should have always been written to the json, but was missed in the to_json method. cc: Dave Airlie <airlied@gmail.com> Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-09-24templates/index.mako: handle missing subtests correctlyDylan Baker1-2/+3
This patch fixes the handling of subtests when more than one result is presented. Currently this will raise a KeyError and die horribly. cc: Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-09-24framework/results.py: Fix Subtests classDylan Baker3-8/+29
Sub-classing built-in types in python is an error-prone leap, and many corner cases came out to bit us in this instance. By wrapping a dict inside of another class we can get the desired results, it means more code, but it also means less corner cases. Also change the equality test of two unit tests, since Subtests isn't a dictionary anymore assert_dict_equal wont work anymore. cc: Mark Janes <mark.a.janes@intel.com> Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-09-24framework/tests/results_tests.py: fix copy and paste error in testDylan Baker1-2/+2
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
2015-09-24arb_compute_shader: Add indirect compute dispatch testJordan Justen3-0/+90
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
2015-09-24arb_compute_shader: Add render + compute testJordan Justen3-0/+103
This test verifieds that rendering and compute operations can be interleaved. Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
2015-09-24arb_compute_shader: Check local ids at various sizes using atomic countersJordan Justen3-0/+63
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
2015-09-24arb_compute_shader: Add common functions for checking invocation IDsJordan Justen3-1/+490
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
2015-09-24arb_compute_shader: Add common functions to generate a CS programJordan Justen3-0/+152
These are a very simplified version of the similar image load/store versions. Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Kristian Høgsberg <krh@bitplanet.net>
2015-09-24shader_runner: Add ARB_program_interface_query supportSamuel Iglesias Gonsalvez1-0/+166
v2: - Modify command format. Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com> Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>