From: Xiaojie Yuan <[email protected]>

'adev->in_suspend' code path is missing in gfx_v10_0_gfx_init_queue()

Signed-off-by: Xiaojie Yuan <[email protected]>
Reviewed-by: Hawking Zhang <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 26 ++++++++++++++------------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
index 4e7f64d91d12..9d162d269aca 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
@@ -2903,7 +2903,19 @@ static int gfx_v10_0_gfx_init_queue(struct amdgpu_ring 
*ring)
        struct amdgpu_device *adev = ring->adev;
        struct v10_gfx_mqd *mqd = ring->mqd_ptr;
 
-       if (adev->in_gpu_reset) {
+       if (!adev->in_gpu_reset && !adev->in_suspend) {
+               memset((void *)mqd, 0, sizeof(*mqd));
+               mutex_lock(&adev->srbm_mutex);
+               nv_grbm_select(adev, ring->me, ring->pipe, ring->queue, 0);
+               gfx_v10_0_gfx_mqd_init(ring);
+#ifdef BRING_UP_DEBUG
+               gfx_v10_0_gfx_queue_init_register(ring);
+#endif
+               nv_grbm_select(adev, 0, 0, 0, 0);
+               mutex_unlock(&adev->srbm_mutex);
+               if (adev->gfx.me.mqd_backup[AMDGPU_MAX_GFX_RINGS])
+                       memcpy(adev->gfx.me.mqd_backup[AMDGPU_MAX_GFX_RINGS], 
mqd, sizeof(*mqd));
+       } else if (adev->in_gpu_reset) {
                /* reset mqd with the backup copy */
                if (adev->gfx.me.mqd_backup[AMDGPU_MAX_GFX_RINGS])
                        memcpy(mqd, 
adev->gfx.me.mqd_backup[AMDGPU_MAX_GFX_RINGS], sizeof(*mqd));
@@ -2918,17 +2930,7 @@ static int gfx_v10_0_gfx_init_queue(struct amdgpu_ring 
*ring)
                mutex_unlock(&adev->srbm_mutex);
 #endif
        } else {
-               memset((void *)mqd, 0, sizeof(*mqd));
-               mutex_lock(&adev->srbm_mutex);
-               nv_grbm_select(adev, ring->me, ring->pipe, ring->queue, 0);
-               gfx_v10_0_gfx_mqd_init(ring);
-#ifdef BRING_UP_DEBUG
-               gfx_v10_0_gfx_queue_init_register(ring);
-#endif
-               nv_grbm_select(adev, 0, 0, 0, 0);
-               mutex_unlock(&adev->srbm_mutex);
-               if (adev->gfx.me.mqd_backup[AMDGPU_MAX_GFX_RINGS])
-                       memcpy(adev->gfx.me.mqd_backup[AMDGPU_MAX_GFX_RINGS], 
mqd, sizeof(*mqd));
+               amdgpu_ring_clear_ring(ring);
        }
 
        return 0;
-- 
2.20.1

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to