Am 15.03.22 um 08:11 schrieb Guchun Chen:
Use amdgpu_bo_free_kernel instead of amdgpu_bo_unref to
perform a clean unreference job.

I think the subject is incorrect. This doesn't leak the pin count, but rather produces a warning that the BO needs a proper cleanup.

Signed-off-by: Guchun Chen <guchun.c...@amd.com>

With the subject line re-validated the patch is Reviewed-by: Christian König <christian.koe...@amd.com>

---
  drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c 
b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index 431742eb7811..e4e7e6dbc6fb 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -1721,7 +1721,7 @@ static int gmc_v9_0_sw_fini(void *handle)
        amdgpu_gem_force_release(adev);
        amdgpu_vm_manager_fini(adev);
        amdgpu_gart_table_vram_free(adev);
-       amdgpu_bo_unref(&adev->gmc.pdb0_bo);
+       amdgpu_bo_free_kernel(&adev->gmc.pdb0_bo, NULL, adev->gmc.ptr_pdb0);
        amdgpu_bo_fini(adev);
return 0;

Reply via email to