This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[COMMIT] A few more tweaks for SPARC Linux
- From: Mark Kettenis <mark dot kettenis at xs4all dot nl>
- To: gdb-patches at sourceware dot org
- Cc: davem at davemloft dot net
- Date: Fri, 22 Apr 2005 22:09:28 +0200 (CEST)
- Subject: [COMMIT] A few more tweaks for SPARC Linux
With the introduction of the tramp_frame stuff (thanks Dave) the
comments were now a bit elaborate, and not completely accurate
anymore. I've removed most of the duplicated information to avoid
things getting out of sync again.
Committed,
Mark
Index: ChangeLog
from Mark Kettenis <kettenis@gnu.org>
* sparc-linux-tdep.c: Reorder includes. Tweak a few comments.
(LINUX32_SIGTRAMP_INSN0, LINUX32_SIGTRAMP_INSN1)
(LINUX32_RT_SIGTRAMP_INSN0, LINUX32_RT_SIGTRAMP_INSN1): Remove
macros.
(sparc32_linux_sigframe, sparc32_linux_rt_sigframe): Hardcode
instructions.
* sparc64-linux-tdep.c: Reorder includes. Tweak a few comments.
(LINUX64_RT_SIGTRAMP_INSN0, LINUX64_RT_SIGTRAMP_INSN1): Remove
macros.
(sparc64_linux_rt_sigframe): Hardcode instructions.
* Makefile.in (sparc-linux-tdep.o, sparc64-linux-tdep.o): Update
dependencies.
Index: sparc-linux-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/sparc-linux-tdep.c,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -p -r1.12 -r1.13
--- sparc-linux-tdep.c 22 Apr 2005 19:06:23 -0000 1.12
+++ sparc-linux-tdep.c 22 Apr 2005 19:52:21 -0000 1.13
@@ -23,7 +23,6 @@
#include "floatformat.h"
#include "frame.h"
#include "frame-unwind.h"
-#include "tramp-frame.h"
#include "gdbarch.h"
#include "gdbcore.h"
#include "osabi.h"
@@ -31,11 +30,17 @@
#include "solib-svr4.h"
#include "symtab.h"
#include "trad-frame.h"
+#include "tramp-frame.h"
#include "sparc-tdep.h"
/* Signal trampoline support. */
+static void sparc32_linux_sigframe_init (const struct tramp_frame *self,
+ struct frame_info *next_frame,
+ struct trad_frame_cache *this_cache,
+ CORE_ADDR func);
+
/* GNU/Linux has two flavors of signals. Normal signal handlers, and
"realtime" (RT) signals. The RT signals can provide additional
information to the signal handler if the SA_SIGINFO flag is set
@@ -45,58 +50,36 @@
/* When the sparc Linux kernel calls a signal handler and the
SA_RESTORER flag isn't set, the return address points to a bit of
- code on the stack. This function returns whether the PC appears to
- be within this bit of code.
-
- The instruction sequence for normal signals is
- mov __NR_sigreturn, %g1 ! hex: 0x821020d8
- ta 0x10 ! hex: 0x91d02010
+ code on the stack. This code checks whether the PC appears to be
+ within this bit of code.
+ The instruction sequence for normal signals is encoded below.
Checking for the code sequence should be somewhat reliable, because
the effect is to call the system call sigreturn. This is unlikely
- to occur anywhere other than a signal trampoline.
-
- It kind of sucks that we have to read memory from the process in
- order to identify a signal trampoline, but there doesn't seem to be
- any other way. However, sparc32_linux_pc_in_sigtramp arranges to
- only call us if no function name could be identified, which should
- be the case since the code is on the stack. */
-
-#define LINUX32_SIGTRAMP_INSN0 0x821020d8 /* mov __NR_sigreturn, %g1 */
-#define LINUX32_SIGTRAMP_INSN1 0x91d02010 /* ta 0x10 */
-
-/* The instruction sequence for RT signals is
- mov __NR_rt_sigreturn, %g1 ! hex: 0x82102065
- ta {0x10,0x6d} ! hex: 0x91d02010
-
- The effect is to call the system call rt_sigreturn.
- Note that 64-bit binaries only use this RT signal return method. */
+ to occur anywhere other than a signal trampoline. */
-#define LINUX32_RT_SIGTRAMP_INSN0 0x82102065
-#define LINUX32_RT_SIGTRAMP_INSN1 0x91d02010
-
-static void sparc32_linux_sigframe_init (const struct tramp_frame *self,
- struct frame_info *next_frame,
- struct trad_frame_cache *this_cache,
- CORE_ADDR func);
-
-static const struct tramp_frame sparc32_linux_sigframe = {
+static const struct tramp_frame sparc32_linux_sigframe =
+{
SIGTRAMP_FRAME,
4,
{
- { LINUX32_SIGTRAMP_INSN0, -1 },
- { LINUX32_SIGTRAMP_INSN1, -1 },
+ { 0x821020d8, -1 }, /* mov __NR_sugreturn, %g1 */
+ { 0x91d02010, -1 }, /* ta 0x10 */
{ TRAMP_SENTINEL_INSN, -1 }
},
sparc32_linux_sigframe_init
};
-static const struct tramp_frame sparc32_linux_rt_sigframe = {
+/* The instruction sequence for RT signals is slightly different. The
+ effect is to call the system call rt_sigreturn. */
+
+static const struct tramp_frame sparc32_linux_rt_sigframe =
+{
SIGTRAMP_FRAME,
4,
{
- { LINUX32_RT_SIGTRAMP_INSN0, -1 },
- { LINUX32_RT_SIGTRAMP_INSN1, -1 },
+ { 0x82102065, -1 }, /* mov __NR_rt_sigreturn, %g1 */
+ { 0x91d02010, -1 }, /* ta 0x10 */
{ TRAMP_SENTINEL_INSN, -1 }
},
sparc32_linux_sigframe_init
@@ -115,15 +98,15 @@ sparc32_linux_sigframe_init (const struc
if (self == &sparc32_linux_rt_sigframe)
base += 128;
- /* Offsets from <bits/sigcontext.h> */
+ /* Offsets from <bits/sigcontext.h>. */
- trad_frame_set_reg_addr (this_cache, SPARC32_PSR_REGNUM, base + 0x00);
- trad_frame_set_reg_addr (this_cache, SPARC32_PC_REGNUM, base + 0x04);
- trad_frame_set_reg_addr (this_cache, SPARC32_NPC_REGNUM, base + 0x08);
- trad_frame_set_reg_addr (this_cache, SPARC32_Y_REGNUM, base + 0x0c);
+ trad_frame_set_reg_addr (this_cache, SPARC32_PSR_REGNUM, base + 0);
+ trad_frame_set_reg_addr (this_cache, SPARC32_PC_REGNUM, base + 4);
+ trad_frame_set_reg_addr (this_cache, SPARC32_NPC_REGNUM, base + 8);
+ trad_frame_set_reg_addr (this_cache, SPARC32_Y_REGNUM, base + 12);
/* Since %g0 is always zero, keep the identity encoding. */
- addr = base + 0x14;
+ addr = base + 20;
for (regnum = SPARC_G1_REGNUM; regnum <= SPARC_O7_REGNUM; regnum++)
{
trad_frame_set_reg_addr (this_cache, regnum, addr);
@@ -138,7 +121,6 @@ sparc32_linux_sigframe_init (const struc
}
trad_frame_set_id (this_cache, frame_id_build (base, func));
}
-
static void
Index: sparc64-linux-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/sparc64-linux-tdep.c,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -p -r1.7 -r1.8
--- sparc64-linux-tdep.c 22 Apr 2005 19:06:23 -0000 1.7
+++ sparc64-linux-tdep.c 22 Apr 2005 19:52:21 -0000 1.8
@@ -22,37 +22,32 @@
#include "defs.h"
#include "frame.h"
#include "frame-unwind.h"
-#include "tramp-frame.h"
#include "gdbarch.h"
#include "osabi.h"
#include "solib-svr4.h"
#include "symtab.h"
#include "trad-frame.h"
+#include "tramp-frame.h"
#include "sparc64-tdep.h"
-/* The instruction sequence for RT signals is
- mov __NR_rt_sigreturn, %g1 ! hex: 0x82102065
- ta 0x6d ! hex: 0x91d0206d
-
- The effect is to call the system call rt_sigreturn.
- Note that 64-bit binaries only use this RT signal return method. */
-
-#define LINUX64_RT_SIGTRAMP_INSN0 0x82102065
-#define LINUX64_RT_SIGTRAMP_INSN1 0x91d0206d
-
+/* Signal trampoline support. */
static void sparc64_linux_sigframe_init (const struct tramp_frame *self,
struct frame_info *next_frame,
struct trad_frame_cache *this_cache,
CORE_ADDR func);
-static const struct tramp_frame sparc64_linux_rt_sigframe = {
+/* See sparc-linux-tdep.c for details. Note that 64-bit binaries only
+ use RT signals. */
+
+static const struct tramp_frame sparc64_linux_rt_sigframe =
+{
SIGTRAMP_FRAME,
4,
{
- { LINUX64_RT_SIGTRAMP_INSN0, -1 },
- { LINUX64_RT_SIGTRAMP_INSN1, -1 },
+ { 0x82102065, -1 }, /* mov __NR_rt_sigreturn, %g1 */
+ { 0x91d0206d, -1 }, /* ta 0x6d */
{ TRAMP_SENTINEL_INSN, -1 }
},
sparc64_linux_sigframe_init
@@ -70,21 +65,21 @@ sparc64_linux_sigframe_init (const struc
base = frame_unwind_register_unsigned (next_frame, SPARC_O1_REGNUM);
base += 128;
- /* Offsets from <bits/sigcontext.h> */
+ /* Offsets from <bits/sigcontext.h>. */
/* Since %g0 is always zero, keep the identity encoding. */
- addr = base + 0x08;
+ addr = base + 8;
for (regnum = SPARC_G1_REGNUM; regnum <= SPARC_O7_REGNUM; regnum++)
{
trad_frame_set_reg_addr (this_cache, regnum, addr);
addr += 8;
}
- trad_frame_set_reg_addr (this_cache, SPARC64_STATE_REGNUM, addr + 0x00);
- trad_frame_set_reg_addr (this_cache, SPARC64_PC_REGNUM, addr + 0x08);
- trad_frame_set_reg_addr (this_cache, SPARC64_NPC_REGNUM, addr + 0x10);
- trad_frame_set_reg_addr (this_cache, SPARC64_Y_REGNUM, addr + 0x18);
- trad_frame_set_reg_addr (this_cache, SPARC64_FPRS_REGNUM, addr + 0x1c);
+ trad_frame_set_reg_addr (this_cache, SPARC64_STATE_REGNUM, addr + 0);
+ trad_frame_set_reg_addr (this_cache, SPARC64_PC_REGNUM, addr + 8);
+ trad_frame_set_reg_addr (this_cache, SPARC64_NPC_REGNUM, addr + 16);
+ trad_frame_set_reg_addr (this_cache, SPARC64_Y_REGNUM, addr + 24);
+ trad_frame_set_reg_addr (this_cache, SPARC64_FPRS_REGNUM, addr + 28);
addr = frame_unwind_register_unsigned (next_frame, SPARC_SP_REGNUM);
if (addr & 1)
@@ -99,6 +94,7 @@ sparc64_linux_sigframe_init (const struc
trad_frame_set_id (this_cache, frame_id_build (base, func));
}
+
static void
sparc64_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch)
{
@@ -119,6 +115,7 @@ sparc64_linux_init_abi (struct gdbarch_i
set_gdbarch_fetch_tls_load_module_address (gdbarch,
svr4_fetch_objfile_link_map);
}
+
/* Provide a prototype to silence -Wmissing-prototypes. */
extern void _initialize_sparc64_linux_tdep (void);
Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/gdb/Makefile.in,v
retrieving revision 1.718
retrieving revision 1.719
diff -u -p -r1.718 -r1.719
--- Makefile.in 21 Apr 2005 22:04:03 -0000 1.718
+++ Makefile.in 22 Apr 2005 19:59:07 -0000 1.719
@@ -2483,7 +2483,8 @@ sentinel-frame.o: sentinel-frame.c $(def
ser-e7kpc.o: ser-e7kpc.c $(defs_h) $(serial_h) $(gdb_string_h)
ser-go32.o: ser-go32.c $(defs_h) $(gdbcmd_h) $(serial_h) $(gdb_string_h)
serial.o: serial.c $(defs_h) $(serial_h) $(gdb_string_h) $(gdbcmd_h)
-ser-base.o: ser-base.c $(defs_h) $(serial_h) $(ser_base_h) $(event_loop_h)
+ser-base.o: ser-base.c $(defs_h) $(serial_h) $(ser_base_h) $(event_loop_h) \
+ $(gdb_string_h)
ser-pipe.o: ser-pipe.c $(defs_h) $(serial_h) $(ser_base_h) $(ser_unix_h) \
$(gdb_vfork_h) $(gdb_string_h)
ser-tcp.o: ser-tcp.c $(defs_h) $(serial_h) $(ser_base_h) $(ser_unix_h) \
@@ -2567,8 +2568,7 @@ sparc64-linux-nat.o: sparc64-linux-nat.c
$(sparc_nat_h)
sparc64-linux-tdep.o: sparc64-linux-tdep.c $(defs_h) $(frame_h) \
$(frame_unwind_h) $(gdbarch_h) $(osabi_h) $(solib_svr4_h) \
- $(symtab_h) $(trad_frame_h) \
- $(sparc64_tdep_h)
+ $(symtab_h) $(trad_frame_h) $(tramp_frame_h) $(sparc64_tdep_h)
sparc64-nat.o: sparc64-nat.c $(defs_h) $(gdbarch_h) $(sparc64_tdep_h) \
$(sparc_nat_h)
sparc64nbsd-nat.o: sparc64nbsd-nat.c $(defs_h) $(regcache_h) $(target_h) \
@@ -2590,7 +2590,7 @@ sparc64-tdep.o: sparc64-tdep.c $(defs_h)
$(sparc64_tdep_h)
sparc-linux-tdep.o: sparc-linux-tdep.c $(defs_h) $(floatformat_h) $(frame_h) \
$(frame_unwind_h) $(gdbarch_h) $(gdbcore_h) $(osabi_h) $(regcache_h) \
- $(solib_svr4_h) $(symtab_h) $(trad_frame_h) \
+ $(solib_svr4_h) $(symtab_h) $(trad_frame_h) $(tramp_frame_h) \
$(sparc_tdep_h)
sparc-nat.o: sparc-nat.c $(defs_h) $(inferior_h) $(regcache_h) $(target_h) \
$(gdb_assert_h) $(gdb_string_h) $(gdb_wait_h) $(sparc_tdep_h) \