s390/ftrace: use HAVE_FUNCTION_GRAPH_RET_ADDR_PTR

Make the call chain more reliable by tagging the ftrace stack entries
with the stack pointer that is associated with the return address.

Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

diff --git a/arch/s390/include/asm/ftrace.h b/arch/s390/include/asm/ftrace.h
index 6e0ed03387854c26a3852724cbbae74943843ea6..68d362f8d6c17bafa112b08837dffd0dc8c47ad3 100644 (file)
--- a/arch/s390/include/asm/ftrace.h
+++ b/arch/s390/include/asm/ftrace.h
@@ -11,6 +11,8 @@
 #define MCOUNT_RETURN_FIXUP    18
 #endif
 
+#define HAVE_FUNCTION_GRAPH_RET_ADDR_PTR
+
 #ifndef __ASSEMBLY__
 
 #ifdef CONFIG_CC_IS_CLANG

diff --git a/arch/s390/kernel/entry.h b/arch/s390/kernel/entry.h
index c3816ae108b085afca4a9326ac2d0eb9c3f3b6a2..20420c2b8a146964e2018edc25fd1393d2823f45 100644 (file)
--- a/arch/s390/kernel/entry.h
+++ b/arch/s390/kernel/entry.h
@@ -65,7 +65,7 @@ int setup_profiling_timer(unsigned int multiplier);
 void __init time_init(void);
 int pfn_is_nosave(unsigned long);
 void s390_early_resume(void);
-unsigned long prepare_ftrace_return(unsigned long parent, unsigned long ip);
+unsigned long prepare_ftrace_return(unsigned long parent, unsigned long sp, unsigned long ip);
 
 struct s390_mmap_arg_struct;
 struct fadvise64_64_args;

diff --git a/arch/s390/kernel/ftrace.c b/arch/s390/kernel/ftrace.c
index 39b13d71a8fe6dc2979e8a8320ae62b675b8ee9b..1bb85f60c0dd515efcda8a3f8812aa5ebc3e3018 100644 (file)
--- a/arch/s390/kernel/ftrace.c
+++ b/arch/s390/kernel/ftrace.c
@@ -201,17 +201,18 @@ device_initcall(ftrace_plt_init);
  * Hook the return address and push it in the stack of return addresses
  * in current thread info.
  */
-unsigned long prepare_ftrace_return(unsigned long parent, unsigned long ip)
+unsigned long prepare_ftrace_return(unsigned long ra, unsigned long sp,
+                                   unsigned long ip)
 {
        if (unlikely(ftrace_graph_is_dead()))
                goto out;
        if (unlikely(atomic_read(&current->tracing_graph_pause)))
                goto out;
        ip -= MCOUNT_INSN_SIZE;
-       if (!function_graph_enter(parent, ip, 0, NULL))
-               parent = (unsigned long) return_to_handler;
+       if (!function_graph_enter(ra, ip, 0, (void *) sp))
+               ra = (unsigned long) return_to_handler;
 out:
-       return parent;
+       return ra;
 }
 NOKPROBE_SYMBOL(prepare_ftrace_return);
 

diff --git a/arch/s390/kernel/mcount.S b/arch/s390/kernel/mcount.S
index 09ae6da0aaa5fb01d1c4976fa2df2964c465cc85..9e1660a6b9db6e49be93e4b145bb639dee52d773 100644 (file)
--- a/arch/s390/kernel/mcount.S
+++ b/arch/s390/kernel/mcount.S
@@ -65,8 +65,8 @@ ENTRY(ftrace_caller)
        .globl ftrace_graph_caller
 ftrace_graph_caller:
        j       ftrace_graph_caller_end
-       lg      %r2,(STACK_PTREGS_GPRS+14*8)(%r15)
-       lg      %r3,(STACK_PTREGS_PSW+8)(%r15)
+       lmg     %r2,%r3,(STACK_PTREGS_GPRS+14*8)(%r15)
+       lg      %r4,(STACK_PTREGS_PSW+8)(%r15)
        brasl   %r14,prepare_ftrace_return
        stg     %r2,(STACK_PTREGS_GPRS+14*8)(%r15)
 ftrace_graph_caller_end:

diff --git a/arch/s390/kernel/unwind_bc.c b/arch/s390/kernel/unwind_bc.c
index cf5a630f3aa9878748b44ab9300b8059c671d4e1..57fd4e902f1f49eaf3125c29482f0e32ab8cce23 100644 (file)
--- a/arch/s390/kernel/unwind_bc.c
+++ b/arch/s390/kernel/unwind_bc.c
@@ -84,7 +84,7 @@ bool unwind_next_frame(struct unwind_state *state)
        /* Decode any ftrace redirection */
        if (ip == (unsigned long) return_to_handler)
                ip = ftrace_graph_ret_addr(state->task, &state->graph_idx,
-                                          ip, NULL);
+                                          ip, (void *) sp);
 #endif
 
        /* Update unwind state */