From: jbeulich@novell.com
Subject: Disallow all accesses to the local APIC page
Patch-mainline: n/a
References: bnc#191115
--- head.orig/arch/x86/include/asm/apic.h 2012-04-10 17:07:20.000000000 +0200
+++ head/arch/x86/include/asm/apic.h 2012-04-11 17:12:53.000000000 +0200
@@ -9,7 +9,9 @@
#include <asm/processor.h>
#include <asm/apicdef.h>
#include <linux/atomic.h>
+#ifndef CONFIG_XEN
#include <asm/fixmap.h>
+#endif
#include <asm/mpspec.h>
#include <asm/msr.h>
@@ -47,6 +49,7 @@ static inline void generic_apic_probe(vo
#ifdef CONFIG_X86_LOCAL_APIC
extern unsigned int apic_verbosity;
+#ifndef CONFIG_XEN
extern int local_apic_timer_c2_ok;
extern int disable_apic;
@@ -120,6 +123,8 @@ extern u64 native_apic_icr_read(void);
extern int x2apic_mode;
+#endif /* CONFIG_XEN */
+
#ifdef CONFIG_X86_X2APIC
/*
* Make previous memory operations globally visible before
@@ -291,16 +296,19 @@ static inline void disable_local_APIC(vo
struct apic {
char *name;
+#ifndef CONFIG_XEN
int (*probe)(void);
int (*acpi_madt_oem_check)(char *oem_id, char *oem_table_id);
int (*apic_id_valid)(int apicid);
int (*apic_id_registered)(void);
+#endif
u32 irq_delivery_mode;
u32 irq_dest_mode;
const struct cpumask *(*target_cpus)(void);
+#ifndef CONFIG_XEN
int disable_esr;
int dest_logical;
@@ -319,8 +327,10 @@ struct apic {
void (*setup_portio_remap)(void);
int (*check_phys_apicid_present)(int phys_apicid);
void (*enable_apic_mode)(void);
+#endif
int (*phys_pkg_id)(int cpuid_apic, int index_msb);
+#ifndef CONFIG_XEN
/*
* When one of the next two hooks returns 1 the apic
* is switched to this. Essentially they are additional
@@ -335,6 +345,7 @@ struct apic {
unsigned int (*cpu_mask_to_apicid)(const struct cpumask *cpumask);
unsigned int (*cpu_mask_to_apicid_and)(const struct cpumask *cpumask,
const struct cpumask *andmask);
+#endif
/* ipi */
void (*send_IPI_mask)(const struct cpumask *mask, int vector);
@@ -344,6 +355,7 @@ struct apic {
void (*send_IPI_all)(int vector);
void (*send_IPI_self)(int vector);
+#ifndef CONFIG_XEN
/* wakeup_secondary_cpu */
int (*wakeup_secondary_cpu)(int apicid, unsigned long start_eip);
@@ -383,6 +395,7 @@ struct apic {
*/
int (*x86_32_numa_cpu_node)(int cpu);
#endif
+#endif /* CONFIG_XEN */
};
/*
@@ -392,6 +405,8 @@ struct apic {
*/
extern struct apic *apic;
+#ifndef CONFIG_XEN
+
/*
* APIC drivers are probed based on how they are listed in the .apicdrivers
* section. So the order is important and enforced by the ordering
@@ -510,6 +525,7 @@ static inline void default_wait_for_init
extern void generic_bigsmp_probe(void);
+#endif /* CONFIG_XEN */
#ifdef CONFIG_X86_LOCAL_APIC
@@ -526,6 +542,8 @@ static inline const struct cpumask *defa
#endif
}
+#ifndef CONFIG_XEN
+
DECLARE_EARLY_PER_CPU(u16, x86_bios_cpu_apicid);
@@ -637,6 +655,8 @@ extern int default_cpu_present_to_apicid
extern int default_check_phys_apicid_present(int phys_apicid);
#endif
+#endif /* CONFIG_XEN */
+
#endif /* CONFIG_X86_LOCAL_APIC */
#endif /* _ASM_X86_APIC_H */
--- head.orig/arch/x86/include/asm/apicdef.h 2012-03-19 00:15:34.000000000 +0100
+++ head/arch/x86/include/asm/apicdef.h 2012-02-10 13:36:33.000000000 +0100
@@ -17,6 +17,8 @@
*/
#define IO_APIC_SLOT_SIZE 1024
+#ifndef CONFIG_XEN
+
#define APIC_ID 0x20
#define APIC_LVR 0x30
@@ -147,6 +149,16 @@
#define XAPIC_ENABLE (1UL << 11)
#define X2APIC_ENABLE (1UL << 10)
+#else /* CONFIG_XEN */
+
+enum {
+ APIC_DEST_ALLBUT = 0x1,
+ APIC_DEST_SELF,
+ APIC_DEST_ALLINC
+};
+
+#endif /* CONFIG_XEN */
+
#ifdef CONFIG_X86_32
# define MAX_IO_APICS 64
# define MAX_LOCAL_APIC 256
@@ -155,6 +167,8 @@
# define MAX_LOCAL_APIC 32768
#endif
+#ifndef CONFIG_XEN
+
/*
* All x86-64 systems are xAPIC compatible.
* In the following, "apicid" is a physical APIC ID.
@@ -425,6 +439,8 @@ struct local_apic {
#undef u32
+#endif /* CONFIG_XEN */
+
#ifdef CONFIG_X86_32
#define BAD_APICID 0xFFu
#else
--- head.orig/arch/x86/include/mach-xen/asm/fixmap.h 2012-02-09 12:32:50.000000000 +0100
+++ head/arch/x86/include/mach-xen/asm/fixmap.h 2011-09-09 15:56:55.000000000 +0200
@@ -17,7 +17,6 @@
#ifndef __ASSEMBLY__
#include <linux/kernel.h>
#include <asm/acpi.h>
-#include <asm/apicdef.h>
#include <asm/page.h>
#ifdef CONFIG_X86_32
#include <linux/threads.h>
@@ -86,10 +85,10 @@ enum fixed_addresses {
#ifdef CONFIG_PROVIDE_OHCI1394_DMA_INIT
FIX_OHCI1394_BASE,
#endif
+#ifndef CONFIG_XEN
#ifdef CONFIG_X86_LOCAL_APIC
FIX_APIC_BASE, /* local (CPU) APIC) -- required for SMP or not */
#endif
-#ifndef CONFIG_XEN
#ifdef CONFIG_X86_IO_APIC
FIX_IO_APIC_BASE_0,
FIX_IO_APIC_BASE_END = FIX_IO_APIC_BASE_0 + MAX_IO_APICS - 1,
--- head.orig/arch/x86/include/mach-xen/asm/smp.h 2012-02-09 12:32:50.000000000 +0100
+++ head/arch/x86/include/mach-xen/asm/smp.h 2011-03-03 16:50:42.000000000 +0100
@@ -15,7 +15,7 @@
# include <asm/io_apic.h>
# endif
#endif
-#include <asm/thread_info.h>
+#include <linux/thread_info.h>
#include <asm/cpumask.h>
#include <asm/cpufeature.h>
@@ -210,7 +210,7 @@ extern unsigned disabled_cpus __cpuinitd
#include <asm/smp-processor-id.h>
-#ifdef CONFIG_X86_LOCAL_APIC
+#if defined(CONFIG_X86_LOCAL_APIC) && !defined(CONFIG_XEN)
#ifndef CONFIG_X86_64
static inline int logical_smp_processor_id(void)
--- head.orig/arch/x86/kernel/acpi/boot.c 2011-04-13 14:08:57.000000000 +0200
+++ head/arch/x86/kernel/acpi/boot.c 2012-02-10 13:36:19.000000000 +0100
@@ -74,14 +74,14 @@ int acpi_sci_override_gsi __initdata;
int acpi_skip_timer_override __initdata;
int acpi_use_timer_override __initdata;
int acpi_fix_pin2_polarity __initdata;
-#else
-#define acpi_skip_timer_override 0
-#define acpi_fix_pin2_polarity 0
-#endif
#ifdef CONFIG_X86_LOCAL_APIC
static u64 acpi_lapic_addr __initdata = APIC_DEFAULT_PHYS_BASE;
#endif
+#else
+#define acpi_skip_timer_override 0
+#define acpi_fix_pin2_polarity 0
+#endif
#ifndef __HAVE_ARCH_CMPXCHG
#warning ACPI uses CMPXCHG, i486 and later hardware
@@ -187,6 +187,7 @@ static int __init acpi_parse_madt(struct
return -ENODEV;
}
+#ifndef CONFIG_XEN
if (madt->address) {
acpi_lapic_addr = (u64) madt->address;
@@ -194,7 +195,6 @@ static int __init acpi_parse_madt(struct
madt->address);
}
-#ifndef CONFIG_XEN
default_acpi_madt_oem_check(madt->header.oem_id,
madt->header.oem_table_id);
#endif
@@ -306,6 +306,7 @@ static int __init
acpi_parse_lapic_addr_ovr(struct acpi_subtable_header * header,
const unsigned long end)
{
+#ifndef CONFIG_XEN
struct acpi_madt_local_apic_override *lapic_addr_ovr = NULL;
lapic_addr_ovr = (struct acpi_madt_local_apic_override *)header;
@@ -314,6 +315,7 @@ acpi_parse_lapic_addr_ovr(struct acpi_su
return -EINVAL;
acpi_lapic_addr = lapic_addr_ovr->address;
+#endif
return 0;
}
--- head.orig/arch/x86/kernel/irq-xen.c 2012-04-11 13:26:23.000000000 +0200
+++ head/arch/x86/kernel/irq-xen.c 2012-02-10 13:36:13.000000000 +0100
@@ -18,9 +18,9 @@
#include <asm/mce.h>
#include <asm/hw_irq.h>
+#ifndef CONFIG_XEN
atomic_t irq_err_count;
-#ifndef CONFIG_XEN
/* Function pointer for generic interrupt vector handling */
void (*x86_platform_ipi_callback)(void) = NULL;
#endif
@@ -61,6 +61,7 @@ int arch_show_interrupts(struct seq_file
seq_printf(p, "%10u ", irq_stats(j)->__nmi_count);
seq_printf(p, " Non-maskable interrupts\n");
#ifdef CONFIG_X86_LOCAL_APIC
+#ifndef CONFIG_XEN
seq_printf(p, "%*s: ", prec, "LOC");
for_each_online_cpu(j)
seq_printf(p, "%10u ", irq_stats(j)->apic_timer_irqs);
@@ -74,6 +75,7 @@ int arch_show_interrupts(struct seq_file
for_each_online_cpu(j)
seq_printf(p, "%10u ", irq_stats(j)->apic_perf_irqs);
seq_printf(p, " Performance monitoring interrupts\n");
+#endif
seq_printf(p, "%*s: ", prec, "IWI");
for_each_online_cpu(j)
seq_printf(p, "%10u ", irq_stats(j)->apic_irq_work_irqs);
@@ -136,10 +138,12 @@ int arch_show_interrupts(struct seq_file
seq_printf(p, "%10u ", per_cpu(mce_poll_count, j));
seq_printf(p, " Machine check polls\n");
#endif
+#ifndef CONFIG_XEN
seq_printf(p, "%*s: %10u\n", prec, "ERR", atomic_read(&irq_err_count));
#if defined(CONFIG_X86_IO_APIC)
seq_printf(p, "%*s: %10u\n", prec, "MIS", atomic_read(&irq_mis_count));
#endif
+#endif
return 0;
}
@@ -185,12 +189,16 @@ u64 arch_irq_stat_cpu(unsigned int cpu)
u64 arch_irq_stat(void)
{
+#ifndef CONFIG_XEN
u64 sum = atomic_read(&irq_err_count);
#ifdef CONFIG_X86_IO_APIC
sum += atomic_read(&irq_mis_count);
#endif
return sum;
+#else
+ return 0;
+#endif
}
--- head.orig/drivers/xen/core/smpboot.c 2012-03-22 16:24:55.000000000 +0100
+++ head/drivers/xen/core/smpboot.c 2012-03-22 16:25:18.000000000 +0100
@@ -285,7 +285,7 @@ void __init smp_prepare_cpus(unsigned in
* Here we can be sure that there is an IO-APIC in the system. Let's
* go and set it up:
*/
- if (!skip_ioapic_setup && nr_ioapics)
+ if (cpu_has_apic && !skip_ioapic_setup && nr_ioapics)
setup_IO_APIC();
#endif
}