尽管有障碍,打开CL没有同步

时间:2013-06-15 19:03:12

标签: opencl pyopencl

我刚开始通过Python的PyOpenCL接口使用OpenCL。我试图创建一个非常简单的“循环”程序,其中每个内核中每个循环的结果取决于上一个循环周期中另一个内核的输出,但我遇到同步问题:

__kernel void part1(__global float* a, __global float* c)
{
    unsigned int i = get_global_id(0);

    c[i] = 0;
    barrier(CLK_GLOBAL_MEM_FENCE);

    if (i < 9)
    {
        for(int t = 0; t < 2; t++){
            c[i] = c[i+1] + a[i];
            barrier(CLK_GLOBAL_MEM_FENCE);
       }
    }
}

主机应用程序

import pyopencl as cl
from numpy import *

ctx = cl.create_some_context()
queue = cl.CommandQueue(ctx)

#read in the OpenCL source file as a string
f = open('recurrent.cl', 'r')
fstr = "".join(f.readlines())

#create the program
program = cl.Program(ctx, fstr).build()

mf = cl.mem_flags

#initialize client side (CPU) arrays
a = array(range(10), dtype=float32)

#create OpenCL buffers
a_buf = cl.Buffer(ctx, mf.READ_ONLY | mf.COPY_HOST_PTR, hostbuf=a)
dest_buf = cl.Buffer(ctx, mf.WRITE_ONLY, a.nbytes)

#execute program
program.part1(queue, a.shape, None, a_buf, dest_buf)
c = empty_like(a)
cl.enqueue_read_buffer(queue, dest_buf, c).wait()

print "a", a
print "c", c

结果是

a [ 0.  1.  2.  3.  4.  5.  6.  7.  8.  9.]
c [  0.   1.   5.   3.   4.  18.  13.   7.   8.   0.]

如您所见,某些结果值是正确的。例如。第三个位置= 5 = 3 + 2但是例如第二个位置是2 = 0 + 2.因此尽管有障碍,但总和超过了不同时间点的其他线程的结果。我认为障碍会确保所有线程都已到达并将结果写入全局内存?

这可能非常简单,我希望得到任何提示和评论!

PS:我在使用英特尔SDK的Sandy Bridge CPU上运行它。

2 个答案:

答案 0 :(得分:2)

我想我现在有了答案。 OpenCL代码实际上完全没问题。但是,如果所有线程都在一个工作组中,则障碍只会起作用。情况并非如此,通过使用get_local_id(0)读取local_id(如Huseyin所建议的),可以轻松检查。在我的例子中,主机为每个线程创建了一个工作组 - 而不是将所有线程放在一个工作组中。性能方面有意义,比较

Questions about global and local work size

但是,在我们的例子中,我们需要确保数据在线程之间是同步的,因此它们都应该在一个工作组中。为此,我们需要更改程序1的执行,

program.part1(queue, a.shape, None, a_buf, dest_buf)

第二个参数引用作业的global_size(所以创建的线程数),而第三个参数似乎引用local_size,即每个工作组的线程数。因此,这一行应该是

program.part1(queue, a.shape, a.shape, a_buf, dest_buf)

这会创建一个包含所有线程的工作组(但要注意一个工作组中允许的最大工作者大小!)。现在,代码仍然无效。最后一个问题是关于OpenCL代码中的障碍:id = 10的最后一个线程没有看到循环中的障碍,因此所有线程都在等待最后一个线程进入障碍(尽管我想知道为什么这样做没有'抛出异常?)。所以我们只需要减少线程总数(去掉最后一个),

program.part1(queue, (a.shape[0]-1,), (a.shape[0]-1,), a_buf, dest_buf)

有效!学到了一些教训......

再次感谢Huseyin! blue2script

答案 1 :(得分:0)

编辑:用户blue2script是正确的,这是“障碍没有被所有本地线程击中”的问题。最重要的是,屏障无法在计算单元/工作组之间同步。

我的回答没有添加任何内容,也没有解决任何问题。所以不要在下面的内核函数中看到if。这是错的。


不完全

 __kernel void part1(__global float* a, __global float* c)
 {
      unsigned int i = get_global_id(0);

      c[i] = 0;
      barrier(CLK_GLOBAL_MEM_FENCE);

      if (i < 9)
      {
          for(int t = 0; t < 2; t++)
          {
              c[i] = c[i+1] + a[i];//c[i+1] is neighbour thread's variable
                                   //and there is no guarantee that
                                   //which one(ith or (i+1)st) computes first
                                   //so you need to get a copy of c[] first
              barrier(CLK_GLOBAL_MEM_FENCE);//thats why this line is not helping
          }
      }
 }

使用全球

 __kernel void part1(__global float* a, __global float* c,__global float* d)
 {
      unsigned int i = get_global_id(0);

      c[i] = 0;
      d[i]=c[i]; 
      barrier(CLK_GLOBAL_MEM_FENCE);

      if (i < 9)
      {
          for(int t = 0; t < 2; t++)
          {
              d[i] = c[i+1] + a[i];//it is guaranteed that no neighbour thread can
                                   //change this threads d[i] element before/after
                                   //execution
              barrier(CLK_GLOBAL_MEM_FENCE);
              c[i]=d[i];
              barrier(CLK_GLOBAL_MEM_FENCE);
          }
      }
      barrier(CLK_GLOBAL_MEM_FENCE);

 }

使用本地人(工作组大小为256,工作总大小是其中的倍数):

 __kernel void part1(__global float* a, __global float* c)
 {
      unsigned int i = get_global_id(0);
      unsigned int Li=get_local_id(0);
      __local d[256];
      c[i] = 0;
      barrier(CLK_GLOBAL_MEM_FENCE);
      d[Li]=c[i]; 
      barrier(CLK_LOCAL_MEM_FENCE);

      if (i < 9)
      {
          for(int t = 0; t < 2; t++)
          {
              d[Li] = c[i+1] + a[i];//it is guaranteed that no neighbour thread can
                                   //change this threads d[i] element before/after
                                   //execution

             barrier(CLK_LOCAL_MEM_FENCE);
             c[i]=d[Li]; //guaranteed they dont interfere each other
             barrier(CLK_LOCAL_MEM_FENCE);
          }
      }

 }

工作组:

enter image description here

使用私人

 __kernel void part1(__global float* a, __global float* c)
 {
      unsigned int i = get_global_id(0);
      unsigned int Li=get_local_id(0);
      __private f1;
      c[i] = 0;

      if (i < 9)
      {
          for(int t = 0; t < 2; t++)
          {
              f1 = c[i+1] + a[i];

             barrier(CLK_GLOBAL_MEM_FENCE);
             c[i]=f1; //guaranteed they dont interfere each other
             barrier(CLK_GLOBAL_MEM_FENCE);
          }
      }

 }