From a2061976f9244eecfd3335c982b2edb3c31e34f6 Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Thu, 22 Mar 2001 14:59:09 +0000 Subject: [PATCH] Cleanup remote protocol section. Add notes on ``!'', ``q'' and the non-use of the continue address. --- gdb/ChangeLog | 5 ++++ gdb/TODO | 80 ++++++++++++++++++++++++++++++++++----------------- 2 files changed, 58 insertions(+), 27 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 3090c95cc3..322ac96bc8 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2001-03-22 Andrew Cagney + + * TODO (Remote Protocol Support): Review. Add notes about ``!'', + non-use of a continue address and typical use of ``q'' packet. + 2001-03-22 Eli Zaretskii Make DJGPP use the new unified support for hardware diff --git a/gdb/TODO b/gdb/TODO index afbcd5e2f7..d3d8919934 100644 --- a/gdb/TODO +++ b/gdb/TODO @@ -325,16 +325,7 @@ to any specific release. -- -Revised UDP support (was: Re: [Fwd: [patch] UDP transport support]) -http://sourceware.cygnus.com/ml/gdb-patches/2000-04/msg00000.html - -(Broken) support for GDB's remote protocol across UDP is to be -included in the follow-on release. - -It should be noted that UDP can only work when the [Gg] packet fits in -a single UDP packet. - -There is also much debate over the merit of this. +Rename read_register{,_pid}() to read_unsigned_register{,_pid}(). -- @@ -918,19 +909,6 @@ Robert Lipe writes: > practical matter, the current thread support is somewhat more annoying > than when GDB was thread-unaware. --- - -Migrate qfThreadInfo packet -> qThreadInfo. (Andrew Cagney) - -Add support for packet enable/disable commands with these thread -packets. General cleanup. - -[PATCH] Document the ThreadInfo remote protocol queries -http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00832.html - -[PATCH] "info threads" queries for remote.c -http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00831.html - -- Language Support @@ -972,6 +950,32 @@ Get DEC/Compaq to contribute their Modula-3 support. -- +Revised UDP support (was: Re: [Fwd: [patch] UDP transport support]) +http://sourceware.cygnus.com/ml/gdb-patches/2000-04/msg00000.html + +(Broken) support for GDB's remote protocol across UDP is to be +included in the follow-on release. + +It should be noted that UDP can only work when the [Gg] packet fits in +a single UDP packet. + +There is also much debate over the merit of this. + +-- + +Migrate qfThreadInfo packet -> qThreadInfo. (Andrew Cagney) + +Add support for packet enable/disable commands with these thread +packets. General cleanup. + +[PATCH] Document the ThreadInfo remote protocol queries +http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00832.html + +[PATCH] "info threads" queries for remote.c +http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00831.html + +-- + Remote protocol doco feedback. Too much feedback to mention needs to be merged in (901660). Search @@ -998,6 +1002,32 @@ fixed so that it builds on 16 bit machines. -- +Fix the ``!'' packet. + +JT reported that the existing targets do, in fact return ``OK'' so it +is possible to merge remote and extended-remote targets. + +-- + +Drop ``
'' from the [SsCc] packets. + +I don't think that GDB generates them so having it in the protocol is +silly. + +-- + +Fix doco on the ``q'' packet. + +It has evolved into a generic RPC. The notes should reflect this and, +perhaps, the ``Q'' packet can be deprecated. + +The doco should mention that ``OK'' is a valid packet response. + +The doco should explain why ``OK'' needs to be a valid packet +response. + +-- + Add the cycle step command. http://sourceware.cygnus.com/ml/gdb/2000-q1/msg00237.html @@ -1014,10 +1044,6 @@ on the fly - should it be returned in the ``T'' packet? Underlying problem is that the register file is target endian. If the target endianess changes gdb doesn't know. --- - -Rename read_register{,_pid}() to read_unsigned_register{,_pid}(). - -- Symbol Support