Re: [PATCH 1/2] drm/ttm: add a way to bo_wait for either the last read or last write

On Fri, Aug 12, 2011 at 7:21 PM, Jerome Glisse j.glisse@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