]> git.hungrycats.org Git - linux/commitdiff
drm/amdgpu/mes11: move pipe reset to mes use_mmio patch
authorAlex Deucher <alexander.deucher@amd.com>
Wed, 3 Jun 2026 08:28:43 +0000 (16:28 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 17 Jun 2026 19:51:34 +0000 (15:51 -0400)
This makes the code flows cleaner and it's only supported on
the use_mmio path.

v2: fix typo
v3: fix typo
v4: directly clear ACTIVE and DEQUEUE_REQUEST (Shaoyun Liu)

Reviewed-by: Jesse Zhang <jesse.zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
drivers/gpu/drm/amd/amdgpu/mes_v11_0.c

index c1efb778ebcbc3575736862207815730a3e90a6a..9fcb2781468b16efbfd908d242a32240105d307f 100644 (file)
@@ -6814,69 +6814,6 @@ static void gfx_v11_0_emit_mem_sync(struct amdgpu_ring *ring)
        amdgpu_ring_write(ring, gcr_cntl); /* GCR_CNTL */
 }
 
-static bool gfx_v11_pipe_reset_support(struct amdgpu_device *adev)
-{
-       /* Disable the pipe reset until the CPFW fully support it.*/
-       dev_warn_once(adev->dev, "The CPFW hasn't support pipe reset yet.\n");
-       return false;
-}
-
-
-static int gfx_v11_reset_gfx_pipe(struct amdgpu_ring *ring)
-{
-       struct amdgpu_device *adev = ring->adev;
-       uint32_t reset_pipe = 0, clean_pipe = 0;
-       int r;
-
-       if (!gfx_v11_pipe_reset_support(adev))
-               return -EOPNOTSUPP;
-
-       gfx_v11_0_set_safe_mode(adev, 0);
-       mutex_lock(&adev->srbm_mutex);
-       soc21_grbm_select(adev, ring->me, ring->pipe, ring->queue, 0);
-
-       switch (ring->pipe) {
-       case 0:
-               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
-                                          PFP_PIPE0_RESET, 1);
-               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
-                                          ME_PIPE0_RESET, 1);
-               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
-                                          PFP_PIPE0_RESET, 0);
-               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
-                                          ME_PIPE0_RESET, 0);
-               break;
-       case 1:
-               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
-                                          PFP_PIPE1_RESET, 1);
-               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
-                                          ME_PIPE1_RESET, 1);
-               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
-                                          PFP_PIPE1_RESET, 0);
-               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
-                                          ME_PIPE1_RESET, 0);
-               break;
-       default:
-               break;
-       }
-
-       WREG32_SOC15(GC, 0, regCP_ME_CNTL, reset_pipe);
-       WREG32_SOC15(GC, 0, regCP_ME_CNTL, clean_pipe);
-
-       r = (RREG32(SOC15_REG_OFFSET(GC, 0, regCP_GFX_RS64_INSTR_PNTR1)) << 2) -
-                                               RS64_FW_UC_START_ADDR_LO;
-       soc21_grbm_select(adev, 0, 0, 0, 0);
-       mutex_unlock(&adev->srbm_mutex);
-       gfx_v11_0_unset_safe_mode(adev, 0);
-
-       dev_info(adev->dev, "The ring %s pipe reset to the ME firmware start PC: %s\n", ring->name,
-                       r == 0 ? "successfully" : "failed");
-       /* FIXME: Sometimes driver can't cache the ME firmware start PC correctly,
-        * so the pipe reset status relies on the later gfx ring test result.
-        */
-       return 0;
-}
-
 static int gfx_v11_0_reset_kgq(struct amdgpu_ring *ring,
                               unsigned int vmid,
                               struct amdgpu_fence *timedout_fence)
@@ -6888,13 +6825,8 @@ static int gfx_v11_0_reset_kgq(struct amdgpu_ring *ring,
        amdgpu_ring_reset_helper_begin(ring, timedout_fence);
 
        r = amdgpu_mes_reset_legacy_queue(ring->adev, ring, vmid, use_mmio, 0);
-       if (r) {
-
-               dev_warn(adev->dev, "reset via MES failed and try pipe reset %d\n", r);
-               r = gfx_v11_reset_gfx_pipe(ring);
-               if (r)
-                       return r;
-       }
+       if (r)
+               return r;
 
        if (use_mmio) {
                r = gfx_v11_0_kgq_init_queue(ring, true);
@@ -6913,158 +6845,6 @@ static int gfx_v11_0_reset_kgq(struct amdgpu_ring *ring,
        return amdgpu_ring_reset_helper_end(ring, timedout_fence);
 }
 
-/*
- * With MEC pipe reset asserted, clear CP_HQD_ACTIVE / CP_HQD_DEQUEUE_REQUEST for
- * every queue on (me, pipe). HQDs must be torn down while pipe reset stays
- * asserted; only then clear the pipe reset bit.
- * Caller must hold adev->srbm_mutex.
- */
-static void gfx_v11_0_clear_hqds_on_mec_pipe(struct amdgpu_device *adev, u32 me,
-                                            u32 pipe)
-{
-       unsigned int q;
-
-       for (q = 0; q < adev->gfx.mec.num_queue_per_pipe; q++) {
-               soc21_grbm_select(adev, me, pipe, q, 0);
-               /* Start from a clean HQD dequeue state before forcing HQD inactive. */
-               WREG32_SOC15(GC, 0, regCP_HQD_ACTIVE, 0);
-               WREG32_SOC15(GC, 0, regCP_HQD_DEQUEUE_REQUEST, 0);
-       }
-}
-
-static int gfx_v11_0_reset_compute_pipe(struct amdgpu_device *adev,
-                                          u32 me, u32 pipe, u32 queue)
-{
-       uint32_t reset_val, clean_val;
-       int r;
-
-       if (!gfx_v11_pipe_reset_support(adev))
-               return -EOPNOTSUPP;
-
-       gfx_v11_0_set_safe_mode(adev, 0);
-       mutex_lock(&adev->srbm_mutex);
-       soc21_grbm_select(adev, me, pipe, queue, 0);
-
-       if (adev->gfx.rs64_enable) {
-               reset_val = RREG32_SOC15(GC, 0, regCP_MEC_RS64_CNTL);
-               clean_val = reset_val;
-
-               switch (pipe) {
-               case 0:
-                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE0_RESET, 1);
-                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE0_RESET, 0);
-                       break;
-               case 1:
-                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE1_RESET, 1);
-                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE1_RESET, 0);
-                       break;
-               case 2:
-                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE2_RESET, 1);
-                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE2_RESET, 0);
-                       break;
-               case 3:
-                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE3_RESET, 1);
-                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
-                                                 MEC_PIPE3_RESET, 0);
-                       break;
-               default:
-                       break;
-               }
-               WREG32_SOC15(GC, 0, regCP_MEC_RS64_CNTL, reset_val);
-               gfx_v11_0_clear_hqds_on_mec_pipe(adev, me, pipe);
-               WREG32_SOC15(GC, 0, regCP_MEC_RS64_CNTL, clean_val);
-               r = (RREG32_SOC15(GC, 0, regCP_MEC_RS64_INSTR_PNTR) << 2) -
-                                       RS64_FW_UC_START_ADDR_LO;
-       } else {
-               reset_val = RREG32_SOC15(GC, 0, regCP_MEC_CNTL);
-               clean_val = reset_val;
-
-               if (me == 1) {
-                       switch (pipe) {
-                       case 0:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE0_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE0_RESET, 0);
-                               break;
-                       case 1:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE1_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE1_RESET, 0);
-                               break;
-                       case 2:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE2_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE2_RESET, 0);
-                               break;
-                       case 3:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE3_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME1_PIPE3_RESET, 0);
-                               break;
-                       default:
-                               break;
-                       }
-                       /* mec1 fw pc: CP_MEC1_INSTR_PNTR */
-               } else {
-                       switch (pipe) {
-                       case 0:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE0_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE0_RESET, 0);
-                               break;
-                       case 1:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE1_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE1_RESET, 0);
-                               break;
-                       case 2:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE2_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE2_RESET, 0);
-                               break;
-                       case 3:
-                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE3_RESET, 1);
-                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
-                                                         MEC_ME2_PIPE3_RESET, 0);
-                               break;
-                       default:
-                               break;
-                       }
-                       /* mec2 fw pc: CP:CP_MEC2_INSTR_PNTR */
-               }
-               WREG32_SOC15(GC, 0, regCP_MEC_CNTL, reset_val);
-               gfx_v11_0_clear_hqds_on_mec_pipe(adev, me, pipe);
-               WREG32_SOC15(GC, 0, regCP_MEC_CNTL, clean_val);
-               r = RREG32(SOC15_REG_OFFSET(GC, 0, regCP_MEC1_INSTR_PNTR));
-       }
-
-       soc21_grbm_select(adev, 0, 0, 0, 0);
-       mutex_unlock(&adev->srbm_mutex);
-       gfx_v11_0_unset_safe_mode(adev, 0);
-
-       dev_dbg(adev->dev, "MEC pipe me%u pipe%u queue%u resets to MEC FW start PC: %s\n",
-               me, pipe, queue, r == 0 ? "successfully" : "failed");
-       /*FIXME:Sometimes driver can't cache the MEC firmware start PC correctly, so the pipe
-        * reset status relies on the compute ring test result.
-        */
-       return 0;
-}
-
 static int gfx_v11_0_reset_kcq(struct amdgpu_ring *ring,
                               unsigned int vmid,
                               struct amdgpu_fence *timedout_fence)
@@ -7076,12 +6856,8 @@ static int gfx_v11_0_reset_kcq(struct amdgpu_ring *ring,
        amdgpu_ring_reset_helper_begin(ring, timedout_fence);
 
        r = amdgpu_mes_reset_legacy_queue(ring->adev, ring, vmid, use_mmio, 0);
-       if (r) {
-               dev_warn(adev->dev, "fail(%d) to reset kcq and try pipe reset\n", r);
-               r = gfx_v11_0_reset_compute_pipe(adev, ring->me, ring->pipe, ring->queue);
-               if (r)
-                       return r;
-       }
+       if (r)
+               return r;
 
        if (use_mmio) {
                r = gfx_v11_0_kcq_init_queue(ring, true);
index ac6d4f2773363c2320985110b7fddf6cc52b11c8..820ee7a1d0b6c63f3f94aa0d4ee992cabb721c05 100644 (file)
@@ -392,6 +392,233 @@ static int mes_v11_0_remove_hw_queue(struct amdgpu_mes *mes,
                        offsetof(union MESAPI__REMOVE_QUEUE, api_status));
 }
 
+static bool mes_v11_0_pipe_reset_support(struct amdgpu_device *adev)
+{
+       /* Disable the pipe reset until the CPFW fully support it.*/
+       dev_warn_once(adev->dev, "The CPFW hasn't support pipe reset yet.\n");
+       return false;
+}
+static int mes_v11_0_reset_gfx_pipe_mmio(struct amdgpu_device *adev,
+                                        u32 me, u32 pipe, u32 queue)
+{
+       uint32_t reset_pipe = 0, clean_pipe = 0;
+       int r;
+
+       if (!mes_v11_0_pipe_reset_support(adev))
+               return -EOPNOTSUPP;
+
+       amdgpu_gfx_rlc_enter_safe_mode(adev, 0);
+       mutex_lock(&adev->srbm_mutex);
+       soc21_grbm_select(adev, me, pipe, queue, 0);
+
+       switch (pipe) {
+       case 0:
+               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
+                                          PFP_PIPE0_RESET, 1);
+               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
+                                          ME_PIPE0_RESET, 1);
+               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
+                                          PFP_PIPE0_RESET, 0);
+               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
+                                          ME_PIPE0_RESET, 0);
+               break;
+       case 1:
+               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
+                                          PFP_PIPE1_RESET, 1);
+               reset_pipe = REG_SET_FIELD(reset_pipe, CP_ME_CNTL,
+                                          ME_PIPE1_RESET, 1);
+               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
+                                          PFP_PIPE1_RESET, 0);
+               clean_pipe = REG_SET_FIELD(clean_pipe, CP_ME_CNTL,
+                                          ME_PIPE1_RESET, 0);
+               break;
+       default:
+               break;
+       }
+
+       WREG32_SOC15(GC, 0, regCP_ME_CNTL, reset_pipe);
+       WREG32_SOC15(GC, 0, regCP_ME_CNTL, clean_pipe);
+
+       r = (RREG32(SOC15_REG_OFFSET(GC, 0, regCP_GFX_RS64_INSTR_PNTR1)) << 2) -
+                                               RS64_FW_UC_START_ADDR_LO;
+       soc21_grbm_select(adev, 0, 0, 0, 0);
+       mutex_unlock(&adev->srbm_mutex);
+       amdgpu_gfx_rlc_exit_safe_mode(adev, 0);
+
+       dev_info(adev->dev, "The gfx pipe reset to the ME firmware start PC: %s\n",
+                       r == 0 ? "successfully" : "failed");
+       /* FIXME: Sometimes driver can't cache the ME firmware start PC correctly,
+        * so the pipe reset status relies on the later gfx ring test result.
+        */
+       return 0;
+}
+
+/*
+ * With MEC pipe reset asserted, clear CP_HQD_ACTIVE / CP_HQD_DEQUEUE_REQUEST for
+ * every queue on (me, pipe). HQDs must be torn down while pipe reset stays
+ * asserted; only then clear the pipe reset bit.
+ * Caller must hold adev->srbm_mutex.
+ */
+static void mes_v11_0_clear_hqds_on_mec_pipe(struct amdgpu_device *adev, u32 me,
+                                            u32 pipe)
+{
+       unsigned int q;
+
+       for (q = 0; q < adev->gfx.mec.num_queue_per_pipe; q++) {
+               soc21_grbm_select(adev, me, pipe, q, 0);
+               /* Start from a clean HQD dequeue state before forcing HQD inactive. */
+               WREG32_SOC15(GC, 0, regCP_HQD_ACTIVE, 0);
+               WREG32_SOC15(GC, 0, regCP_HQD_DEQUEUE_REQUEST, 0);
+       }
+}
+
+static int mes_v11_0_reset_compute_pipe_mmio(struct amdgpu_device *adev,
+                                            u32 me, u32 pipe, u32 queue)
+{
+       uint32_t reset_val, clean_val;
+       int r;
+
+       if (!mes_v11_0_pipe_reset_support(adev))
+               return -EOPNOTSUPP;
+
+       amdgpu_gfx_rlc_enter_safe_mode(adev, 0);
+       mutex_lock(&adev->srbm_mutex);
+       soc21_grbm_select(adev, me, pipe, queue, 0);
+
+       if (adev->gfx.rs64_enable) {
+               reset_val = RREG32_SOC15(GC, 0, regCP_MEC_RS64_CNTL);
+               clean_val = reset_val;
+
+               switch (pipe) {
+               case 0:
+                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE0_RESET, 1);
+                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE0_RESET, 0);
+                       break;
+               case 1:
+                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE1_RESET, 1);
+                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE1_RESET, 0);
+                       break;
+               case 2:
+                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE2_RESET, 1);
+                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE2_RESET, 0);
+                       break;
+               case 3:
+                       reset_val = REG_SET_FIELD(reset_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE3_RESET, 1);
+                       clean_val = REG_SET_FIELD(clean_val, CP_MEC_RS64_CNTL,
+                                                 MEC_PIPE3_RESET, 0);
+                       break;
+               default:
+                       break;
+               }
+               WREG32_SOC15(GC, 0, regCP_MEC_RS64_CNTL, reset_val);
+               mes_v11_0_clear_hqds_on_mec_pipe(adev, me, pipe);
+               WREG32_SOC15(GC, 0, regCP_MEC_RS64_CNTL, clean_val);
+               r = (RREG32_SOC15(GC, 0, regCP_MEC_RS64_INSTR_PNTR) << 2) -
+                                       RS64_FW_UC_START_ADDR_LO;
+       } else {
+               reset_val = RREG32_SOC15(GC, 0, regCP_MEC_CNTL);
+               clean_val = reset_val;
+
+               if (me == 1) {
+                       switch (pipe) {
+                       case 0:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE0_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE0_RESET, 0);
+                               break;
+                       case 1:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE1_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE1_RESET, 0);
+                               break;
+                       case 2:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE2_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE2_RESET, 0);
+                               break;
+                       case 3:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE3_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME1_PIPE3_RESET, 0);
+                               break;
+                       default:
+                               break;
+                       }
+                       /* mec1 fw pc: CP_MEC1_INSTR_PNTR */
+               } else {
+                       switch (pipe) {
+                       case 0:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE0_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE0_RESET, 0);
+                               break;
+                       case 1:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE1_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE1_RESET, 0);
+                               break;
+                       case 2:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE2_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE2_RESET, 0);
+                               break;
+                       case 3:
+                               reset_val = REG_SET_FIELD(reset_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE3_RESET, 1);
+                               clean_val = REG_SET_FIELD(clean_val, CP_MEC_CNTL,
+                                                         MEC_ME2_PIPE3_RESET, 0);
+                               break;
+                       default:
+                               break;
+                       }
+                       /* mec2 fw pc: CP:CP_MEC2_INSTR_PNTR */
+               }
+               WREG32_SOC15(GC, 0, regCP_MEC_CNTL, reset_val);
+               mes_v11_0_clear_hqds_on_mec_pipe(adev, me, pipe);
+               WREG32_SOC15(GC, 0, regCP_MEC_CNTL, clean_val);
+               r = RREG32(SOC15_REG_OFFSET(GC, 0, regCP_MEC1_INSTR_PNTR));
+       }
+
+       soc21_grbm_select(adev, 0, 0, 0, 0);
+       mutex_unlock(&adev->srbm_mutex);
+       amdgpu_gfx_rlc_exit_safe_mode(adev, 0);
+
+       dev_dbg(adev->dev, "MEC pipe me%u pipe%u queue%u resets to MEC FW start PC: %s\n",
+               me, pipe, queue, r == 0 ? "successfully" : "failed");
+       /*FIXME:Sometimes driver can't cache the MEC firmware start PC correctly, so the pipe
+        * reset status relies on the compute ring test result.
+        */
+       return 0;
+}
+
+static int mes_v11_0_reset_pipe_mmio(struct amdgpu_mes *mes, uint32_t queue_type,
+                                    uint32_t me_id, uint32_t pipe_id,
+                                    uint32_t queue_id, uint32_t vmid)
+{
+       struct amdgpu_device *adev = mes->adev;
+
+       if (queue_type == AMDGPU_RING_TYPE_GFX)
+               return mes_v11_0_reset_gfx_pipe_mmio(adev, me_id, pipe_id, queue_id);
+       else if (queue_type == AMDGPU_RING_TYPE_COMPUTE)
+               return mes_v11_0_reset_compute_pipe_mmio(adev, me_id, pipe_id, queue_id);
+       else
+               return -EOPNOTSUPP;
+}
+
 static int mes_v11_0_reset_queue_mmio(struct amdgpu_mes *mes, uint32_t queue_type,
                                      uint32_t me_id, uint32_t pipe_id,
                                      uint32_t queue_id, uint32_t vmid)
@@ -764,10 +991,16 @@ static int mes_v11_0_reset_hw_queue(struct amdgpu_mes *mes,
 {
        union MESAPI__RESET mes_reset_queue_pkt;
 
-       if (input->use_mmio)
-               return mes_v11_0_reset_queue_mmio(mes, input->queue_type,
-                                                 input->me_id, input->pipe_id,
-                                                 input->queue_id, input->vmid);
+       if (input->use_mmio) {
+               int r = mes_v11_0_reset_queue_mmio(mes, input->queue_type,
+                                                  input->me_id, input->pipe_id,
+                                                  input->queue_id, input->vmid);
+               if (r)
+                       return mes_v11_0_reset_pipe_mmio(mes, input->queue_type,
+                                                        input->me_id, input->pipe_id,
+                                                        input->queue_id, input->vmid);
+               return 0;
+       }
 
        memset(&mes_reset_queue_pkt, 0, sizeof(mes_reset_queue_pkt));