Commit e8e89622 authored by Daniel Vetter's avatar Daniel Vetter Committed by Dave Airlie

drm/ttm: fix fence locking in ttm_buffer_object_transfer

Noticed while reviewing the fence locking in the radeon pageflip

v2: Instead of grabbing the bdev->fence_lock in object_transfer just
move the single callsite of that function a few lines, so that it is
protected by the fence_lock. Suggested by Jerome Glisse.

v3: Fix typo in commit message.
Reviewed-by: default avatarJerome Glisse <>
Signed-off-by: default avatarDaniel Vetter <>
Signed-off-by: default avatarDave Airlie <>
parent c75be259
......@@ -654,11 +654,13 @@ int ttm_bo_move_accel_cleanup(struct ttm_buffer_object *bo,
set_bit(TTM_BO_PRIV_FLAG_MOVING, &bo->priv_flags);
/* ttm_buffer_object_transfer accesses bo->sync_obj */
ret = ttm_buffer_object_transfer(bo, &ghost_obj);
if (tmp_obj)
ret = ttm_buffer_object_transfer(bo, &ghost_obj);
if (ret)
return ret;
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment