Skip to content

Commit 70b5937

Browse files
committed
x86/xen: Consolidate XEN-MSI init
X86 cannot store the irq domain pointer in struct device without breaking XEN because the irq domain pointer takes precedence over arch_*_msi_irqs() fallbacks. To achieve this XEN MSI interrupt management needs to be wrapped into an irq domain. Move the x86_msi ops setup into a single function to prepare for this. Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Reviewed-by: Juergen Gross <jgross@suse.com> Link: https://lore.kernel.org/r/20200826112333.420224092@linutronix.de
1 parent 7d4d892 commit 70b5937

1 file changed

Lines changed: 32 additions & 19 deletions

File tree

arch/x86/pci/xen.c

Lines changed: 32 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,10 @@ static void xen_initdom_restore_msi_irqs(struct pci_dev *dev)
372372
WARN(ret && ret != -ENOSYS, "restore_msi -> %d\n", ret);
373373
}
374374
}
375-
#endif
375+
#else /* CONFIG_XEN_DOM0 */
376+
#define xen_initdom_setup_msi_irqs NULL
377+
#define xen_initdom_restore_msi_irqs NULL
378+
#endif /* !CONFIG_XEN_DOM0 */
376379

377380
static void xen_teardown_msi_irqs(struct pci_dev *dev)
378381
{
@@ -404,7 +407,31 @@ static void xen_teardown_msi_irq(unsigned int irq)
404407
WARN_ON_ONCE(1);
405408
}
406409

407-
#endif
410+
static __init void xen_setup_pci_msi(void)
411+
{
412+
if (xen_pv_domain()) {
413+
if (xen_initial_domain()) {
414+
x86_msi.setup_msi_irqs = xen_initdom_setup_msi_irqs;
415+
x86_msi.restore_msi_irqs = xen_initdom_restore_msi_irqs;
416+
} else {
417+
x86_msi.setup_msi_irqs = xen_setup_msi_irqs;
418+
}
419+
x86_msi.teardown_msi_irqs = xen_pv_teardown_msi_irqs;
420+
pci_msi_ignore_mask = 1;
421+
} else if (xen_hvm_domain()) {
422+
x86_msi.setup_msi_irqs = xen_hvm_setup_msi_irqs;
423+
x86_msi.teardown_msi_irqs = xen_teardown_msi_irqs;
424+
} else {
425+
WARN_ON_ONCE(1);
426+
return;
427+
}
428+
429+
x86_msi.teardown_msi_irq = xen_teardown_msi_irq;
430+
}
431+
432+
#else /* CONFIG_PCI_MSI */
433+
static inline void xen_setup_pci_msi(void) { }
434+
#endif /* CONFIG_PCI_MSI */
408435

409436
int __init pci_xen_init(void)
410437
{
@@ -421,12 +448,7 @@ int __init pci_xen_init(void)
421448
/* Keep ACPI out of the picture */
422449
acpi_noirq_set();
423450

424-
#ifdef CONFIG_PCI_MSI
425-
x86_msi.setup_msi_irqs = xen_setup_msi_irqs;
426-
x86_msi.teardown_msi_irq = xen_teardown_msi_irq;
427-
x86_msi.teardown_msi_irqs = xen_pv_teardown_msi_irqs;
428-
pci_msi_ignore_mask = 1;
429-
#endif
451+
xen_setup_pci_msi();
430452
return 0;
431453
}
432454

@@ -446,10 +468,7 @@ static void __init xen_hvm_msi_init(void)
446468
((eax & XEN_HVM_CPUID_APIC_ACCESS_VIRT) && boot_cpu_has(X86_FEATURE_APIC)))
447469
return;
448470
}
449-
450-
x86_msi.setup_msi_irqs = xen_hvm_setup_msi_irqs;
451-
x86_msi.teardown_msi_irqs = xen_teardown_msi_irqs;
452-
x86_msi.teardown_msi_irq = xen_teardown_msi_irq;
471+
xen_setup_pci_msi();
453472
}
454473
#endif
455474

@@ -482,13 +501,7 @@ int __init pci_xen_initial_domain(void)
482501
{
483502
int irq;
484503

485-
#ifdef CONFIG_PCI_MSI
486-
x86_msi.setup_msi_irqs = xen_initdom_setup_msi_irqs;
487-
x86_msi.teardown_msi_irq = xen_teardown_msi_irq;
488-
x86_msi.teardown_msi_irqs = xen_teardown_pv_msi_irqs;
489-
x86_msi.restore_msi_irqs = xen_initdom_restore_msi_irqs;
490-
pci_msi_ignore_mask = 1;
491-
#endif
504+
xen_setup_pci_msi();
492505
__acpi_register_gsi = acpi_register_gsi_xen;
493506
__acpi_unregister_gsi = NULL;
494507
/*

0 commit comments

Comments
 (0)