powerpc: Remove more redundant VSX save/tests
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Wed, 16 Aug 2017 06:01:18 +0000 (16:01 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 16 Aug 2017 12:35:04 +0000 (22:35 +1000)
__giveup_vsx/save_vsx are completely equivalent to testing MSR_FP
and MSR_VEC and calling the corresponding giveup/save function so
just remove the spurious VSX cases. Also add WARN_ONs checking that
we never have VSX enabled without the two other.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/process.c

index ff522bf75d53dd9ca65a986224e28ca0a654f7c4..cc5bae4bba7becc8245ce6a2ab135fc5b8721d71 100644 (file)
@@ -355,14 +355,6 @@ static void giveup_vsx(struct task_struct *tsk)
        msr_check_and_clear(MSR_FP|MSR_VEC|MSR_VSX);
 }
 
-static void save_vsx(struct task_struct *tsk)
-{
-       if (tsk->thread.regs->msr & MSR_FP)
-               save_fpu(tsk);
-       if (tsk->thread.regs->msr & MSR_VEC)
-               save_altivec(tsk);
-}
-
 void enable_kernel_vsx(void)
 {
        unsigned long cpumsr;
@@ -411,7 +403,6 @@ static int restore_vsx(struct task_struct *tsk)
 }
 #else
 static inline int restore_vsx(struct task_struct *tsk) { return 0; }
-static inline void save_vsx(struct task_struct *tsk) { }
 #endif /* CONFIG_VSX */
 
 #ifdef CONFIG_SPE
@@ -491,6 +482,8 @@ void giveup_all(struct task_struct *tsk)
        msr_check_and_set(msr_all_available);
        check_if_tm_restore_required(tsk);
 
+       WARN_ON((usermsr & MSR_VSX) && !((usermsr & MSR_FP) && (usermsr & MSR_VEC)));
+
 #ifdef CONFIG_PPC_FPU
        if (usermsr & MSR_FP)
                __giveup_fpu(tsk);
@@ -499,10 +492,6 @@ void giveup_all(struct task_struct *tsk)
        if (usermsr & MSR_VEC)
                __giveup_altivec(tsk);
 #endif
-#ifdef CONFIG_VSX
-       if (usermsr & MSR_VSX)
-               __giveup_vsx(tsk);
-#endif
 #ifdef CONFIG_SPE
        if (usermsr & MSR_SPE)
                __giveup_spe(tsk);
@@ -561,19 +550,13 @@ void save_all(struct task_struct *tsk)
 
        msr_check_and_set(msr_all_available);
 
-       /*
-        * Saving the way the register space is in hardware, save_vsx boils
-        * down to a save_fpu() and save_altivec()
-        */
-       if (usermsr & MSR_VSX) {
-               save_vsx(tsk);
-       } else {
-               if (usermsr & MSR_FP)
-                       save_fpu(tsk);
+       WARN_ON((usermsr & MSR_VSX) && !((usermsr & MSR_FP) && (usermsr & MSR_VEC)));
 
-               if (usermsr & MSR_VEC)
-                       save_altivec(tsk);
-       }
+       if (usermsr & MSR_FP)
+               save_fpu(tsk);
+
+       if (usermsr & MSR_VEC)
+               save_altivec(tsk);
 
        if (usermsr & MSR_SPE)
                __giveup_spe(tsk);