Skip to content

Commit a107aeb

Browse files
ChristianKoenigAMDalexdeucher
authored andcommitted
drm/amdgpu: partially revert "revert to old status lock handling v3"
The CI systems are pointing out list corruptions, so we still need to fix something here. Keep the asserts, but revert the lock changes for now. Fixes: 59e4405 ("drm/amdgpu: revert to old status lock handling v3") Signed-off-by: Christian König <[email protected]> Acked-by: Alex Deucher <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent ddbfac1 commit a107aeb

File tree

4 files changed

+105
-68
lines changed

4 files changed

+105
-68
lines changed

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -726,12 +726,12 @@ amdgpu_userq_bo_validate(struct amdgpu_device *adev, struct drm_exec *exec,
726726
struct amdgpu_bo *bo;
727727
int ret;
728728

729-
spin_lock(&vm->invalidated_lock);
729+
spin_lock(&vm->status_lock);
730730
while (!list_empty(&vm->invalidated)) {
731731
bo_va = list_first_entry(&vm->invalidated,
732732
struct amdgpu_bo_va,
733733
base.vm_status);
734-
spin_unlock(&vm->invalidated_lock);
734+
spin_unlock(&vm->status_lock);
735735

736736
bo = bo_va->base.bo;
737737
ret = drm_exec_prepare_obj(exec, &bo->tbo.base, 2);
@@ -748,9 +748,9 @@ amdgpu_userq_bo_validate(struct amdgpu_device *adev, struct drm_exec *exec,
748748
if (ret)
749749
return ret;
750750

751-
spin_lock(&vm->invalidated_lock);
751+
spin_lock(&vm->status_lock);
752752
}
753-
spin_unlock(&vm->invalidated_lock);
753+
spin_unlock(&vm->status_lock);
754754

755755
return 0;
756756
}

0 commit comments

Comments
 (0)