From patchwork Tue Nov 6 19:31:09 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Baldwin X-Patchwork-Id: 30046 Received: (qmail 6822 invoked by alias); 6 Nov 2018 19:32:08 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 6778 invoked by uid 89); 6 Nov 2018 19:32:07 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-25.2 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, SPF_HELO_PASS, SPF_SOFTFAIL autolearn=ham version=3.3.2 spammy=7.0, conditionals, sk:archite X-HELO: mail.baldwin.cx Received: from bigwig.baldwin.cx (HELO mail.baldwin.cx) (96.47.65.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 06 Nov 2018 19:32:05 +0000 Received: from ralph.com (ralph.baldwin.cx [66.234.199.215]) by mail.baldwin.cx (Postfix) with ESMTPSA id E063610AFCD for ; Tue, 6 Nov 2018 14:32:02 -0500 (EST) From: John Baldwin To: gdb-patches@sourceware.org Subject: [PATCH] Update the conditionals in fbsd-nat.h so they are always honored. Date: Tue, 6 Nov 2018 11:31:09 -0800 Message-Id: <20181106193109.21548-1-jhb@FreeBSD.org> X-IsSubscribed: yes Not all of the architecture-specific FreeBSD target files were including the right headers to enable conditionals in fbsd-nat.h after the C++ target conversion. As a result, certain operations like 'info auxv' and 'p $_siginfo' were not working for some native targets (noticed on RISC-V). Fix this in a couple of ways: 1) Declare fbsd_nat_target::xfer_partial unconditionally and only use conditionals in the function body for individual target objects. Originally this function was only used to read the ELF auxiliary vector, so the entire function was conditional on a macro required for that object (KERN_AUXV_PROC). However, xfer_partial has since grown support for additional objects. Making the function unconditional avoids needing to add the right header to fbsd-nat.h and allows each target object to use independent requirements. This did require using a more explicit conditional test for the $_siginfo support. Removing the "outer" KERN_PROC_AUXV test enabled $_siginfo for all kernels with PT_LWPINFO, but some older kernels (FreeBSD 6.0) exposed PT_LWPINFO with a different siginfo format. Instead use an explicit test for when the current siginfo format was adopted (shipped in FreeBSD 7.0). This actually enables $_siginfo on a wider range of kernels as KERN_PROC_AUXV wasn't introduced until FreeBSD 9.1/10.0. 2) Include in fbsd-nat.h for the definition of TDP_RFPPWAIT that governs support for fork following. gdb/ChangeLog: * fbsd-nat.c [__FreeBSD_version >= 700009] (USE_SIGINFO): Macro defined. (union sigval32, struct siginfo32, fbsd_siginfo_size) (fbsd_convert_siginfo): Make conditional on USE_SIGINFO instead of KERN_PROC_AUXV and PT_LWPINFO. (fbsd_nat_target::xfer_partial): Define method unconditionally. Make TARGET_OBJECT_SIGNAL_INFO conditional on USE_SIGINFO. Make TARGET_OBJECT_AUXV conditional on KERN_PROC_AUXV. Make TARGET_OBJECT_FREEBSD_VMMAP and TARGET_OBJECT_FREEBSD_PS_STRINGS conditional on KERN_PROC_VMMAP and KERN_PROC_PS_STRINGS. * fbsd-nat.h: Include . (fbsd_nat_target::xfer_partial): Declare method unconditionally. --- gdb/ChangeLog | 16 ++++++++++++++++ gdb/fbsd-nat.c | 19 +++++++++++++++---- gdb/fbsd-nat.h | 3 +-- 3 files changed, 32 insertions(+), 6 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 68ecef7728..7b4b8d74c2 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,19 @@ +2018-11-06 John Baldwin + + * fbsd-nat.c [__FreeBSD_version >= 700009] (USE_SIGINFO): Macro + defined. + (union sigval32, struct siginfo32, fbsd_siginfo_size) + (fbsd_convert_siginfo): Make conditional on USE_SIGINFO instead + of KERN_PROC_AUXV and PT_LWPINFO. + (fbsd_nat_target::xfer_partial): Define method unconditionally. + Make TARGET_OBJECT_SIGNAL_INFO conditional on USE_SIGINFO. + Make TARGET_OBJECT_AUXV conditional on KERN_PROC_AUXV. + Make TARGET_OBJECT_FREEBSD_VMMAP and + TARGET_OBJECT_FREEBSD_PS_STRINGS conditional on KERN_PROC_VMMAP + and KERN_PROC_PS_STRINGS. + * fbsd-nat.h: Include . + (fbsd_nat_target::xfer_partial): Declare method unconditionally. + 2018-11-06 John Baldwin * riscv-fbsd-nat.c (getregs_supplies): Return true for diff --git a/gdb/fbsd-nat.c b/gdb/fbsd-nat.c index 107a729b3c..097322e1ec 100644 --- a/gdb/fbsd-nat.c +++ b/gdb/fbsd-nat.c @@ -532,9 +532,17 @@ fbsd_nat_target::info_proc (const char *args, enum info_proc_what what) return true; } -#ifdef KERN_PROC_AUXV +/* + * The current layout of siginfo_t on FreeBSD was adopted in SVN + * revision 153154 which shipped in FreeBSD versions 7.0 and later. + * Don't bother supporting the older layout on older kernels. The + * older format was also never used in core dump notes. + */ +#if __FreeBSD_version >= 700009 +#define USE_SIGINFO +#endif -#ifdef PT_LWPINFO +#ifdef USE_SIGINFO /* Return the size of siginfo for the current inferior. */ #ifdef __LP64__ @@ -676,7 +684,7 @@ fbsd_nat_target::xfer_partial (enum target_object object, switch (object) { -#ifdef PT_LWPINFO +#ifdef USE_SIGINFO case TARGET_OBJECT_SIGNAL_INFO: { struct ptrace_lwpinfo pl; @@ -708,6 +716,7 @@ fbsd_nat_target::xfer_partial (enum target_object object, return TARGET_XFER_OK; } #endif +#ifdef KERN_PROC_AUXV case TARGET_OBJECT_AUXV: { gdb::byte_vector buf_storage; @@ -749,6 +758,8 @@ fbsd_nat_target::xfer_partial (enum target_object object, } return TARGET_XFER_E_IO; } +#endif +#if defined(KERN_PROC_VMMAP) && defined(KERN_PROC_PS_STRINGS) case TARGET_OBJECT_FREEBSD_VMMAP: case TARGET_OBJECT_FREEBSD_PS_STRINGS: { @@ -804,13 +815,13 @@ fbsd_nat_target::xfer_partial (enum target_object object, *xfered_len = len; return TARGET_XFER_OK; } +#endif default: return inf_ptrace_target::xfer_partial (object, annex, readbuf, writebuf, offset, len, xfered_len); } } -#endif #ifdef PT_LWPINFO static int debug_fbsd_lwp; diff --git a/gdb/fbsd-nat.h b/gdb/fbsd-nat.h index 3e34600e15..1bc5d58928 100644 --- a/gdb/fbsd-nat.h +++ b/gdb/fbsd-nat.h @@ -21,6 +21,7 @@ #define FBSD_NAT_H #include "inf-ptrace.h" +#include #ifdef TRAP_BRKPT /* MIPS does not set si_code for SIGTRAP. sparc64 reports @@ -41,14 +42,12 @@ public: bool info_proc (const char *, enum info_proc_what) override; -#ifdef KERN_PROC_AUXV enum target_xfer_status xfer_partial (enum target_object object, const char *annex, gdb_byte *readbuf, const gdb_byte *writebuf, ULONGEST offset, ULONGEST len, ULONGEST *xfered_len) override; -#endif #ifdef PT_LWPINFO bool thread_alive (ptid_t ptid) override;