* Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > static struct irq_2_iommu *irq_2_iommu_alloc(unsigned int irq) > { > - return irq_2_iommu_alloc_node(irq, cpu_to_node(boot_cpu_id)); > + int node; > + > +#ifdef CONFIG_SMP > + node = irq_to_desc(irq)->node; > +#else > + node = cpu_to_node(boot_cpu_id); > +#endif > + > + return irq_2_iommu_alloc_node(irq, node); > } To avoid the ifdef, shouldnt there be an new irq_node(irq) primitive that maps to desc->node on SMP and does the right thing on UP? Ingo -- 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