Re: [PATCH 1/9] drm/amdgpu: Fix error handling in amdgpu_vm_init

From: zhoucm1 <david1.zhou-5C7GfCeVMHo@public.gmane.org>
To: Felix Kuehling <Felix.Kuehling-5C7GfCeVMHo@public.gmane.org>,
	amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [PATCH 1/9] drm/amdgpu: Fix error handling in amdgpu_vm_init
Date: Mon, 28 Aug 2017 10:51:24 +0800	[thread overview]
Message-ID: <7e24c6b5-6f8f-88c7-d958-65062df27177@amd.com> (raw)
In-Reply-To: <1503731949-22742-2-git-send-email-Felix.Kuehling-5C7GfCeVMHo@public.gmane.org>


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 <Felix.Kuehling@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>

> ---
>   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
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx