[PATCH 1/2] PCI: Make pci_msi_setup_pci_dev() non-static for use by arch code
mpe at ellerman.id.au
Wed Aug 19 10:44:46 AEST 2015
Thanks for the patches.
On Tue, 2015-08-18 at 18:13 -0300, Guilherme G. Piccoli wrote:
> Commit 1851617cd2 ("PCI/MSI: Disable MSI at enumeration even if kernel
> doesn't support MSI") changed the location of the code that disables
> MSI/MSI-X interrupts at PCI probe time in devices that have this flag set.
> It moved the code from pci_msi_init_pci_dev() to a new function named
> pci_msi_setup_pci_dev(), called by pci_setup_device().
> Since then, the pSeries platform of the powerpc architecture needs to
> disable MSI at PCI probe time manually, as the code flow doesn't
> reach pci_setup_device().
> For doing so, it wants to call
> pci_msi_setup_pci_dev(). This patch makes the required function
> non-static, so that it will be called on PCI probe path on powerpc pSeries
> platform in next patch.
I didn't follow that entirely, I think you mean something like:
The pseries PCI probing code does not call pci_setup_device(), so since
commit 1851617cd2 pci_msi_setup_pci_dev() is not called and MSIs are left
enabled, which is a bug.
To fix this the pseries PCI probe should manually call
pci_msi_setup_pci_dev(), so make it non-static.
Does that look OK?
Also you haven't CC'ed the original author of the commit, or the PCI
maintainer, or the relevant lists.
That would be:
Michael S. Tsirkin <mst at redhat.com>
Bjorn Helgaas <bhelgaas at google.com>
linux-pci at vger.kernel.org
linux-kernel at vger.kernel.org
And finally both patches should have a fixes line, such as:
Fixes: 1851617cd2da ("PCI/MSI: Disable MSI at enumeration even if kernel doesn't support MSI")
More information about the Linuxppc-dev