Signed-off-by: Jan Beulich --- a/xen/arch/x86/apic.c +++ b/xen/arch/x86/apic.c @@ -953,18 +953,17 @@ no_apic: return -1; } -void x2apic_setup(void) +void x2apic_ap_setup(void) +{ + if ( x2apic_enabled ) + __enable_x2apic(); +} + +void __init x2apic_bsp_setup(void) { struct IO_APIC_route_entry **ioapic_entries = NULL; uint64_t msr_content; - if ( smp_processor_id() != 0 ) - { - if ( x2apic_enabled ) - __enable_x2apic(); - return; - } - if ( !cpu_has_x2apic ) return; --- a/xen/arch/x86/genapic/x2apic.c +++ b/xen/arch/x86/genapic/x2apic.c @@ -27,7 +27,7 @@ #include #include -static int x2apic_phys; /* By default we use logical cluster mode. */ +static int __initdata x2apic_phys; /* By default we use logical cluster mode. */ boolean_param("x2apic_phys", x2apic_phys); static void init_apic_ldr_x2apic_phys(void) @@ -134,7 +134,7 @@ static const struct genapic apic_x2apic_ .send_IPI_self = send_IPI_self_x2apic }; -const struct genapic *apic_x2apic_probe(void) +const struct genapic *__init apic_x2apic_probe(void) { return x2apic_phys ? &apic_x2apic_phys : &apic_x2apic_cluster; } --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -1190,7 +1190,7 @@ void __init __start_xen(unsigned long mb init_cpu_to_node(); - x2apic_setup(); + x2apic_bsp_setup(); init_IRQ(); --- a/xen/arch/x86/smpboot.c +++ b/xen/arch/x86/smpboot.c @@ -195,7 +195,7 @@ void smp_callin(void) * update until we finish. We are free to set up this CPU: first the APIC. */ Dprintk("CALLIN, before setup_local_APIC().\n"); - x2apic_setup(); + x2apic_ap_setup(); setup_local_APIC(); map_cpu_to_logical_apicid(); --- a/xen/include/asm-x86/apic.h +++ b/xen/include/asm-x86/apic.h @@ -25,7 +25,8 @@ extern int apic_verbosity; extern int x2apic_enabled; extern int directed_eoi_enabled; -void x2apic_setup(void); +void x2apic_bsp_setup(void); +void x2apic_ap_setup(void); const struct genapic *apic_x2apic_probe(void); /*