]> xenbits.xensource.com Git - xen.git/commitdiff
x86: kill msix_flush_writes()
authorKeir Fraser <keir.fraser@citrix.com>
Thu, 21 Jan 2010 15:13:00 +0000 (15:13 +0000)
committerKeir Fraser <keir.fraser@citrix.com>
Thu, 21 Jan 2010 15:13:00 +0000 (15:13 +0000)
The (only) two callers of it don't need it, as the MSI-X case of
msi_set_mask_bit() already does the necessary readl().

Signed-off-by: Jan Beulich <jbeulich@novell.com>
xen/arch/x86/msi.c

index 67f48a3555e88c47661eba7f0b28ee8a93755d4d..06b53a202f8069c40db52a2e1ca6020ef86268ae 100644 (file)
@@ -327,27 +327,6 @@ static void msix_set_enable(struct pci_dev *dev, int enable)
     }
 }
 
-static void msix_flush_writes(unsigned int irq)
-{
-    struct msi_desc *entry = irq_desc[irq].msi_desc;
-
-    BUG_ON(!entry || !entry->dev);
-    switch (entry->msi_attrib.type) {
-    case PCI_CAP_ID_MSI:
-        /* nothing to do */
-        break;
-    case PCI_CAP_ID_MSIX:
-    {
-        int offset = PCI_MSIX_ENTRY_VECTOR_CTRL_OFFSET;
-        readl(entry->mask_base + offset);
-        break;
-    }
-    default:
-        BUG();
-        break;
-    }
-}
-
 int msi_maskable_irq(const struct msi_desc *entry)
 {
     BUG_ON(!entry);
@@ -409,13 +388,11 @@ static int msi_get_mask_bit(const struct msi_desc *entry)
 void mask_msi_irq(unsigned int irq)
 {
     msi_set_mask_bit(irq, 1);
-    msix_flush_writes(irq);
 }
 
 void unmask_msi_irq(unsigned int irq)
 {
     msi_set_mask_bit(irq, 0);
-    msix_flush_writes(irq);
 }
 
 static struct msi_desc* alloc_msi_entry(void)