No, just bare metal, I assumed your problem was with compute ring test failure which I didn't see. Can you please recheck if reverting this still failing on SRIOV ? If so we obviously need to keep looking how to fix it.
Thanks, Andrey ________________________________________ From: Yu, Xiangliang Sent: 25 January 2018 20:59:45 To: Grodzovsky, Andrey; [email protected] Cc: Deucher, Alexander; Grodzovsky, Andrey; Koenig, Christian Subject: RE: [PATCH] Revert "drm/amdgpu/gfx8: Fix compute ring failure after resetting" Did you test reset case in sriov? > -----Original Message----- > From: amd-gfx [mailto:[email protected]] On Behalf > Of Andrey Grodzovsky > Sent: Friday, January 26, 2018 7:07 AM > To: [email protected] > Cc: Deucher, Alexander <[email protected]>; Grodzovsky, > Andrey <[email protected]>; Yu, Xiangliang > <[email protected]>; Koenig, Christian <[email protected]> > Subject: [PATCH] Revert "drm/amdgpu/gfx8: Fix compute ring failure after > resetting" > > This reverts commit 75737cb4eb78c7f185e4700b4aa20cf7a3381aca. > > Fixes GFX ring test failure after HW reset. > No compute ring test failures were observed with the change reverted. > So seems like whatever problem that change was addressing is not present > anymore. > > Signed-off-by: Andrey Grodzovsky <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > index 1207f36..8a65b53 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c > @@ -4847,6 +4847,9 @@ static int gfx_v8_0_kcq_init_queue(struct > amdgpu_ring *ring) > /* reset MQD to a clean status */ > if (adev->gfx.mec.mqd_backup[mqd_idx]) > memcpy(mqd, adev- > >gfx.mec.mqd_backup[mqd_idx], sizeof(struct vi_mqd_allocation)); > + /* reset ring buffer */ > + ring->wptr = 0; > + amdgpu_ring_clear_ring(ring); > } else { > amdgpu_ring_clear_ring(ring); > } > @@ -4921,13 +4924,6 @@ static int gfx_v8_0_kiq_resume(struct > amdgpu_device *adev) > /* Test KCQs */ > for (i = 0; i < adev->gfx.num_compute_rings; i++) { > ring = &adev->gfx.compute_ring[i]; > - if (adev->in_gpu_reset) { > - /* move reset ring buffer to here to workaround > - * compute ring test failed > - */ > - ring->wptr = 0; > - amdgpu_ring_clear_ring(ring); > - } > ring->ready = true; > r = amdgpu_ring_test_ring(ring); > if (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
