Call set_gdbarch_get_siginfo_type in linux_init_abi
authorYao Qi <yao.qi@linaro.org>
Wed, 24 Jun 2015 13:53:03 +0000 (14:53 +0100)
committerYao Qi <yao.qi@linaro.org>
Wed, 24 Jun 2015 13:53:03 +0000 (14:53 +0100)
linux_get_siginfo_type is installed to many linux gdbarch.  This patch
is to move this to a common area linux-tdep.c:linux_init_abi, so that
linux_get_siginfo_type is installed to every linux gdbarch.  If some
linux gdbarch needs its own version, please override it in
$ARCH_linux_init_abi.  In the testsuite, we enable siginfo related
tests for all linux targets.

gdb:

2015-06-24  Yao Qi  <yao.qi@linaro.org>

* aarch64-linux-tdep.c (aarch64_linux_init_abi): Don't call
set_gdbarch_get_siginfo_type.
* amd64-linux-tdep.c (amd64_linux_init_abi_common): Likewise.
* arm-linux-tdep.c (arm_linux_init_abi): Likewise.
* i386-linux-tdep.c (i386_linux_init_abi): Likewise.
* m68klinux-tdep.c (m68k_linux_init_abi): Likewise.
* ppc-linux-tdep.c (ppc_linux_init_abi): Likewise.
* s390-linux-tdep.c (s390_gdbarch_init): Likewise.
* tilegx-linux-tdep.c (tilegx_linux_init_abi): Likewise.
* linux-tdep.c (linux_get_siginfo_type): Change it to static.
(linux_init_abi): Call set_gdbarch_get_siginfo_type.
* linux-tdep.h (linux_get_siginfo_type): Remove the declaration.

gdb/testsuite:

2015-06-24  Yao Qi  <yao.qi@linaro.org>

* lib/gdb.exp (supports_get_siginfo_type): Return 1 for all
linux targets.

13 files changed:
gdb/ChangeLog
gdb/aarch64-linux-tdep.c
gdb/amd64-linux-tdep.c
gdb/arm-linux-tdep.c
gdb/i386-linux-tdep.c
gdb/linux-tdep.c
gdb/linux-tdep.h
gdb/m68klinux-tdep.c
gdb/ppc-linux-tdep.c
gdb/s390-linux-tdep.c
gdb/testsuite/ChangeLog
gdb/testsuite/lib/gdb.exp
gdb/tilegx-linux-tdep.c

index 0d0fe10..3603241 100644 (file)
@@ -1,3 +1,18 @@
+2015-06-24  Yao Qi  <yao.qi@linaro.org>
+
+       * aarch64-linux-tdep.c (aarch64_linux_init_abi): Don't call
+       set_gdbarch_get_siginfo_type.
+       * amd64-linux-tdep.c (amd64_linux_init_abi_common): Likewise.
+       * arm-linux-tdep.c (arm_linux_init_abi): Likewise.
+       * i386-linux-tdep.c (i386_linux_init_abi): Likewise.
+       * m68klinux-tdep.c (m68k_linux_init_abi): Likewise.
+       * ppc-linux-tdep.c (ppc_linux_init_abi): Likewise.
+       * s390-linux-tdep.c (s390_gdbarch_init): Likewise.
+       * tilegx-linux-tdep.c (tilegx_linux_init_abi): Likewise.
+       * linux-tdep.c (linux_get_siginfo_type): Change it to static.
+       (linux_init_abi): Call set_gdbarch_get_siginfo_type.
+       * linux-tdep.h (linux_get_siginfo_type): Remove the declaration.
+
 2015-06-24  Gary Benson  <gbenson@redhat.com>
 
        * common/buffer.c (stdint.h): Do not include.
index 2f93846..f8cf43e 100644 (file)
@@ -962,7 +962,6 @@ aarch64_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
   /* Shared library handling.  */
   set_gdbarch_skip_trampoline_code (gdbarch, find_solib_trampoline_target);
 
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
   tramp_frame_prepend_unwinder (gdbarch, &aarch64_linux_rt_sigframe);
 
   /* Enable longjmp.  */
index 2a8de82..e9be9cf 100644 (file)
@@ -1835,8 +1835,6 @@ amd64_linux_init_abi_common(struct gdbarch_info info, struct gdbarch *gdbarch)
   set_gdbarch_displaced_step_location (gdbarch,
                                        linux_displaced_step_location);
 
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
-
   set_gdbarch_process_record (gdbarch, i386_process_record);
   set_gdbarch_process_record_signal (gdbarch, amd64_linux_record_signal);
 }
index fd0ef5b..6273027 100644 (file)
@@ -1440,8 +1440,6 @@ arm_linux_init_abi (struct gdbarch_info info,
     (gdbarch, arm_linux_iterate_over_regset_sections);
   set_gdbarch_core_read_description (gdbarch, arm_linux_core_read_description);
 
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
-
   /* Displaced stepping.  */
   set_gdbarch_displaced_step_copy_insn (gdbarch,
                                        arm_linux_displaced_step_copy_insn);
index 7c2e322..2022927 100644 (file)
@@ -994,8 +994,6 @@ i386_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
   set_xml_syscall_file_name (gdbarch, XML_SYSCALL_FILENAME_I386);
   set_gdbarch_get_syscall_number (gdbarch,
                                   i386_linux_get_syscall_number);
-
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
 }
 
 /* Provide a prototype to silence -Wmissing-prototypes.  */
index c81f71b..ff3ada7 100644 (file)
@@ -243,7 +243,7 @@ get_linux_inferior_data (void)
 /* This function is suitable for architectures that don't
    extend/override the standard siginfo structure.  */
 
-struct type *
+static struct type *
 linux_get_siginfo_type (struct gdbarch *gdbarch)
 {
   struct linux_gdbarch_data *linux_gdbarch_data;
@@ -2440,6 +2440,7 @@ linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
   set_gdbarch_vsyscall_range (gdbarch, linux_vsyscall_range);
   set_gdbarch_infcall_mmap (gdbarch, linux_infcall_mmap);
   set_gdbarch_infcall_munmap (gdbarch, linux_infcall_munmap);
+  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
 }
 
 /* Provide a prototype to silence -Wmissing-prototypes.  */
index ff45286..9beb6f6 100644 (file)
@@ -29,8 +29,6 @@ typedef char *(*linux_collect_thread_registers_ftype) (const struct regcache *,
                                                       bfd *, char *, int *,
                                                       enum gdb_signal);
 
-struct type *linux_get_siginfo_type (struct gdbarch *);
-
 extern enum gdb_signal linux_gdb_signal_from_target (struct gdbarch *gdbarch,
                                                     int signal);
 
index 03de4ea..c60a879 100644 (file)
@@ -422,8 +422,6 @@ m68k_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
   /* Enable TLS support.  */
   set_gdbarch_fetch_tls_load_module_address (gdbarch,
                                              svr4_fetch_objfile_link_map);
-
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
 }
 
 /* Provide a prototype to silence -Wmissing-prototypes.  */
index 3849ca6..4bdd13e 100644 (file)
@@ -1801,8 +1801,6 @@ ppc_linux_init_abi (struct gdbarch_info info,
   set_gdbarch_displaced_step_location (gdbarch,
                                       linux_displaced_step_location);
 
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
-
   /* Support reverse debugging.  */
   set_gdbarch_process_record (gdbarch, ppc_process_record);
   set_gdbarch_process_record_signal (gdbarch, ppc_linux_record_signal);
index 0650c27..2460aa1 100644 (file)
@@ -3329,8 +3329,6 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
   set_gdbarch_fetch_tls_load_module_address (gdbarch,
                                             svr4_fetch_objfile_link_map);
 
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
-
   /* SystemTap functions.  */
   set_gdbarch_stap_register_prefixes (gdbarch, stap_register_prefixes);
   set_gdbarch_stap_register_indirection_prefixes (gdbarch,
index c209fe6..8825f4c 100644 (file)
@@ -1,5 +1,10 @@
 2015-06-24  Yao Qi  <yao.qi@linaro.org>
 
+       * lib/gdb.exp (supports_get_siginfo_type): Return 1 for all
+       linux targets.
+
+2015-06-24  Yao Qi  <yao.qi@linaro.org>
+
        * lib/gdb.exp (supports_get_siginfo_type): New proc.
        * gdb.base/siginfo-obj.exp: Invoke supports_get_siginfo_type.
        * gdb.base/siginfo-thread.exp: Likewise.
index 761f7d1..c0e0cb1 100644 (file)
@@ -2000,9 +2000,7 @@ gdb_caching_proc support_complex_tests {
 # return 0.
 
 proc supports_get_siginfo_type {} {
-    if { [istarget "i?86-*-linux*"]
-        || [istarget "x86_64-*-linux*"]
-        || [istarget "arm*-*-linux*"] } {
+    if { [istarget "*-*-linux*"] } {
        return 1
     } else {
        return 0
index 2de86e5..b3e34fe 100644 (file)
@@ -133,8 +133,6 @@ tilegx_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
   /* Shared library handling.  */
   set_gdbarch_skip_trampoline_code (gdbarch, find_solib_trampoline_target);
   set_gdbarch_skip_solib_resolver (gdbarch, glibc_skip_solib_resolver);
-
-  set_gdbarch_get_siginfo_type (gdbarch, linux_get_siginfo_type);
 }
 
 /* Provide a prototype to silence -Wmissing-prototypes.  */