diff options
author | Christian König <christian.koenig@amd.com> | 2016-09-16 07:11:45 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-09-28 16:16:22 -0400 |
commit | 2a82ec21b9ba342dc90e054458e430f577beca89 (patch) | |
tree | aed39b86a5d9bcbe78ba256adce8324829bb70d3 | |
parent | 2698f6206f53bc23a40a1412b304225a6d62d120 (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.c | 8 |
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 | ||
1640 | error_unreserve: | ||
1641 | amdgpu_bo_unreserve(vm->page_directory); | ||
1642 | |||
1639 | error_free_page_directory: | 1643 | error_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); |