2004-04-26 Orjan Friberg <orjanf@axis.com>

	From Paul Koning <pkoning@equallogic.com>:
	* breakpoint.c (free_valchain): New function.
	(insert_bp_location, delete_breakpoint): Use free_valchain.
	(remove_breakpoint): Do not remove the valchain.
	(bpstat_stop_status): If not stopped by watchpoint, skip
	watchpoints when generating stop status list.
	* infrun.c (handle_inferior_event): Make
	stepped_after_stopped_by_watchpoint a global variable.
	* remote.c (remote_stopped_data_address): Return watch data
	address rather than zero if stepped_after_stopped_by_watchpoint is
	set.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index af8808bd5fcce299d13a96236f0a7377c243f7ce..ccc04d24a48f1e4afdb4630140dbac48fcbc9ecf 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,17 @@
+2004-04-26  Orjan Friberg <orjanf@axis.com>
+
+       From Paul Koning <pkoning@equallogic.com>:
+       * breakpoint.c (free_valchain): New function.
+       (insert_bp_location, delete_breakpoint): Use free_valchain.
+       (remove_breakpoint): Do not remove the valchain.
+       (bpstat_stop_status): If not stopped by watchpoint, skip
+       watchpoints when generating stop status list.
+       * infrun.c (handle_inferior_event): Make
+       stepped_after_stopped_by_watchpoint a global variable.
+       * remote.c (remote_stopped_data_address): Return watch data
+       address rather than zero if stepped_after_stopped_by_watchpoint is
+       set.
+
 2004-04-25  Michael Chastain  <mec.gnu@mindspring.com>
 
        Fix PR gdb/1626.

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index c38720a8197aa186e8cd24103d567b2899d96c45..95b08144487de3fc975242218ad23da4cb544ebe 100644 (file)
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -746,6 +746,23 @@ insert_catchpoint (struct ui_out *uo, void *args)
   return 0;
 }
 
+/* Helper routine: free the value chain for a breakpoint (watchpoint).  */
+
+static void free_valchain (struct bp_location *b)
+{
+  struct value *v;
+  struct value *n;
+
+  /* Free the saved value chain.  We will construct a new one
+     the next time the watchpoint is inserted.  */
+  for (v = b->owner->val_chain; v; v = n)
+    {
+      n = v->next;
+      value_free (v);
+    }
+  b->owner->val_chain = NULL;
+}
+
 /* Insert a low-level "breakpoint" of some type.  BPT is the breakpoint.
    Any error messages are printed to TMP_ERROR_STREAM; and DISABLED_BREAKS,
    PROCESS_WARNING, and HW_BREAKPOINT_ERROR are used to report problems.
@@ -920,6 +937,8 @@ insert_bp_location (struct bp_location *bpt,
 
       if (within_current_scope)
        {
+         free_valchain (bpt);
+
          /* Evaluate the expression and cut the chain of values
             produced off from the value chain.
 
@@ -1505,15 +1524,6 @@ remove_breakpoint (struct bp_location *b, insertion_state_t is)
       if ((is == mark_uninserted) && (b->inserted))
        warning ("Could not remove hardware watchpoint %d.",
                 b->owner->number);
-
-      /* Free the saved value chain.  We will construct a new one
-         the next time the watchpoint is inserted.  */
-      for (v = b->owner->val_chain; v; v = n)
-       {
-         n = v->next;
-         value_free (v);
-       }
-      b->owner->val_chain = NULL;
     }
   else if ((b->owner->type == bp_catch_fork ||
            b->owner->type == bp_catch_vfork ||
@@ -2616,10 +2626,15 @@ bpstat_stop_status (CORE_ADDR bp_addr, ptid_t ptid)
     if (!breakpoint_enabled (b) && b->enable_state != bp_permanent)
       continue;
 
+    /* Hardware watchpoints are treated as non-existent if the reason we 
+       stopped wasn't a hardware watchpoint (we didn't stop on some data 
+       address).  Otherwise gdb won't stop on a break instruction in the code
+       (not from a breakpoint) when a hardware watchpoint has been defined.  */
     if (b->type != bp_watchpoint
-       && b->type != bp_hardware_watchpoint
-       && b->type != bp_read_watchpoint
-       && b->type != bp_access_watchpoint
+       && !((b->type == bp_hardware_watchpoint
+             || b->type == bp_read_watchpoint
+             || b->type == bp_access_watchpoint)
+            && target_stopped_data_address () != 0)
        && b->type != bp_hardware_breakpoint
        && b->type != bp_catch_fork
        && b->type != bp_catch_vfork
@@ -6881,6 +6896,8 @@ delete_breakpoint (struct breakpoint *bpt)
   if (bpt->loc->inserted)
     remove_breakpoint (bpt->loc, mark_inserted);
 
+  free_valchain (bpt->loc);
+
   if (breakpoint_chain == bpt)
     breakpoint_chain = bpt->next;
 

diff --git a/gdb/infrun.c b/gdb/infrun.c
index 0b4f05a911ab3016bd941ecdfa4f8c2b14d0db23..f546602b7bad93846d4e635d622f2551060d760f 100644 (file)
--- a/gdb/infrun.c
+++ b/gdb/infrun.c
@@ -1356,6 +1356,8 @@ adjust_pc_after_break (struct execution_control_state *ecs)
    by an event from the inferior, figure out what it means and take
    appropriate action.  */
 
+int stepped_after_stopped_by_watchpoint;
+
 void
 handle_inferior_event (struct execution_control_state *ecs)
 {
@@ -1364,7 +1366,6 @@ handle_inferior_event (struct execution_control_state *ecs)
      isn't used, then you're wrong!  The macro STOPPED_BY_WATCHPOINT,
      defined in the file "config/pa/nm-hppah.h", accesses the variable
      indirectly.  Mutter something rude about the HP merge.  */
-  int stepped_after_stopped_by_watchpoint;
   int sw_single_step_trap_p = 0;
 
   /* Cache the last pid/waitstatus. */

diff --git a/gdb/remote.c b/gdb/remote.c
index 1e173d27b9dadb2c0cf1a992610af96553f45bf7..384f011569396e784178780cf718ee8dcf175953 100644 (file)
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -4640,10 +4640,13 @@ remote_stopped_by_watchpoint (void)
     return remote_stopped_by_watchpoint_p;
 }
 
+extern int stepped_after_stopped_by_watchpoint;
+
 static CORE_ADDR
 remote_stopped_data_address (void)
 {
-  if (remote_stopped_by_watchpoint ())
+  if (remote_stopped_by_watchpoint ()
+      || stepped_after_stopped_by_watchpoint)
     return remote_watch_data_address;
   return (CORE_ADDR)0;
 }