On Fri, Nov 27, 2020 at 04:09:07PM +0800, Su, Jinzhou (Joe) wrote: > If there are 2 gfx rings, the doorbell lower range of second ring > will override the first ring. > > Signed-off-by: Jinzhou.Su <Jinzhou.Su@xxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > index ffbda6680a68..f33e54b01d3a 100644 > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c > @@ -6357,8 +6357,11 @@ static int gfx_v10_0_gfx_mqd_init(struct amdgpu_ring *ring) > DOORBELL_EN, 0); > mqd->cp_rb_doorbell_control = tmp; > > - /* set doorbell range */ > - gfx_v10_0_cp_gfx_set_doorbell(adev, ring); > + /*if there are 2 gfx rings, set the lower doorbell range of the first ring, > + *otherwise the range of the second ring will override the first ring */ > + if (ring->doorbell_index == adev->doorbell_index.gfx_ring0 << 1) Maybe you can compare if ring->name is "gfx_0.0.0" here. Anyway, patch is Reviewed-by: Huang Rui <ray.huang@xxxxxxx> > + gfx_v10_0_cp_gfx_set_doorbell(adev, ring); > + > /* reset read and write pointers, similar to CP_RB0_WPTR/_RPTR */ > ring->wptr = 0; > mqd->cp_gfx_hqd_rptr = RREG32_SOC15(GC, 0, mmCP_GFX_HQD_RPTR); > -- > 2.17.1 > _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
