x86: clean up apic->acpi_madt_oem_check methods
authorIngo Molnar <mingo@elte.hu>
Wed, 28 Jan 2009 02:43:47 +0000 (03:43 +0100)
committerIngo Molnar <mingo@elte.hu>
Wed, 28 Jan 2009 22:20:12 +0000 (23:20 +0100)
Impact: refactor code

x86 subarchitectures each defined a "acpi_madt_oem_check()" method,
which could be an inline function, or an extern, or a static function,
and which was also the name of a genapic field.

Untangle this namespace spaghetti by setting ->acpi_madt_oem_check()
to NULL on those subarchitectures that have no detection quirks,
and rename the other ones (summit, es7000) that do.

Also change default_acpi_madt_oem_check() to handle NULL entries,
and clean its control flow up as well.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
13 files changed:
arch/x86/include/asm/es7000/mpparse.h
arch/x86/include/asm/genapic.h
arch/x86/include/asm/mach-default/mach_mpparse.h
arch/x86/include/asm/mach-generic/mach_mpparse.h
arch/x86/include/asm/summit/mpparse.h
arch/x86/kernel/acpi/boot.c
arch/x86/kernel/genapic_64.c
arch/x86/mach-generic/bigsmp.c
arch/x86/mach-generic/default.c
arch/x86/mach-generic/es7000.c
arch/x86/mach-generic/numaq.c
arch/x86/mach-generic/probe.c
arch/x86/mach-generic/summit.c

index c1629b090ec23ea5aaf6102d10d6dda54fcd85e1..30692c4ae859616068c51b2ca232331e0b02ab35 100644 (file)
@@ -9,7 +9,7 @@ extern void unmap_unisys_acpi_oem_table(unsigned long oem_addr);
 extern void setup_unisys(void);
 
 #ifndef CONFIG_X86_GENERICARCH
-extern int acpi_madt_oem_check(char *oem_id, char *oem_table_id);
+extern int default_acpi_madt_oem_check(char *oem_id, char *oem_table_id);
 extern int mps_oem_check(struct mpc_table *mpc, char *oem, char *productid);
 #endif
 
index 26c5e824a7176b89500d254a3d38ce9dca1e8ea2..108abdf6953b1455db92b4cb733d3b97f6b7fee4 100644 (file)
@@ -103,7 +103,7 @@ extern struct genapic apic_flat;
 extern struct genapic apic_physflat;
 extern struct genapic apic_x2apic_cluster;
 extern struct genapic apic_x2apic_phys;
-extern int acpi_madt_oem_check(char *, char *);
+extern int default_acpi_madt_oem_check(char *, char *);
 
 extern void apic_send_IPI_self(int vector);
 
index c70a263d68cd4add7b79a0cbe604f7f6073959e4..8fa01770ba6249f65b2e39c1692f5f1591ada345 100644 (file)
@@ -8,7 +8,7 @@ mps_oem_check(struct mpc_table *mpc, char *oem, char *productid)
 }
 
 /* Hook from generic ACPI tables.c */
-static inline int acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+static inline int default_acpi_madt_oem_check(char *oem_id, char *oem_table_id)
 {
        return 0;
 }
index 9444ab8dca94fd38cc2f687abb02d21998f36fa1..f497d96c76bb264b9ea770ca3196ecefe876dcd2 100644 (file)
@@ -4,6 +4,6 @@
 
 extern int mps_oem_check(struct mpc_table *, char *, char *);
 
-extern int acpi_madt_oem_check(char *, char *);
+extern int default_acpi_madt_oem_check(char *, char *);
 
 #endif /* _ASM_X86_MACH_GENERIC_MACH_MPPARSE_H */
index 380e86c02363ba0ccea1070d2f8235787ce0a41e..555ed8238e94116c506c2ca3179ba19a75bd2611 100644 (file)
@@ -27,7 +27,7 @@ static inline int mps_oem_check(struct mpc_table *mpc, char *oem,
 }
 
 /* Hook from generic ACPI tables.c */
-static inline int acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+static inline int summit_acpi_madt_oem_check(char *oem_id, char *oem_table_id)
 {
        if (!strncmp(oem_id, "IBM", 3) &&
            (!strncmp(oem_table_id, "SERVIGIL", 8)
index 4cb5964f149979deb409d85b32d1019ed18c25b6..314fe0dddef42afa1df22b4e6108457c2c926d63 100644 (file)
@@ -239,7 +239,8 @@ static int __init acpi_parse_madt(struct acpi_table_header *table)
                       madt->address);
        }
 
-       acpi_madt_oem_check(madt->header.oem_id, madt->header.oem_table_id);
+       default_acpi_madt_oem_check(madt->header.oem_id,
+                                   madt->header.oem_table_id);
 
        return 0;
 }
index 2b986389a24fd35b8336f99bc0cb2a7de650ef27..060945b8eec4bbf9993831ae676f5cf15aec8abf 100644 (file)
@@ -68,7 +68,7 @@ void apic_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 default_acpi_madt_oem_check(char *oem_id, char *oem_table_id)
 {
        int i;
 
index 13e82bc4dae626a65218bee16d62a8ebb0fd2915..22c3608b80dd079b5a62aa04939973584894de2d 100644 (file)
@@ -61,7 +61,7 @@ struct genapic apic_bigsmp = {
 
        .name                           = "bigsmp",
        .probe                          = probe_bigsmp,
-       .acpi_madt_oem_check            = acpi_madt_oem_check,
+       .acpi_madt_oem_check            = NULL,
        .apic_id_registered             = apic_id_registered,
 
        .int_delivery_mode              = INT_DELIVERY_MODE,
index d5fec764fb405f1b0101257e0ce179087f5ba7c0..cfec3494a967f8368e368ae63a4b715c4ac8d700 100644 (file)
@@ -28,7 +28,7 @@ struct genapic apic_default = {
 
        .name                           = "default",
        .probe                          = probe_default,
-       .acpi_madt_oem_check            = acpi_madt_oem_check,
+       .acpi_madt_oem_check            = NULL,
        .apic_id_registered             = apic_id_registered,
 
        .int_delivery_mode              = INT_DELIVERY_MODE,
index 4a404ea5f928a4bf9f821ca00e202d6323245e3d..23fe6f1c9691f09aedef7b3ed2c7b63791b29e8d 100644 (file)
@@ -57,7 +57,7 @@ mps_oem_check(struct mpc_table *mpc, char *oem, char *productid)
 
 #ifdef CONFIG_ACPI
 /* Hook from generic ACPI tables.c */
-static int __init acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+static int __init es7000_acpi_madt_oem_check(char *oem_id, char *oem_table_id)
 {
        unsigned long oem_addr = 0;
        int check_dsdt;
@@ -81,7 +81,7 @@ static int __init acpi_madt_oem_check(char *oem_id, char *oem_table_id)
        return ret;
 }
 #else
-static int __init acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+static int __init es7000_acpi_madt_oem_check(char *oem_id, char *oem_table_id)
 {
        return 0;
 }
@@ -104,7 +104,7 @@ struct genapic apic_es7000 = {
 
        .name                           = "es7000",
        .probe                          = probe_es7000,
-       .acpi_madt_oem_check            = acpi_madt_oem_check,
+       .acpi_madt_oem_check            = es7000_acpi_madt_oem_check,
        .apic_id_registered             = apic_id_registered,
 
        .int_delivery_mode              = INT_DELIVERY_MODE,
index fa486ca49c0a37706c22d217382be9135d2ce46c..9691b4e1654dd7f84826b5588a4553b47577b1f2 100644 (file)
@@ -31,12 +31,6 @@ static int probe_numaq(void)
        return found_numaq;
 }
 
-/* Hook from generic ACPI tables.c */
-static int acpi_madt_oem_check(char *oem_id, char *oem_table_id)
-{
-       return 0;
-}
-
 static void vector_allocation_domain(int cpu, cpumask_t *retmask)
 {
        /* Careful. Some cpus do not strictly honor the set of cpus
@@ -54,7 +48,7 @@ struct genapic apic_numaq = {
 
        .name                           = "NUMAQ",
        .probe                          = probe_numaq,
-       .acpi_madt_oem_check            = acpi_madt_oem_check,
+       .acpi_madt_oem_check            = NULL,
        .apic_id_registered             = apic_id_registered,
 
        .int_delivery_mode              = INT_DELIVERY_MODE,
index 82bf0f520fb6442bc7db5205577cba8c7cf62882..a21e2b1a701110209a767767f397e420c9dd357f 100644 (file)
@@ -128,20 +128,24 @@ int __init mps_oem_check(struct mpc_table *mpc, char *oem, char *productid)
        return 0;
 }
 
-int __init acpi_madt_oem_check(char *oem_id, char *oem_table_id)
+int __init default_acpi_madt_oem_check(char *oem_id, char *oem_table_id)
 {
        int i;
+
        for (i = 0; apic_probe[i]; ++i) {
-               if (apic_probe[i]->acpi_madt_oem_check(oem_id, oem_table_id)) {
-                       if (!cmdline_apic) {
-                               apic = apic_probe[i];
-                               if (x86_quirks->update_genapic)
-                                       x86_quirks->update_genapic();
-                               printk(KERN_INFO "Switched to APIC driver `%s'.\n",
-                                      apic->name);
-                       }
-                       return 1;
+               if (!apic_probe[i]->acpi_madt_oem_check)
+                       continue;
+               if (!apic_probe[i]->acpi_madt_oem_check(oem_id, oem_table_id))
+                       continue;
+
+               if (!cmdline_apic) {
+                       apic = apic_probe[i];
+                       if (x86_quirks->update_genapic)
+                               x86_quirks->update_genapic();
+                       printk(KERN_INFO "Switched to APIC driver `%s'.\n",
+                              apic->name);
                }
+               return 1;
        }
        return 0;
 }
index 479c1d4097790758267486b48c75aee5fa9dd5f5..0eea9fbb2a50a6d7b650e9b8a34242d281156296 100644 (file)
@@ -41,7 +41,7 @@ struct genapic apic_summit = {
 
        .name                           = "summit",
        .probe                          = probe_summit,
-       .acpi_madt_oem_check            = acpi_madt_oem_check,
+       .acpi_madt_oem_check            = summit_acpi_madt_oem_check,
        .apic_id_registered             = apic_id_registered,
 
        .int_delivery_mode              = INT_DELIVERY_MODE,