Commit d3a247bf authored by Cyrill Gorcunov's avatar Cyrill Gorcunov Committed by Ingo Molnar

x86, apic: Slim down stack usage in early_init_lapic_mapping()

As far as I see there is no external poking of mp_lapic_addr in
this procedure which could lead to unpredited changes and
require local storage unit for it. Lets use it plain forward.
Signed-off-by: default avatarCyrill Gorcunov <gorcunov@openvz.org>
Cc: Yinghai Lu <yinghai@kernel.org>
LKML-Reference: <20090826171324.GC4548@lenovo>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent ffc43836
...@@ -1562,8 +1562,6 @@ no_apic: ...@@ -1562,8 +1562,6 @@ no_apic:
#ifdef CONFIG_X86_64 #ifdef CONFIG_X86_64
void __init early_init_lapic_mapping(void) void __init early_init_lapic_mapping(void)
{ {
unsigned long phys_addr;
/* /*
* If no local APIC can be found then go out * If no local APIC can be found then go out
* : it means there is no mpatable and MADT * : it means there is no mpatable and MADT
...@@ -1571,11 +1569,9 @@ void __init early_init_lapic_mapping(void) ...@@ -1571,11 +1569,9 @@ void __init early_init_lapic_mapping(void)
if (!smp_found_config) if (!smp_found_config)
return; return;
phys_addr = mp_lapic_addr; set_fixmap_nocache(FIX_APIC_BASE, mp_lapic_addr);
set_fixmap_nocache(FIX_APIC_BASE, phys_addr);
apic_printk(APIC_VERBOSE, "mapped APIC to %16lx (%16lx)\n", apic_printk(APIC_VERBOSE, "mapped APIC to %16lx (%16lx)\n",
APIC_BASE, phys_addr); APIC_BASE, mp_lapic_addr);
/* /*
* Fetch the APIC ID of the BSP in case we have a * Fetch the APIC ID of the BSP in case we have a
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment