On 2017年08月26日 15:19, Felix Kuehling wrote:
Make sure vm->root.bo is not left reserved if amdgpu_bo_kmap fails.

Change-Id: If3687b39a50b0ffe7f8be2ea6e927fa2ca0f9e45
Signed-off-by: Felix Kuehling <[email protected]>
Reviewed-by: Chunming Zhou <[email protected]>

---
  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 9 +++------
  1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index e57a72e..70d7632 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -2556,14 +2556,11 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct 
amdgpu_vm *vm,
                goto error_free_root;
vm->last_eviction_counter = atomic64_read(&adev->num_evictions);
-
-       if (vm->use_cpu_for_update) {
+       if (vm->use_cpu_for_update)
                r = amdgpu_bo_kmap(vm->root.bo, NULL);
-               if (r)
-                       goto error_free_root;
-       }
-
        amdgpu_bo_unreserve(vm->root.bo);
+       if (r)
+               goto error_free_root;
vm->vm_context = vm_context;
        if (vm_context == AMDGPU_VM_CONTEXT_COMPUTE) {

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to