Re: [PATCH v2] PCI: Add ATS-disable quirk for AMD Stoney GPUs

[Samuel Sieb]

On 06/15/2017 07:04 AM, Joerg Roedel wrote:
> Hi Bjorn,
>
> On Fri, Apr 07, 2017 at 04:32:18PM +0200, Joerg Roedel wrote:
> > From: Joerg Roedel <jroedel@xxxxxxx>
> >
> > ATS is broken on this hardware and causes IOMMU stalls and
> > system failure. Disable ATS on these devices to make them
> > usable again with IOMMU enabled.
> >
> > Note that the commit in the Fixes-tag is not buggy, it
> > just uncovers the problem in the hardware by increasing
> > the ATS-flush rate.
> >
> > Fixes: b1516a14657a ('iommu/amd: Implement flush queue')
> > Signed-off-by: Joerg Roedel <jroedel@xxxxxxx>
> > ---
> >   drivers/pci/quirks.c | 19 +++++++++++++++++++
> >   1 file changed, 19 insertions(+)
>
> Any more objections on this patch? Please let me know if you want to
> have something changed.

The other patch seems to fix this issue without disabling ATS.  Isn't 
that better?