drm/amdgpu: revert to old status lock handling v3

Browse source

It turned out that protecting the status of each bo_va with a
spinlock was just hiding problems instead of solving them.

Revert the whole approach, add a separate stats_lock and lockdep
assertions that the correct reservation lock is held all over the place.

This not only allows for better checks if a state transition is properly
protected by a lock, but also switching back to using list macros to
iterate over the state of lists protected by the dma_resv lock of the
root PD.

v2: re-add missing check
v3: split into two patches

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

...

This commit is contained in:

Christian König 2025-08-27 11:45:45 +02:00 • committed by Alex Deucher

parent 0aa09d8a6c

commit 59e4405e9e

4 changed files with 93 additions and 102 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

8

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

View file

168

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

View file

15

drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h

View file

4

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

View file