2009-07-31 Michael Snyder <msnyder@vmware.com>

	* remote.c (store_register_using_P): Make error msg more verbose.
	(store_registers_using_G): Ditto.
	(fetch_register_using_p): Ditto.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8a62b36f5af855c643a71aad0184414aa8d9156c..c8c06f274fbf59f45da7ebd4a66d682f23d852c7 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -2,7 +2,11 @@
 
        * remote.c (store_register_using_P): Call putpkt and getpkt
        directly instead of calling remote_send.
-       (store_register_using_G): Ditto.
+       (store_registers_using_G): Ditto.
+       (fetch_register_using_p): Ditto.
+
+       * remote.c (store_register_using_P): Make error msg more verbose.
+       (store_registers_using_G): Ditto.
        (fetch_register_using_p): Ditto.
 
 2009-07-31  Ulrich Weigand  <uweigand@de.ibm.com>

diff --git a/gdb/remote.c b/gdb/remote.c
index d120f8c4b105780881033f5c1a1b0fdac807b743..e30e69969dd59f4391a4536b9b401a83c16acb66 100644 (file)
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -4885,8 +4885,10 @@ fetch_register_using_p (struct regcache *regcache, struct packet_reg *reg)
     case PACKET_UNKNOWN:
       return 0;
     case PACKET_ERROR:
-      error (_("Could not fetch register \"%s\""),
-            gdbarch_register_name (get_regcache_arch (regcache), reg->regnum));
+      error (_("Could not fetch register \"%s\"; remote failure reply '%s'"),
+            gdbarch_register_name (get_regcache_arch (regcache), 
+                                   reg->regnum), 
+            buf);
     }
 
   /* If this register is unfetchable, tell the regcache.  */
@@ -5151,8 +5153,8 @@ store_register_using_P (const struct regcache *regcache,
     case PACKET_OK:
       return 1;
     case PACKET_ERROR:
-      error (_("Could not write register \"%s\""),
-            gdbarch_register_name (gdbarch, reg->regnum));
+      error (_("Could not write register \"%s\"; remote failure reply '%s'"),
+            gdbarch_register_name (gdbarch, reg->regnum), rs->buf);
     case PACKET_UNKNOWN:
       return 0;
     default:
@@ -5195,7 +5197,8 @@ store_registers_using_G (const struct regcache *regcache)
   putpkt (rs->buf);
   getpkt (&rs->buf, &rs->buf_size, 0);
   if (packet_check_result (rs->buf) == PACKET_ERROR)
-    error (_("Could not write registers"));
+    error (_("Could not write registers; remote failure reply '%s'"), 
+          rs->buf);
 }
 
 /* Store register REGNUM, or all registers if REGNUM == -1, from the contents