Commit 30f2daed authored by Pierre-Eric Pelloux-Prayer's avatar Pierre-Eric Pelloux-Prayer Committed by Alex Deucher
Browse files

drm/amdgpu: add missing lock in amdgpu_benchmark_do_move 



Taking the entity lock is required to guarantee the ordering of
execution. The next commit will add a check that the lock is
held.

Signed-off-by: default avatarPierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3d9eea6d

drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c

+2 −0
Original line number Diff line number Diff line
@@ -35,6 +35,7 @@ static int amdgpu_benchmark_do_move(struct amdgpu_device *adev, unsigned size, 
	struct dma_fence *fence;

	int i, r;



	mutex_lock(&adev->mman.default_entity.lock);

	stime = ktime_get();

	for (i = 0; i < n; i++) {

		r = amdgpu_copy_buffer(adev, &adev->mman.default_entity,
@@ -47,6 +48,7 @@ static int amdgpu_benchmark_do_move(struct amdgpu_device *adev, unsigned size, 
		if (r)

			goto exit_do_move;

	}

	mutex_unlock(&adev->mman.default_entity.lock);



exit_do_move:

	etime = ktime_get();