[PATCH] Remove safe_smp_processor_id()

And replace all users with ordinary smp_processor_id.  The function
was originally added to get some basic oops information out even
if the GS register was corrupted. However that didn't
work for some anymore because printk is needed to print the oops
and it uses smp_processor_id() already. Also GS register corruptions
are not particularly common anymore.

This also helps the Xen port which would otherwise need to
do this in a special way because it can't access the local APIC.

Cc: Chris Wright <chrisw@sous-sol.org>

Signed-off-by: Andi Kleen <ak@suse.de>

1 files changed, 0 insertions, 23 deletions

diff --git a/arch/x86_64/kernel/smp.c b/arch/x86_64/kernel/smp.c
index 2df988bbf312..4f67697f5036 100644
--- a/arch/x86_64/kernel/smp.c
+++ b/arch/x86_64/kernel/smp.c
@@ -522,26 +522,3 @@ asmlinkage void smp_call_function_interrupt(void)
         }
 }
 
-int safe_smp_processor_id(void)
-{
-        unsigned apicid, i;
-
-        if (disable_apic || !apic_mapped)
-                return 0;
-
-        apicid = hard_smp_processor_id();
-        if (apicid < NR_CPUS && x86_cpu_to_apicid[apicid] == apicid)
-                return apicid;
-
-        for (i = 0; i < NR_CPUS; ++i) {
-                if (x86_cpu_to_apicid[i] == apicid)
-                        return i;
-        }
-
-        /* No entries in x86_cpu_to_apicid?  Either no MPS|ACPI,
-         * or called too early.  Either way, we must be CPU 0. */
-        if (x86_cpu_to_apicid[0] == BAD_APICID)
-                return 0;
-
-        return 0; /* Should not happen */
-}