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

Christian König ckoenig.leichtzumerken at gmail.com
Mon Oct 22 10:10:03 UTC 2018


Am 22.10.18 um 11:47 schrieb Rex Zhu:
> 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: Christian König <christian.koenig 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