lkml.org 
[lkml]   [2022]   [Aug]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.19 149/158] arm64/sme: Dont flush SVE register state when allocating SME storage
    Date
    From: Mark Brown <broonie@kernel.org>

    commit 826a4fdd2ada9e5923c58bdd168f31a42e958ffc upstream.

    Currently when taking a SME access trap we allocate storage for the SVE
    register state in order to be able to handle storage of streaming mode SVE.
    Due to the original usage in a purely SVE context the SVE register state
    allocation this also flushes the register state for SVE if storage was
    already allocated but in the SME context this is not desirable. For a SME
    access trap to be taken the task must not be in streaming mode so either
    there already is SVE register state present for regular SVE mode which would
    be corrupted or the task does not have TIF_SVE and the flush is redundant.

    Fix this by adding a flag to sve_alloc() indicating if we are in a SVE
    context and need to flush the state. Freshly allocated storage is always
    zeroed either way.

    Fixes: 8bd7f91c03d8 ("arm64/sme: Implement traps and syscall handling for SME")
    Signed-off-by: Mark Brown <broonie@kernel.org>
    Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
    Link: https://lore.kernel.org/r/20220817182324.638214-4-broonie@kernel.org
    Signed-off-by: Will Deacon <will@kernel.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    arch/arm64/include/asm/fpsimd.h | 4 ++--
    arch/arm64/kernel/fpsimd.c | 10 ++++++----
    arch/arm64/kernel/ptrace.c | 6 +++---
    arch/arm64/kernel/signal.c | 2 +-
    4 files changed, 12 insertions(+), 10 deletions(-)

    --- a/arch/arm64/include/asm/fpsimd.h
    +++ b/arch/arm64/include/asm/fpsimd.h
    @@ -153,7 +153,7 @@ struct vl_info {

    #ifdef CONFIG_ARM64_SVE

    -extern void sve_alloc(struct task_struct *task);
    +extern void sve_alloc(struct task_struct *task, bool flush);
    extern void fpsimd_release_task(struct task_struct *task);
    extern void fpsimd_sync_to_sve(struct task_struct *task);
    extern void fpsimd_force_sync_to_sve(struct task_struct *task);
    @@ -256,7 +256,7 @@ size_t sve_state_size(struct task_struct

    #else /* ! CONFIG_ARM64_SVE */

    -static inline void sve_alloc(struct task_struct *task) { }
    +static inline void sve_alloc(struct task_struct *task, bool flush) { }
    static inline void fpsimd_release_task(struct task_struct *task) { }
    static inline void sve_sync_to_fpsimd(struct task_struct *task) { }
    static inline void sve_sync_from_fpsimd_zeropad(struct task_struct *task) { }
    --- a/arch/arm64/kernel/fpsimd.c
    +++ b/arch/arm64/kernel/fpsimd.c
    @@ -716,10 +716,12 @@ size_t sve_state_size(struct task_struct
    * do_sve_acc() case, there is no ABI requirement to hide stale data
    * written previously be task.
    */
    -void sve_alloc(struct task_struct *task)
    +void sve_alloc(struct task_struct *task, bool flush)
    {
    if (task->thread.sve_state) {
    - memset(task->thread.sve_state, 0, sve_state_size(task));
    + if (flush)
    + memset(task->thread.sve_state, 0,
    + sve_state_size(task));
    return;
    }

    @@ -1389,7 +1391,7 @@ void do_sve_acc(unsigned long esr, struc
    return;
    }

    - sve_alloc(current);
    + sve_alloc(current, true);
    if (!current->thread.sve_state) {
    force_sig(SIGKILL);
    return;
    @@ -1440,7 +1442,7 @@ void do_sme_acc(unsigned long esr, struc
    return;
    }

    - sve_alloc(current);
    + sve_alloc(current, false);
    sme_alloc(current);
    if (!current->thread.sve_state || !current->thread.za_state) {
    force_sig(SIGKILL);
    --- a/arch/arm64/kernel/ptrace.c
    +++ b/arch/arm64/kernel/ptrace.c
    @@ -882,7 +882,7 @@ static int sve_set_common(struct task_st
    * state and ensure there's storage.
    */
    if (target->thread.svcr != old_svcr)
    - sve_alloc(target);
    + sve_alloc(target, true);
    }

    /* Registers: FPSIMD-only case */
    @@ -912,7 +912,7 @@ static int sve_set_common(struct task_st
    goto out;
    }

    - sve_alloc(target);
    + sve_alloc(target, true);
    if (!target->thread.sve_state) {
    ret = -ENOMEM;
    clear_tsk_thread_flag(target, TIF_SVE);
    @@ -1082,7 +1082,7 @@ static int za_set(struct task_struct *ta

    /* Ensure there is some SVE storage for streaming mode */
    if (!target->thread.sve_state) {
    - sve_alloc(target);
    + sve_alloc(target, false);
    if (!target->thread.sve_state) {
    clear_thread_flag(TIF_SME);
    ret = -ENOMEM;
    --- a/arch/arm64/kernel/signal.c
    +++ b/arch/arm64/kernel/signal.c
    @@ -307,7 +307,7 @@ static int restore_sve_fpsimd_context(st
    fpsimd_flush_task_state(current);
    /* From now, fpsimd_thread_switch() won't touch thread.sve_state */

    - sve_alloc(current);
    + sve_alloc(current, true);
    if (!current->thread.sve_state) {
    clear_thread_flag(TIF_SVE);
    return -ENOMEM;

    \
     
     \ /
      Last update: 2022-08-29 13:35    [W:4.216 / U:0.108 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site