[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080718183800.GA26109@sgi.com>
Date: Fri, 18 Jul 2008 13:38:00 -0500
From: Jack Steiner <steiner@....com>
To: Ingo Molnar <mingo@...e.hu>
Cc: tglx@...utronix.de, linux-kernel@...r.kernel.org,
"H. Peter Anvin" <hpa@...or.com>
Subject: Re: [PATCH] - Earlier detection of system type (UV or non-UV)
A patch is coming that requires earlier detection of system type (UV or non-UV)
Move the logic that detects whether running on UV to acpi_boot_table_init().
Signed-off-by: Jack Steiner <steiner@....com>
---
V2 patch
- Added appropriate checks for CONFIG_ACPI to make it compile on
configs w/o ACPI.
- Eliminated check for x86_64.
arch/x86/kernel/acpi/boot.c | 1 +
arch/x86/kernel/genapic_64.c | 17 ++++++++++++-----
arch/x86/mm/srat_64.c | 2 +-
include/asm-x86/genapic_32.h | 1 +
include/asm-x86/genapic_64.h | 3 +++
5 files changed, 18 insertions(+), 6 deletions(-)
Index: linux/arch/x86/kernel/acpi/boot.c
===================================================================
--- linux.orig/arch/x86/kernel/acpi/boot.c 2008-07-18 13:15:03.000000000 -0500
+++ linux/arch/x86/kernel/acpi/boot.c 2008-07-18 13:22:07.000000000 -0500
@@ -1653,6 +1653,7 @@ int __init acpi_boot_table_init(void)
return error;
}
+ acpi_table_parse(ACPI_SIG_MADT, acpi_early_madt_oem_check);
acpi_table_parse(ACPI_SIG_BOOT, acpi_parse_sbf);
/*
Index: linux/arch/x86/kernel/genapic_64.c
===================================================================
--- linux.orig/arch/x86/kernel/genapic_64.c 2008-07-18 13:15:03.000000000 -0500
+++ linux/arch/x86/kernel/genapic_64.c 2008-07-18 13:17:36.000000000 -0500
@@ -66,16 +66,23 @@ void send_IPI_self(int vector)
__send_IPI_shortcut(APIC_DEST_SELF, vector, APIC_DEST_PHYSICAL);
}
-int __init acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+int __init acpi_early_madt_oem_check(struct acpi_table_header *table)
{
- if (!strcmp(oem_id, "SGI")) {
- if (!strcmp(oem_table_id, "UVL"))
+#ifdef CONFIG_ACPI
+ if (!strcmp(table->oem_id, "SGI")) {
+ if (!strcmp(table->oem_table_id, "UVL"))
uv_system_type = UV_LEGACY_APIC;
- else if (!strcmp(oem_table_id, "UVX"))
+ else if (!strcmp(table->oem_table_id, "UVX"))
uv_system_type = UV_X2APIC;
- else if (!strcmp(oem_table_id, "UVH"))
+ else if (!strcmp(table->oem_table_id, "UVH"))
uv_system_type = UV_NON_UNIQUE_APIC;
}
+#endif
+ return 0;
+}
+
+int __init acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+{
return 0;
}
Index: linux/arch/x86/mm/srat_64.c
===================================================================
--- linux.orig/arch/x86/mm/srat_64.c 2008-07-18 13:15:03.000000000 -0500
+++ linux/arch/x86/mm/srat_64.c 2008-07-18 13:15:07.000000000 -0500
@@ -138,7 +138,7 @@ acpi_numa_processor_affinity_init(struct
return;
}
- if (is_uv_system())
+ if (get_uv_system_type() >= UV_X2APIC)
apic_id = (pa->apic_id << 8) | pa->local_sapic_eid;
else
apic_id = pa->apic_id;
Index: linux/include/asm-x86/genapic_32.h
===================================================================
--- linux.orig/include/asm-x86/genapic_32.h 2008-07-16 17:45:26.000000000 -0500
+++ linux/include/asm-x86/genapic_32.h 2008-07-18 13:22:38.000000000 -0500
@@ -118,6 +118,7 @@ enum uv_system_type {UV_NONE, UV_LEGACY_
#define get_uv_system_type() UV_NONE
#define is_uv_system() 0
#define uv_wakeup_secondary(a, b) 1
+#define acpi_early_madt_oem_check(a) 0
#endif
Index: linux/include/asm-x86/genapic_64.h
===================================================================
--- linux.orig/include/asm-x86/genapic_64.h 2008-07-18 13:15:03.000000000 -0500
+++ linux/include/asm-x86/genapic_64.h 2008-07-18 13:15:07.000000000 -0500
@@ -39,6 +39,9 @@ enum uv_system_type {UV_NONE, UV_LEGACY_
extern enum uv_system_type get_uv_system_type(void);
extern int is_uv_system(void);
+struct acpi_table_header;
+extern int acpi_early_madt_oem_check(struct acpi_table_header *);
+
extern struct genapic apic_x2apic_uv_x;
DECLARE_PER_CPU(int, x2apic_extra_bits);
extern void uv_cpu_init(void);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists