On Tue, Jul 6, 2021 at 7:16 AM Sasha Levin <sashal@xxxxxxxxxx> wrote: > > From: Yifan Zhang <yifan1.zhang@xxxxxxx> > > [ Upstream commit 631003101c516ea29a74aee59666708857b9a805 ] > > If GC has entered CGPG, ringing doorbell > first page doesn't wakeup GC. > Enlarge CP_MEC_DOORBELL_RANGE_UPPER to workaround this issue. > > Signed-off-by: Yifan Zhang <yifan1.zhang@xxxxxxx> > Reviewed-by: Felix Kuehling <Felix.Kuehling@xxxxxxx> > Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx> This should be dropped. It was already reverted. Alex > --- > drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 6 +++++- > 1 file changed, 5 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 516467e962b7..c09225d065c2 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c > @@ -3673,8 +3673,12 @@ static int gfx_v9_0_kiq_init_register(struct amdgpu_ring *ring) > if (ring->use_doorbell) { > WREG32_SOC15(GC, 0, mmCP_MEC_DOORBELL_RANGE_LOWER, > (adev->doorbell_index.kiq * 2) << 2); > + /* If GC has entered CGPG, ringing doorbell > first page doesn't > + * wakeup GC. Enlarge CP_MEC_DOORBELL_RANGE_UPPER to workaround > + * this issue. > + */ > WREG32_SOC15(GC, 0, mmCP_MEC_DOORBELL_RANGE_UPPER, > - (adev->doorbell_index.userqueue_end * 2) << 2); > + (adev->doorbell.size - 4)); > } > > WREG32_SOC15_RLC(GC, 0, mmCP_HQD_PQ_DOORBELL_CONTROL, > -- > 2.30.2 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx