On Wed, Nov 03, 2021 at 03:40:55PM +0100, Jan Beulich wrote: > While commit XXXXXXXXXXXX ("x86/IOMMU: mark IOMMU / intremap not in use > when ACPI tables are missing") deals with apic_x2apic_probe() as called > from x2apic_bsp_setup(), the check_x2apic_preenabled() path is similarly > affected: The call needs to occur after acpi_boot_init() (which is what > calls acpi_iommu_init()), such that iommu_intremap getting disabled > there can be properly taken into account by apic_x2apic_probe(). > > Signed-off-by: Jan Beulich <jbeul...@suse.com>
LGTM. I cannot find any dependency between acpi_boot_init and having initialized the apic. Reviewed-by: Roger Pau Monné <roger....@citrix.com> Thanks, Roger.