[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