[Piglit] [PATCH 1/1] cl: skip get image info test if there's no image support

Tom Stellard tom at stellard.net
Mon May 2 15:08:59 UTC 2016


On Mon, May 02, 2016 at 10:01:43AM -0400, Jan Vesely wrote:
> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
> ---
>  tests/cl/api/get-image-info.c | 25 +++++++++++++++++++++++--
>  1 file changed, 23 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/cl/api/get-image-info.c b/tests/cl/api/get-image-info.c
> index 2d653b1..a8b5bec 100644
> --- a/tests/cl/api/get-image-info.c
> +++ b/tests/cl/api/get-image-info.c
> @@ -46,6 +46,22 @@ PIGLIT_CL_API_TEST_CONFIG_BEGIN
>  
>  PIGLIT_CL_API_TEST_CONFIG_END
>  
> +static bool context_has_image_support(const piglit_cl_context ctx)
> +{
> +	int ret = 0;
> +	unsigned i;
> +	for(i = 0; i < ctx->num_devices; i++) {
> +		int *image_support =
> +			piglit_cl_get_device_info(ctx->device_ids[i],
> +						CL_DEVICE_IMAGE_SUPPORT);
> +		if (image_support)
> +			ret |= *image_support;
> +
> +		free(image_support);
> +	}
> +	return ret;
> +}
> +
>  
>  enum piglit_result
>  piglit_cl_test(const int argc,
> @@ -63,12 +79,17 @@ piglit_cl_test(const int argc,
>  		.image_channel_data_type = CL_FLOAT,
>  	};
>  
> +	if (!context_has_image_support(env->context)) {
> +		fprintf(stderr, "No device with image support found!\n");
> +		return PIGLIT_SKIP;
> +	}
> +
>  	size_t param_value_size;
>  	void* param_value;
> -	
> +
>  	int num_image_infos = PIGLIT_CL_ENUM_NUM(cl_image_info, env->version);
>  	const cl_image_info* image_infos = PIGLIT_CL_ENUM_ARRAY(cl_image_info);
> -	
> +
>  #if defined CL_VERSION_1_2
>  	if(env->version >= 12) {
>  		cl_image_desc image_desc = {
> -- 
> 2.5.5
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list