arm64/fp: Reindent fpsimd_save()

Currently all the active code in fpsimd_save() is inside a check for
TIF_FOREIGN_FPSTATE. Reduce the indentation level by changing to return
from the function if TIF_FOREIGN_FPSTATE is set.

Signed-off-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20211019172247.3045838-2-broonie@kernel.org
Signed-off-by: Will Deacon <will@kernel.org>

diff --git a/arch/arm64/kernel/fpsimd.c b/arch/arm64/kernel/fpsimd.c
index ff4962750b3d0f66327167eaf0686a8efdfc9a29..995f8801602bb2bbf0839c1317905a98e54d3f7a 100644 (file)
--- a/arch/arm64/kernel/fpsimd.c
+++ b/arch/arm64/kernel/fpsimd.c
@@ -308,24 +308,26 @@ static void fpsimd_save(void)
        WARN_ON(!system_supports_fpsimd());
        WARN_ON(!have_cpu_fpsimd_context());
 
-       if (!test_thread_flag(TIF_FOREIGN_FPSTATE)) {
-               if (IS_ENABLED(CONFIG_ARM64_SVE) &&
-                   test_thread_flag(TIF_SVE)) {
-                       if (WARN_ON(sve_get_vl() != last->sve_vl)) {
-                               /*
-                                * Can't save the user regs, so current would
-                                * re-enter user with corrupt state.
-                                * There's no way to recover, so kill it:
-                                */
-                               force_signal_inject(SIGKILL, SI_KERNEL, 0, 0);
-                               return;
-                       }
-
-                       sve_save_state((char *)last->sve_state +
-                                               sve_ffr_offset(last->sve_vl),
-                                      &last->st->fpsr);
-               } else
-                       fpsimd_save_state(last->st);
+       if (test_thread_flag(TIF_FOREIGN_FPSTATE))
+               return;
+
+       if (IS_ENABLED(CONFIG_ARM64_SVE) &&
+           test_thread_flag(TIF_SVE)) {
+               if (WARN_ON(sve_get_vl() != last->sve_vl)) {
+                       /*
+                        * Can't save the user regs, so current would
+                        * re-enter user with corrupt state.
+                        * There's no way to recover, so kill it:
+                        */
+                       force_signal_inject(SIGKILL, SI_KERNEL, 0, 0);
+                       return;
+               }
+
+               sve_save_state((char *)last->sve_state +
+                                       sve_ffr_offset(last->sve_vl),
+                              &last->st->fpsr);
+       } else {
+               fpsimd_save_state(last->st);
        }
 }