Skip to content

Commit 62d131e

Browse files
James Hogangregkh
authored andcommitted
MIPS: Negate error syscall return in trace
commit 4f32a39d49b25eaa66d2420f1f03d371ea4cd906 upstream. The sys_exit trace event takes a single return value for the system call, which MIPS passes the value of the $v0 (result) register, however MIPS returns positive error codes in $v0 with $a3 specifying that $v0 contains an error code. As a result erroring system calls are traced returning positive error numbers that can't always be distinguished from success. Use regs_return_value() to negate the error code if $a3 is set. Fixes: 1d7bf99 ("MIPS: ftrace: Add support for syscall tracepoints.") Signed-off-by: James Hogan <james.hogan@imgtec.com> Cc: Steven Rostedt <rostedt@goodmis.org> Cc: Ingo Molnar <mingo@redhat.com> Cc: linux-mips@linux-mips.org Patchwork: https://patchwork.linux-mips.org/patch/16651/ Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org> Signed-off-by: Ralf Baechle <ralf@linux-mips.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 4d9f109 commit 62d131e

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

arch/mips/kernel/ptrace.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -927,7 +927,7 @@ asmlinkage void syscall_trace_leave(struct pt_regs *regs)
927927
audit_syscall_exit(regs);
928928

929929
if (unlikely(test_thread_flag(TIF_SYSCALL_TRACEPOINT)))
930-
trace_sys_exit(regs, regs->regs[2]);
930+
trace_sys_exit(regs, regs_return_value(regs));
931931

932932
if (test_thread_flag(TIF_SYSCALL_TRACE))
933933
tracehook_report_syscall_exit(regs, 0);

0 commit comments

Comments
 (0)