Commit: a21dae9c0bbd529cff3265dabf7585497092a42a - binutils_os216 (git)

GNU Binutils with patches for OS216

Commit MetaInfo

• Revision: a21dae9c0bbd529cff3265dabf7585497092a42a (tree)
• Time: 2008-06-22 11:36:30
• Author: Michael Snyder <msnyder@vmwa...>
• Commiter: Michael Snyder

Log Message

2008-06-21 Michael Snyder <[e-mail]>

Add support for 'R' packet (restart).
* gdbfreeplay-back.c (handle_special_case). Add support for 'R'.
Remove support for "monitor gdbfreeplay-echo".

Change Summary

• modified: gdb/gdbserver/ChangeLog
• modified: gdb/gdbserver/gdbfreeplay-back.c

Incremental Difference

--- a/gdb/gdbserver/ChangeLog
+++ b/gdb/gdbserver/ChangeLog
@@ -1,5 +1,9 @@
 2008-06-21  Michael Snyder  <[e-mail]>
 
+	Add support for 'R' packet (restart).
+	* gdbfreeplay-back.c (handle_special_case).  Add support for 'R'.
+	Remove support for "monitor gdbfreeplay-echo".
+
 	Add support for 'O' packet output.
 	* remote-breakpoint.h (PLAY_O_PACKETS): New enum const.
 	* gdbfreeplay-back.c (StopFrame): Discard gpos, replace with Opos.

--- a/gdb/gdbserver/gdbfreeplay-back.c
+++ b/gdb/gdbserver/gdbfreeplay-back.c
@@ -700,23 +700,16 @@ handle_special_case (FILE *infile, int fd, char *request)
       return OK;
     }
 
-  /* Handle "monitor gdbfreeplay-echo"
-     (just to get a handle on the 'O' message).  */
-  if ((p = strstr (request, monitor_echo)) != NULL)
+  /* Handle 'R' (restart) request.
+     This is an extended-remote request that is not really used any more,
+     but we can send it out-of-band (using "maint packet") to effectively
+     set the replay buffer back to the beginning.  */
+  if (strstr (request, "$R#52") != NULL)
     {
-      /* OK, this will be an ascii-fied string.  */
-      p += strlen (monitor_echo);
-      /* Skip spaces */
-      while (p[0] == '2' && p[1] == '0')
-	p += 2;	/* skip a space */
-
-      while (p[0] && p[0] != '#')
-	{
-	  c  = hex_to_int (*p++) << 4;
-	  c += hex_to_int (*p++);
-	  fputc (c, stdout);
-	}
-      fprintf (stdout, "\n");
+      /* Reset replay buffer to beginning.  */
+      /* NOTE: gdb doesn't know about target state changing, so
+	 if you use this, you must accompany it with "flushregs".  */
+      cur_frame = 0;
       return OK;
     }