At least of hand sounds like it makes sense. But one comment below.
Am 26.04.2018 um 04:06 schrieb Deng, Emily:
Ping...
-----Original Message-----
From: Emily Deng [mailto:[email protected]]
Sent: Wednesday, April 25, 2018 11:26 AM
To: [email protected]
Cc: Deng, Emily <[email protected]>
Subject: [PATCH] drm/amdgpu: For sriov reset, move IB test into exclusive
mode
When put the IB test out of exclusive mode, and do sriov reset, the IB test
will randomly fail. As out of exclusive mode it uses kiq to do read and write
registers, but as it has world switch, the kiq read and write time will be
random, sometimes it will beyond the MAX_KIQ_REG_WAIT and then the
read or write register will fail, which will result the IB test fail. So move
the
Looks like your commit message was cut of.
With that fixed the patch is Reviewed-by: Christian König
<[email protected]>.
Regards,
Christian.
Signed-off-by: Emily Deng <[email protected]>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 211f1af..dc921d6 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -3156,20 +3156,19 @@ static int amdgpu_device_reset_sriov(struct
amdgpu_device *adev,
/* now we are okay to resume SMC/CP/SDMA */
r = amdgpu_device_ip_reinit_late_sriov(adev);
- amdgpu_virt_release_full_gpu(adev, true);
if (r)
goto error;
amdgpu_irq_gpu_reset_resume_helper(adev);
r = amdgpu_ib_ring_tests(adev);
+error:
+ amdgpu_virt_release_full_gpu(adev, true);
if (!r && adev->virt.gim_feature &
AMDGIM_FEATURE_GIM_FLR_VRAMLOST) {
atomic_inc(&adev->vram_lost_counter);
r = amdgpu_device_handle_vram_lost(adev);
}
-error:
-
return r;
}
--
2.7.4
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
