diff --git a/xen/arch/x86/acpi/boot.c b/xen/arch/x86/acpi/boot.c index 24702d041c..cc4bbc0284 100644 --- a/xen/arch/x86/acpi/boot.c +++ b/xen/arch/x86/acpi/boot.c @@ -742,8 +742,6 @@ int __init acpi_boot_init(void) acpi_mmcfg_init(); - acpi_iommu_init(); - erst_init(); acpi_hest_init(); diff --git a/xen/arch/x86/setup.c b/xen/arch/x86/setup.c index 723bd49c76..f40a9fe5d3 100644 --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -1664,15 +1664,30 @@ void __init noreturn __start_xen(unsigned long mbi_p) dmi_scan_machine(); - generic_apic_probe(); - mmio_ro_ranges = rangeset_new(NULL, "r/o mmio ranges", RANGESETF_prettyprint_hex); xsm_multiboot_init(module_map, mbi); + /* + * IOMMU-related ACPI table parsing may require some of the system domains + * to be usable, e.g. for pci_hide_device()'s use of dom_xen. + */ setup_system_domains(); + /* + * IOMMU-related ACPI table parsing has to happen before APIC probing, for + * check_x2apic_preenabled() to be able to observe respective findings, in + * particular iommu_intremap having got turned off. + */ + acpi_iommu_init(); + + /* + * APIC probing needs to happen before general ACPI table parsing, as e.g. + * generic_bigsmp_probe() may occur only afterwards. + */ + generic_apic_probe(); + acpi_boot_init(); if ( smp_found_config ) diff --git a/xen/drivers/passthrough/x86/iommu.c b/xen/drivers/passthrough/x86/iommu.c index b2284ae001..f11280f4b2 100644 --- a/xen/drivers/passthrough/x86/iommu.c +++ b/xen/drivers/passthrough/x86/iommu.c @@ -43,14 +43,17 @@ bool __read_mostly iommu_intpost; void __init acpi_iommu_init(void) { - int ret; + int ret = -ENODEV; if ( !iommu_enable && !iommu_intremap ) return; - ret = acpi_dmar_init(); - if ( ret == -ENODEV ) - ret = acpi_ivrs_init(); + if ( !acpi_disabled ) + { + ret = acpi_dmar_init(); + if ( ret == -ENODEV ) + ret = acpi_ivrs_init(); + } if ( ret ) {