<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-2022-jp"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr"> Please add a comment above the CP_PQ_WPTR_POLL_CNTL write to explain that it's to disable the polling. WIth that fixed: Acked-by: Alex Deucher <alexander.deucher@amd.com> </div> <hr style="display:inline-block;width:98%" tabindex="-1"> <div id="divRplyFwdMsg" dir="ltr">From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Liu, Monk <Monk.Liu@amd.com> Sent: Tuesday, February 27, 2018 12:31:35 AM To: Alex Deucher Cc: amd-gfx list Subject: RE: [PATCH 17/22] drm/amdgpu: disable GFX ring and disable PQ wptr in hw_fini <div> </div> </div> <div class="BodyFragment"> <div class="PlainText">This is only for virtualization, for bare-metal it go another path -----Original Message----- From: Alex Deucher [<a href="mailto:alexdeucher@gmail.com">mailto:alexdeucher@gmail.com</a>] Sent: 2018年2月27日 1:40 To: Liu, Monk <Monk.Liu@amd.com> Cc: amd-gfx list <amd-gfx@lists.freedesktop.org> Subject: Re: [PATCH 17/22] drm/amdgpu: disable GFX ring and disable PQ wptr in hw_fini On Mon, Feb 26, 2018 at 12:18 AM, Monk Liu <Monk.Liu@amd.com> wrote: > otherwise there will be DMAR reading error comes out from CP since GFX > is still alive and CPC's WPTR_POLL is still enabled, which would lead > to DMAR read error. > > fix: > we can hault CPG after hw_fini, but cannot halt CPC becaues KIQ stil > need to be alive to let RLCV invoke, but its WPTR_POLL could be > disabled. > > Change-Id: Ia60ee54901531f737d09063bf2037630e7c94771 > Signed-off-by: Monk Liu <Monk.Liu@amd.com> Is this handled properly for bare metal as well? Alex > --- > drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > index e9cc03e..793db9f 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > @@ -2961,7 +2961,8 @@ static int gfx_v9_0_hw_fini(void *handle) > gfx_v9_0_kcq_disable(&adev->gfx.kiq.ring, > &adev->gfx.compute_ring[i]); > > if (amdgpu_sriov_vf(adev)) { > - pr_debug("For SRIOV client, shouldn't do anything.\n"); > + gfx_v9_0_cp_gfx_enable(adev, false); > + WREG32_FIELD15(GC, 0, CP_PQ_WPTR_POLL_CNTL, EN, 0); > return 0; > } > gfx_v9_0_cp_enable(adev, false); > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </div> </div> </body> </html>