[Beignet] [PATCH V2] fix enqueue_built_in_kernels bug. add image_1d_to_1d builtin kernel name.

Zhigang Gong zhigang.gong at linux.intel.com
Wed Jun 25 08:26:12 PDT 2014


Right, it turns out a real bug. I just sent a patch out to fix it.
Please check it out and have a try.

On Wed, Jun 25, 2014 at 09:41:10AM +0000, Luo, Xionghu wrote:
> hi zhigang,
> Just found this bug can be reproduced under debug build. 
> 
> Luo Xionghu
> Best Regards
> 
> 
> -----Original Message-----
> From: Zhigang Gong [mailto:zhigang.gong at linux.intel.com] 
> Sent: Wednesday, June 25, 2014 3:08 PM
> To: Luo, Xionghu
> Cc: beignet at lists.freedesktop.org
> Subject: Re: [Beignet] [PATCH V2] fix enqueue_built_in_kernels bug. add image_1d_to_1d builtin kernel name.
> 
> LGTM, will push latter, thanks.
> 
> On Wed, Jun 25, 2014 at 09:56:44AM +0800, xionghu.luo at intel.com wrote:
> > From: Luo <xionghu.luo at intel.com>
> > 
> > need asign the length to a local variable.
> > 
> > v2: bug was a false alarm. the case passed after regenerate cmake folder.
> > 
> > Signed-off-by Luo <xionghu.luo at intel.com>
> > ---
> >  src/cl_gt_device.h | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/src/cl_gt_device.h b/src/cl_gt_device.h index 
> > d7855cd..63c9047 100644
> > --- a/src/cl_gt_device.h
> > +++ b/src/cl_gt_device.h
> > @@ -87,6 +87,7 @@ DECL_INFO_STRING(built_in_kernels, "__cl_copy_region_align4;"
> >                                     "__cl_copy_region_unalign_dst_offset;"
> >                                     "__cl_copy_region_unalign_src_offset;"
> >                                     "__cl_copy_buffer_rect;"
> > +                                   "__cl_copy_image_1d_to_1d;"
> >                                     "__cl_copy_image_2d_to_2d;"
> >                                     "__cl_copy_image_3d_to_2d;"
> >                                     "__cl_copy_image_2d_to_3d;"
> > --
> > 1.8.1.2
> > 
> > _______________________________________________
> > Beignet mailing list
> > Beignet at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/beignet
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list