powerpc/booke: Don't set DABR on 64-bit BookE, use DAC1 instead
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Tue, 8 Sep 2009 14:16:58 +0000 (14:16 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Fri, 11 Sep 2009 01:27:59 +0000 (11:27 +1000)
Also remove a duplicate setting of it in the context switch path
on BookE.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/process.c

index 678ff13..0a32164 100644 (file)
@@ -284,14 +284,13 @@ int set_dabr(unsigned long dabr)
                return ppc_md.set_dabr(dabr);
 
        /* XXX should we have a CPU_FTR_HAS_DABR ? */
-#if defined(CONFIG_PPC64) || defined(CONFIG_6xx)
-       mtspr(SPRN_DABR, dabr);
-#endif
-
 #if defined(CONFIG_BOOKE)
        mtspr(SPRN_DAC1, dabr);
+#elif defined(CONFIG_PPC_BOOK3S)
+       mtspr(SPRN_DABR, dabr);
 #endif
 
+
        return 0;
 }
 
@@ -372,15 +371,16 @@ struct task_struct *__switch_to(struct task_struct *prev,
 
 #endif /* CONFIG_SMP */
 
-       if (unlikely(__get_cpu_var(current_dabr) != new->thread.dabr))
-               set_dabr(new->thread.dabr);
-
 #if defined(CONFIG_BOOKE)
        /* If new thread DAC (HW breakpoint) is the same then leave it */
        if (new->thread.dabr)
                set_dabr(new->thread.dabr);
+#else
+       if (unlikely(__get_cpu_var(current_dabr) != new->thread.dabr))
+               set_dabr(new->thread.dabr);
 #endif
 
+
        new_thread = &new->thread;
        old_thread = &current->thread;