diff options
author | Andreas Arnez <arnez@linux.vnet.ibm.com> | 2014-09-10 12:42:14 +0000 |
---|---|---|
committer | Andreas Krebbel <krebbel@linux.vnet.ibm.com> | 2014-09-30 09:14:39 +0200 |
commit | f968fe80b09ddc88538db2250b367fc0074739fa (patch) | |
tree | 074ff20465431c4dc63eebb7d95a5dbcdb6a26c8 /gdb/linux-tdep.c | |
parent | 174ad59a8eff83977a367d2cbb358c00fc470ee9 (diff) | |
download | gdb-f968fe80b09ddc88538db2250b367fc0074739fa.zip gdb-f968fe80b09ddc88538db2250b367fc0074739fa.tar.gz gdb-f968fe80b09ddc88538db2250b367fc0074739fa.tar.bz2 |
Linux targets: drop fall back to target method for 'make_corefile_notes'
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)<collect>:
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.
Diffstat (limited to 'gdb/linux-tdep.c')
-rw-r--r-- | gdb/linux-tdep.c | 36 |
1 files changed, 11 insertions, 25 deletions
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, |