[Piglit] [PATCH] cl: Add tests for global atomc_add()

Jan Vesely jan.vesely at rutgers.edu
Fri Sep 19 13:19:53 PDT 2014


On Fri, 2014-09-19 at 14:23 -0400, Tom Stellard wrote:
> ---
>  .../builtin/atomic/atomic_add-global.program_test  | 46 ++++++++++++++
>  .../execute/builtin/atomic/atomic_add-local.cl     | 70 ----------------------
>  .../builtin/atomic/atomic_add-local.program_test   | 46 ++++++++++++++
>  .../program/execute/builtin/atomic/atomic_add.inc  | 23 +++++++
>  4 files changed, 115 insertions(+), 70 deletions(-)
>  create mode 100644 tests/cl/program/execute/builtin/atomic/atomic_add-global.program_test
>  delete mode 100644 tests/cl/program/execute/builtin/atomic/atomic_add-local.cl
>  create mode 100644 tests/cl/program/execute/builtin/atomic/atomic_add-local.program_test
>  create mode 100644 tests/cl/program/execute/builtin/atomic/atomic_add.inc
> 
> diff --git a/tests/cl/program/execute/builtin/atomic/atomic_add-global.program_test b/tests/cl/program/execute/builtin/atomic/atomic_add-global.program_test
> new file mode 100644
> index 0000000..26c6752
> --- /dev/null
> +++ b/tests/cl/program/execute/builtin/atomic/atomic_add-global.program_test
> @@ -0,0 +1,46 @@
> +[config]
> +name: Scalar Data Type Load (int)
> +
> +program_source_file: atomic_add.inc
> +build_options: -D ATOMIC_ADDRSPACE=global
> +clc_version_min: 11
> +
> +[test]
> +name: simple int
> +kernel_name: simple_int
> +dimensions: 1
> +global_size: 1 0 0
> +local_size:  1 0 0
> +arg_out: 0 buffer int[2] -4 1
> +arg_in:  1 buffer int[1] 0
> +arg_in:  2 int           -4
> +arg_in:  3 int           5
> +
> +[test]
> +name: simple uint
> +kernel_name: simple_uint
> +dimensions: 1
> +global_size: 1 0 0
> +local_size:  1 0 0
> +arg_out: 0 buffer uint[2] 4 9
> +arg_in:  1 buffer uint[1] 0
> +arg_in:  2 uint           4
> +arg_in:  3 uint           5
> +
> +[test]
> +name: threads
> +kernel_name: threads_int
> +dimensions: 1
> +global_size: 8 0 0
> +local_size:  8 0 0
> +arg_out: 0 buffer int[1] 28
> +arg_in:  1 buffer int[1] 0
> +
> +[test]
> +name: threads

this test shares name with the above test.

> +kernel_name: threads_uint
> +dimensions: 1
> +global_size: 8 0 0
> +local_size:  8 0 0
> +arg_out: 0 buffer uint[1] 28
> +arg_in:  1 buffer uint[1] 0
> diff --git a/tests/cl/program/execute/builtin/atomic/atomic_add-local.cl b/tests/cl/program/execute/builtin/atomic/atomic_add-local.cl
> deleted file mode 100644
> index c155fbb..0000000
> --- a/tests/cl/program/execute/builtin/atomic/atomic_add-local.cl
> +++ /dev/null
> @@ -1,70 +0,0 @@
> -/*!
> -[config]
> -name: atomic_add local
> -clc_version_min: 11
> -
> -[test]
> -name: simple int
> -kernel_name: simple_int
> -dimensions: 1
> -global_size: 1 0 0
> -local_size:  1 0 0
> -arg_out: 0 buffer int[2] -4 1
> -arg_in:  1 buffer int[1] NULL
> -arg_in:  2 int           -4
> -arg_in:  3 int           5
> -
> -[test]
> -name: simple uint
> -kernel_name: simple_uint
> -dimensions: 1
> -global_size: 1 0 0
> -local_size:  1 0 0
> -arg_out: 0 buffer uint[2] 4 9
> -arg_in:  1 buffer uint[1] NULL
> -arg_in:  2 uint           4
> -arg_in:  3 uint           5
> -
> -[test]
> -name: threads
> -kernel_name: threads_int
> -dimensions: 1
> -global_size: 8 0 0
> -local_size:  8 0 0
> -arg_out: 0 buffer int[1] 28
> -arg_in:  1 buffer int[1] NULL
> -
> -[test]
> -name: threads
> -kernel_name: threads_uint
> -dimensions: 1
> -global_size: 8 0 0
> -local_size:  8 0 0
> -arg_out: 0 buffer uint[1] 28
> -arg_in:  1 buffer uint[1] NULL
> -
> -!*/
> -
> -#define SIMPLE_TEST(TYPE) \
> -kernel void simple_##TYPE(global TYPE *out, local TYPE *mem, TYPE initial, TYPE value) { \
> -	*mem = initial; \
> -	TYPE a = atomic_add(mem, value); \
> -	out[0] = a; \
> -	out[1] = *mem; \
> -}
> -
> -#define THREADS_TEST(TYPE) \
> -kernel void threads_##TYPE(global TYPE *out, local TYPE *mem) { \
> -	*mem = 0; \
> -	barrier(CLK_LOCAL_MEM_FENCE); \
> -	TYPE id = get_local_id(0); \
> -	atomic_add(mem, id); \
> -	barrier(CLK_LOCAL_MEM_FENCE); \
> -	*out = *mem; \
> -}
> -
> -SIMPLE_TEST(int)
> -SIMPLE_TEST(uint)
> -
> -THREADS_TEST(int)
> -THREADS_TEST(uint)
> diff --git a/tests/cl/program/execute/builtin/atomic/atomic_add-local.program_test b/tests/cl/program/execute/builtin/atomic/atomic_add-local.program_test
> new file mode 100644
> index 0000000..75479d1
> --- /dev/null
> +++ b/tests/cl/program/execute/builtin/atomic/atomic_add-local.program_test
> @@ -0,0 +1,46 @@
> +[config]
> +name: Scalar Data Type Load (int)
> +
> +program_source_file: atomic_add.inc
> +build_options: -D ATOMIC_ADDRSPACE=local
> +clc_version_min: 11
> +
> +[test]
> +name: simple int
> +kernel_name: simple_int
> +dimensions: 1
> +global_size: 1 0 0
> +local_size:  1 0 0
> +arg_out: 0 buffer int[2] -4 1
> +arg_in:  1 buffer int[1] NULL
> +arg_in:  2 int           -4
> +arg_in:  3 int           5
> +
> +[test]
> +name: simple uint
> +kernel_name: simple_uint
> +dimensions: 1
> +global_size: 1 0 0
> +local_size:  1 0 0
> +arg_out: 0 buffer uint[2] 4 9
> +arg_in:  1 buffer uint[1] NULL
> +arg_in:  2 uint           4
> +arg_in:  3 uint           5
> +
> +[test]
> +name: threads
> +kernel_name: threads_int
> +dimensions: 1
> +global_size: 8 0 0
> +local_size:  8 0 0
> +arg_out: 0 buffer int[1] 28
> +arg_in:  1 buffer int[1] NULL
> +
> +[test]
> +name: threads

same here
this test shares name with the above test.

> +kernel_name: threads_uint
> +dimensions: 1
> +global_size: 8 0 0
> +local_size:  8 0 0
> +arg_out: 0 buffer uint[1] 28
> +arg_in:  1 buffer uint[1] NULL
> diff --git a/tests/cl/program/execute/builtin/atomic/atomic_add.inc b/tests/cl/program/execute/builtin/atomic/atomic_add.inc
> new file mode 100644
> index 0000000..4db4046
> --- /dev/null
> +++ b/tests/cl/program/execute/builtin/atomic/atomic_add.inc
> @@ -0,0 +1,23 @@
> +#define SIMPLE_TEST(TYPE) \
> +kernel void simple_##TYPE(global TYPE *out, ATOMIC_ADDRSPACE TYPE *mem, TYPE initial, TYPE value) { \
> +	*mem = initial; \
> +	TYPE a = atomic_add(mem, value); \
> +	out[0] = a; \
> +	out[1] = *mem; \
> +}
> +
> +#define THREADS_TEST(TYPE) \
> +kernel void threads_##TYPE(global TYPE *out, ATOMIC_ADDRSPACE TYPE *mem) { \
> +	*mem = 0; \
> +	barrier(CLK_LOCAL_MEM_FENCE); \
> +	TYPE id = get_local_id(0); \
> +	atomic_add(mem, id); \
> +	barrier(CLK_LOCAL_MEM_FENCE); \

Is CLK_LOCAL_MEM_FENCE necessary? shouldn't barrier(0) work as well?

> +	*out = *mem; \
> +}
> +
> +SIMPLE_TEST(int)
> +SIMPLE_TEST(uint)
> +
> +THREADS_TEST(int)
> +THREADS_TEST(uint)

-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140919/d353ab11/attachment.sig>


More information about the Piglit mailing list