From 1f8cf22072102afa4eb142425aeda8ab71b70233 Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Mon, 4 Jun 2012 16:23:57 +0000 Subject: [PATCH] 2012-06-04 Pedro Alves * arch-utils.c (default_gdb_signal_from_target): Delete. * arch-utils.h (default_gdb_signal_from_target): Delete. * corelow.c (core_open) : Extended comment. Check gdbarch_gdb_signal_from_target_p. * gdbarch.sh (gdb_signal_from_target): Make it an M method (with predicate). * gdbarch.h: Regenerate. * gdbarch.c: Regenerate. --- gdb/ChangeLog | 11 +++++++++++ gdb/arch-utils.c | 9 --------- gdb/arch-utils.h | 3 --- gdb/corelow.c | 13 +++++++++---- gdb/gdbarch.c | 15 ++++++++++++--- gdb/gdbarch.h | 5 +++-- gdb/gdbarch.sh | 5 ++--- 7 files changed, 37 insertions(+), 24 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ec6d26a70a1..5317e17f3eb 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,14 @@ +2012-06-04 Pedro Alves + + * arch-utils.c (default_gdb_signal_from_target): Delete. + * arch-utils.h (default_gdb_signal_from_target): Delete. + * corelow.c (core_open) : Extended comment. Check + gdbarch_gdb_signal_from_target_p. + * gdbarch.sh (gdb_signal_from_target): Make it an M method (with + predicate). + * gdbarch.h: Regenerate. + * gdbarch.c: Regenerate. + 2012-06-04 Pedro Alves * gdbarch.sh (gdb_signal_from_target): Mention that the diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c index 98cf4b8f787..e683a2d42b0 100644 --- a/gdb/arch-utils.c +++ b/gdb/arch-utils.c @@ -793,15 +793,6 @@ default_gen_return_address (struct gdbarch *gdbarch, error (_("This architecture has no method to collect a return address.")); } -enum gdb_signal -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); -} - /* */ /* -Wmissing-prototypes */ diff --git a/gdb/arch-utils.h b/gdb/arch-utils.h index ba71ec3ac93..7c398b3e882 100644 --- a/gdb/arch-utils.h +++ b/gdb/arch-utils.h @@ -172,7 +172,4 @@ 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_target (struct gdbarch *, - int); - #endif diff --git a/gdb/corelow.c b/gdb/corelow.c index 8c3cd5c9210..dd625604c93 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -441,11 +441,16 @@ core_open (char *filename, int from_tty) if (siggy > 0) { /* If we don't have a CORE_GDBARCH to work with, assume a native - core. */ + core (map gdb_signal from host signals). If we do have + CORE_GDBARCH to work with, but no gdb_signal_from_target + implementation for that gdbarch, as a fallback measure, + assume the host signal mapping. It'll be correct for native + cores, but most likely incorrect for cross-cores. */ enum gdb_signal sig = (core_gdbarch != NULL - ? gdbarch_gdb_signal_from_target (core_gdbarch, - siggy) - : gdb_signal_from_host (siggy)); + && gdbarch_gdb_signal_from_target_p (core_gdbarch) + ? gdbarch_gdb_signal_from_target (core_gdbarch, + siggy) + : gdb_signal_from_host (siggy)); printf_filtered (_("Program terminated with signal %d, %s.\n"), siggy, gdb_signal_to_string (sig)); diff --git a/gdb/gdbarch.c b/gdb/gdbarch.c index 3ebe835e9fb..9d7b67a9c78 100644 --- a/gdb/gdbarch.c +++ b/gdb/gdbarch.c @@ -427,7 +427,7 @@ struct gdbarch startup_gdbarch = 0, /* sofun_address_maybe_missing */ 0, /* process_record */ 0, /* process_record_signal */ - default_gdb_signal_from_target, /* gdb_signal_from_target */ + 0, /* gdb_signal_from_target */ 0, /* get_siginfo_type */ 0, /* record_special_symbol */ 0, /* get_syscall_number */ @@ -536,7 +536,6 @@ 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_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 +726,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_target, invalid_p == 0 */ + /* Skip verify of gdb_signal_from_target, has predicate. */ /* Skip verify of get_siginfo_type, has predicate. */ /* Skip verify of record_special_symbol, has predicate. */ /* Skip verify of get_syscall_number, has predicate. */ @@ -995,6 +994,9 @@ gdbarch_dump (struct gdbarch *gdbarch, struct ui_file *file) fprintf_unfiltered (file, "gdbarch_dump: gcore_bfd_target = %s\n", pstring (gdbarch->gcore_bfd_target)); + fprintf_unfiltered (file, + "gdbarch_dump: gdbarch_gdb_signal_from_target_p() = %d\n", + gdbarch_gdb_signal_from_target_p (gdbarch)); fprintf_unfiltered (file, "gdbarch_dump: gdb_signal_from_target = <%s>\n", host_address_to_string (gdbarch->gdb_signal_from_target)); @@ -3787,6 +3789,13 @@ set_gdbarch_process_record_signal (struct gdbarch *gdbarch, gdbarch->process_record_signal = process_record_signal; } +int +gdbarch_gdb_signal_from_target_p (struct gdbarch *gdbarch) +{ + gdb_assert (gdbarch != NULL); + return gdbarch->gdb_signal_from_target != NULL; +} + enum gdb_signal gdbarch_gdb_signal_from_target (struct gdbarch *gdbarch, int signo) { diff --git a/gdb/gdbarch.h b/gdb/gdbarch.h index da449c2a654..e4e7abfdfe3 100644 --- a/gdb/gdbarch.h +++ b/gdb/gdbarch.h @@ -945,8 +945,9 @@ extern void set_gdbarch_process_record_signal (struct gdbarch *gdbarch, gdbarch_ header (the nm-*.h files), the host header, or similar headers. 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. */ + (target_wait, target_resume, etc.). */ + +extern int gdbarch_gdb_signal_from_target_p (struct gdbarch *gdbarch); 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); diff --git a/gdb/gdbarch.sh b/gdb/gdbarch.sh index 5cefdab8d3f..0f58defb383 100755 --- a/gdb/gdbarch.sh +++ b/gdb/gdbarch.sh @@ -778,9 +778,8 @@ M:int:process_record_signal:struct regcache *regcache, enum gdb_signal signal:re # header (the nm-*.h files), the host header, or similar # headers. 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 +# (target_wait, target_resume, etc.). +M:enum gdb_signal:gdb_signal_from_target:int signo:signo # Extra signal info inspection. # -- 2.30.2