summary refs log tree commit diff
diff options
context:
space:
mode:
authorVasily Gorbik <gor@linux.ibm.com>2019-11-26 17:40:04 +0100
committerVasily Gorbik <gor@linux.ibm.com>2019-11-30 10:52:46 +0100
commitadcfb8cdc910bdd0b5d52d2ba88103af93dc43d3 (patch)
tree9fc78b22bcf4396d244d4a034df3fc3a2e9524db
parentd497b7ec836d2c900993f1c43b2ddff5f8a6b129 (diff)
downloadlinux-adcfb8cdc910bdd0b5d52d2ba88103af93dc43d3.tar.gz
s390/unwind: always inline get_stack_pointer
Always inline get_stack_pointer() to avoid potential problems
due to compiler inlining decisions, i.e. getting stack pointer of
get_stack_pointer() itself which is later reused.

Reviewed-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
-rw-r--r--arch/s390/include/asm/stacktrace.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/s390/include/asm/stacktrace.h b/arch/s390/include/asm/stacktrace.h
index 4f3dd1c86c0d..4725315a9cb1 100644
--- a/arch/s390/include/asm/stacktrace.h
+++ b/arch/s390/include/asm/stacktrace.h
@@ -33,8 +33,8 @@ static inline bool on_stack(struct stack_info *info,
 	return addr >= info->begin && addr + len <= info->end;
 }
 
-static inline unsigned long get_stack_pointer(struct task_struct *task,
-					      struct pt_regs *regs)
+static __always_inline unsigned long get_stack_pointer(struct task_struct *task,
+						       struct pt_regs *regs)
 {
 	if (regs)
 		return (unsigned long) kernel_stack_pointer(regs);