* [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled @ 2026-09-10 9:04 Grzegorz Jaszczyk 2026-09-10 15:23 ` Dave Hansen 2026-09-11 12:51 ` Thomas Gleixner 0 siblings, 2 replies; 6+ messages in thread From: Grzegorz Jaszczyk @ 2026-09-10 9:04 UTC (permalink / raw) To: tglx Cc: linux-kernel, dmaluka, vineethrp, chuanxiao.dong, Grzegorz Jaszczyk, Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Eric Dumazet, Melody Wang During early boot, the generic x86 kernel defaults to the MMIO-based APIC driver (apic_physflat). However, if the kernel is booted (e.g. via kexec) when x2apic is already enabled in hardware, the MMIO interface to the APIC is disabled. Normally, ACPI MADT probing would install an x2APIC driver early. However, if ACPI is disabled (e.g., CONFIG_ACPI is not set, as in crashdump kernels), x86_64_probe_apic() does not run until late_time_init() via apic_intr_mode_init(). This creates a window between local_irq_enable() and late_time_init() where interrupts are enabled, but the APIC driver pointer still points to apic_physflat. Because check_x2apic() detected hardware x2APIC mode and set x2apic_mode = 1, init_apic_mappings() skipped mapping the APIC fixmap. If a pending interrupt (e.g., left in IRR across kexec on secondary CPUs) fires during this window, native_apic_mem_eoi() is invoked, which attempts to write to the unmapped APIC EOI register, triggering an immediate kernel page fault (#PF). To prevent this, check if x2apic is enabled in hardware during early APIC call setup in apic_setup_apic_calls() and switch the default APIC driver from apic_physflat to apic_x2apic_phys immediately. This ensures that any early EOI writes use safe MSR-based accesses. The driver can still be upgraded later during the normal APIC probe phase. Signed-off-by: Grzegorz Jaszczyk <jaszczyk@chromium.org> --- arch/x86/kernel/apic/init.c | 2 ++ arch/x86/kernel/apic/local.h | 5 +++++ arch/x86/kernel/apic/x2apic_phys.c | 10 ++++++++++ 3 files changed, 17 insertions(+) diff --git a/arch/x86/kernel/apic/init.c b/arch/x86/kernel/apic/init.c index 821e2e536f19..c48323f725e7 100644 --- a/arch/x86/kernel/apic/init.c +++ b/arch/x86/kernel/apic/init.c @@ -82,6 +82,8 @@ static __init void update_static_calls(void) void __init apic_setup_apic_calls(void) { + x2apic_phys_early_init(); + /* Ensure that the default APIC has native_eoi populated */ apic->native_eoi = apic->eoi; update_static_calls(); diff --git a/arch/x86/kernel/apic/local.h b/arch/x86/kernel/apic/local.h index 090dd71837aa..0c33eaa3b03a 100644 --- a/arch/x86/kernel/apic/local.h +++ b/arch/x86/kernel/apic/local.h @@ -20,6 +20,11 @@ void x2apic_send_IPI_all(int vector); void x2apic_send_IPI_allbutself(int vector); void x2apic_send_IPI_self(int vector); extern u32 x2apic_max_apicid; +#ifdef CONFIG_X86_X2APIC +void x2apic_phys_early_init(void); +#else +static inline void x2apic_phys_early_init(void) { } +#endif /* IPI */ diff --git a/arch/x86/kernel/apic/x2apic_phys.c b/arch/x86/kernel/apic/x2apic_phys.c index 090647cc5a78..c41f47aed7ab 100644 --- a/arch/x86/kernel/apic/x2apic_phys.c +++ b/arch/x86/kernel/apic/x2apic_phys.c @@ -155,3 +155,13 @@ static struct apic apic_x2apic_phys __ro_after_init = { }; apic_driver(apic_x2apic_phys); + +void __init x2apic_phys_early_init(void) +{ + if (x2apic_enabled()) { + apic = &apic_x2apic_phys; + if (apic->x2apic_set_max_apicid) + apic->max_apic_id = x2apic_max_apicid; + pr_info("Switched default APIC to: %s\n", apic->name); + } +} -- 2.55.0.1003.g10538fe699-goog ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled 2026-09-10 9:04 [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled Grzegorz Jaszczyk @ 2026-09-10 15:23 ` Dave Hansen 2026-09-11 15:11 ` Grzegorz Jaszczyk 2026-09-11 12:51 ` Thomas Gleixner 1 sibling, 1 reply; 6+ messages in thread From: Dave Hansen @ 2026-09-10 15:23 UTC (permalink / raw) To: Grzegorz Jaszczyk, tglx Cc: linux-kernel, dmaluka, vineethrp, chuanxiao.dong, Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Eric Dumazet, Melody Wang On 9/10/26 02:04, Grzegorz Jaszczyk wrote: > During early boot, the generic x86 kernel defaults to the MMIO-based > APIC driver (apic_physflat). However, if the kernel is booted (e.g. via > kexec) when x2apic is already enabled in hardware, the MMIO interface to > the APIC is disabled. There's also some new hardware that locks the hardware in x2apic mode. I wonder if it has a similar problem. > Normally, ACPI MADT probing would install an x2APIC driver early. > However, if ACPI is disabled (e.g., CONFIG_ACPI is not set, as in > crashdump kernels), x86_64_probe_apic() does not run until late_time_init() > via apic_intr_mode_init(). Is this "default_acpi_madt_oem_check()" that uses apic_install_driver()? > This creates a window between local_irq_enable() and late_time_init() > where interrupts are enabled, but the APIC driver pointer still points > to apic_physflat. Because check_x2apic() detected hardware x2APIC mode > and set x2apic_mode = 1, init_apic_mappings() skipped mapping the APIC > fixmap. If a pending interrupt (e.g., left in IRR across kexec on > secondary CPUs) fires during this window, native_apic_mem_eoi() is > invoked, which attempts to write to the unmapped APIC EOI register, > triggering an immediate kernel page fault (#PF). Long-term, it seems like reducing the use of x2apic_mode would be really nice. > To prevent this, check if x2apic is enabled in hardware during early APIC > call setup in apic_setup_apic_calls() and switch the default APIC driver > from apic_physflat to apic_x2apic_phys immediately. This ensures that any > early EOI writes use safe MSR-based accesses. The driver can still be > upgraded later during the normal APIC probe phase. > > > diff --git a/arch/x86/kernel/apic/init.c b/arch/x86/kernel/apic/init.c > index 821e2e536f19..c48323f725e7 100644 > --- a/arch/x86/kernel/apic/init.c > +++ b/arch/x86/kernel/apic/init.c > @@ -82,6 +82,8 @@ static __init void update_static_calls(void) > > void __init apic_setup_apic_calls(void) > { > + x2apic_phys_early_init(); > + > /* Ensure that the default APIC has native_eoi populated */ > apic->native_eoi = apic->eoi; > update_static_calls(); This seems a bit ad-hoc. > apic_driver(apic_x2apic_phys); > + > +void __init x2apic_phys_early_init(void) > +{ > + if (x2apic_enabled()) { > + apic = &apic_x2apic_phys; > + if (apic->x2apic_set_max_apicid) > + apic->max_apic_id = x2apic_max_apicid; > + pr_info("Switched default APIC to: %s\n", apic->name); > + } > +} This seems to be a subset of what apic_install_driver() and apic_x2apic_phys->probe() do, down do the pr_info(). Even if those are overkill for this, it would be nice to use them for consistency if they function. It also seems like some of the more obscure apic drivers depend on x2apic_mode. They also seem like they might be selected by the MADT search. Could this end up overwriting those? Maybe this is too much work for the task at hand, but this does seem to show that there's some information missing from 'struct apic'. Say apic_physflat had an ->initialized() that was false until its MMIO was unmapped and apic_x2apic_phys->initialized() pointed over to: x2apic_enabled(). apic_setup_apic_calls() { // Search for an APIC driver that is initialized: for_each(drv) { if (!drv->initialized()) continue; apic_install_driver(drv); } } Basically, I'm wondering if it's worth continuing to special-case x2apic handling or whether it should just be a part of the apic driver infrastructure. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled 2026-09-10 15:23 ` Dave Hansen @ 2026-09-11 15:11 ` Grzegorz Jaszczyk 0 siblings, 0 replies; 6+ messages in thread From: Grzegorz Jaszczyk @ 2026-09-11 15:11 UTC (permalink / raw) To: Dave Hansen Cc: tglx, linux-kernel, dmaluka, vineethrp, chuanxiao.dong, Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Eric Dumazet, Melody Wang, jaszczk On Thu, Sep 10, 2026 at 5:23 PM Dave Hansen <dave.hansen@intel.com> wrote: > > On 9/10/26 02:04, Grzegorz Jaszczyk wrote: > > During early boot, the generic x86 kernel defaults to the MMIO-based > > APIC driver (apic_physflat). However, if the kernel is booted (e.g. via > > kexec) when x2apic is already enabled in hardware, the MMIO interface to > > the APIC is disabled. > > There's also some new hardware that locks the hardware in x2apic mode. I > wonder if it has a similar problem. You are probably referring to "x86/apic: Don't disable x2APIC if locked" - if so, it seems it will have the same problem whenever ACPI MADT probing doesn't run early (e.g. in CONFIG_ACPI=n crashdump kernels). In both cases (locked by hardware/BIOS or handed over via kexec), the CPU starts with x2APIC enabled. As Intel SDM section "11.12.2 x2APIC Register Availability" states: "In x2APIC mode, the memory mapped interface is not available and any access to the MMIO interface will behave similar to that of a legacy xAPIC in globally disabled state." > > > Normally, ACPI MADT probing would install an x2APIC driver early. > > However, if ACPI is disabled (e.g., CONFIG_ACPI is not set, as in > > crashdump kernels), x86_64_probe_apic() does not run until late_time_init() > > via apic_intr_mode_init(). > > Is this "default_acpi_madt_oem_check()" that uses apic_install_driver()? Yes > > apic_driver(apic_x2apic_phys); > > + > > +void __init x2apic_phys_early_init(void) > > +{ > > + if (x2apic_enabled()) { > > + apic = &apic_x2apic_phys; > > + if (apic->x2apic_set_max_apicid) > > + apic->max_apic_id = x2apic_max_apicid; > > + pr_info("Switched default APIC to: %s\n", apic->name); > > + } > > +} > > This seems to be a subset of what apic_install_driver() and > apic_x2apic_phys->probe() do, down do the pr_info(). > > Even if those are overkill for this, it would be nice to use them for > consistency if they function. Ok, I can use apic_install_driver() instead. When it comes to apic_x2apic_phys->probe() it can't really be used in current form because at this early stage x2apic_phys_probe() will return 0, since x2apic_mode is set quite late in setup.c (check_x2apic()). > > It also seems like some of the more obscure apic drivers depend on > x2apic_mode. They also seem like they might be selected by the MADT > search. Could this end up overwriting those? It should be the other way around: MADT is called after apic_setup_apic_calls() and it will overwirte it as normal: just this time instead of overwriting apic_physflat it will overwrite apic_x2apic_phys. > > Maybe this is too much work for the task at hand, but this does seem to > show that there's some information missing from 'struct apic'. Say > apic_physflat had an ->initialized() that was false until its MMIO was > unmapped and apic_x2apic_phys->initialized() pointed over to: > x2apic_enabled(). > > apic_setup_apic_calls() > { > // Search for an APIC driver that is initialized: > for_each(drv) { > if (!drv->initialized()) > continue; > > apic_install_driver(drv); > } > } > > Basically, I'm wondering if it's worth continuing to special-case x2apic > handling or whether it should just be a part of the apic driver > infrastructure. Yeah, something like this will work. Following your suggestion I was thinking about something like this: void __init apic_setup_apic_calls(void) { + struct apic **drv; + + for (drv = __apicdrivers; drv < __apicdrivers_end; drv++) { + if ((*drv)->early_probe && (*drv)->early_probe()) { + apic_install_driver(*drv); + break; + } + } and as apic_x2apic_phys.early_probe() we could use existing x2apic_enabled(). And in apic_physflat, an early_probe returning !x2apic_enabled(). But I see that Thomas also responded, and I think he is looking into a different approach. Thanks, Grzegorz ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled 2026-09-10 9:04 [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled Grzegorz Jaszczyk 2026-09-10 15:23 ` Dave Hansen @ 2026-09-11 12:51 ` Thomas Gleixner 2026-09-11 15:17 ` Grzegorz Jaszczyk 1 sibling, 1 reply; 6+ messages in thread From: Thomas Gleixner @ 2026-09-11 12:51 UTC (permalink / raw) To: Grzegorz Jaszczyk Cc: linux-kernel, dmaluka, vineethrp, chuanxiao.dong, Grzegorz Jaszczyk, Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Eric Dumazet, Melody Wang On Thu, Sep 10 2026 at 09:04, Grzegorz Jaszczyk wrote: > To prevent this, check if x2apic is enabled in hardware during early APIC > call setup in apic_setup_apic_calls() and switch the default APIC driver > from apic_physflat to apic_x2apic_phys immediately. This ensures that any > early EOI writes use safe MSR-based accesses. The driver can still be > upgraded later during the normal APIC probe phase. That's really the wrong place. Initializing those calls has absolutely nothing to do with the driver probing. There is quite some historical ballast in this whole APIC driver probing, which I never came around to clean up. I'll have a look how this can be done sanely without replicating half of the existing code just to paper over the real issue. Thanks, tglx ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled 2026-09-11 12:51 ` Thomas Gleixner @ 2026-09-11 15:17 ` Grzegorz Jaszczyk 2026-09-16 21:20 ` Thomas Gleixner 0 siblings, 1 reply; 6+ messages in thread From: Grzegorz Jaszczyk @ 2026-09-11 15:17 UTC (permalink / raw) To: Thomas Gleixner Cc: linux-kernel, dmaluka, vineethrp, chuanxiao.dong, Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Eric Dumazet, Melody Wang On Fri, Sep 11, 2026 at 2:51 PM Thomas Gleixner <tglx@kernel.org> wrote: > > On Thu, Sep 10 2026 at 09:04, Grzegorz Jaszczyk wrote: > > To prevent this, check if x2apic is enabled in hardware during early APIC > > call setup in apic_setup_apic_calls() and switch the default APIC driver > > from apic_physflat to apic_x2apic_phys immediately. This ensures that any > > early EOI writes use safe MSR-based accesses. The driver can still be > > upgraded later during the normal APIC probe phase. > > That's really the wrong place. Initializing those calls has absolutely > nothing to do with the driver probing. > > There is quite some historical ballast in this whole APIC driver > probing, which I never came around to clean up. > > I'll have a look how this can be done sanely without replicating half of > the existing code just to paper over the real issue. > Ok, understood. Thank you. Let me know when you have something to look at or test. Thanks, Grzegorz ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled 2026-09-11 15:17 ` Grzegorz Jaszczyk @ 2026-09-16 21:20 ` Thomas Gleixner 0 siblings, 0 replies; 6+ messages in thread From: Thomas Gleixner @ 2026-09-16 21:20 UTC (permalink / raw) To: Grzegorz Jaszczyk Cc: linux-kernel, dmaluka, vineethrp, chuanxiao.dong, Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin, Eric Dumazet, Melody Wang On Fri, Sep 11 2026 at 17:17, Grzegorz Jaszczyk wrote: > On Fri, Sep 11, 2026 at 2:51 PM Thomas Gleixner <tglx@kernel.org> wrote: >> >> On Thu, Sep 10 2026 at 09:04, Grzegorz Jaszczyk wrote: >> > To prevent this, check if x2apic is enabled in hardware during early APIC >> > call setup in apic_setup_apic_calls() and switch the default APIC driver >> > from apic_physflat to apic_x2apic_phys immediately. This ensures that any >> > early EOI writes use safe MSR-based accesses. The driver can still be >> > upgraded later during the normal APIC probe phase. >> >> That's really the wrong place. Initializing those calls has absolutely >> nothing to do with the driver probing. >> >> There is quite some historical ballast in this whole APIC driver >> probing, which I never came around to clean up. >> >> I'll have a look how this can be done sanely without replicating half of >> the existing code just to paper over the real issue. >> > Ok, understood. Thank you. Let me know when you have something to look > at or test. It actually needs a larger overhaul because when the x2apic_phys driver is installed early on then its probe function will prevent installing the cluster driver. So your claim that any other driver can still be selected is only true when ACPI invokes the MADT check function, but not via the probe mechanism. I'm not really happy about the untested below, but something like that should solve your problem and still keep all options open. That patch needs to be split up in more digestable pieces. FYI, I'm on vacation and only sporadically at the keyboard. So if you get it working feel free to polish it up and submit the whole pile with a "Suggested-by: tglx..." tag. It definitely needs some eyeballs and testing time in tip. Thanks, tglx --- --- a/arch/x86/include/asm/apic.h +++ b/arch/x86/include/asm/apic.h @@ -133,7 +133,8 @@ extern void sync_Arb_IDs(void); extern void init_bsp_APIC(void); extern void apic_intr_mode_select(void); extern void apic_intr_mode_init(void); -extern void init_apic_mappings(void); +void apic_early_init(void); +void apic_finalize_early_init(void); void register_lapic_address(unsigned long address); extern void setup_boot_APIC_clock(void); extern void setup_secondary_APIC_clock(void); @@ -254,7 +255,6 @@ static inline u32 native_apic_msr_read(u #define x2apic_mode (0) #define x2apic_supported() (0) #endif /* !CONFIG_X86_X2APIC */ -extern void __init check_x2apic(void); struct irq_data; @@ -288,7 +288,8 @@ struct apic { u32 disable_esr : 1, dest_mode_logical : 1, x2apic_set_max_apicid : 1, - nmi_to_offline_cpu : 1; + nmi_to_offline_cpu : 1, + probed : 1; u32 (*calc_dest_apicid)(unsigned int cpu); @@ -366,9 +367,6 @@ extern int lapic_can_unplug_cpu(void); #ifdef CONFIG_X86_LOCAL_APIC extern struct apic_override __x86_apic_override; -void __init apic_setup_apic_calls(void); -void __init apic_install_driver(struct apic *driver); - #define apic_update_callback(_callback, _fn) { \ __x86_apic_override._callback = _fn; \ apic->_callback = _fn; \ --- a/arch/x86/kernel/apic/apic.c +++ b/arch/x86/kernel/apic/apic.c @@ -1869,7 +1869,7 @@ static __init void try_to_enable_x2apic( x2apic_enable(); } -void __init check_x2apic(void) +static __init struct apic *check_x2apic(void) { if (x2apic_enabled()) { pr_info("x2apic: enabled by BIOS, switching to x2apic ops\n"); @@ -1879,15 +1879,19 @@ void __init check_x2apic(void) else x2apic_state = X2APIC_ON; apic_read_boot_cpu_id(true); + /* Force the initial driver to X2APIC */ + return &apic_x2apic_phys; } else if (!boot_cpu_has(X86_FEATURE_X2APIC)) { x2apic_state = X2APIC_DISABLED; } + /* Keep the default driver */ + return NULL; } #else /* CONFIG_X86_X2APIC */ -void __init check_x2apic(void) +static __init struct apic *check_x2apic(void) { if (!apic_is_x2apic_enabled()) - return; + return NULL; /* * Checkme: Can we simply turn off x2APIC here instead of disabling the APIC? */ @@ -1896,6 +1900,7 @@ void __init check_x2apic(void) apic_is_disabled = true; setup_clear_cpu_cap(X86_FEATURE_APIC); + return &apic_noop; } static inline void try_to_enable_x2apic(int remap_mode) { } @@ -2064,10 +2069,22 @@ static bool __init detect_init_APIC(void } #endif -/** - * init_apic_mappings - initialize APIC mappings - */ -void __init init_apic_mappings(void) +void __init apic_early_init(void) +{ + struct apic *driver; + + if (acpi_mps_check()) { + apic_is_disabled = true; + setup_clear_cpu_cap(X86_FEATURE_APIC); + driver = &apic_noop; + } else { + driver = check_x2apic(); + } + + apic_install_default_driver(driver); +} + +void __init apic_finalize_early_init(void) { if (apic_validate_deadline_timer()) pr_info("TSC deadline timer available\n"); --- a/arch/x86/kernel/apic/init.c +++ b/arch/x86/kernel/apic/init.c @@ -80,19 +80,8 @@ static __init void update_static_calls(v update_call(wakeup_secondary_cpu_64); } -void __init apic_setup_apic_calls(void) +static void __init __apic_install_driver(struct apic *driver) { - /* Ensure that the default APIC has native_eoi populated */ - apic->native_eoi = apic->eoi; - update_static_calls(); - pr_info("Static calls initialized\n"); -} - -void __init apic_install_driver(struct apic *driver) -{ - if (apic == driver) - return; - apic = driver; if (IS_ENABLED(CONFIG_X86_X2APIC) && apic->x2apic_set_max_apicid) @@ -105,6 +94,24 @@ void __init apic_install_driver(struct a /* Apply any already installed callback overrides */ restore_override_callbacks(); update_static_calls(); +} - pr_info("Switched APIC routing to: %s\n", driver->name); +void __init apic_install_default_driver(struct apic *driver) +{ + if (driver && driver != apic) { + __apic_install_driver(driver); + } else { + /* Ensure that the default APIC has the native_eoi() callback populated */ + apic->native_eoi = apic->eoi; + update_static_calls(); + } + pr_info("Set default APIC routing to: %s\n", apic->name); +} + +void __init apic_install_driver(struct apic *driver) +{ + if (apic != driver) + __apic_install_driver(driver); + apic->probed = true; + pr_info("Switched APIC routing to: %s\n", apic->name); } --- a/arch/x86/kernel/apic/local.h +++ b/arch/x86/kernel/apic/local.h @@ -13,12 +13,16 @@ #include <asm/irq_vectors.h> #include <asm/apic.h> +void __init apic_install_default_driver(struct apic *driver); +void __init apic_install_driver(struct apic *driver); + /* X2APIC */ u32 x2apic_get_apic_id(u32 id); void x2apic_send_IPI_all(int vector); void x2apic_send_IPI_allbutself(int vector); void x2apic_send_IPI_self(int vector); +extern struct apic apic_x2apic_phys; extern u32 x2apic_max_apicid; /* IPI */ --- a/arch/x86/kernel/apic/x2apic_phys.c +++ b/arch/x86/kernel/apic/x2apic_phys.c @@ -7,7 +7,6 @@ int x2apic_phys; -static struct apic apic_x2apic_phys; u32 x2apic_max_apicid __ro_after_init = UINT_MAX; void __init x2apic_set_max_apicid(u32 apicid) @@ -113,7 +112,7 @@ static int x2apic_phys_probe(void) if (x2apic_phys || x2apic_fadt_phys()) return 1; - return apic == &apic_x2apic_phys; + return apic == &apic_x2apic_phys && apic->probed; } u32 x2apic_get_apic_id(u32 id) @@ -121,7 +120,7 @@ u32 x2apic_get_apic_id(u32 id) return id; } -static struct apic apic_x2apic_phys __ro_after_init = { +struct apic apic_x2apic_phys __ro_after_init = { .name = "physical x2apic", .probe = x2apic_phys_probe, --- a/arch/x86/kernel/setup.c +++ b/arch/x86/kernel/setup.c @@ -993,14 +993,7 @@ void __init setup_arch(char **cmdline_p) x86_report_nx(); - apic_setup_apic_calls(); - - if (acpi_mps_check()) { -#ifdef CONFIG_X86_LOCAL_APIC - apic_is_disabled = true; -#endif - setup_clear_cpu_cap(X86_FEATURE_APIC); - } + apic_early_init(); e820__finish_early_params(); @@ -1062,7 +1055,6 @@ void __init setup_arch(char **cmdline_p) /* max_low_pfn get updated here */ find_low_pfn_range(); #else - check_x2apic(); /* How many end-of-memory variables you have, grandma! */ /* need this before calling reserve_initrd */ @@ -1242,7 +1234,7 @@ void __init setup_arch(char **cmdline_p) x86_init.mpparse.parse_smp_cfg(); /* Last opportunity to detect and map the local APIC */ - init_apic_mappings(); + apic_finalize_early_init(); topology_init_possible_cpus(); ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-09-16 21:20 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2026-09-10 9:04 [PATCH] x86/apic: Switch to x2apic driver early if x2apic is enabled Grzegorz Jaszczyk 2026-09-10 15:23 ` Dave Hansen 2026-09-11 15:11 ` Grzegorz Jaszczyk 2026-09-11 12:51 ` Thomas Gleixner 2026-09-11 15:17 ` Grzegorz Jaszczyk 2026-09-16 21:20 ` Thomas Gleixner
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®