On Fri, Aug 12, 2011 at 7:21 PM, Jerome Glisse <j.glisse at gmail.com> wrote: >> diff --git a/drivers/gpu/drm/ttm/ttm_execbuf_util.c >> b/drivers/gpu/drm/ttm/ttm_execbuf_util.c >> index 3832fe1..0438296 100644 >> --- a/drivers/gpu/drm/ttm/ttm_execbuf_util.c >> +++ b/drivers/gpu/drm/ttm/ttm_execbuf_util.c >> @@ -223,6 +223,14 @@ void ttm_eu_fence_buffer_objects(struct list_head >> *list, void *sync_obj) >> ? ? ? ? ? ? ? ?bo = entry->bo; >> ? ? ? ? ? ? ? ?entry->old_sync_obj = bo->sync_obj; > > Here we should set entry->old_sync_obj_read & > entry->old_sync_obj_write to NULL so > we don't get bite by uninitialized memory (if caller fail or forget to do so)
OK, thanks. It's fixed in the attached patch. There are no other changes. Marek -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-drm-ttm-add-a-way-to-bo_wait-for-either-the-last-rea.patch Type: text/x-diff Size: 18479 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20110813/5a52dcc2/attachment.patch>
