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 --- tests/cl/api/create-context.c | 16 +--------------- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/tests/cl/api/create-context.c b/tests/cl/api/create-context.c index f4929f0..551f8a2 100644 --- a/tests/cl/api/create-context.c +++ b/tests/cl/api/create-context.c @@ -139,7 +139,7 @@ piglit_cl_test(const int argc, bool found_invalid_platform = false; cl_platform_id* platform_ids; unsigned int num_platform_ids; - cl_platform_id invalid_platform_id; + cl_platform_id invalid_platform_id = NULL; //TODO: test also CL_CONTEXT_INTEROP_USER_SYNC cl_context_properties context_properties[] = { @@ -160,20 +160,6 @@ piglit_cl_test(const int argc, 0 }; - /* Find invalid platform_id */ - invalid_platform_id = 0; - num_platform_ids = piglit_cl_get_platform_ids(&platform_ids); - while(!found_invalid_platform) { - found_invalid_platform = true; - invalid_platform_id = (cl_platform_id)1; - for(i = 0; i < num_platform_ids; i++) { - if(invalid_platform_id == platform_ids[i]) { - found_invalid_platform = false; - break; - } - } - } - free(platform_ids); invalid_platform_context_properties[1] = (cl_context_properties)invalid_platform_id; -- 2.5.0 _______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
