diff options
author | Christian König <christian.koenig@amd.com> | 2017-09-06 10:55:16 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-09-12 14:27:48 -0400 |
commit | aebc5e6f50f770ec9392c3ca804f18b30797dfa7 (patch) | |
tree | 9101bec930d4a1778616057ffbce270f0ee90330 /drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | |
parent | 9cca0b8e5df0ac438c65eec5044bfa089d16fbbe (diff) |
drm/amdgpu: rework amdgpu_cs_find_mapping
Use the VM instead of the BO list to find the BO for a virtual address.
This fixes UVD/VCE in physical mode with VM local BOs.
Signed-off-by: Christian König <christian.koenig@amd.com>
Acked-by: Leo Liu <leo.liu@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 | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c index 545531db66db..758bbb9e77f3 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | |||
@@ -2086,6 +2086,7 @@ static void amdgpu_vm_bo_insert_map(struct amdgpu_device *adev, | |||
2086 | struct amdgpu_vm *vm = bo_va->base.vm; | 2086 | struct amdgpu_vm *vm = bo_va->base.vm; |
2087 | struct amdgpu_bo *bo = bo_va->base.bo; | 2087 | struct amdgpu_bo *bo = bo_va->base.bo; |
2088 | 2088 | ||
2089 | mapping->bo_va = bo_va; | ||
2089 | list_add(&mapping->list, &bo_va->invalids); | 2090 | list_add(&mapping->list, &bo_va->invalids); |
2090 | amdgpu_vm_it_insert(mapping, &vm->va); | 2091 | amdgpu_vm_it_insert(mapping, &vm->va); |
2091 | 2092 | ||
@@ -2263,6 +2264,7 @@ int amdgpu_vm_bo_unmap(struct amdgpu_device *adev, | |||
2263 | 2264 | ||
2264 | list_del(&mapping->list); | 2265 | list_del(&mapping->list); |
2265 | amdgpu_vm_it_remove(mapping, &vm->va); | 2266 | amdgpu_vm_it_remove(mapping, &vm->va); |
2267 | mapping->bo_va = NULL; | ||
2266 | trace_amdgpu_vm_bo_unmap(bo_va, mapping); | 2268 | trace_amdgpu_vm_bo_unmap(bo_va, mapping); |
2267 | 2269 | ||
2268 | if (valid) | 2270 | if (valid) |
@@ -2348,6 +2350,7 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, | |||
2348 | if (tmp->last > eaddr) | 2350 | if (tmp->last > eaddr) |
2349 | tmp->last = eaddr; | 2351 | tmp->last = eaddr; |
2350 | 2352 | ||
2353 | tmp->bo_va = NULL; | ||
2351 | list_add(&tmp->list, &vm->freed); | 2354 | list_add(&tmp->list, &vm->freed); |
2352 | trace_amdgpu_vm_bo_unmap(NULL, tmp); | 2355 | trace_amdgpu_vm_bo_unmap(NULL, tmp); |
2353 | } | 2356 | } |
@@ -2374,6 +2377,19 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, | |||
2374 | } | 2377 | } |
2375 | 2378 | ||
2376 | /** | 2379 | /** |
2380 | * amdgpu_vm_bo_lookup_mapping - find mapping by address | ||
2381 | * | ||
2382 | * @vm: the requested VM | ||
2383 | * | ||
2384 | * Find a mapping by it's address. | ||
2385 | */ | ||
2386 | struct amdgpu_bo_va_mapping *amdgpu_vm_bo_lookup_mapping(struct amdgpu_vm *vm, | ||
2387 | uint64_t addr) | ||
2388 | { | ||
2389 | return amdgpu_vm_it_iter_first(&vm->va, addr, addr); | ||
2390 | } | ||
2391 | |||
2392 | /** | ||
2377 | * amdgpu_vm_bo_rmv - remove a bo to a specific vm | 2393 | * amdgpu_vm_bo_rmv - remove a bo to a specific vm |
2378 | * | 2394 | * |
2379 | * @adev: amdgpu_device pointer | 2395 | * @adev: amdgpu_device pointer |
@@ -2398,6 +2414,7 @@ void amdgpu_vm_bo_rmv(struct amdgpu_device *adev, | |||
2398 | list_for_each_entry_safe(mapping, next, &bo_va->valids, list) { | 2414 | list_for_each_entry_safe(mapping, next, &bo_va->valids, list) { |
2399 | list_del(&mapping->list); | 2415 | list_del(&mapping->list); |
2400 | amdgpu_vm_it_remove(mapping, &vm->va); | 2416 | amdgpu_vm_it_remove(mapping, &vm->va); |
2417 | mapping->bo_va = NULL; | ||
2401 | trace_amdgpu_vm_bo_unmap(bo_va, mapping); | 2418 | trace_amdgpu_vm_bo_unmap(bo_va, mapping); |
2402 | list_add(&mapping->list, &vm->freed); | 2419 | list_add(&mapping->list, &vm->freed); |
2403 | } | 2420 | } |