[Public]

Acked-by: Alex Deucher <alexander.deuc...@amd.com>
________________________________
From: Yifan Zha <yifan....@amd.com>
Sent: Friday, June 28, 2024 4:42 AM
To: amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>; Deucher, 
Alexander <alexander.deuc...@amd.com>; Zhang, Hawking <hawking.zh...@amd.com>
Cc: Chen, Horace <horace.c...@amd.com>; Chang, HaiJun <haijun.ch...@amd.com>; 
Zha, YiFan(Even) <yifan....@amd.com>
Subject: [PATCH] drm/amdgpu: Set no_hw_access when VF request full GPU fails

[Why]
If VF request full GPU access and the request failed,
the VF driver can get stuck accessing registers for an extended period during
the unload of KMS.

[How]
Set no_hw_access flag when VF request for full GPU access fails
This prevents further hardware access attempts, avoiding the prolonged
stuck state.

Signed-off-by: Yifan Zha <yifan....@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
index ccb3d041c2b2..111c380f929b 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
@@ -86,8 +86,10 @@ int amdgpu_virt_request_full_gpu(struct amdgpu_device *adev, 
bool init)

         if (virt->ops && virt->ops->req_full_gpu) {
                 r = virt->ops->req_full_gpu(adev, init);
-               if (r)
+               if (r) {
+                       adev->no_hw_access = true;
                         return r;
+               }

                 adev->virt.caps &= ~AMDGPU_SRIOV_CAPS_RUNTIME;
         }
--
2.25.1

Reply via email to