It is actually safe to probe system_call() in entry_64.S, but only till
.Lsyscall_exit. To allow this, convert .Lsyscall_exit to a non-local
symbol __system_call() and blacklist that symbol, rather than
system_call().

Signed-off-by: Naveen N. Rao <naveen.n....@linux.vnet.ibm.com>
---
 arch/powerpc/kernel/entry_64.S | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/powerpc/kernel/entry_64.S b/arch/powerpc/kernel/entry_64.S
index 380361c0bb6a..e030ce34dd66 100644
--- a/arch/powerpc/kernel/entry_64.S
+++ b/arch/powerpc/kernel/entry_64.S
@@ -176,7 +176,7 @@ system_call:                        /* label this so stack 
traces look sane */
        mtctr   r12
        bctrl                   /* Call handler */
 
-.Lsyscall_exit:
+__system_call:
        std     r3,RESULT(r1)
        CURRENT_THREAD_INFO(r12, r1)
 
@@ -294,12 +294,12 @@ END_FTR_SECTION_IFSET(CPU_FTR_HAS_PPR)
        blt+    system_call
 
        /* Return code is already in r3 thanks to do_syscall_trace_enter() */
-       b       .Lsyscall_exit
+       b       __system_call
 
 
 .Lsyscall_enosys:
        li      r3,-ENOSYS
-       b       .Lsyscall_exit
+       b       __system_call
        
 .Lsyscall_exit_work:
 #ifdef CONFIG_PPC_BOOK3S
@@ -388,7 +388,7 @@ END_FTR_SECTION_IFSET(CPU_FTR_HAS_PPR)
        b       .       /* prevent speculative execution */
 #endif
 _ASM_NOKPROBE_SYMBOL(system_call_common);
-_ASM_NOKPROBE_SYMBOL(system_call);
+_ASM_NOKPROBE_SYMBOL(__system_call);
 
 /* Save non-volatile GPRs, if not already saved. */
 _GLOBAL(save_nvgprs)
@@ -413,38 +413,38 @@ _GLOBAL(save_nvgprs)
 _GLOBAL(ppc_fork)
        bl      save_nvgprs
        bl      sys_fork
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ppc_vfork)
        bl      save_nvgprs
        bl      sys_vfork
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ppc_clone)
        bl      save_nvgprs
        bl      sys_clone
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ppc32_swapcontext)
        bl      save_nvgprs
        bl      compat_sys_swapcontext
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ppc64_swapcontext)
        bl      save_nvgprs
        bl      sys_swapcontext
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ppc_switch_endian)
        bl      save_nvgprs
        bl      sys_switch_endian
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ret_from_fork)
        bl      schedule_tail
        REST_NVGPRS(r1)
        li      r3,0
-       b       .Lsyscall_exit
+       b       __system_call
 
 _GLOBAL(ret_from_kernel_thread)
        bl      schedule_tail
@@ -456,7 +456,7 @@ _GLOBAL(ret_from_kernel_thread)
 #endif
        blrl
        li      r3,0
-       b       .Lsyscall_exit
+       b       __system_call
 
 /*
  * This routine switches between two different tasks.  The process
-- 
2.12.1

Reply via email to