lkml.org 
[lkml]   [2019]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.20 281/352] arm64/sve: ptrace: Fix SVE_PT_REGS_OFFSET definition
    Date
    4.20-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    [ Upstream commit ee1b465b303591d3a04d403122bbc0d7026520fb ]

    SVE_PT_REGS_OFFSET is supposed to indicate the offset for skipping
    over the ptrace NT_ARM_SVE header (struct user_sve_header) to the
    start of the SVE register data proper.

    However, currently SVE_PT_REGS_OFFSET is defined in terms of struct
    sve_context, which is wrong: that structure describes the SVE
    header in the signal frame, not in the ptrace regset.

    This patch fixes the definition to use the ptrace header structure
    struct user_sve_header instead.

    By good fortune, the two structures are the same size anyway, so
    there is no functional or ABI change.

    Signed-off-by: Dave Martin <Dave.Martin@arm.com>
    Signed-off-by: Will Deacon <will.deacon@arm.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    arch/arm64/include/uapi/asm/ptrace.h | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/arch/arm64/include/uapi/asm/ptrace.h b/arch/arm64/include/uapi/asm/ptrace.h
    index a36227fdb084..65ef8b0fdb0e 100644
    --- a/arch/arm64/include/uapi/asm/ptrace.h
    +++ b/arch/arm64/include/uapi/asm/ptrace.h
    @@ -131,7 +131,7 @@ struct user_sve_header {

    /* Offset from the start of struct user_sve_header to the register data */
    #define SVE_PT_REGS_OFFSET \
    - ((sizeof(struct sve_context) + (SVE_VQ_BYTES - 1)) \
    + ((sizeof(struct user_sve_header) + (SVE_VQ_BYTES - 1)) \
    / SVE_VQ_BYTES * SVE_VQ_BYTES)

    /*
    --
    2.19.1


    \
     
     \ /
      Last update: 2019-02-11 16:54    [W:3.056 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site