Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1611345

RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS

Show all headers | View raw

> -----Original Message-----
> From: Joerg Roedel [mailto:jroedel@suse.de]
> Sent: Tuesday, March 28, 2017 4:29 PM
> To: Deucher, Alexander
> Cc: 'Joerg Roedel'; Bjorn Helgaas; linux-pci@vger.kernel.org; linux-
> kernel@vger.kernel.org; Daniel Drake; Nath, Arindam
> Subject: Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS
> 
> On Tue, Mar 28, 2017 at 08:18:26PM +0000, Deucher, Alexander wrote:
> > > -----Original Message-----
> > > From: Joerg Roedel [mailto:joro@8bytes.org]
> > > Sent: Tuesday, March 28, 2017 8:17 AM
> > > To: Bjorn Helgaas
> > > Cc: linux-pci@vger.kernel.org; linux-kernel@vger.kernel.org; Joerg
> Roedel;
> > > Daniel Drake; Deucher, Alexander
> > > Subject: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS
> > >
> > > From: Joerg Roedel <jroedel@suse.de>
> > >
> > > ATS is broken on these devices. Under invalidation load, the
> > > GPU does not reply to invalidations anymore, causing
> > > Completion-wait loop timeouts on the AMD IOMMU driver side.
> > > Fix it by not enabling ATS on these devices.
> > >
> > > Note that below mentioned commit is not broken, it just
> > > triggers the issue because it might cause invalidation
> > > storms on devices.
> > >
> > > Fixes: b1516a14657a ('iommu/amd: Implement flush queue')
> > > Reported-by: Daniel Drake <drake@endlessm.com>
> > > Cc: Daniel Drake <drake@endlessm.com>
> > > Cc: Alexander Deucher <Alexander.Deucher@amd.com>
> > > Signed-off-by: Joerg Roedel <jroedel@suse.de>
> >
> > Did you see Arindam's patch from yesterday[1]?  Not sure which is the
> proper fix, maybe both?
> 
> Arindam's patch makes sense on its own, but not as a fix for this issue.
> It lowers the invalidation load on the GPU, but there are still ways to
> trigger a high invalidation rate on the device. So it might hide the
> issue, but not fix it.
> 
> We need to disable ATS on the device if it doesn't work reliably.

The question is, could the problem stem from flushing an entity that didn't request it, or should that not matter?  I guess it shouldn't matter otherwise we'd see this on other platforms like Carrizo as well.

Alex

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS Joerg Roedel <joro@8bytes.org> - 2017-03-28 14:20 +0200
  RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS "Deucher, Alexander" <Alexander.Deucher@amd.com> - 2017-03-28 22:20 +0200
    Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS Joerg Roedel <jroedel@suse.de> - 2017-03-28 22:30 +0200
      RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS "Deucher, Alexander" <Alexander.Deucher@amd.com> - 2017-03-28 22:40 +0200
        Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS 'Joerg Roedel' <jroedel@suse.de> - 2017-03-28 23:00 +0200
          RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS "Deucher, Alexander" <Alexander.Deucher@amd.com> - 2017-03-28 23:20 +0200
            Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS 'Joerg Roedel' <jroedel@suse.de> - 2017-03-29 00:30 +0200
              RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS "Nath, Arindam" <Arindam.Nath@amd.com> - 2017-03-29 09:20 +0200
                Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS 'Joerg Roedel' <jroedel@suse.de> - 2017-03-29 11:50 +0200
                RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS "Nath, Arindam" <Arindam.Nath@amd.com> - 2017-03-29 11:50 +0200
              RE: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS "Deucher, Alexander" <Alexander.Deucher@amd.com> - 2017-03-29 18:30 +0200
  Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS Bjorn Helgaas <helgaas@kernel.org> - 2017-04-04 18:50 +0200
    Re: [PATCH] PCI: Blacklist AMD Stoney GPU devices for ATS Joerg Roedel <jroedel@suse.de> - 2017-04-07 14:40 +0200

csiph-web