[Beignet] [PATCH V2 2/4] Add the internal used kernels for buffer copy

[junyan . he]

From: Junyan He junyan...@linux.intel.com

Add internal used kernels for buffer copy. The align
1 4 16 is seperated into three kernels to improve
performance. The CMakeList is also updated.

Signed-off-by: Junyan He junyan...@linux.intel.com
---
 src/CMakeLists.txt  |   18 ++
 src/kernels/cl_internal_copy_buf_align1.cl  |6 ++
 src/kernels/cl_internal_copy_buf_align16.cl |9 +
 src/kernels/cl_internal_copy_buf_align4.cl  |6 ++
 4 files changed, 39 insertions(+)
 create mode 100644 src/kernels/cl_internal_copy_buf_align1.cl
 create mode 100644 src/kernels/cl_internal_copy_buf_align16.cl
 create mode 100644 src/kernels/cl_internal_copy_buf_align4.cl

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 3fc8689..de3b6d2 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -4,7 +4,25 @@ include_directories(${CMAKE_CURRENT_SOURCE_DIR}
 ${CMAKE_CURRENT_SOURCE_DIR}/../include
 ${MESA_SOURCE_INCLUDES})
 
+macro (MakeKernelBinStr KERNEL_PATH KERNEL_FILES)
+foreach (KF ${KERNEL_FILES})
+  set (input_file ${KERNEL_PATH}/${KF}.cl)
+  set (output_file ${KERNEL_PATH}/${KF}_str.c)
+  list (APPEND KERNEL_STR_FILES ${output_file})
+  add_custom_command(
+OUTPUT ${output_file}
+COMMAND rm -rf ${output_file}
+COMMAND ${CMAKE_CURRENT_BINARY_DIR}/../backend/src/gbe_bin_generater -s 
${input_file} -o${output_file}
+MAIN_DEPENDENCY ${input_file})
+endforeach (KF)
+endmacro (MakeKernelBinStr)
+
+set (KERNEL_STR_FILES)
+set (KERNEL_NAMES cl_internal_copy_buf_align1 cl_internal_copy_buf_align4 
cl_internal_copy_buf_align16)
+MakeKernelBinStr (${CMAKE_CURRENT_SOURCE_DIR}/kernels/ ${KERNEL_NAMES})
+
 set(OPENCL_SRC
+${KERNEL_STR_FILES}
 cl_api.c
 cl_alloc.c
 cl_kernel.c
diff --git a/src/kernels/cl_internal_copy_buf_align1.cl 
b/src/kernels/cl_internal_copy_buf_align1.cl
new file mode 100644
index 000..ac8bddc
--- /dev/null
+++ b/src/kernels/cl_internal_copy_buf_align1.cl
@@ -0,0 +1,6 @@
+kernel void __cl_cpy_region_align1 ( global char* src, unsigned int src_offset,
+ global char* dst, unsigned int dst_offset 
)
+{
+int i = get_global_id(0);
+dst[i+dst_offset] = src[i+src_offset];
+}
diff --git a/src/kernels/cl_internal_copy_buf_align16.cl 
b/src/kernels/cl_internal_copy_buf_align16.cl
new file mode 100644
index 000..dc80966
--- /dev/null
+++ b/src/kernels/cl_internal_copy_buf_align16.cl
@@ -0,0 +1,9 @@
+kernel void __cl_cpy_region_align16 ( global float* src, unsigned int 
src_offset,
+  global float* dst, unsigned int 
dst_offset )
+{
+int i = get_global_id(0) * 4;
+dst[i+dst_offset] = src[i+src_offset];
+dst[i+dst_offset + 1] = src[i+src_offset + 1];
+dst[i+dst_offset + 2] = src[i+src_offset + 2];
+dst[i+dst_offset + 3] = src[i+src_offset + 3];
+}
diff --git a/src/kernels/cl_internal_copy_buf_align4.cl 
b/src/kernels/cl_internal_copy_buf_align4.cl
new file mode 100644
index 000..2c16e60
--- /dev/null
+++ b/src/kernels/cl_internal_copy_buf_align4.cl
@@ -0,0 +1,6 @@
+kernel void __cl_cpy_region_align4 ( global float* src, unsigned int 
src_offset,
+ global float* dst, unsigned int 
dst_offset )
+{
+int i = get_global_id(0);
+dst[i+dst_offset] = src[i+src_offset];
+}
-- 
1.7.9.5

___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet

Re: [Beignet] [PATCH V2 2/4] Add the internal used kernels for buffer copy

[Simon Richter]

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

On 09.10.2013 09:55, junyan...@inbox.com wrote:

 +macro (MakeKernelBinStr KERNEL_PATH KERNEL_FILES) +foreach (KF
 ${KERNEL_FILES}) +  set (input_file ${KERNEL_PATH}/${KF}.cl) +  set
 (output_file ${KERNEL_PATH}/${KF}_str.c) +  list (APPEND
 KERNEL_STR_FILES ${output_file}) +  add_custom_command( +OUTPUT
 ${output_file} +COMMAND rm -rf ${output_file} +COMMAND
 ${CMAKE_CURRENT_BINARY_DIR}/../backend/src/gbe_bin_generater -s
 ${input_file} -o${output_file} +MAIN_DEPENDENCY ${input_file}) 
 +endforeach (KF) +endmacro (MakeKernelBinStr)

This fails to compile from clean tree with -j, because there is no
dependency ordering:

/bin/sh: 1: ../backend/src/gbe_bin_generater: not found

   Simon
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Icedove - http://www.enigmail.net/

iJwEAQECAAYFAlJVIVgACgkQ0sfeulffv7tEkAQAlLisoaKm8oMnkluxxVg8lXhM
JKPOgpE3DQzylIY8QfEUErcpTwTvDXfAwXSLCjDeIHnp5wtd3+VQ9sT+KD+NWefa
oAjp/y5Phjp37qLON/bB6iMx6ZnIVPtz6IlpUD3RdMUwC3ghefRku5VM//FtPERT
1HS+4qto+nbaLkyFj+c=
=4uO4
-END PGP SIGNATURE-
___
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet