[PATCH] drm/amdgpu: Fix hive reference count leak

[Liu, Shaoyun]

[AMD Official Use Only - General]

Looks good to me .

-----Original Message-----
From: Chander, Vignesh <Vignesh.Chander at amd.com>
Sent: Friday, September 9, 2022 12:52 PM
To: amd-gfx at lists.freedesktop.org
Cc: Liu, Shaoyun <Shaoyun.Liu at amd.com>; Chander, Vignesh <Vignesh.Chander at amd.com>
Subject: [PATCH] drm/amdgpu: Fix hive reference count leak

both get_xgmi_hive and put_xgmi_hive can be skipped since the reset domain is not necessary for VF

Signed-off-by: Vignesh Chander <Vignesh.Chander at amd.com>
Reviewed-by: Shaoyun Liu <Shaoyun.Liu at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index e21804362995..943c9e750575 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -2451,9 +2451,9 @@ static int amdgpu_device_ip_init(struct amdgpu_device *adev)
         */
        if (adev->gmc.xgmi.num_physical_nodes > 1) {
                if (amdgpu_xgmi_add_device(adev) == 0) {
-                       struct amdgpu_hive_info *hive = amdgpu_get_xgmi_hive(adev);
-
                        if (!amdgpu_sriov_vf(adev)) {
+                               struct amdgpu_hive_info *hive = amdgpu_get_xgmi_hive(adev);
+
                                if (!hive->reset_domain ||
                                    !amdgpu_reset_get_reset_domain(hive->reset_domain)) {
                                        r = -ENOENT;
--
2.25.1