From 22203bbfb7c46db2901fa8a10d604bfc38dabfc7 Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Thu, 24 May 2012 17:03:28 +0000 Subject: [PATCH 1/1] 2012-05-24 Pedro Alves PR gdb/7205 * arch-utils.c (default_gdb_signal_to_host): Rename to ... (default_gdb_signal_to_target): ... this. Add comment. (default_gdb_signal_from_host): Rename to ... (default_gdb_signal_from_target): ... this. Add comment. * arch-utils.h (default_gdb_signal_to_host): Rename to ... (default_gdb_signal_to_target): ... this. (default_gdb_signal_from_host): Rename to ... (default_gdb_signal_from_target): ... this. * corelow.c (core_open): Adjust to naming change. Replace comment. * gdbarch.sh (gdb_signal_from_host): Rename to ... (gdb_signal_from_target): ... this. Adjust to default_gdb_signal_from_host naming change. Extend comment. (gdb_signal_to_host): Rename to ... (gdb_signal_to_target): ... this. Adjust to default_gdb_signal_to_host naming change. * gdbarch.h, gdbarch.c: Renegerate. --- gdb/ChangeLog | 21 +++++++++++++++++++++ gdb/arch-utils.c | 5 ++++- gdb/arch-utils.h | 4 ++-- gdb/corelow.c | 10 ++++------ gdb/gdbarch.c | 26 +++++++++++++------------- gdb/gdbarch.h | 15 +++++++++------ gdb/gdbarch.sh | 9 ++++++--- 7 files changed, 59 insertions(+), 31 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index fd2f414ba4..3c1a167304 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,24 @@ +2012-05-24 Pedro Alves + + PR gdb/7205 + + * arch-utils.c (default_gdb_signal_to_host): Rename to ... + (default_gdb_signal_to_target): ... this. Add comment. + (default_gdb_signal_from_host): Rename to ... + (default_gdb_signal_from_target): ... this. Add comment. + * arch-utils.h (default_gdb_signal_to_host): Rename to ... + (default_gdb_signal_to_target): ... this. + (default_gdb_signal_from_host): Rename to ... + (default_gdb_signal_from_target): ... this. + * corelow.c (core_open): Adjust to naming change. Replace comment. + * gdbarch.sh (gdb_signal_from_host): Rename to ... + (gdb_signal_from_target): ... this. Adjust to + default_gdb_signal_from_host naming change. Extend comment. + (gdb_signal_to_host): Rename to ... + (gdb_signal_to_target): ... this. Adjust to + default_gdb_signal_to_host naming change. + * gdbarch.h, gdbarch.c: Renegerate. + 2012-05-24 Pedro Alves PR gdb/7205 diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c index 803ce1ca26..98cf4b8f78 100644 --- a/gdb/arch-utils.c +++ b/gdb/arch-utils.c @@ -794,8 +794,11 @@ default_gen_return_address (struct gdbarch *gdbarch, } enum gdb_signal -default_gdb_signal_from_host (struct gdbarch *gdbarch, int signo) +default_gdb_signal_from_target (struct gdbarch *gdbarch, int signo) { + /* Lacking a better mapping, assume host signal numbers. If + debugging a cross-core, most likely this translation will be + incorrect. */ return gdb_signal_from_host (signo); } diff --git a/gdb/arch-utils.h b/gdb/arch-utils.h index 1b8fc6f3da..ba71ec3ac9 100644 --- a/gdb/arch-utils.h +++ b/gdb/arch-utils.h @@ -172,7 +172,7 @@ extern void default_gen_return_address (struct gdbarch *gdbarch, extern const char *default_auto_charset (void); extern const char *default_auto_wide_charset (void); -extern enum gdb_signal default_gdb_signal_from_host (struct gdbarch *, - int); +extern enum gdb_signal default_gdb_signal_from_target (struct gdbarch *, + int); #endif diff --git a/gdb/corelow.c b/gdb/corelow.c index f566b2d89b..8c3cd5c921 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -440,13 +440,11 @@ core_open (char *filename, int from_tty) siggy = bfd_core_file_failing_signal (core_bfd); if (siggy > 0) { - /* NOTE: gdb_signal_from_host() converts a target signal - value into gdb's internal signal value. Unfortunately gdb's - internal value is called ``gdb_signal'' and this function - got the name ..._from_host(). */ + /* If we don't have a CORE_GDBARCH to work with, assume a native + core. */ enum gdb_signal sig = (core_gdbarch != NULL - ? gdbarch_gdb_signal_from_host (core_gdbarch, - siggy) + ? gdbarch_gdb_signal_from_target (core_gdbarch, + siggy) : gdb_signal_from_host (siggy)); printf_filtered (_("Program terminated with signal %d, %s.\n"), diff --git a/gdb/gdbarch.c b/gdb/gdbarch.c index 1287485e20..3ebe835e9f 100644 --- a/gdb/gdbarch.c +++ b/gdb/gdbarch.c @@ -260,7 +260,7 @@ struct gdbarch int sofun_address_maybe_missing; gdbarch_process_record_ftype *process_record; gdbarch_process_record_signal_ftype *process_record_signal; - gdbarch_gdb_signal_from_host_ftype *gdb_signal_from_host; + gdbarch_gdb_signal_from_target_ftype *gdb_signal_from_target; gdbarch_get_siginfo_type_ftype *get_siginfo_type; gdbarch_record_special_symbol_ftype *record_special_symbol; gdbarch_get_syscall_number_ftype *get_syscall_number; @@ -427,7 +427,7 @@ struct gdbarch startup_gdbarch = 0, /* sofun_address_maybe_missing */ 0, /* process_record */ 0, /* process_record_signal */ - default_gdb_signal_from_host, /* gdb_signal_from_host */ + default_gdb_signal_from_target, /* gdb_signal_from_target */ 0, /* get_siginfo_type */ 0, /* record_special_symbol */ 0, /* get_syscall_number */ @@ -536,7 +536,7 @@ gdbarch_alloc (const struct gdbarch_info *info, gdbarch->displaced_step_free_closure = NULL; gdbarch->displaced_step_location = NULL; gdbarch->relocate_instruction = NULL; - gdbarch->gdb_signal_from_host = default_gdb_signal_from_host; + gdbarch->gdb_signal_from_target = default_gdb_signal_from_target; gdbarch->has_shared_address_space = default_has_shared_address_space; gdbarch->fast_tracepoint_valid_at = default_fast_tracepoint_valid_at; gdbarch->auto_charset = default_auto_charset; @@ -727,7 +727,7 @@ verify_gdbarch (struct gdbarch *gdbarch) /* Skip verify of sofun_address_maybe_missing, invalid_p == 0 */ /* Skip verify of process_record, has predicate. */ /* Skip verify of process_record_signal, has predicate. */ - /* Skip verify of gdb_signal_from_host, invalid_p == 0 */ + /* Skip verify of gdb_signal_from_target, invalid_p == 0 */ /* Skip verify of get_siginfo_type, has predicate. */ /* Skip verify of record_special_symbol, has predicate. */ /* Skip verify of get_syscall_number, has predicate. */ @@ -996,8 +996,8 @@ gdbarch_dump (struct gdbarch *gdbarch, struct ui_file *file) "gdbarch_dump: gcore_bfd_target = %s\n", pstring (gdbarch->gcore_bfd_target)); fprintf_unfiltered (file, - "gdbarch_dump: gdb_signal_from_host = <%s>\n", - host_address_to_string (gdbarch->gdb_signal_from_host)); + "gdbarch_dump: gdb_signal_from_target = <%s>\n", + host_address_to_string (gdbarch->gdb_signal_from_target)); fprintf_unfiltered (file, "gdbarch_dump: gen_return_address = <%s>\n", host_address_to_string (gdbarch->gen_return_address)); @@ -3788,20 +3788,20 @@ set_gdbarch_process_record_signal (struct gdbarch *gdbarch, } enum gdb_signal -gdbarch_gdb_signal_from_host (struct gdbarch *gdbarch, int signo) +gdbarch_gdb_signal_from_target (struct gdbarch *gdbarch, int signo) { gdb_assert (gdbarch != NULL); - gdb_assert (gdbarch->gdb_signal_from_host != NULL); + gdb_assert (gdbarch->gdb_signal_from_target != NULL); if (gdbarch_debug >= 2) - fprintf_unfiltered (gdb_stdlog, "gdbarch_gdb_signal_from_host called\n"); - return gdbarch->gdb_signal_from_host (gdbarch, signo); + fprintf_unfiltered (gdb_stdlog, "gdbarch_gdb_signal_from_target called\n"); + return gdbarch->gdb_signal_from_target (gdbarch, signo); } void -set_gdbarch_gdb_signal_from_host (struct gdbarch *gdbarch, - gdbarch_gdb_signal_from_host_ftype gdb_signal_from_host) +set_gdbarch_gdb_signal_from_target (struct gdbarch *gdbarch, + gdbarch_gdb_signal_from_target_ftype gdb_signal_from_target) { - gdbarch->gdb_signal_from_host = gdb_signal_from_host; + gdbarch->gdb_signal_from_target = gdb_signal_from_target; } int diff --git a/gdb/gdbarch.h b/gdb/gdbarch.h index 582bae6338..5d73d7272c 100644 --- a/gdb/gdbarch.h +++ b/gdb/gdbarch.h @@ -939,12 +939,15 @@ typedef int (gdbarch_process_record_signal_ftype) (struct gdbarch *gdbarch, stru extern int gdbarch_process_record_signal (struct gdbarch *gdbarch, struct regcache *regcache, enum gdb_signal signal); extern void set_gdbarch_process_record_signal (struct gdbarch *gdbarch, gdbarch_process_record_signal_ftype *process_record_signal); -/* Signal translation: translate inferior's signal (host's) number into - GDB's representation. */ - -typedef enum gdb_signal (gdbarch_gdb_signal_from_host_ftype) (struct gdbarch *gdbarch, int signo); -extern enum gdb_signal gdbarch_gdb_signal_from_host (struct gdbarch *gdbarch, int signo); -extern void set_gdbarch_gdb_signal_from_host (struct gdbarch *gdbarch, gdbarch_gdb_signal_from_host_ftype *gdb_signal_from_host); +/* Signal translation: translate inferior's signal (target's) number + into GDB's representation. This is mainly used when cross-debugging + core files --- "Live" targets hide the translation behind the target + interface (target_wait, target_resume, etc.). The default is to do + the translation using host signal numbers. */ + +typedef enum gdb_signal (gdbarch_gdb_signal_from_target_ftype) (struct gdbarch *gdbarch, int signo); +extern enum gdb_signal gdbarch_gdb_signal_from_target (struct gdbarch *gdbarch, int signo); +extern void set_gdbarch_gdb_signal_from_target (struct gdbarch *gdbarch, gdbarch_gdb_signal_from_target_ftype *gdb_signal_from_target); /* Extra signal info inspection. diff --git a/gdb/gdbarch.sh b/gdb/gdbarch.sh index 273c87a2bd..cc1fe654a0 100755 --- a/gdb/gdbarch.sh +++ b/gdb/gdbarch.sh @@ -772,9 +772,12 @@ M:int:process_record:struct regcache *regcache, CORE_ADDR addr:regcache, addr # Return -1 if something goes wrong, 0 otherwise. M:int:process_record_signal:struct regcache *regcache, enum gdb_signal signal:regcache, signal -# Signal translation: translate inferior's signal (host's) number into -# GDB's representation. -m:enum gdb_signal:gdb_signal_from_host:int signo:signo::default_gdb_signal_from_host::0 +# Signal translation: translate inferior's signal (target's) number +# into GDB's representation. This is mainly used when cross-debugging +# core files --- "Live" targets hide the translation behind the target +# interface (target_wait, target_resume, etc.). The default is to do +# the translation using host signal numbers. +m:enum gdb_signal:gdb_signal_from_target:int signo:signo::default_gdb_signal_from_target::0 # Extra signal info inspection. # -- 2.34.1