Thread (65 messages) 65 messages, 5 authors, 2014-11-06

[Patch Part2 v4 20/31] PCI/MSI: Kill redundant calling for irq_set_msi_desc() for MSIx interrupts

From: bhelgaas@google.com (Bjorn Helgaas)
Date: 2014-11-06 04:04:48
Also in: linux-acpi, linux-pci, lkml

On Wed, Nov 5, 2014 at 6:32 PM, Yijing Wang [off-list ref] wrote:
On 2014/11/6 6:45, Bjorn Helgaas wrote:
quoted
On Tue, Nov 04, 2014 at 08:01:54PM +0800, Jiang Liu wrote:
quoted
It's arch_setup_msi_irq()/arch_setup_msi_irqs()'s responsibility to call
irq_set_msi_desc() to associate IRQ descriptors and MSI descriptors,
so kill the redundant call of irq_set_msi_desc() for MSIx interrupts
in PCI MSI core.
"MSI-X" in English text, "msix" in code.

The default arch_setup_msi_irq() in drivers/pci/msi.c doesn't call
irq_set_msi_desc().  Does it happen somewhere inside chip->setup_irq()?
Yes.

I also found this.
http://www.spinics.net/lists/linux-pci/msg34256.html
Yes, and I asked the same question then :)

It's just impractical to review things like this that make assumptions
about lots of code scattered all over the place with no direct linkage
to the change.

Bjorn
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help