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:[email protected]] On Behalf
> Of Christian K?nig
> Sent: Monday, April 02, 2018 11:23 PM
> To: Alex Deucher <[email protected]>; StDenis, Tom
> <[email protected]>
> Cc: Deucher, Alexander <[email protected]>; Huang, Ray
> <[email protected]>; [email protected]
> 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 <[email protected]>
> wrote:
> >> This commit
> >>
> >> [root@raven linux]# git bisect bad
> >> c704d7a15d292e6e222b7e990ed7e9f41617cdd8 is the first bad commit
> >> commit c704d7a15d292e6e222b7e990ed7e9f41617cdd8
> >> Author: Alex Deucher <[email protected]>
> >> 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 <[email protected]>
> >>      Reviewed-by: Christian König <[email protected]>
> >>      Signed-off-by: Alex Deucher <[email protected]>
> >>
> >> :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
> >> [email protected]
> >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 
> _______________________________________________
> 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

Reply via email to