[Beignet] [PATCH] only release cmrt device when it is already created

Yang, Rong R rong.r.yang at intel.com
Fri Apr 29 02:05:47 UTC 2016


LGTM, pushed, thanks.

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Guo Yejun
> Sent: Thursday, April 28, 2016 7:48
> To: beignet at lists.freedesktop.org
> Cc: Guo, Yejun <yejun.guo at intel.com>
> Subject: [Beignet] [PATCH] only release cmrt device when it is already
> created
> 
> this patch fixed the issue at
> https://bugs.freedesktop.org/show_bug.cgi?id=95136
> 
> Signed-off-by: Guo Yejun <yejun.guo at intel.com>
> ---
>  src/cl_api.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/cl_api.c b/src/cl_api.c index 3902592..881ea6d 100644
> --- a/src/cl_api.c
> +++ b/src/cl_api.c
> @@ -279,7 +279,8 @@ cl_int
>  clReleaseDevice(cl_device_id device)
>  {
>  #ifdef HAS_CMRT
> -  cmrt_destroy_device(device);
> +  if (device->cmrt_device != NULL)
> +    cmrt_destroy_device(device);
>  #endif
> 
>    // XXX stub for C++ Bindings
> --
> 1.9.1
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list