regression on raven (gmc9)

[Deng, Emily]

Hi Alex and Christian,
    Those patches are important for vega10 SRIOV, without this, vega10 run many games will hang on SRIOV, 
how about to add  "if (amdgpu_sriov_vf(adev) && adev->asic_type == CHIP_VEGA10)" to separate the 
bare metal and SRIOV?

Best Wishes,
Emily Deng




> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Christian K?nig
> Sent: Monday, April 02, 2018 11:23 PM
> To: Alex Deucher <alexdeucher at gmail.com>; StDenis, Tom
> <Tom.StDenis at amd.com>
> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>; Huang, Ray
> <Ray.Huang at amd.com>; amd-gfx at lists.freedesktop.org
> Subject: Re: regression on raven (gmc9)
> 
> Am 02.04.2018 um 17:00 schrieb Alex Deucher:
> > On Mon, Apr 2, 2018 at 7:17 AM, Tom St Denis <tstdenis at amd.com>
> wrote:
> >> This commit
> >>
> >> [root at raven linux]# git bisect bad
> >> c704d7a15d292e6e222b7e990ed7e9f41617cdd8 is the first bad commit
> >> commit c704d7a15d292e6e222b7e990ed7e9f41617cdd8
> >> Author: Alex Deucher <alexander.deucher at amd.com>
> >> Date:   Tue Mar 27 17:10:56 2018 -0500
> >>
> >>      drm/amdgpu/gmc9: use amdgpu_ring_emit_reg_write_reg_wait in gpu
> >> tlb flush
> >>
> >>      Use amdgpu_ring_emit_reg_write_reg_wait.  On engines that support
> it,
> >>      it provides a write and wait in a single packet which avoids a missed
> >>      ack if a world switch happens between the request and waiting for the
> >>      ack.
> >>
> >>      Reviewed-by: Huang Rui <ray.huang at amd.com>
> >>      Reviewed-by: Christian König <christian.koenig at amd.com>
> >>      Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> >>
> >> :040000 040000 a08a12a316d09ea231337136cc7aa076d4da44a8
> >> 521c8a18b0c63e185e416a267619a92822d7f268 M      drivers
> >>
> >> Causes my raven1 to lockup on starting X with a gfx ring stall.
> > I've reverted the patches.
> 
> Yeah, you need a specialized firmware version for this which Raven probably
> doesn't have.
> 
> Anyway the root cause of the issue which should be fixed by this patch set
> was found in the meantime, so the whole patch set shouldn't be necessary
> any more.
> 
> Christian.
> 
> >
> > Alex
> >
> >> Tom
> >> _______________________________________________
> >> amd-gfx mailing list
> >> amd-gfx at lists.freedesktop.org
> >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx