diff options
author | Christian König <christian.koenig@amd.com> | 2018-08-31 05:08:06 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2018-09-02 11:16:44 -0400 |
commit | 5d35ed4832dab334e076a24c18a52776c2f24911 (patch) | |
tree | 65226fc3025e1281810adaced06a8902d62ca80b /drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | |
parent | b995795bf09b6bb7847a2a9fc8e6b5b4ab0ce20c (diff) |
drm/amdgpu: fix idle state and bulk_moveable flag
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>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c index f31fa351caba..d59222fb5931 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | |||
@@ -156,12 +156,15 @@ static void amdgpu_vm_bo_base_init(struct amdgpu_vm_bo_base *base, | |||
156 | return; | 156 | return; |
157 | list_add_tail(&base->bo_list, &bo->va); | 157 | list_add_tail(&base->bo_list, &bo->va); |
158 | 158 | ||
159 | if (bo->tbo.type == ttm_bo_type_kernel) | ||
160 | list_move(&base->vm_status, &vm->relocated); | ||
161 | |||
162 | if (bo->tbo.resv != vm->root.base.bo->tbo.resv) | 159 | if (bo->tbo.resv != vm->root.base.bo->tbo.resv) |
163 | return; | 160 | return; |
164 | 161 | ||
162 | vm->bulk_moveable = false; | ||
163 | if (bo->tbo.type == ttm_bo_type_kernel) | ||
164 | list_move(&base->vm_status, &vm->relocated); | ||
165 | else | ||
166 | list_move(&base->vm_status, &vm->idle); | ||
167 | |||
165 | if (bo->preferred_domains & | 168 | if (bo->preferred_domains & |
166 | amdgpu_mem_type_to_domain(bo->tbo.mem.mem_type)) | 169 | amdgpu_mem_type_to_domain(bo->tbo.mem.mem_type)) |
167 | return; | 170 | return; |
@@ -1121,7 +1124,7 @@ restart: | |||
1121 | struct amdgpu_vm_bo_base, | 1124 | struct amdgpu_vm_bo_base, |
1122 | vm_status); | 1125 | vm_status); |
1123 | bo_base->moved = false; | 1126 | bo_base->moved = false; |
1124 | list_del_init(&bo_base->vm_status); | 1127 | list_move(&bo_base->vm_status, &vm->idle); |
1125 | 1128 | ||
1126 | bo = bo_base->bo->parent; | 1129 | bo = bo_base->bo->parent; |
1127 | if (!bo) | 1130 | if (!bo) |
@@ -2646,7 +2649,6 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct amdgpu_vm *vm, | |||
2646 | return r; | 2649 | return r; |
2647 | 2650 | ||
2648 | vm->pte_support_ats = false; | 2651 | vm->pte_support_ats = false; |
2649 | vm->bulk_moveable = true; | ||
2650 | 2652 | ||
2651 | if (vm_context == AMDGPU_VM_CONTEXT_COMPUTE) { | 2653 | if (vm_context == AMDGPU_VM_CONTEXT_COMPUTE) { |
2652 | vm->use_cpu_for_update = !!(adev->vm_manager.vm_update_mode & | 2654 | vm->use_cpu_for_update = !!(adev->vm_manager.vm_update_mode & |