diff options
author | Pedro Alves <palves@redhat.com> | 2016-02-24 22:52:06 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2016-02-24 22:52:06 +0000 |
commit | e7ad2f145c05bc60b1cd2796c8f3b72f02e3e730 (patch) | |
tree | b977a7b52d8dbb2c626494ded7fabab6b1c0c729 /gdb/ChangeLog | |
parent | 338435ef105ff51e967571ad073830ec1eb5a4ab (diff) | |
download | binutils-gdb-e7ad2f145c05bc60b1cd2796c8f3b72f02e3e730.tar.gz |
Handle MIPS Linux SIGTRAP siginfo.si_code values
This unbreaks pending/delayed breakpoints handling, as well as
hardware watchpoints, on MIPS.
Ref: https://sourceware.org/ml/gdb-patches/2016-02/msg00681.html
The MIPS kernel reports SI_KERNEL for all kernel generated traps,
instead of TRAP_BRKPT / TRAP_HWBKPT, but GDB isn't aware of this.
Basically, this commit:
- Folds watchpoints logic into check_stopped_by_breakpoint, and
renames it to save_stop_reason.
- Adds GDB_ARCH_IS_TRAP_HWBKPT.
- Makes MIPS set both GDB_ARCH_IS_TRAP_BRPT and
GDB_ARCH_IS_TRAP_HWBKPT to SI_KERNEL. In save_stop_reason, we
handle the case of the same si_code returning true for both
TRAP_BRPT and TRAP_HWBKPT by looking at what the debug registers
say.
Tested on x86-64 Fedora 20, native and gdbserver.
gdb/ChangeLog:
2016-02-24 Pedro Alves <palves@redhat.com>
* linux-nat.c (save_sigtrap) Delete.
(stop_wait_callback): Call save_stop_reason instead of
save_sigtrap.
(check_stopped_by_breakpoint): Rename to ...
(save_stop_reason): ... this. Bits of save_sigtrap folded here.
Use GDB_ARCH_IS_TRAP_HWBKPT and handle ambiguous
GDB_ARCH_IS_TRAP_BRKPT / GDB_ARCH_IS_TRAP_HWBKPT. Factor out
common code between the USE_SIGTRAP_SIGINFO and
!USE_SIGTRAP_SIGINFO blocks.
(linux_nat_filter_event): Call save_stop_reason instead of
save_sigtrap.
* nat/linux-ptrace.h: Check for both SI_KERNEL and TRAP_BRKPT
si_code for MIPS.
* nat/linux-ptrace.h: Fix "TRAP_HWBPT" typo in x86 table. Add
comments on MIPS behavior.
(GDB_ARCH_IS_TRAP_HWBKPT): Define for all archs.
gdb/gdbserver/ChangeLog:
2016-02-24 Pedro Alves <palves@redhat.com>
* linux-low.c (check_stopped_by_breakpoint): Rename to ...
(save_stop_reason): ... this. Use GDB_ARCH_IS_TRAP_HWBKPT and
handle ambiguous GDB_ARCH_IS_TRAP_BRKPT / GDB_ARCH_IS_TRAP_HWBKPT.
Factor out common code between the USE_SIGTRAP_SIGINFO and
!USE_SIGTRAP_SIGINFO blocks.
(linux_low_filter_event): Call save_stop_reason instead of
check_stopped_by_breakpoint and check_stopped_by_watchpoint.
Update comments.
(linux_wait_1): Update comments.
Diffstat (limited to 'gdb/ChangeLog')
-rw-r--r-- | gdb/ChangeLog | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 95cabdb74f4..0d6ea60bf72 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,22 @@ +2016-02-24 Pedro Alves <palves@redhat.com> + + * linux-nat.c (save_sigtrap) Delete. + (stop_wait_callback): Call save_stop_reason instead of + save_sigtrap. + (check_stopped_by_breakpoint): Rename to ... + (save_stop_reason): ... this. Bits of save_sigtrap folded here. + Use GDB_ARCH_IS_TRAP_HWBKPT and handle ambiguous + GDB_ARCH_IS_TRAP_BRKPT / GDB_ARCH_IS_TRAP_HWBKPT. Factor out + common code between the USE_SIGTRAP_SIGINFO and + !USE_SIGTRAP_SIGINFO blocks. + (linux_nat_filter_event): Call save_stop_reason instead of + save_sigtrap. + * nat/linux-ptrace.h: Check for both SI_KERNEL and TRAP_BRKPT + si_code for MIPS. + * nat/linux-ptrace.h: Fix "TRAP_HWBPT" typo in x86 table. Add + comments on MIPS behavior. + (GDB_ARCH_IS_TRAP_HWBKPT): Define for all archs. + 2016-02-24 Marcin KoĆcielnicki <koriakin@0x04.net> * rs6000-tdep.c (rs6000_frame_cache): Initialize frame and pc to 0 |