Commit c1806d78 authored by Philip Yang's avatar Philip Yang Committed by Alex Deucher
Browse files

drm/amdgpu: Use vm status_lock to protect vm idle list 



Use vm_status_lock to protect all vm_status state transitions to allow
them to happen without a reservation lock in unlocked page table
updates.

Signed-off-by: default avatarPhilip Yang <Philip.Yang@amd.com>
Acked-by: default avatarFelix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent b38e77cb

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

+3 −1
Original line number Diff line number Diff line
@@ -211,7 +211,9 @@ static void amdgpu_vm_bo_moved(struct amdgpu_vm_bo_base *vm_bo) 
 */

static void amdgpu_vm_bo_idle(struct amdgpu_vm_bo_base *vm_bo)

{

	spin_lock(&vm_bo->vm->status_lock);

	list_move(&vm_bo->vm_status, &vm_bo->vm->idle);

	spin_unlock(&vm_bo->vm->status_lock);

	vm_bo->moved = false;

}
@@ -2554,6 +2556,7 @@ void amdgpu_debugfs_vm_bo_info(struct amdgpu_vm *vm, struct seq_file *m) 
	unsigned int total_done_objs = 0;

	unsigned int id = 0;



	spin_lock(&vm->status_lock);

	seq_puts(m, "\tIdle BOs:\n");

	list_for_each_entry_safe(bo_va, tmp, &vm->idle, base.vm_status) {

		if (!bo_va->base.bo)
@@ -2591,7 +2594,6 @@ void amdgpu_debugfs_vm_bo_info(struct amdgpu_vm *vm, struct seq_file *m) 
	id = 0;



	seq_puts(m, "\tInvalidated BOs:\n");

	spin_lock(&vm->status_lock);

	list_for_each_entry_safe(bo_va, tmp, &vm->invalidated, base.vm_status) {

		if (!bo_va->base.bo)

			continue;