Commit 1a33860d authored Jul 26, 2017 by Christian König Committed by Alex Deucher Aug 17, 2017

drm/ttm: remove nonsense wait in ttm_bo_cleanup_refs_and_unlock



With shared reservation objects the assumption that no fence
could have been added isn't true any more.

Additional to that the BO is about to be destroyed, so removing the
fences now has no advantage whatsoever.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

parent 8824c751

drivers/gpu/drm/ttm/ttm_bo.c

+0 −7

Original line number	Diff line number	Diff line
		@@ -505,13 +505,6 @@ static int ttm_bo_cleanup_refs_and_unlock(struct ttm_buffer_object *bo,
		spin_unlock(&glob->lru_lock);
		return 0;
		}

		/*
		* remove sync_obj with ttm_bo_wait, the wait should be
		* finished, and no new wait object should have been added.
		*/
		ret = ttm_bo_wait(bo, false, true);
		WARN_ON(ret);
		}

		if (ret \|\| unlikely(list_empty(&bo->ddestroy))) {

Admin message