Thanks for the patch, I have some comments as below:

> -----Original Message-----
> From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of
> xionghu....@intel.com
> Sent: Friday, April 25, 2014 5:37 AM
> To: beignet@lists.freedesktop.org
> Cc: Luo
> Subject: [Beignet] [PATCH] refine the event implementation and fix some
bugs.
> 
> From: Luo <xionghu....@intel.com>
> 
> 1 remove useless data element;
> 2 fix some logic bugs;
> 3 add implementation for clEnqueueMarkerWithWaitList.

According to the above statement, this patch should be split to 3 patches.
And clEnqueueMarkerWithWaitList is a opencl-1.2 function, should you submit
the patch
to the opencl-1.2 branch?

The policy is that if this patch is for both master/opencl-1.2 branch, you
don't need to add any tag to the patch subject.
If a patch is for the opencl-1.2 only, please put the [opencl-1.2] at the
patch subject.

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

Reply via email to