drm/amdgpu:need som change on vega10 mailbox

if sriov gpu reset is invoked by job timeout, it is run
in a global work-queue which is very slow and better not call
msleep ortherwise it takes long time to get back CPU.

so make below changes:

1: Change msleep 1 to mdelay 5
2: Ignore the ack fail from pf after time out,
   because VF FLR will clear ack, sometime VF FLR is done
   prior to the beginning of poll_ack so we can ignore this ack

TODO:
Put job_timedout (and the following gpu reset) in a driver thread,
instead of the global work_struct.

Signed-off-by: Monk Liu <Monk.Liu@amd.com>
Reviewed-by: Xiangliang Yu <Xiangliang.Yu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

1 files changed, 5 insertions, 5 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c b/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c
index 7bdc51b02326..f0d64f13abbc 100644
--- a/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c
+++ b/drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c
@@ -398,8 +398,8 @@ static int xgpu_vi_poll_ack(struct amdgpu_device *adev)
                         r = -ETIME;
                         break;
                 }
-                msleep(1);
-                timeout -= 1;
+                mdelay(5);
+                timeout -= 5;
 
                 reg = RREG32_NO_KIQ(mmMAILBOX_CONTROL);
         }
@@ -418,8 +418,8 @@ static int xgpu_vi_poll_msg(struct amdgpu_device *adev, enum idh_event event)
                         r = -ETIME;
                         break;
                 }
-                msleep(1);
-                timeout -= 1;
+                mdelay(5);
+                timeout -= 5;
 
                 r = xgpu_vi_mailbox_rcv_msg(adev, event);
         }
@@ -447,7 +447,7 @@ static int xgpu_vi_send_access_requests(struct amdgpu_device *adev,
                 request == IDH_REQ_GPU_RESET_ACCESS) {
                 r = xgpu_vi_poll_msg(adev, IDH_READY_TO_ACCESS_GPU);
                 if (r)
-                        return r;
+                        pr_err("Doesn't get ack from pf, continue\n");
         }
 
         return 0;