[PATCH] drm/amdgpu: Reverse the sequence of ctx_mgr_fini and vm_fini in amdgpu_driver_postclose_kms

Zhang, Jerry(Junwei) Jerry.Zhang at amd.com
Tue Oct 23 02:17:05 UTC 2018


On 10/22/2018 05:47 PM, Rex Zhu wrote:
> csa buffer will be created per ctx, when ctx fini,
> the csa buffer and va will be released. so need to
> do ctx_mgr fin before vm fini.
>
> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
Reviewed-by: Junwei Zhang <Jerry.Zhang at amd.com>

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
> index 27de848..f2ef9a1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
> @@ -1054,8 +1054,8 @@ void amdgpu_driver_postclose_kms(struct drm_device *dev,
>   	pasid = fpriv->vm.pasid;
>   	pd = amdgpu_bo_ref(fpriv->vm.root.base.bo);
>   
> -	amdgpu_vm_fini(adev, &fpriv->vm);
>   	amdgpu_ctx_mgr_fini(&fpriv->ctx_mgr);
> +	amdgpu_vm_fini(adev, &fpriv->vm);
>   
>   	if (pasid)
>   		amdgpu_pasid_free_delayed(pd->tbo.resv, pasid);



More information about the amd-gfx mailing list