| Age | Commit message (Collapse) | Author | Files | Lines |
|---|
|
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>
|
|
v2: (Jan Vesely comments)
fix grammar
if-def clReleaseMemObject(memobj[2])
test() return a status
use subtests
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Fix resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Serge Martin (EdB) <edb+piglit@sigluy.net>
|
|
OpenCL objects are just pointers in the end, using random pointer value
is likely to make an application crash.
Also, all others tests use NULL as an invalid object.
See also Francisco Jerez comments:
http://lists.freedesktop.org/archives/mesa-dev/2014-November/070520.html
http://lists.freedesktop.org/archives/mesa-dev/2013-October/046830.html
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
v2:
spaces/tabs formating
clReleaseMemObject the buffer
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
this query is only valid for a custom device or a build-in kernel
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
v2:
add the test to cl.py (Jan Vesely)
add checks on returned values (Jan Vesely)
disable CL_KERNEL_ARG_INFO_NOT_AVAILABLE test
v3:
fix compilation without 1.2 headers (Jan Vesely)
keep testing CL_KERNEL_ARG_INFO_NOT_AVAILABLE (Jan Vesely)
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
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>
|
|
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
Reviewed-by: EdB <edb@sigluy.net>
|
|
|
|
Although CL tests are not built with MSVC, this is the single instance
of void pointer arithmetic in them, so updating it to not use void
pointer arithmetic will enable us to use -Werror=pointer-arith option
universally in a follow on change.
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
This reverts commit ff8898df3c78324b2d8945c855a9d2e0034f5525.
This broke cl tests, and it's a very trivial self contained patch, so
we'll just revert it.
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
reported-by: Michel Dänzer <michel@daenzer.net>
|
|
It really doesn't make sense to have static tests in the generated tests
directory, this patch adds them to the tests directory and adds the
appropriate code to cl.py for this change
Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
|
|
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
|
|
Fix resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
This fix https://bugs.freedesktop.org/show_bug.cgi?id=86361
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
The specs say 2.5 ulp, but we only take integer values
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Fix resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
Fix Coverity "resource leak" defect.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Fix resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Fix uninitialized scalar variable defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
|
|
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
This patch silences Clang sometimes-uninitialized warnings.
compile-program.c:312:6: warning: variable 'kernel' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
if(!piglit_cl_check_error(errNo, CL_SUCCESS)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compile-program.c:332:18: note: uninitialized use occurs here
clReleaseKernel(kernel);
^~~~~~
compile-program.c:312:3: note: remove the 'if' if its condition is always false
if(!piglit_cl_check_error(errNo, CL_SUCCESS)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compile-program.c:302:5: warning: variable 'kernel' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
if(!piglit_cl_check_error(errNo, CL_SUCCESS)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compile-program.c:332:18: note: uninitialized use occurs here
clReleaseKernel(kernel);
^~~~~~
compile-program.c:302:2: note: remove the 'if' if its condition is always false
if(!piglit_cl_check_error(errNo, CL_SUCCESS)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compile-program.c:114:18: note: initialize the variable 'kernel' to silence this warning
cl_kernel kernel;
^
= NULL
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: EdB <edb+piglit@sigluy.net>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
because -cl-strict-aliasing options argument to
clBuildProgram is no longer supported in OpenCL 1.1.
|
|
|
|
|
|
|
|
v2:
Use clBuildProgram to create to program to attach kernel to
|
|
If there is only one device associed with the kernel and device arg is NULL
you don't have to trigger CL_INVALID_DEVICE
v2:
Test for CL_SUCCESS instead of disabling it (Jan Vesely)
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
v2:
- Fix filename in doxygen comments
- Remove redundant binary status check.
v3:
- Add test cases for passing binary programs to clBuildProgram() and
clCompileProgram().
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
|
|
Tests both usage and non-usage of return value.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests both usage of and no usage of return value.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests for both usage-of and non-usage-of the return value
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests both usage of return and no usage of return.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests both usage of return and no usage of return variants.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests both usage and non-usage of return value.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests both usage of return values and no usage in separate files.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Tests both usage of return value and no-return-usage variants.
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Adds two separate files, one which tests atomic_add without usage
of the return values, and another which tests with return usage.
At least the Radeon HD (EG/SI) line has two separate sets of instructions
depending on whether the return value is used or not.
v2: Only use return variant for atomic_add-global-return
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
|
|
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
These use lte kernels
Also fixes duplicate subtest names
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Reviewed-by: Tom Stellard <thomas.stellard@amd.com>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Signed-off-by: Aaron Watry <awatry@gmail.com>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
|
Signed-off-by: Aaron Watry <awatry@gmail.com>
|
|
v2: Properly test signed/unsigned values
Signed-off-by: Aaron Watry <awatry@gmail.com>
|
|
Fix clang sometimes-uninitialized warning.
program-tester.c:1523:12: warning: variable 'main_argument_type' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
} else if(regex_match(main_argument, "\\.bin$")) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
program-tester.c:1542:9: note: uninitialized use occurs here
switch(main_argument_type) {
^~~~~~~~~~~~~~~~~~
program-tester.c:1523:9: note: remove the 'if' if its condition is always true
} else if(regex_match(main_argument, "\\.bin$")) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
program-tester.c:1455:2: note: variable 'main_argument_type' is declared here
enum main_argument_type_t {
^
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
|
