Merge tag 'for-3.16' of git://git.linaro.org/people/ard.biesheuvel/linux-arm into upstream

FPSIMD register bank context switching and crypto algorithms
optimisations for arm64 from Ard Biesheuvel.

* tag 'for-3.16' of git://git.linaro.org/people/ard.biesheuvel/linux-arm:
  arm64/crypto: AES-ECB/CBC/CTR/XTS using ARMv8 NEON and Crypto Extensions
  arm64: pull in <asm/simd.h> from asm-generic
  arm64/crypto: AES in CCM mode using ARMv8 Crypto Extensions
  arm64/crypto: AES using ARMv8 Crypto Extensions
  arm64/crypto: GHASH secure hash using ARMv8 Crypto Extensions
  arm64/crypto: SHA-224/SHA-256 using ARMv8 Crypto Extensions
  arm64/crypto: SHA-1 using ARMv8 Crypto Extensions
  arm64: add support for kernel mode NEON in interrupt context
  arm64: defer reloading a task's FPSIMD state to userland resume
  arm64: add abstractions for FPSIMD state manipulation
  asm-generic: allow generic unaligned access if the arch supports it

Conflicts:
	arch/arm64/include/asm/thread_info.h

7 files changed, 205 insertions, 33 deletions

diff --git a/arch/arm64/kernel/entry-fpsimd.S b/arch/arm64/kernel/entry-fpsimd.S
index 6a27cd6dbfa6..d358ccacfc00 100644
--- a/arch/arm64/kernel/entry-fpsimd.S
+++ b/arch/arm64/kernel/entry-fpsimd.S
@@ -41,3 +41,27 @@ ENTRY(fpsimd_load_state)
         fpsimd_restore x0, 8
         ret
 ENDPROC(fpsimd_load_state)
+
+#ifdef CONFIG_KERNEL_MODE_NEON
+
+/*
+ * Save the bottom n FP registers.
+ *
+ * x0 - pointer to struct fpsimd_partial_state
+ */
+ENTRY(fpsimd_save_partial_state)
+        fpsimd_save_partial x0, 1, 8, 9
+        ret
+ENDPROC(fpsimd_load_partial_state)
+
+/*
+ * Load the bottom n FP registers.
+ *
+ * x0 - pointer to struct fpsimd_partial_state
+ */
+ENTRY(fpsimd_load_partial_state)
+        fpsimd_restore_partial x0, 8, 9
+        ret
+ENDPROC(fpsimd_load_partial_state)
+
+#endif
diff --git a/arch/arm64/kernel/entry.S b/arch/arm64/kernel/entry.S
index a670d0a98c89..bf017f4ffb4f 100644
--- a/arch/arm64/kernel/entry.S
+++ b/arch/arm64/kernel/entry.S
@@ -562,7 +562,7 @@ fast_work_pending:
         str     x0, [sp, #S_X0]                 // returned x0
 work_pending:
         tbnz    x1, #TIF_NEED_RESCHED, work_resched
-        /* TIF_SIGPENDING or TIF_NOTIFY_RESUME case */
+        /* TIF_SIGPENDING, TIF_NOTIFY_RESUME or TIF_FOREIGN_FPSTATE case */
         ldr     x2, [sp, #S_PSTATE]
         mov     x0, sp                          // 'regs'
         tst     x2, #PSR_MODE_MASK              // user mode regs?
diff --git a/arch/arm64/kernel/fpsimd.c b/arch/arm64/kernel/fpsimd.c
index 4aef42a04bdc..ad8aebb1cdef 100644
--- a/arch/arm64/kernel/fpsimd.c
+++ b/arch/arm64/kernel/fpsimd.c
@@ -35,6 +35,60 @@
 #define FPEXC_IDF       (1 << 7)
 
 /*
+ * In order to reduce the number of times the FPSIMD state is needlessly saved
+ * and restored, we need to keep track of two things:
+ * (a) for each task, we need to remember which CPU was the last one to have
+ *     the task's FPSIMD state loaded into its FPSIMD registers;
+ * (b) for each CPU, we need to remember which task's userland FPSIMD state has
+ *     been loaded into its FPSIMD registers most recently, or whether it has
+ *     been used to perform kernel mode NEON in the meantime.
+ *
+ * For (a), we add a 'cpu' field to struct fpsimd_state, which gets updated to
+ * the id of the current CPU everytime the state is loaded onto a CPU. For (b),
+ * we add the per-cpu variable 'fpsimd_last_state' (below), which contains the
+ * address of the userland FPSIMD state of the task that was loaded onto the CPU
+ * the most recently, or NULL if kernel mode NEON has been performed after that.
+ *
+ * With this in place, we no longer have to restore the next FPSIMD state right
+ * when switching between tasks. Instead, we can defer this check to userland
+ * resume, at which time we verify whether the CPU's fpsimd_last_state and the
+ * task's fpsimd_state.cpu are still mutually in sync. If this is the case, we
+ * can omit the FPSIMD restore.
+ *
+ * As an optimization, we use the thread_info flag TIF_FOREIGN_FPSTATE to
+ * indicate whether or not the userland FPSIMD state of the current task is
+ * present in the registers. The flag is set unless the FPSIMD registers of this
+ * CPU currently contain the most recent userland FPSIMD state of the current
+ * task.
+ *
+ * For a certain task, the sequence may look something like this:
+ * - the task gets scheduled in; if both the task's fpsimd_state.cpu field
+ *   contains the id of the current CPU, and the CPU's fpsimd_last_state per-cpu
+ *   variable points to the task's fpsimd_state, the TIF_FOREIGN_FPSTATE flag is
+ *   cleared, otherwise it is set;
+ *
+ * - the task returns to userland; if TIF_FOREIGN_FPSTATE is set, the task's
+ *   userland FPSIMD state is copied from memory to the registers, the task's
+ *   fpsimd_state.cpu field is set to the id of the current CPU, the current
+ *   CPU's fpsimd_last_state pointer is set to this task's fpsimd_state and the
+ *   TIF_FOREIGN_FPSTATE flag is cleared;
+ *
+ * - the task executes an ordinary syscall; upon return to userland, the
+ *   TIF_FOREIGN_FPSTATE flag will still be cleared, so no FPSIMD state is
+ *   restored;
+ *
+ * - the task executes a syscall which executes some NEON instructions; this is
+ *   preceded by a call to kernel_neon_begin(), which copies the task's FPSIMD
+ *   register contents to memory, clears the fpsimd_last_state per-cpu variable
+ *   and sets the TIF_FOREIGN_FPSTATE flag;
+ *
+ * - the task gets preempted after kernel_neon_end() is called; as we have not
+ *   returned from the 2nd syscall yet, TIF_FOREIGN_FPSTATE is still set so
+ *   whatever is in the FPSIMD registers is not saved to memory, but discarded.
+ */
+static DEFINE_PER_CPU(struct fpsimd_state *, fpsimd_last_state);
+
+/*
  * Trapped FP/ASIMD access.
  */
 void do_fpsimd_acc(unsigned int esr, struct pt_regs *regs)
@@ -72,43 +126,137 @@ void do_fpsimd_exc(unsigned int esr, struct pt_regs *regs)
 
 void fpsimd_thread_switch(struct task_struct *next)
 {
-        /* check if not kernel threads */
-        if (current->mm)
+        /*
+         * Save the current FPSIMD state to memory, but only if whatever is in
+         * the registers is in fact the most recent userland FPSIMD state of
+         * 'current'.
+         */
+        if (current->mm && !test_thread_flag(TIF_FOREIGN_FPSTATE))
                 fpsimd_save_state(&current->thread.fpsimd_state);
-        if (next->mm)
-                fpsimd_load_state(&next->thread.fpsimd_state);
+
+        if (next->mm) {
+                /*
+                 * If we are switching to a task whose most recent userland
+                 * FPSIMD state is already in the registers of *this* cpu,
+                 * we can skip loading the state from memory. Otherwise, set
+                 * the TIF_FOREIGN_FPSTATE flag so the state will be loaded
+                 * upon the next return to userland.
+                 */
+                struct fpsimd_state *st = &next->thread.fpsimd_state;
+
+                if (__this_cpu_read(fpsimd_last_state) == st
+                    && st->cpu == smp_processor_id())
+                        clear_ti_thread_flag(task_thread_info(next),
+                                             TIF_FOREIGN_FPSTATE);
+                else
+                        set_ti_thread_flag(task_thread_info(next),
+                                           TIF_FOREIGN_FPSTATE);
+        }
 }
 
 void fpsimd_flush_thread(void)
 {
-        preempt_disable();
         memset(&current->thread.fpsimd_state, 0, sizeof(struct fpsimd_state));
-        fpsimd_load_state(&current->thread.fpsimd_state);
+        set_thread_flag(TIF_FOREIGN_FPSTATE);
+}
+
+/*
+ * Save the userland FPSIMD state of 'current' to memory, but only if the state
+ * currently held in the registers does in fact belong to 'current'
+ */
+void fpsimd_preserve_current_state(void)
+{
+        preempt_disable();
+        if (!test_thread_flag(TIF_FOREIGN_FPSTATE))
+                fpsimd_save_state(&current->thread.fpsimd_state);
+        preempt_enable();
+}
+
+/*
+ * Load the userland FPSIMD state of 'current' from memory, but only if the
+ * FPSIMD state already held in the registers is /not/ the most recent FPSIMD
+ * state of 'current'
+ */
+void fpsimd_restore_current_state(void)
+{
+        preempt_disable();
+        if (test_and_clear_thread_flag(TIF_FOREIGN_FPSTATE)) {
+                struct fpsimd_state *st = &current->thread.fpsimd_state;
+
+                fpsimd_load_state(st);
+                this_cpu_write(fpsimd_last_state, st);
+                st->cpu = smp_processor_id();
+        }
+        preempt_enable();
+}
+
+/*
+ * Load an updated userland FPSIMD state for 'current' from memory and set the
+ * flag that indicates that the FPSIMD register contents are the most recent
+ * FPSIMD state of 'current'
+ */
+void fpsimd_update_current_state(struct fpsimd_state *state)
+{
+        preempt_disable();
+        fpsimd_load_state(state);
+        if (test_and_clear_thread_flag(TIF_FOREIGN_FPSTATE)) {
+                struct fpsimd_state *st = &current->thread.fpsimd_state;
+
+                this_cpu_write(fpsimd_last_state, st);
+                st->cpu = smp_processor_id();
+        }
         preempt_enable();
 }
 
+/*
+ * Invalidate live CPU copies of task t's FPSIMD state
+ */
+void fpsimd_flush_task_state(struct task_struct *t)
+{
+        t->thread.fpsimd_state.cpu = NR_CPUS;
+}
+
 #ifdef CONFIG_KERNEL_MODE_NEON
 
+static DEFINE_PER_CPU(struct fpsimd_partial_state, hardirq_fpsimdstate);
+static DEFINE_PER_CPU(struct fpsimd_partial_state, softirq_fpsimdstate);
+
 /*
  * Kernel-side NEON support functions
  */
-void kernel_neon_begin(void)
+void kernel_neon_begin_partial(u32 num_regs)
 {
-        /* Avoid using the NEON in interrupt context */
-        BUG_ON(in_interrupt());
-        preempt_disable();
+        if (in_interrupt()) {
+                struct fpsimd_partial_state *s = this_cpu_ptr(
+                        in_irq() ? &hardirq_fpsimdstate : &softirq_fpsimdstate);
 
-        if (current->mm)
-                fpsimd_save_state(&current->thread.fpsimd_state);
+                BUG_ON(num_regs > 32);
+                fpsimd_save_partial_state(s, roundup(num_regs, 2));
+        } else {
+                /*
+                 * Save the userland FPSIMD state if we have one and if we
+                 * haven't done so already. Clear fpsimd_last_state to indicate
+                 * that there is no longer userland FPSIMD state in the
+                 * registers.
+                 */
+                preempt_disable();
+                if (current->mm &&
+                    !test_and_set_thread_flag(TIF_FOREIGN_FPSTATE))
+                        fpsimd_save_state(&current->thread.fpsimd_state);
+                this_cpu_write(fpsimd_last_state, NULL);
+        }
 }
-EXPORT_SYMBOL(kernel_neon_begin);
+EXPORT_SYMBOL(kernel_neon_begin_partial);
 
 void kernel_neon_end(void)
 {
-        if (current->mm)
-                fpsimd_load_state(&current->thread.fpsimd_state);
-
-        preempt_enable();
+        if (in_interrupt()) {
+                struct fpsimd_partial_state *s = this_cpu_ptr(
+                        in_irq() ? &hardirq_fpsimdstate : &softirq_fpsimdstate);
+                fpsimd_load_partial_state(s);
+        } else {
+                preempt_enable();
+        }
 }
 EXPORT_SYMBOL(kernel_neon_end);
 
@@ -120,12 +268,12 @@ static int fpsimd_cpu_pm_notifier(struct notifier_block *self,
 {
         switch (cmd) {
         case CPU_PM_ENTER:
-                if (current->mm)
+                if (current->mm && !test_thread_flag(TIF_FOREIGN_FPSTATE))
                         fpsimd_save_state(&current->thread.fpsimd_state);
                 break;
         case CPU_PM_EXIT:
                 if (current->mm)
-                        fpsimd_load_state(&current->thread.fpsimd_state);
+                        set_thread_flag(TIF_FOREIGN_FPSTATE);
                 break;
         case CPU_PM_ENTER_FAILED:
         default:
diff --git a/arch/arm64/kernel/process.c b/arch/arm64/kernel/process.c
index d04eb871cb0e..9f2d6020b6c2 100644
--- a/arch/arm64/kernel/process.c
+++ b/arch/arm64/kernel/process.c
@@ -206,7 +206,7 @@ void release_thread(struct task_struct *dead_task)
 
 int arch_dup_task_struct(struct task_struct *dst, struct task_struct *src)
 {
-        fpsimd_save_state(&current->thread.fpsimd_state);
+        fpsimd_preserve_current_state();
         *dst = *src;
         return 0;
 }
diff --git a/arch/arm64/kernel/ptrace.c b/arch/arm64/kernel/ptrace.c
index 4b58e812cf67..32d52d3b079c 100644
--- a/arch/arm64/kernel/ptrace.c
+++ b/arch/arm64/kernel/ptrace.c
@@ -518,6 +518,7 @@ static int fpr_set(struct task_struct *target, const struct user_regset *regset,
                 return ret;
 
         target->thread.fpsimd_state.user_fpsimd = newstate;
+        fpsimd_flush_task_state(target);
         return ret;
 }
 
@@ -765,6 +766,7 @@ static int compat_vfp_set(struct task_struct *target,
                 uregs->fpcr = fpscr & VFP_FPSCR_CTRL_MASK;
         }
 
+        fpsimd_flush_task_state(target);
         return ret;
 }
 
diff --git a/arch/arm64/kernel/signal.c b/arch/arm64/kernel/signal.c
index 2ba72a11629f..6357b9c6c90e 100644
--- a/arch/arm64/kernel/signal.c
+++ b/arch/arm64/kernel/signal.c
@@ -51,7 +51,7 @@ static int preserve_fpsimd_context(struct fpsimd_context __user *ctx)
         int err;
 
         /* dump the hardware registers to the fpsimd_state structure */
-        fpsimd_save_state(fpsimd);
+        fpsimd_preserve_current_state();
 
         /* copy the FP and status/control registers */
         err = __copy_to_user(ctx->vregs, fpsimd->vregs, sizeof(fpsimd->vregs));
@@ -86,11 +86,8 @@ static int restore_fpsimd_context(struct fpsimd_context __user *ctx)
         __get_user_error(fpsimd.fpcr, &ctx->fpcr, err);
 
         /* load the hardware registers from the fpsimd_state structure */
-        if (!err) {
-                preempt_disable();
-                fpsimd_load_state(&fpsimd);
-                preempt_enable();
-        }
+        if (!err)
+                fpsimd_update_current_state(&fpsimd);
 
         return err ? -EFAULT : 0;
 }
@@ -433,4 +430,8 @@ asmlinkage void do_notify_resume(struct pt_regs *regs,
                 clear_thread_flag(TIF_NOTIFY_RESUME);
                 tracehook_notify_resume(regs);
         }
+
+        if (thread_flags & _TIF_FOREIGN_FPSTATE)
+                fpsimd_restore_current_state();
+
 }
diff --git a/arch/arm64/kernel/signal32.c b/arch/arm64/kernel/signal32.c
index 050c1c2af777..3491c638f172 100644
--- a/arch/arm64/kernel/signal32.c
+++ b/arch/arm64/kernel/signal32.c
@@ -222,7 +222,7 @@ static int compat_preserve_vfp_context(struct compat_vfp_sigframe __user *frame)
          * Note that this also saves V16-31, which aren't visible
          * in AArch32.
          */
-        fpsimd_save_state(fpsimd);
+        fpsimd_preserve_current_state();
 
         /* Place structure header on the stack */
         __put_user_error(magic, &frame->magic, err);
@@ -285,11 +285,8 @@ static int compat_restore_vfp_context(struct compat_vfp_sigframe __user *frame)
          * We don't need to touch the exception register, so
          * reload the hardware state.
          */
-        if (!err) {
-                preempt_disable();
-                fpsimd_load_state(&fpsimd);
-                preempt_enable();
-        }
+        if (!err)
+                fpsimd_update_current_state(&fpsimd);
 
         return err ? -EFAULT : 0;
 }