[Beignet] [PATCH 3/4] Fix a event leak in create context

Zhigang Gong zhigang.gong at linux.intel.com
Wed Oct 7 23:22:19 PDT 2015


Nice catch, this patch LGTM.

On Thu, Sep 24, 2015 at 05:13:26PM +0800, Pan Xiuli wrote:
> We get an event out of NDRangeKernel, and we don't release it.
> As an gpgpu event it can also make drm buffer leak, to avoid
> potenial error we just release it.w
> 
> Signed-off-by: Pan Xiuli <xiuli.pan at intel.com>
> ---
>  src/cl_device_id.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/cl_device_id.c b/src/cl_device_id.c
> index 78d2cf4..a3d3fc4 100644
> --- a/src/cl_device_id.c
> +++ b/src/cl_device_id.c
> @@ -622,6 +622,7 @@ cl_self_test(cl_device_id device, cl_self_test_res atomic_in_l3_flag)
>                    // Atomic fail need to test SLM again with atomic in L3 feature disabled.
>                    tested = 0;
>                  }
> +                clReleaseEvent(kernel_finished);
>                }
>              }
>              clReleaseMemObject(buffer);
> -- 
> 2.1.4
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list