Re: [patch] PCI: disable MSI on more ATI NorthBridges

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: David Miller
Date: Thursday, October 18, 2007 - 4:46 am

From: "Shane Huang" <Shane.Huang@amd.com>
Date: Thu, 18 Oct 2007 18:37:59 +0800


This logic seems backwards, to me.  "shoot first, ask questions later"
To me this it not how to approach this problem.

Once you turn MSI off, there is next to no incentive to fix the
problem because users aren't running into it any longer.

The only two devices in that bug report which should be using MSI
would be the SATA controller and the broadcom ethernet NIC.  And by
the failed bootup logs provided by the user the problem is clearly
with the SATA controller.

One common problem we're finding is that some devices have a hardware
bug where setting INTX_DISABLE in the PCI COMMAND register masks MSI
interrupts too.

I mention this because the user in that report mentions that the
kernel upgrade causes the failure, and one thing we started doing not
too long ago was to set the INTX_DISABLE bit when MSI is enabled for a
device.

So maybe this SATA controller has this problem too.  It is easy to
test, simply comment out all of the pci_intx() function calls in
drivers/pci/msi.c and perform a test boot with MSI enabled.

I would rather you approach analysis of these kinds of MSI bugs in
this manner, instead of disabling MSI wholesale.  Because with the
latter approach it is nearly guarenteed that the real reason will only
be discovered with an extremely low priority.

Thank you.
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch] PCI: disable MSI on more ATI NorthBridges, Shane Huang, (Thu Oct 18, 2:14 am)
Re: [patch] PCI: disable MSI on more ATI NorthBridges, David Miller, (Thu Oct 18, 3:19 am)
RE: [patch] PCI: disable MSI on more ATI NorthBridges, Shane Huang, (Thu Oct 18, 3:37 am)
Re: [patch] PCI: disable MSI on more ATI NorthBridges, David Miller, (Thu Oct 18, 4:46 am)
Re: [patch] PCI: disable MSI on more ATI NorthBridges, Daniel Barkalow, (Fri Oct 19, 10:42 am)
[patch] PCI: disable the MSI of AMD RS690, Shane Huang, (Thu Jan 24, 3:59 am)
[patch] PCI: modify SB700 SATA MSI quirk, Shane Huang, (Thu Jan 24, 4:12 am)
Re: [patch] PCI: disable the MSI of AMD RS690, Brice Goglin, (Thu Jan 24, 4:15 am)
Re: [patch] PCI: modify SB700 SATA MSI quirk, Tejun Heo, (Thu Jan 24, 5:19 pm)
RE: [patch] PCI: modify SB700 SATA MSI quirk, Shane Huang, (Thu Jan 24, 8:26 pm)
Re: [patch] PCI: modify SB700 SATA MSI quirk, Tejun Heo, (Thu Jan 24, 8:35 pm)
RE: [patch] PCI: modify SB700 SATA MSI quirk, Shane Huang, (Thu Jan 24, 8:48 pm)
Re: [patch] PCI: modify SB700 SATA MSI quirk, Greg KH, (Thu Jan 24, 9:33 pm)
RE: [patch] PCI: disable the MSI of AMD RS690, Shane Huang, (Fri Jan 25, 3:39 am)