diff options
author | Catalin Marinas <catalin.marinas@arm.com> | 2014-08-29 16:08:02 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-01-22 21:56:46 +0000 |
commit | 598e8b9c75705238d4e0c7ff8532ca4acfefdcde (patch) | |
tree | 0eb746ff3c2d4c68d2b3cb2402f5762e95d0d898 | |
parent | 078511edeb9c152d10e75233b535d730918aa213 (diff) |
arm64: Add brackets around user_stack_pointer()v3.10/topic/arm64-perf
Commit 5f888a1d33 (ARM64: perf: support dwarf unwinding in compat mode)
changes user_stack_pointer() to return the compat SP for 32-bit tasks
but without brackets around the whole definition, with possible issues
on the call sites (noticed with a subsequent fix for KSTK_ESP).
Fixes: 5f888a1d33c4 (ARM64: perf: support dwarf unwinding in compat mode)
Reported-by: Sudeep Holla <sudeep.holla@arm.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
(cherry picked from commit 2520d039728b2a3c5ae7f79fe2a0e9d182855b12)
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | arch/arm64/include/asm/ptrace.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm64/include/asm/ptrace.h b/arch/arm64/include/asm/ptrace.h index b8e7cf199ce6..47d0ab177f7e 100644 --- a/arch/arm64/include/asm/ptrace.h +++ b/arch/arm64/include/asm/ptrace.h @@ -132,7 +132,7 @@ struct pt_regs { (!((regs)->pstate & PSR_F_BIT)) #define user_stack_pointer(regs) \ - (!compat_user_mode(regs)) ? ((regs)->sp) : ((regs)->compat_sp) + (!compat_user_mode(regs) ? (regs)->sp : (regs)->compat_sp) /* * Are the current registers suitable for user mode? (used to maintain |