From patchwork Mon Dec 10 04:23:52 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Dmitry V. Levin" X-Patchwork-Id: 1010268 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 43CrJ437Nvz9s5c for ; Mon, 10 Dec 2018 15:51:16 +1100 (AEDT) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=altlinux.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 43CrJ41Wl1zDqcQ for ; Mon, 10 Dec 2018 15:51:16 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=altlinux.org X-Original-To: linuxppc-dev@lists.ozlabs.org Delivered-To: linuxppc-dev@lists.ozlabs.org Authentication-Results: lists.ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=altlinux.org (client-ip=194.107.17.57; helo=vmicros1.altlinux.org; envelope-from=ldv@altlinux.org; receiver=) Authentication-Results: lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=altlinux.org Received: from vmicros1.altlinux.org (vmicros1.altlinux.org [194.107.17.57]) by lists.ozlabs.org (Postfix) with ESMTP id 43CqhX3gYgzDqZS for ; Mon, 10 Dec 2018 15:23:55 +1100 (AEDT) Received: from mua.local.altlinux.org (mua.local.altlinux.org [192.168.1.14]) by vmicros1.altlinux.org (Postfix) with ESMTP id 68CEF72CC66; Mon, 10 Dec 2018 07:23:52 +0300 (MSK) Received: by mua.local.altlinux.org (Postfix, from userid 508) id 5497F7CEFE9; Mon, 10 Dec 2018 07:23:52 +0300 (MSK) Date: Mon, 10 Dec 2018 07:23:52 +0300 From: "Dmitry V. Levin" To: Oleg Nesterov , Andy Lutomirski Subject: [PATCH v5 00/25] ptrace: add PTRACE_GET_SYSCALL_INFO request Message-ID: <20181210042352.GA6092@altlinux.org> MIME-Version: 1.0 Content-Disposition: inline X-Mailman-Approved-At: Mon, 10 Dec 2018 15:49:38 +1100 X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-s390@vger.kernel.org, Rich Felker , linux-ia64@vger.kernel.org, linux-sh@vger.kernel.org, Alexey Brodkin , Heiko Carstens , linux-api@vger.kernel.org, "James E.J. Bottomley" , Max Filippov , Guo Ren , Ralf Baechle , linux-kselftest@vger.kernel.org, "H. Peter Anvin" , Breno Leitao , Russell King , linux-riscv@lists.infradead.org, Vincent Chen , Shuah Khan , Thomas Gleixner , Paul Mackerras , Jonas Bonn , Elvira Khabirova , sparclinux@vger.kernel.org, linux-arch@vger.kernel.org, Paul Moore , Yoshinori Sato , linux-xtensa@linux-xtensa.org, Helge Deller , x86@kernel.org, linux-kernel@vger.kernel.org, Eugene Syromyatnikov , linux-alpha@vger.kernel.org, Ingo Molnar , Geert Uytterhoeven , Catalin Marinas , James Hogan , Matt Turner , linux-snps-arc@lists.infradead.org, strace-devel@lists.strace.io, uclinux-h8-devel@lists.sourceforge.jp, Fenghua Yu , Albert Ou , Kees Cook , Arnd Bergmann , Jann Horn , Will Deacon , Jeff Dike , Mark Salter , linux-um@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, Aurelien Jacquiot , linux-c6x-dev@linux-c6x.org, linux-m68k@lists.linux-m68k.org, nios2-dev@lists.rocketboards.org, Borislav Petkov , Greentime Hu , Ley Foon Tan , Stafford Horne , Stefan Kristiansson , Guan Xuetao , linux-arm-kernel@lists.infradead.org, Richard Henderson , Chris Zankel , Michal Simek , Tony Luck , Will Drewry , linux-parisc@vger.kernel.org, Vineet Gupta , linux-mips@vger.kernel.org, Eric Paris , Richard Kuo , Paul Burton , linux-audit@redhat.com, Palmer Dabbelt , Richard Weinberger , Martin Schwidefsky , Ivan Kokshaysky , linux-hexagon@vger.kernel.org, "David S. Miller" , openrisc@lists.librecores.org Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: "Linuxppc-dev" PTRACE_GET_SYSCALL_INFO is a generic ptrace API that lets ptracer obtain details of the syscall the tracee is blocked in. There are two reasons for a special syscall-related ptrace request. Firstly, with the current ptrace API there are cases when ptracer cannot retrieve necessary information about syscalls. Some examples include: * The notorious int-0x80-from-64-bit-task issue. See [1] for details. In short, if a 64-bit task performs a syscall through int 0x80, its tracer has no reliable means to find out that the syscall was, in fact, a compat syscall, and misidentifies it. * Syscall-enter-stop and syscall-exit-stop look the same for the tracer. Common practice is to keep track of the sequence of ptrace-stops in order not to mix the two syscall-stops up. But it is not as simple as it looks; for example, strace had a (just recently fixed) long-standing bug where attaching strace to a tracee that is performing the execve system call led to the tracer identifying the following syscall-exit-stop as syscall-enter-stop, which messed up all the state tracking. * Since the introduction of commit 84d77d3f06e7e8dea057d10e8ec77ad71f721be3 ("ptrace: Don't allow accessing an undumpable mm"), both PTRACE_PEEKDATA and process_vm_readv become unavailable when the process dumpable flag is cleared. On such architectures as ia64 this results in all syscall arguments being unavailable for the tracer. Secondly, ptracers also have to support a lot of arch-specific code for obtaining information about the tracee. For some architectures, this requires a ptrace(PTRACE_PEEKUSER, ...) invocation for every syscall argument and return value. PTRACE_GET_SYSCALL_INFO returns the following structure: struct ptrace_syscall_info { __u8 op; /* PTRACE_SYSCALL_INFO_* */ __u8 __pad0[3]; __u32 arch; __u64 instruction_pointer; __u64 stack_pointer; __u64 frame_pointer; union { struct { __u64 nr; __u64 args[6]; } entry; struct { __s64 rval; __u8 is_error; __u8 __pad1[7]; } exit; struct { __u64 nr; __u64 args[6]; __u32 ret_data; __u8 __pad2[4]; } seccomp; }; }; The structure was chosen according to [2], except for the following changes: * seccomp substructure was added as a superset of entry substructure; * the type of nr field was changed from int to __u64 because syscall numbers are, as a practical matter, 64 bits; * stack_pointer and frame_pointer fields were added along with instruction_pointer field since they are readily available and can save the tracer from extra PTRACE_GETREGS/PTRACE_GETREGSET calls; * arch is always initialized to aid with tracing system calls such as execve(); * instruction_pointer, stack_pointer, and frame_pointer are always initialized so they could be easily obtained for non-syscall stops; * a boolean is_error field was added along with rval field, this way the tracer can more reliably distinguish a return value from an error value. strace has been ported to PTRACE_GET_SYSCALL_INFO, you can find it at [3] and [4]. [1] https://lore.kernel.org/lkml/CA+55aFzcSVmdDj9Lh_gdbz1OzHyEm6ZrGPBDAJnywm2LF_eVyg@mail.gmail.com/ [2] https://lore.kernel.org/lkml/CAObL_7GM0n80N7J_DFw_eQyfLyzq+sf4y2AvsCCV88Tb3AwEHA@mail.gmail.com/ [3] https://github.com/strace/strace/commits/ldv/PTRACE_GET_SYSCALL_INFO [4] https://gitlab.com/strace/strace/commits/ldv/PTRACE_GET_SYSCALL_INFO Notes: v5: * Merge separate series and patches into the single series. * Change PTRACE_EVENTMSG_SYSCALL_{ENTRY,EXIT} values as requested by Oleg. * Change struct ptrace_syscall_info: generalize instruction_pointer, stack_pointer, and frame_pointer fields by moving them from ptrace_syscall_info.{entry,seccomp} substructures to ptrace_syscall_info and initializing them for all stops. * Add PTRACE_SYSCALL_INFO_NONE, assign it to ptrace_syscall_info.op when not in a syscall stop, so e.g. "strace -i" could use the same PTRACE_SYSCALL_INFO_SECCOMP interface to obtain instruction_pointer when the tracee is in a signal stop. * Patch all remaining architectures to provide all necessary syscall_get_* functions. * Make available for all architectures: do not conditionalize on CONFIG_HAVE_ARCH_TRACEHOOK since all syscall_get_* functions are implemented on all architectures. * Add a test for PTRACE_GET_SYSCALL_INFO to selftests/ptrace. v4: * Do not introduce task_struct.ptrace_event, use child->last_siginfo->si_code instead. * Implement PTRACE_SYSCALL_INFO_SECCOMP and ptrace_syscall_info.seccomp support along with PTRACE_SYSCALL_INFO_{ENTRY,EXIT} and ptrace_syscall_info.{entry,exit}. v3: * Change struct ptrace_syscall_info. * Support PTRACE_EVENT_SECCOMP by adding ptrace_event to task_struct. * Add proper defines for ptrace_syscall_info.op values. * Rename PT_SYSCALL_IS_ENTERING and PT_SYSCALL_IS_EXITING to PTRACE_EVENTMSG_SYSCALL_ENTRY and PTRACE_EVENTMSG_SYSCALL_EXIT * and move them to uapi. v2: * Do not use task->ptrace. * Replace entry_info.is_compat with entry_info.arch, use syscall_get_arch(). * Use addr argument of sys_ptrace to get expected size of the struct; return full size of the struct. Dmitry V. Levin (23): alpha: define remaining syscall_get_* functions Move EM_ARCOMPACT and EM_ARCV2 to uapi/linux/elf-em.h arc: define syscall_get_arch() c6x: define syscall_get_arch() elf-em.h: add EM_CSKY csky: define syscall_get_arch() h8300: define remaining syscall_get_* functions Move EM_HEXAGON to uapi/linux/elf-em.h hexagon: define remaining syscall_get_* functions Move EM_NDS32 to uapi/linux/elf-em.h nds32: define syscall_get_arch() nios2: define syscall_get_arch() m68k: add asm/syscall.h mips: define syscall_get_error() parisc: define syscall_get_error() powerpc: define syscall_get_error() riscv: define syscall_get_arch() Move EM_XTENSA to uapi/linux/elf-em.h xtensa: define syscall_get_* functions Move EM_UNICORE to uapi/linux/elf-em.h unicore32: add asm/syscall.h syscall_get_arch: add "struct task_struct *" argument selftests/ptrace: add a test case for PTRACE_GET_SYSCALL_INFO Elvira Khabirova (2): powerpc/ptrace: replace ptrace_report_syscall() with a tracehook call ptrace: add PTRACE_GET_SYSCALL_INFO request arch/alpha/include/asm/syscall.h | 29 +- arch/arc/include/asm/elf.h | 6 +- arch/arc/include/asm/syscall.h | 11 + arch/arm/include/asm/syscall.h | 2 +- arch/arm64/include/asm/syscall.h | 4 +- arch/c6x/include/asm/syscall.h | 7 + arch/csky/include/asm/syscall.h | 7 + arch/h8300/include/asm/syscall.h | 18 ++ arch/hexagon/include/asm/elf.h | 6 +- arch/hexagon/include/asm/syscall.h | 20 ++ arch/ia64/include/asm/syscall.h | 2 +- arch/m68k/include/asm/syscall.h | 39 +++ arch/microblaze/include/asm/syscall.h | 2 +- arch/mips/include/asm/syscall.h | 12 +- arch/mips/kernel/ptrace.c | 2 +- arch/nds32/include/asm/elf.h | 3 +- arch/nds32/include/asm/syscall.h | 8 + arch/nios2/include/asm/syscall.h | 6 + arch/openrisc/include/asm/syscall.h | 2 +- arch/parisc/include/asm/syscall.h | 11 +- arch/powerpc/include/asm/syscall.h | 20 +- arch/powerpc/kernel/ptrace.c | 7 +- arch/riscv/include/asm/syscall.h | 10 + arch/s390/include/asm/syscall.h | 4 +- arch/sh/include/asm/syscall_32.h | 2 +- arch/sh/include/asm/syscall_64.h | 2 +- arch/sparc/include/asm/syscall.h | 5 +- arch/unicore32/include/asm/elf.h | 3 +- arch/unicore32/include/asm/syscall.h | 45 +++ arch/x86/include/asm/syscall.h | 8 +- arch/x86/um/asm/syscall.h | 2 +- arch/xtensa/include/asm/elf.h | 2 +- arch/xtensa/include/asm/syscall.h | 69 +++++ include/asm-generic/syscall.h | 5 +- include/linux/tracehook.h | 9 +- include/uapi/linux/audit.h | 16 ++ include/uapi/linux/elf-em.h | 8 + include/uapi/linux/ptrace.h | 39 +++ kernel/auditsc.c | 4 +- kernel/ptrace.c | 99 ++++++- kernel/seccomp.c | 4 +- tools/testing/selftests/ptrace/.gitignore | 1 + tools/testing/selftests/ptrace/Makefile | 2 +- .../selftests/ptrace/get_syscall_info.c | 272 ++++++++++++++++++ 44 files changed, 783 insertions(+), 52 deletions(-) create mode 100644 arch/m68k/include/asm/syscall.h create mode 100644 arch/unicore32/include/asm/syscall.h create mode 100644 tools/testing/selftests/ptrace/get_syscall_info.c