drm/amdgpu: fix idle state and bulk_moveable flag
authorChristian König <christian.koenig@amd.com>
Fri, 31 Aug 2018 09:08:06 +0000 (11:08 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Sun, 2 Sep 2018 15:16:44 +0000 (10:16 -0500)
Add BOs to the idle state again and correctly clear the flag when
new BOs are added.

Signed-off-by: Christian König <christian.koenig@amd.com>
Tested-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c

index f31fa35..d59222f 100644 (file)
@@ -156,12 +156,15 @@ static void amdgpu_vm_bo_base_init(struct amdgpu_vm_bo_base *base,
                return;
        list_add_tail(&base->bo_list, &bo->va);
 
-       if (bo->tbo.type == ttm_bo_type_kernel)
-               list_move(&base->vm_status, &vm->relocated);
-
        if (bo->tbo.resv != vm->root.base.bo->tbo.resv)
                return;
 
+       vm->bulk_moveable = false;
+       if (bo->tbo.type == ttm_bo_type_kernel)
+               list_move(&base->vm_status, &vm->relocated);
+       else
+               list_move(&base->vm_status, &vm->idle);
+
        if (bo->preferred_domains &
            amdgpu_mem_type_to_domain(bo->tbo.mem.mem_type))
                return;
@@ -1121,7 +1124,7 @@ restart:
                                           struct amdgpu_vm_bo_base,
                                           vm_status);
                bo_base->moved = false;
-               list_del_init(&bo_base->vm_status);
+               list_move(&bo_base->vm_status, &vm->idle);
 
                bo = bo_base->bo->parent;
                if (!bo)
@@ -2646,7 +2649,6 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct amdgpu_vm *vm,
                return r;
 
        vm->pte_support_ats = false;
-       vm->bulk_moveable = true;
 
        if (vm_context == AMDGPU_VM_CONTEXT_COMPUTE) {
                vm->use_cpu_for_update = !!(adev->vm_manager.vm_update_mode &