Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* PATCH: Add xmlRegisters= to qsupported query
@ 2010-03-28 20:48 H.J. Lu
  2010-03-28 20:52 ` H.J. Lu
  0 siblings, 1 reply; 17+ messages in thread
From: H.J. Lu @ 2010-03-28 20:48 UTC (permalink / raw)
  To: GDB

Hi,

This patch adds xmlRegisters= to qsupported query.  Any comments?

Thanks.


H.J.
----
gdb/

2010-03-28  H.J. Lu  <hongjiu.lu@intel.com>

	* i386-tdep.c: Include "remote.h".
	(i386_gdbarch_init): Call register_remote_support_xml.

	* remote.c (remote_support_xml): New.
	(register_remote_support_xml): Likewise.
	(remote_query_supported): Support remote_support_xml.

	* remote.h (register_remote_support_xml): New.

gdb/doc/

2010-03-28  H.J. Lu  <hongjiu.lu@intel.com>

	* gdb.texinfo (General Query Packets): Add xmlRegisters.

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 56dbe5d..ad84d2c 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -30603,6 +30603,12 @@ extensions to the remote protocol.  @value{GDBN} does not use such
 extensions unless the stub also reports that it supports them by
 including @samp{multiprocess+} in its @samp{qSupported} reply.
 @xref{multiprocess extensions}, for details.
+
+@item xmlRegisters
+This feature indicates that @value{GDBN} supports supports the XML
+target description.  If the stub sees @samp{xmlRegisters=} with
+target specfic strings separated by comma, it can send @value{GDBN}
+the XML target description.
 @end table
 
 Stubs should ignore any unknown values for
diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c
index 83275ac..d7af84a 100644
--- a/gdb/i386-tdep.c
+++ b/gdb/i386-tdep.c
@@ -44,6 +44,7 @@
 #include "value.h"
 #include "dis-asm.h"
 #include "disasm.h"
+#include "remote.h"
 
 #include "gdb_assert.h"
 #include "gdb_string.h"
@@ -5943,6 +5944,9 @@ i386_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
   set_gdbarch_fast_tracepoint_valid_at (gdbarch,
 					i386_fast_tracepoint_valid_at);
 
+  /* Tell remote stub that we support XML target description.  */
+  register_remote_support_xml ("x86");
+
   return gdbarch;
 }
 
diff --git a/gdb/remote.c b/gdb/remote.c
index 0c791aa..9637f53 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -3464,6 +3464,41 @@ static struct protocol_feature remote_protocol_features[] = {
     PACKET_bs },
 };
 
+static char *remote_support_xml;
+
+/* Register string appended to "xmlRegisters=" in qSupported query.  */
+
+void
+register_remote_support_xml (const char *xml)
+{
+  if (remote_support_xml == NULL)
+    remote_support_xml = xstrdup (xml);
+  else
+    {
+      const char *str = strstr (remote_support_xml, xml);
+      int append;
+
+      if (str)
+	{
+	  if (str != remote_support_xml && *(str - 1) != ',')
+	    append = 0;
+	  else
+	    {
+	      size_t len = strlen (xml);
+	      append = (xml[len] != ','
+			&& xml[len] != ';'
+			&& xml[len] != '\0');
+	    }
+	}
+      else
+	append = 1;
+
+      if (append)
+	remote_support_xml = concat (xml, ",",  remote_support_xml,
+				     NULL);
+    }
+}
+
 static void
 remote_query_supported (void)
 {
@@ -3483,14 +3518,35 @@ remote_query_supported (void)
   if (remote_protocol_packets[PACKET_qSupported].support != PACKET_DISABLE)
     {
       const char *qsupported = gdbarch_qsupported (target_gdbarch);
-      if (qsupported)
+      if (qsupported || remote_support_xml)
 	{
-	  char *q;
+	  char *q, *x;
+	  char *p = NULL;
+	  const char *m;
+
 	  if (rs->extended)
-	    q = concat ("qSupported:multiprocess+;", qsupported, NULL);
+	    m = "multiprocess+;";
 	  else
-	    q = concat ("qSupported:", qsupported, NULL);
+	    m = "";
+
+	  if (remote_support_xml)
+	    {
+	      if (qsupported)
+		p = concat ("xmlRegisters=", remote_support_xml,
+			    ";", NULL);
+	      else
+		p = concat ("xmlRegisters=", remote_support_xml, NULL);
+	      x = p;
+	    }
+	  else
+	    x = "";
+
+	  q = concat ("qSupported:", m, x,
+		      qsupported ? qsupported : "",
+		      NULL);
 	  putpkt (q);
+	  if (p)
+	    xfree (p);
 	  xfree (q);
 	}
       else
diff --git a/gdb/remote.h b/gdb/remote.h
index a82d0d7..d480e26 100644
--- a/gdb/remote.h
+++ b/gdb/remote.h
@@ -66,6 +66,7 @@ extern void (*deprecated_target_wait_loop_hook) (void);
 
 void register_remote_g_packet_guess (struct gdbarch *gdbarch, int bytes,
 				     const struct target_desc *tdesc);
+void register_remote_support_xml (const char *);
 
 void remote_file_put (const char *local_file, const char *remote_file,
 		      int from_tty);


^ permalink raw reply	[flat|nested] 17+ messages in thread
end of thread, other threads:[~2010-03-30 15:09 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-03-28 20:48 PATCH: Add xmlRegisters= to qsupported query H.J. Lu
2010-03-28 20:52 ` H.J. Lu
2010-03-28 21:27   ` Pedro Alves
2010-03-28 22:56     ` H.J. Lu
2010-03-28 23:48     ` H.J. Lu
2010-03-28 23:46   ` H.J. Lu
2010-03-29  6:40     ` Eli Zaretskii
2010-03-29 14:13       ` H.J. Lu
2010-03-29 14:32         ` Eli Zaretskii
2010-03-29 14:47           ` H.J. Lu
2010-03-29 14:55     ` H.J. Lu
2010-03-29 16:19       ` H.J. Lu
2010-03-29 17:49         ` Mark Kettenis
2010-03-29 18:02         ` H.J. Lu
2010-03-30 13:55           ` Pedro Alves
2010-03-30 14:58           ` H.J. Lu
2010-03-30 15:09             ` Pedro Alves

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox