Commit 55eae7de authored by Thomas Gleixner's avatar Thomas Gleixner

x86/x2apic: Move code in conditional region

No point in having try_to_enable_x2apic() outside of the
CONFIG_X86_X2APIC section and having inline functions and more ifdefs
to deal with it. Move the code into the existing ifdef section and
remove the inline cruft.

Fixup the printk about not enabling interrupt remapping as suggested
by Boris.
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Cc: Jiang Liu <jiang.liu@linux.intel.com>
Cc: Joerg Roedel <joro@8bytes.org>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Borislav Petkov <bp@alien8.de>
Link: http://lkml.kernel.org/r/20150115211702.795388613@linutronix.deSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent d524165c
...@@ -186,27 +186,11 @@ static inline int x2apic_enabled(void) ...@@ -186,27 +186,11 @@ static inline int x2apic_enabled(void)
} }
#define x2apic_supported() (cpu_has_x2apic) #define x2apic_supported() (cpu_has_x2apic)
static inline void x2apic_force_phys(void)
{
x2apic_phys = 1;
}
#else #else
static inline void disable_x2apic(void) static inline void disable_x2apic(void) { }
{ static inline void check_x2apic(void) { }
} static inline void enable_x2apic(void) { }
static inline void check_x2apic(void) static inline int x2apic_enabled(void) { return 0; }
{
}
static inline void enable_x2apic(void)
{
}
static inline int x2apic_enabled(void)
{
return 0;
}
static inline void x2apic_force_phys(void)
{
}
#define x2apic_mode (0) #define x2apic_mode (0)
#define x2apic_supported() (0) #define x2apic_supported() (0)
......
...@@ -1561,41 +1561,8 @@ void enable_x2apic(void) ...@@ -1561,41 +1561,8 @@ void enable_x2apic(void)
} }
} }
void __init check_x2apic(void)
{
if (x2apic_enabled()) {
pr_info("x2apic enabled by BIOS, switching to x2apic ops\n");
x2apic_mode = 1;
}
}
#else /* CONFIG_X86_X2APIC */
static int __init validate_x2apic(void)
{
if (!apic_is_x2apic_enabled())
return 0;
/*
* Checkme: Can we simply turn off x2apic here instead of panic?
*/
panic("BIOS has enabled x2apic but kernel doesn't support x2apic, please disable x2apic in BIOS.\n");
}
early_initcall(validate_x2apic);
#endif /* !CONFIG_X86_X2APIC */
static int __init try_to_enable_IR(void)
{
#ifdef CONFIG_X86_IO_APIC
if (!x2apic_enabled() && skip_ioapic_setup) {
pr_info("Skipped enabling intr-remap because of skipping "
"io-apic setup\n");
return -1;
}
#endif
return irq_remapping_enable();
}
static __init void try_to_enable_x2apic(int ir_stat) static __init void try_to_enable_x2apic(int ir_stat)
{ {
#ifdef CONFIG_X86_X2APIC
if (!x2apic_supported()) if (!x2apic_supported())
return; return;
...@@ -1615,7 +1582,7 @@ static __init void try_to_enable_x2apic(int ir_stat) ...@@ -1615,7 +1582,7 @@ static __init void try_to_enable_x2apic(int ir_stat)
* without IR all CPUs can be addressed by IOAPIC/MSI * without IR all CPUs can be addressed by IOAPIC/MSI
* only in physical mode * only in physical mode
*/ */
x2apic_force_phys(); x2apic_phys = 1;
} }
if (!x2apic_mode) { if (!x2apic_mode) {
...@@ -1623,7 +1590,39 @@ static __init void try_to_enable_x2apic(int ir_stat) ...@@ -1623,7 +1590,39 @@ static __init void try_to_enable_x2apic(int ir_stat)
enable_x2apic(); enable_x2apic();
pr_info("Enabled x2apic\n"); pr_info("Enabled x2apic\n");
} }
}
void __init check_x2apic(void)
{
if (x2apic_enabled()) {
pr_info("x2apic: enabled by BIOS, switching to x2apic ops\n");
x2apic_mode = 1;
}
}
#else /* CONFIG_X86_X2APIC */
static int __init validate_x2apic(void)
{
if (!apic_is_x2apic_enabled())
return 0;
/*
* Checkme: Can we simply turn off x2apic here instead of panic?
*/
panic("BIOS has enabled x2apic but kernel doesn't support x2apic, please disable x2apic in BIOS.\n");
}
early_initcall(validate_x2apic);
static inline void try_to_enable_x2apic(int ir_stat) { }
#endif /* !CONFIG_X86_X2APIC */
static int __init try_to_enable_IR(void)
{
#ifdef CONFIG_X86_IO_APIC
if (!x2apic_enabled() && skip_ioapic_setup) {
pr_info("Not enabling interrupt remapping due to skipped IO-APIC setup\n");
return -1;
}
#endif #endif
return irq_remapping_enable();
} }
void __init enable_IR_x2apic(void) void __init enable_IR_x2apic(void)
......
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