From patchwork Fri Sep 12 16:18:35 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Arnez X-Patchwork-Id: 2813 Received: (qmail 20599 invoked by alias); 12 Sep 2014 16:18:42 -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 20589 invoked by uid 89); 12 Sep 2014 16:18:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.4 required=5.0 tests=AWL, BAYES_00, RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: e06smtp10.uk.ibm.com Received: from e06smtp10.uk.ibm.com (HELO e06smtp10.uk.ibm.com) (195.75.94.106) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Fri, 12 Sep 2014 16:18:40 +0000 Received: from /spool/local by e06smtp10.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 12 Sep 2014 17:18:37 +0100 Received: from d06dlp03.portsmouth.uk.ibm.com (9.149.20.15) by e06smtp10.uk.ibm.com (192.168.101.140) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Fri, 12 Sep 2014 17:18:36 +0100 Received: from b06cxnps4075.portsmouth.uk.ibm.com (d06relay12.portsmouth.uk.ibm.com [9.149.109.197]) by d06dlp03.portsmouth.uk.ibm.com (Postfix) with ESMTP id 5A0C81B08051 for ; Fri, 12 Sep 2014 17:19:40 +0100 (BST) Received: from d06av09.portsmouth.uk.ibm.com (d06av09.portsmouth.uk.ibm.com [9.149.37.250]) by b06cxnps4075.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id s8CGIZnw44433598 for ; Fri, 12 Sep 2014 16:18:35 GMT Received: from d06av09.portsmouth.uk.ibm.com (localhost [127.0.0.1]) by d06av09.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id s8CGIZZQ013942 for ; Fri, 12 Sep 2014 10:18:35 -0600 Received: from br87z6lw.boeblingen.de.ibm.com (dyn-9-152-212-196.boeblingen.de.ibm.com [9.152.212.196]) by d06av09.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVin) with ESMTP id s8CGIZFg013939 for ; Fri, 12 Sep 2014 10:18:35 -0600 From: Andreas Arnez To: gdb-patches@sourceware.org Subject: [PATCH 25/26] Linux targets: drop fall back to target method for 'make_corefile_notes' Date: Fri, 12 Sep 2014 18:18:35 +0200 Message-Id: <1410538715-30526-1-git-send-email-arnez@linux.vnet.ibm.com> In-Reply-To: <1410536396-25524-1-git-send-email-arnez@linux.vnet.ibm.com> References: <1410536396-25524-1-git-send-email-arnez@linux.vnet.ibm.com> X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 14091216-4966-0000-0000-0000015175A3 X-IsSubscribed: yes Now that all Linux targets use the regset iterator, the fall back to the deprecated target method is dropped. gdb/ChangeLog: * linux-nat.c (linux_nat_collect_thread_registers): Remove. (linux_nat_make_corefile_notes): Remove. (linux_target_install_ops): Do not set target method 'make_corefile_notes'. * linux-tdep.c (struct linux_corefile_thread_data): Remove field. (linux_corefile_thread_callback): Instead of args->collect, call linux_collect_thread_registers. (linux_make_corefile_notes): Remove 'collect' parameter. Return NULL unless there is a regset iterator. (linux_make_corefile_notes_1): Remove. (linux_init_abi): Replace reference to linux_make_corefile_notes_1 by linux_make_corefile_notes. * linux-tdep.h (linux_make_corefile_notes): Remove prototype. --- gdb/linux-nat.c | 57 -------------------------------------------------------- gdb/linux-tdep.c | 36 +++++++++++------------------------ gdb/linux-tdep.h | 3 --- 3 files changed, 11 insertions(+), 85 deletions(-) diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 0898442..643e9ab 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -4059,62 +4059,6 @@ linux_child_pid_to_exec_file (struct target_ops *self, int pid) return buf; } -/* Records the thread's register state for the corefile note - section. */ - -static char * -linux_nat_collect_thread_registers (const struct regcache *regcache, - ptid_t ptid, bfd *obfd, - char *note_data, int *note_size, - enum gdb_signal stop_signal) -{ - struct gdbarch *gdbarch = get_regcache_arch (regcache); - const struct regset *regset; - int core_regset_p; - gdb_gregset_t gregs; - gdb_fpregset_t fpregs; - - core_regset_p = gdbarch_regset_from_core_section_p (gdbarch); - - if (core_regset_p - && (regset = gdbarch_regset_from_core_section (gdbarch, ".reg", - sizeof (gregs))) - != NULL && regset->collect_regset != NULL) - regset->collect_regset (regset, regcache, -1, &gregs, sizeof (gregs)); - else - fill_gregset (regcache, &gregs, -1); - - note_data = (char *) elfcore_write_prstatus - (obfd, note_data, note_size, ptid_get_lwp (ptid), - gdb_signal_to_host (stop_signal), &gregs); - - if (core_regset_p - && (regset = gdbarch_regset_from_core_section (gdbarch, ".reg2", - sizeof (fpregs))) - != NULL && regset->collect_regset != NULL) - regset->collect_regset (regset, regcache, -1, &fpregs, sizeof (fpregs)); - else - fill_fpregset (regcache, &fpregs, -1); - - note_data = (char *) elfcore_write_prfpreg (obfd, note_data, note_size, - &fpregs, sizeof (fpregs)); - - return note_data; -} - -/* Fills the "to_make_corefile_note" target vector. Builds the note - section for a corefile, and returns it in a malloc buffer. */ - -static char * -linux_nat_make_corefile_notes (struct target_ops *self, - bfd *obfd, int *note_size) -{ - /* FIXME: uweigand/2011-10-06: Once all GNU/Linux architectures have been - converted to gdbarch_core_regset_sections, this function can go away. */ - return linux_make_corefile_notes (target_gdbarch (), obfd, note_size, - linux_nat_collect_thread_registers); -} - /* Implement the to_xfer_partial interface for memory reads using the /proc filesystem. Because we can use a single read() call for /proc, this can be much more efficient than banging away at PTRACE_PEEKTEXT, @@ -4511,7 +4455,6 @@ linux_target_install_ops (struct target_ops *t) t->to_post_startup_inferior = linux_child_post_startup_inferior; t->to_post_attach = linux_child_post_attach; t->to_follow_fork = linux_child_follow_fork; - t->to_make_corefile_notes = linux_nat_make_corefile_notes; super_xfer_partial = t->to_xfer_partial; t->to_xfer_partial = linux_xfer_partial; diff --git a/gdb/linux-tdep.c b/gdb/linux-tdep.c index fcba93b..7a76670 100644 --- a/gdb/linux-tdep.c +++ b/gdb/linux-tdep.c @@ -1211,7 +1211,6 @@ struct linux_corefile_thread_data char *note_data; int *note_size; enum gdb_signal stop_signal; - linux_collect_thread_registers_ftype collect; }; /* Called by gdbthread.c once per thread. Records the thread's @@ -1244,9 +1243,9 @@ linux_corefile_thread_callback (struct thread_info *info, void *data) old_chain = make_cleanup (xfree, siginfo_data); - args->note_data = args->collect (regcache, info->ptid, args->obfd, - args->note_data, args->note_size, - args->stop_signal); + args->note_data = linux_collect_thread_registers + (regcache, info->ptid, args->obfd, args->note_data, + args->note_size, args->stop_signal); /* Don't return anything if we got no register information above, such a core file is useless. */ @@ -1466,12 +1465,11 @@ linux_fill_prpsinfo (struct elf_internal_linux_prpsinfo *p) return 1; } -/* Fills the "to_make_corefile_note" target vector. Builds the note - section for a corefile, and returns it in a malloc buffer. */ +/* Build the note section for a corefile, and return it in a malloc + buffer. */ -char * -linux_make_corefile_notes (struct gdbarch *gdbarch, bfd *obfd, int *note_size, - linux_collect_thread_registers_ftype collect) +static char * +linux_make_corefile_notes (struct gdbarch *gdbarch, bfd *obfd, int *note_size) { struct linux_corefile_thread_data thread_args; struct elf_internal_linux_prpsinfo prpsinfo; @@ -1480,6 +1478,9 @@ linux_make_corefile_notes (struct gdbarch *gdbarch, bfd *obfd, int *note_size, int auxv_len; volatile struct gdb_exception e; + if (! gdbarch_iterate_over_regset_sections_p (gdbarch)) + return NULL; + if (linux_fill_prpsinfo (&prpsinfo)) { if (gdbarch_elfcore_write_linux_prpsinfo_p (gdbarch)) @@ -1514,7 +1515,6 @@ linux_make_corefile_notes (struct gdbarch *gdbarch, bfd *obfd, int *note_size, thread_args.note_data = note_data; thread_args.note_size = note_size; thread_args.stop_signal = find_stop_signal (); - thread_args.collect = collect; iterate_over_threads (linux_corefile_thread_callback, &thread_args); note_data = thread_args.note_data; if (!note_data) @@ -1545,20 +1545,6 @@ linux_make_corefile_notes (struct gdbarch *gdbarch, bfd *obfd, int *note_size, return note_data; } -static char * -linux_make_corefile_notes_1 (struct gdbarch *gdbarch, bfd *obfd, int *note_size) -{ - /* FIXME: uweigand/2011-10-06: Once all GNU/Linux architectures have been - converted to gdbarch_core_regset_sections, we no longer need to fall back - to the target method at this point. */ - - if (!gdbarch_iterate_over_regset_sections_p (gdbarch)) - return target_make_corefile_notes (obfd, note_size); - else - return linux_make_corefile_notes (gdbarch, obfd, note_size, - linux_collect_thread_registers); -} - /* Implementation of `gdbarch_gdb_signal_from_target', as defined in gdbarch.h. This function is not static because it is exported to other -tdep files. */ @@ -1820,7 +1806,7 @@ linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch) set_gdbarch_info_proc (gdbarch, linux_info_proc); set_gdbarch_core_info_proc (gdbarch, linux_core_info_proc); set_gdbarch_find_memory_regions (gdbarch, linux_find_memory_regions); - set_gdbarch_make_corefile_notes (gdbarch, linux_make_corefile_notes_1); + set_gdbarch_make_corefile_notes (gdbarch, linux_make_corefile_notes); set_gdbarch_has_shared_address_space (gdbarch, linux_has_shared_address_space); set_gdbarch_gdb_signal_from_target (gdbarch, diff --git a/gdb/linux-tdep.h b/gdb/linux-tdep.h index 610e2f0..2354603 100644 --- a/gdb/linux-tdep.h +++ b/gdb/linux-tdep.h @@ -29,9 +29,6 @@ typedef char *(*linux_collect_thread_registers_ftype) (const struct regcache *, bfd *, char *, int *, enum gdb_signal); -char *linux_make_corefile_notes (struct gdbarch *, bfd *, int *, - linux_collect_thread_registers_ftype); - struct type *linux_get_siginfo_type (struct gdbarch *); extern enum gdb_signal linux_gdb_signal_from_target (struct gdbarch *gdbarch,