[PATCH] drm/amdgpu: Remove one duplicated ef removal

Mike Lothian mike at fireburn.co.uk
Tue Jul 19 08:31:47 UTC 2022


With this patch applied on Linus's tree 5.18-rc7 I get the following warning:

drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:1367:20: warning:
unused variable 'pd' [-Wunused-variable]
       struct amdgpu_bo *pd = vm->root.bo;
                         ^
1 warning generated.

It does fix the nullpointer though :D


On Fri, 8 Jul 2022 at 02:55, xinhui pan <xinhui.pan at amd.com> wrote:
>
> That has been done in BO release notify.
>
> Signed-off-by: xinhui pan <xinhui.pan at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> index 0f9811d02f61..49536fa8d269 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> @@ -1583,11 +1583,6 @@ void amdgpu_amdkfd_gpuvm_destroy_cb(struct amdgpu_device *adev,
>         if (!process_info)
>                 return;
>
> -       /* Release eviction fence from PD */
> -       amdgpu_bo_reserve(pd, false);
> -       amdgpu_bo_fence(pd, NULL, false);
> -       amdgpu_bo_unreserve(pd);
> -
>         /* Update process info */
>         mutex_lock(&process_info->lock);
>         process_info->n_vms--;
> --
> 2.34.1
>


More information about the amd-gfx mailing list