This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[patch 3/4] Fix hw watchpoints: remove dead code [update]
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: gdb-patches at sourceware dot org
- Date: Sat, 3 Oct 2009 19:11:00 +0200
- Subject: [patch 3/4] Fix hw watchpoints: remove dead code [update]
- References: <20090817194624.GD10694@host0.dyn.jankratochvil.net>
Hi,
just a rediff on top of the patch 1/4.
Thanks,
Jan
------------------------------------------------------------------------------
simple dead code removal. Code would become invalid with the changes anyway.
The testcase is in fact unrelated, just I was surprised "hbreak" functionality
is currently not regression tested anywhere.
gdb/
2009-08-17 Jan Kratochvil <jan.kratochvil@redhat.com>
* i386-nat.c (i386_stopped_by_hwbp): Remove.
gdb/testsuite/
2009-10-03 Jan Kratochvil <jan.kratochvil@redhat.com>
* gdb.base/watchpoint-hw.exp: New variable breakline.
(hbreak, continue to break-at-exit after hbreak): New tests.
--- a/gdb/i386-nat.c
+++ b/gdb/i386-nat.c
@@ -593,27 +593,6 @@ i386_stopped_by_watchpoint (void)
return target_stopped_data_address (¤t_target, &addr);
}
-/* Return non-zero if the inferior has some break/watchpoint that
- triggered. */
-
-static int
-i386_stopped_by_hwbp (void)
-{
- int i;
-
- dr_status_mirror = i386_dr_low.get_status ();
- if (maint_show_dr)
- i386_show_dr ("stopped_by_hwbp", 0, 0, hw_execute);
-
- ALL_DEBUG_REGISTERS(i)
- {
- if (I386_DR_WATCH_HIT (i))
- return 1;
- }
-
- return 0;
-}
-
/* Insert a hardware-assisted breakpoint at BP_TGT->placed_address.
Return 0 on success, EBUSY on failure. */
static int
--- a/gdb/testsuite/gdb.base/watchpoint-hw.exp
+++ b/gdb/testsuite/gdb.base/watchpoint-hw.exp
@@ -53,11 +53,27 @@ if ![runto_main] {
gdb_test "rwatch watchee"
-gdb_breakpoint [gdb_get_line_number "break-at-exit"]
+set breakline [gdb_get_line_number "break-at-exit"]
+gdb_breakpoint $breakline
gdb_test "continue" "Continuing.\r\nHardware read watchpoint \[0-9\]+: watchee\r\n\r\nValue = 0\r\n.*"
# Here should be no repeated notification of the read watchpoint.
gdb_test "continue" \
"Continuing\\.\[ \r\n\]+Breakpoint \[0-9\]+, .*break-at-exit.*" \
- "continue to break-at-exit"
+ "continue to break-at-exit after rwatch"
+
+clean_restart ${test}
+
+if ![runto_main] {
+ untested ${test}.exp
+ return -1
+}
+
+gdb_test "hbreak ${srcfile}:${breakline}" \
+ "Hardware assisted breakpoint 2 at 0x\[0-9a-f\]+: file .*${srcfile}, line ${breakline}\\." \
+ "hbreak"
+
+gdb_test "continue" \
+ "Continuing\\.\[ \r\n\]+Breakpoint \[0-9\]+, .*break-at-exit.*" \
+ "continue to break-at-exit after hbreak"