Runtime: fix a cl_gpgpu_bind_image_for_vme NULL SIGSEGV.

Submitted by Luo, Xionghu on July 21, 2017, 1:43 a.m.

Details

Message ID 894E4BC922C573429354F1EC4342D61C7E087F91@SHSMSX104.ccr.corp.intel.com
State New
Series "Runtime: fix a cl_gpgpu_bind_image_for_vme NULL SIGSEGV."
Headers show

Commit Message

Luo, Xionghu July 21, 2017, 1:43 a.m.
LGTM.
Thanks.

Luo Xionghu
Best Regards

-----Original Message-----
From: Beignet [mailto:beignet-bounces@lists.freedesktop.org] On Behalf Of Yang Rong

Sent: Friday, July 21, 2017 12:11 PM
To: beignet@lists.freedesktop.org
Cc: Yang, Rong R <rong.r.yang@intel.com>
Subject: [Beignet] [PATCH] Runtime: fix a cl_gpgpu_bind_image_for_vme NULL SIGSEGV.

Signed-off-by: Yang Rong <rong.r.yang@intel.com>

---
 src/cl_command_queue.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

--
2.1.4

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

Patch hide | download patch | download mbox

diff --git a/src/cl_command_queue.c b/src/cl_command_queue.c index 43ff8fe..93902c8 100644
--- a/src/cl_command_queue.c
+++ b/src/cl_command_queue.c
@@ -155,7 +155,8 @@  cl_command_queue_bind_image(cl_command_queue queue, cl_kernel k, cl_gpgpu gpgpu,
                           image->w, image->h, image->depth,
                           image->row_pitch, image->slice_pitch, (cl_gpgpu_tiling)image->tiling);
     //We always setup media surface state, so this surface can be used for vme
-    else if( (image->fmt.image_channel_order == CL_R) && (image->fmt.image_channel_data_type == CL_UNORM_INT8) )
+    else if ((image->fmt.image_channel_order == CL_R) && (image->fmt.image_channel_data_type == CL_UNORM_INT8) &&
+             cl_gpgpu_bind_image_for_vme)
       cl_gpgpu_bind_image_for_vme(gpgpu, k->images[i].idx + BTI_WORKAROUND_IMAGE_OFFSET, image->base.bo,
                           image->offset + k->args[id].mem->offset,
                           image->intel_fmt, image->image_type, image->bpp,