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

From: Jack Steiner
Date: Fri Jul 18 2008 - 14:38:20 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>

---

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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/