Re: [PATCH] - Earlier detection of system type (UV or non-UV)

From: Jack Steiner
Date: Fri Jul 18 2008 - 17:28:30 EST


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@xxxxxxx>

---

Argggg. The previous was clearly wrong (but it does compile on the version of
gcc that I'm using).

This patch should be ok.



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 | 2 +-
include/asm-x86/genapic_64.h | 3 +++
5 files changed, 18 insertions(+), 7 deletions(-)

Index: linux/arch/x86/kernel/acpi/boot.c
===================================================================
--- linux.orig/arch/x86/kernel/acpi/boot.c 2008-07-18 15:33:00.000000000 -0500
+++ linux/arch/x86/kernel/acpi/boot.c 2008-07-18 15:33:04.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 15:33:00.000000000 -0500
+++ linux/arch/x86/kernel/genapic_64.c 2008-07-18 15:33:04.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 15:33:00.000000000 -0500
+++ linux/arch/x86/mm/srat_64.c 2008-07-18 15:33:04.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-18 15:33:00.000000000 -0500
+++ linux/include/asm-x86/genapic_32.h 2008-07-18 16:22:55.000000000 -0500
@@ -118,6 +118,6 @@ 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
-
+static inline int __init acpi_early_madt_oem_check(struct acpi_table_header *table) { return 0; }

#endif
Index: linux/include/asm-x86/genapic_64.h
===================================================================
--- linux.orig/include/asm-x86/genapic_64.h 2008-07-18 15:33:00.000000000 -0500
+++ linux/include/asm-x86/genapic_64.h 2008-07-18 15:33:04.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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/