On Sun, 2009-05-10 at 22:36 -0700, David Miller wrote:
> From: Michael Ellerman <michael@xxxxxxxxxxxxxx>
> Date: Mon, 11 May 2009 11:21:51 +1000
>
> > So I guess this device is just silently ignoring that write?
>
> No, it signals a fault on the PCI bus, which is how we noticed
> this problem in the first place.
But the patch doesn't change that? It just removes the readl AFAICS:
diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c
index 6f2e629..3627732 100644
--- a/drivers/pci/msi.c
+++ b/drivers/pci/msi.c
@@ -455,8 +455,6 @@ static int msix_capability_init(struct pci_dev *dev,
entry->msi_attrib.default_irq = dev->irq;
entry->msi_attrib.pos = pos;
entry->mask_base = base;
- entry->masked = readl(base + j * PCI_MSIX_ENTRY_SIZE +
- PCI_MSIX_ENTRY_VECTOR_CTRL_OFFSET);
msix_mask_irq(entry, 1);
Or has msix_mask_irq() changed since 413f81eba?
cheers
Attachment:
signature.asc
Description: This is a digitally signed message part
