Merge i386_update_inferior_debug_regs
authorGary Benson <gbenson@redhat.com>
Thu, 12 Jun 2014 11:01:11 +0000 (12:01 +0100)
committerGary Benson <gbenson@redhat.com>
Wed, 18 Jun 2014 09:15:23 +0000 (10:15 +0100)
This commit synchronizes the i386_update_inferior_debug_regs functions
in i386-nat.c and i386-low.c.

gdb/
2014-06-18  Gary Benson  <gbenson@redhat.com>

* i386-nat.c (i386_update_inferior_debug_regs) <state>:
New parameter.  All uses updated.

gdb/gdbserver/
2014-06-18  Gary Benson  <gbenson@redhat.com>

* i386-low.c (i386_update_inferior_debug_regs) <inf_state>:
Renamed to state.

gdb/ChangeLog
gdb/gdbserver/ChangeLog
gdb/gdbserver/i386-low.c
gdb/i386-nat.c

index 54a573ec700647b78a3a9f1550cf922fe4701224..70b68abf987e8ab30399524c82e1feb963596e83 100644 (file)
@@ -1,3 +1,8 @@
+2014-06-18  Gary Benson  <gbenson@redhat.com>
+
+       * i386-nat.c (i386_update_inferior_debug_regs) <state>:
+       New parameter.  All uses updated.
+
 2014-06-18  Gary Benson  <gbenson@redhat.com>
 
        * i386-nat.c (maint_show_dr): Renamed to debug_hw_points.
index 91dc4983f68f98ec6f222aa12f268842d162e9f5..7c72043089fd8e80f6ac9983d139c501ab68cd0f 100644 (file)
@@ -1,3 +1,8 @@
+2014-06-18  Gary Benson  <gbenson@redhat.com>
+
+       * i386-low.c (i386_update_inferior_debug_regs) <inf_state>:
+       Renamed to state.
+
 2014-06-18  Gary Benson  <gbenson@redhat.com>
 
        * i386-low.c (i386_length_and_rw_bits): Use internal_error
index 54dd4b8eaf2c79190d438401ed33264577899f1c..8c5083828f24358076977775be3c3c711d7026df 100644 (file)
@@ -417,27 +417,27 @@ Invalid value %d of operation in i386_handle_nonaligned_watchpoint.\n"),
   return retval;
 }
 
-/* Update the inferior debug registers state, in INF_STATE, with the
+/* Update the inferior debug registers state, in STATE, with the
    new debug registers state, in NEW_STATE.  */
 
 static void
-i386_update_inferior_debug_regs (struct i386_debug_reg_state *inf_state,
+i386_update_inferior_debug_regs (struct i386_debug_reg_state *state,
                                 struct i386_debug_reg_state *new_state)
 {
   int i;
 
   ALL_DEBUG_REGISTERS (i)
     {
-      if (I386_DR_VACANT (new_state, i) != I386_DR_VACANT (inf_state, i))
+      if (I386_DR_VACANT (new_state, i) != I386_DR_VACANT (state, i))
        i386_dr_low_set_addr (new_state, i);
       else
-       gdb_assert (new_state->dr_mirror[i] == inf_state->dr_mirror[i]);
+       gdb_assert (new_state->dr_mirror[i] == state->dr_mirror[i]);
     }
 
-  if (new_state->dr_control_mirror != inf_state->dr_control_mirror)
+  if (new_state->dr_control_mirror != state->dr_control_mirror)
     i386_dr_low_set_control (new_state);
 
-  *inf_state = *new_state;
+  *state = *new_state;
 }
 
 /* Insert a watchpoint to watch a memory region which starts at
index 65205498d9c541feafb9965d9da448de26326df3..a64cdab90ce2e87c6c448dc744c467a40c18dfa4 100644 (file)
@@ -526,14 +526,13 @@ Invalid value %d of operation in i386_handle_nonaligned_watchpoint.\n"),
   return retval;
 }
 
-/* Update the inferior's debug registers with the new debug registers
-   state, in NEW_STATE, and then update our local mirror to match.  */
+/* Update the inferior debug registers state, in STATE, with the
+   new debug registers state, in NEW_STATE.  */
 
 static void
-i386_update_inferior_debug_regs (struct i386_debug_reg_state *new_state)
+i386_update_inferior_debug_regs (struct i386_debug_reg_state *state,
+                                struct i386_debug_reg_state *new_state)
 {
-  struct i386_debug_reg_state *state
-    = i386_debug_reg_state (ptid_get_pid (inferior_ptid));
   int i;
 
   ALL_DEBUG_REGISTERS (i)
@@ -586,7 +585,7 @@ i386_insert_watchpoint (struct target_ops *self,
     }
 
   if (retval == 0)
-    i386_update_inferior_debug_regs (&local_state);
+    i386_update_inferior_debug_regs (state, &local_state);
 
   if (debug_hw_points)
     i386_show_dr (state, "insert_watchpoint", addr, len, type);
@@ -626,7 +625,7 @@ i386_remove_watchpoint (struct target_ops *self,
     }
 
   if (retval == 0)
-    i386_update_inferior_debug_regs (&local_state);
+    i386_update_inferior_debug_regs (state, &local_state);
 
   if (debug_hw_points)
     i386_show_dr (state, "remove_watchpoint", addr, len, type);
@@ -761,7 +760,7 @@ i386_insert_hw_breakpoint (struct target_ops *self, struct gdbarch *gdbarch,
                                               addr, len_rw) ? EBUSY : 0;
 
   if (retval == 0)
-    i386_update_inferior_debug_regs (&local_state);
+    i386_update_inferior_debug_regs (state, &local_state);
 
   if (debug_hw_points)
     i386_show_dr (state, "insert_hwbp", addr, 1, hw_execute);
@@ -787,7 +786,7 @@ i386_remove_hw_breakpoint (struct target_ops *self, struct gdbarch *gdbarch,
                                               addr, len_rw);
 
   if (retval == 0)
-    i386_update_inferior_debug_regs (&local_state);
+    i386_update_inferior_debug_regs (state, &local_state);
 
   if (debug_hw_points)
     i386_show_dr (state, "remove_hwbp", addr, 1, hw_execute);