On Friday 17 April 2015 02:50:07 Duc Dang wrote: > + > + /* > + * MSIINTn (n is 0..F) indicates if there is a pending MSI interrupt > + * If bit x of this register is set (x is 0..7), one or more interupts > + * corresponding to MSInIRx is set. > + */ > + grp_select = readl(xgene_msi->msi_regs + MSI_INT0 + (msi_grp << 16)); > + while (grp_select) { > + msir_index = ffs(grp_select) - 1; > + /* > + * Calculate MSInIRx address to read to check for interrupts > + * (refer to termination address and data assignment > + * described in xgene_compose_msi_msg function) > + */ > + msir_reg = (msi_grp << 19) + (msir_index << 16); > + msir_val = readl(xgene_msi->msi_regs + MSI_IR0 + msir_reg); > + while (msir_val) { > + intr_index = ffs(msir_val) - 1; > + /* > + * Calculate MSI vector number (refer to the termination > + * address and data assignment described in > + * xgene_compose_msi_msg function) > + */ > + hw_irq = (((msir_index * IRQS_PER_IDX) + intr_index) * > + NR_HW_IRQS) + msi_grp; > + virq = irq_find_mapping(xgene_msi->domain, hw_irq); > + if (virq != 0) > + generic_handle_irq(virq); > + msir_val &= ~(1 << intr_index); > + processed++; > + } > + grp_select &= ~(1 << msir_index); > + } > As the MSI is forwarded to the GIC here, how do you maintain ordering between DMA data getting forwarded from the PCI host bridge to RAM with regard to the MSI handler getting entered from this code? Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html