From patchwork Mon Jan 18 18:37:31 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Baldwin X-Patchwork-Id: 10431 Received: (qmail 11853 invoked by alias); 18 Jan 2016 18:47: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 11840 invoked by uid 89); 18 Jan 2016 18:47:08 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=1.5 required=5.0 tests=AWL, BAYES_50, SPF_HELO_PASS, SPF_SOFTFAIL autolearn=no version=3.3.2 spammy=270, 2.7.0, thread_info, core_pid_to_str X-HELO: bigwig.baldwin.cx Received: from bigwig.baldwin.cx (HELO bigwig.baldwin.cx) (96.47.65.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Mon, 18 Jan 2016 18:47:07 +0000 Received: from ralph.com (c-73-231-226-104.hsd1.ca.comcast.net [73.231.226.104]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id C66DAB9BA for ; Mon, 18 Jan 2016 13:38:27 -0500 (EST) From: John Baldwin To: gdb-patches@sourceware.org Subject: [PATCH v4 4/7] Display per-thread information for threads in FreeBSD cores. Date: Mon, 18 Jan 2016 10:37:31 -0800 Message-Id: <1453142254-20266-5-git-send-email-jhb@FreeBSD.org> In-Reply-To: <1453142254-20266-1-git-send-email-jhb@FreeBSD.org> References: <1453142254-20266-1-git-send-email-jhb@FreeBSD.org> X-IsSubscribed: yes Display the LWP ID of each thread in a FreeBSD core. Extract thread names from the per-thread THRMISC note. gdb/ChangeLog: * fbsd_tdep.c (fbsd_core_pid_to_str): New function. (fbsd_core_thread_name): New function. (fbsd_init_abi): Add "core_pid_to_str" gdbarch method. Add "core_thread_name" gdbarch method. --- gdb/ChangeLog | 7 ++++++ gdb/fbsd-tdep.c | 66 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 73 insertions(+) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index bbac503..8f0fd34 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,12 @@ 2016-01-16 John Baldwin + * fbsd_tdep.c (fbsd_core_pid_to_str): New function. + (fbsd_core_thread_name): New function. + (fbsd_init_abi): Add "core_pid_to_str" gdbarch method. + Add "core_thread_name" gdbarch method. + +2016-01-16 John Baldwin + * corelow.c (core_thread_name): New function. (init_core_ops): Use "core_thread_name" for the "to_thread_name" target op. diff --git a/gdb/fbsd-tdep.c b/gdb/fbsd-tdep.c index 0ef94d6..b4e7eff 100644 --- a/gdb/fbsd-tdep.c +++ b/gdb/fbsd-tdep.c @@ -28,6 +28,70 @@ #include "fbsd-tdep.h" +/* This is how we want PTIDs from core files to be printed. */ + +static char * +fbsd_core_pid_to_str (struct gdbarch *gdbarch, ptid_t ptid) +{ + static char buf[80]; + + if (ptid_get_lwp (ptid) != 0) + { + xsnprintf (buf, sizeof buf, "LWP %ld", ptid_get_lwp (ptid)); + return buf; + } + + return normal_pid_to_str (ptid); +} + +/* Extract the name assigned to a thread from a core. Returns the + string in a static buffer. */ + +static const char * +fbsd_core_thread_name (struct gdbarch *gdbarch, struct thread_info *thr) +{ + static char buf[80]; + struct bfd_section *section; + bfd_size_type size; + char sectionstr[32]; + + if (ptid_get_lwp (thr->ptid) != 0) + { + /* FreeBSD includes a NT_FREEBSD_THRMISC note for each thread + whose contents are defined by a "struct thrmisc" declared in + on FreeBSD. The per-thread name is stored as + a null-terminated string as the first member of the + structure. Rather than define the full structure here, just + extract the null-terminated name from the start of the + note. */ + xsnprintf (sectionstr, sizeof sectionstr, ".thrmisc/%ld", + ptid_get_lwp (thr->ptid)); + section = bfd_get_section_by_name (core_bfd, sectionstr); + if (section != NULL && bfd_section_size (core_bfd, section) > 0) + { + /* Truncate the name if it is longer than "buf". */ + size = bfd_section_size (core_bfd, section); + if (size > sizeof buf - 1) + size = sizeof buf - 1; + if (bfd_get_section_contents (core_bfd, section, buf, (file_ptr) 0, + size) + && buf[0] != '\0') + { + buf[size] = '\0'; + + /* Note that each thread will report the process command + as its thread name instead of an empty name if a name + has not been set explicitly. Return a NULL name in + that case. */ + if (strcmp (buf, elf_tdata (core_bfd)->core->program) != 0) + return buf; + } + } + } + + return NULL; +} + static int find_signalled_thread (struct thread_info *info, void *data) { @@ -132,5 +196,7 @@ fbsd_make_corefile_notes (struct gdbarch *gdbarch, bfd *obfd, int *note_size) void fbsd_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch) { + set_gdbarch_core_pid_to_str (gdbarch, fbsd_core_pid_to_str); + set_gdbarch_core_thread_name (gdbarch, fbsd_core_thread_name); set_gdbarch_make_corefile_notes (gdbarch, fbsd_make_corefile_notes); }