[Beignet] [PATCH] refine bswap utest to cover nsetc fail cases.

Zhigang Gong zhigang.gong at linux.intel.com
Wed Dec 3 22:54:01 PST 2014


LGTM, pushed, thanks.

On Wed, Dec 03, 2014 at 04:51:53PM +0800, xionghu.luo at intel.com wrote:
> From: Luo Xionghu <xionghu.luo at intel.com>
> 
> two bswap call in one block would trigger nsetc failures.
> the fail was fixed in backend already, just update the utest.
> 
> Signed-off-by: Luo Xionghu <xionghu.luo at intel.com>
> ---
>  kernels/compiler_bswap.cl |    1 +
>  utests/compiler_bswap.cpp |    7 +++++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/kernels/compiler_bswap.cl b/kernels/compiler_bswap.cl
> index 9ef0e6b..97313b1 100644
> --- a/kernels/compiler_bswap.cl
> +++ b/kernels/compiler_bswap.cl
> @@ -1,6 +1,7 @@
>  #define TEST_TYPE(TYPE, LENGTH)                                       \
>  kernel void compiler_bswap_##TYPE(global TYPE * src, global TYPE * dst){ \
>     dst[get_global_id(0)]= __builtin_bswap##LENGTH(src[get_global_id(0)]); \
> +   dst[get_global_id(0)]= __builtin_bswap##LENGTH(dst[get_global_id(0)] -1 ); \
>  }
>  
>  
> diff --git a/utests/compiler_bswap.cpp b/utests/compiler_bswap.cpp
> index b5986b9..9475b99 100644
> --- a/utests/compiler_bswap.cpp
> +++ b/utests/compiler_bswap.cpp
> @@ -85,6 +85,13 @@ void test(const char *kernel_name)
>    for (int32_t i = 0; i < (int32_t) n; ++i)
>      cpu(i, cpu_src, cpu_dst);
>  
> +  for (int32_t i = 0; i < (int32_t) n; ++i)
> +    cpu_dst[i] = cpu_dst[i] -1;
> +
> +  // Run on CPU
> +  for (int32_t i = 0; i < (int32_t) n; ++i)
> +    cpu(i, cpu_dst, cpu_dst);
> +
>    OCL_MAP_BUFFER(1);
>   // dump_data(cpu_src, cpu_dst, n);
>  
> -- 
> 1.7.9.5
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list