aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian König <christian.koenig@amd.com>2016-09-16 07:11:45 -0400
committerAlex Deucher <alexander.deucher@amd.com>2016-09-28 16:16:22 -0400
commit2a82ec21b9ba342dc90e054458e430f577beca89 (patch)
treeaed39b86a5d9bcbe78ba256adce8324829bb70d3
parent2698f6206f53bc23a40a1412b304225a6d62d120 (diff)
drm/amdgpu: fix initializing the VM last eviction counter
Close a very small window where something can go wrong. Signed-off-by: Christian König <christian.koenig@amd.com> Acked-by: Edward O'Callaghan <funfunctor@folklore1984.net> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
-rw-r--r--drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 9f4ff29df80d..04aa5a77e953 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -1629,13 +1629,17 @@ int amdgpu_vm_init(struct amdgpu_device *adev, struct amdgpu_vm *vm)
1629 goto error_free_page_directory; 1629 goto error_free_page_directory;
1630 1630
1631 r = amdgpu_vm_clear_bo(adev, vm, vm->page_directory); 1631 r = amdgpu_vm_clear_bo(adev, vm, vm->page_directory);
1632 amdgpu_bo_unreserve(vm->page_directory);
1633 if (r) 1632 if (r)
1634 goto error_free_page_directory; 1633 goto error_unreserve;
1634
1635 vm->last_eviction_counter = atomic64_read(&adev->num_evictions); 1635 vm->last_eviction_counter = atomic64_read(&adev->num_evictions);
1636 amdgpu_bo_unreserve(vm->page_directory);
1636 1637
1637 return 0; 1638 return 0;
1638 1639
1640error_unreserve:
1641 amdgpu_bo_unreserve(vm->page_directory);
1642
1639error_free_page_directory: 1643error_free_page_directory:
1640 amdgpu_bo_unref(&vm->page_directory->shadow); 1644 amdgpu_bo_unref(&vm->page_directory->shadow);
1641 amdgpu_bo_unref(&vm->page_directory); 1645 amdgpu_bo_unref(&vm->page_directory);