From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2230 invoked by alias); 10 May 2006 18:26:50 -0000 Received: (qmail 2221 invoked by uid 22791); 10 May 2006 18:26:49 -0000 X-Spam-Check-By: sourceware.org Received: from nitzan.inter.net.il (HELO nitzan.inter.net.il) (192.114.186.20) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 10 May 2006 18:26:47 +0000 Received: from HOME-C4E4A596F7 (IGLD-80-230-229-172.inter.net.il [80.230.229.172]) by nitzan.inter.net.il (MOS 3.7.3-GA) with ESMTP id DIC62990 (AUTH halo1); Wed, 10 May 2006 21:26:43 +0300 (IDT) Date: Wed, 10 May 2006 18:44:00 -0000 Message-Id: From: Eli Zaretskii To: gdb@sourceware.org In-reply-to: <20060509230123.GA19291@nevyn.them.org> (message from Daniel Jacobowitz on Tue, 9 May 2006 19:01:23 -0400) Subject: Re: [PROPOSAL] Checking for supported packets - revised Reply-to: Eli Zaretskii References: <20060314021526.GA802@nevyn.them.org> <20060321051221.GA15578@nevyn.them.org> <20060330215247.GA9062@nevyn.them.org> <20060331135859.GA27522@nevyn.them.org> <20060331141958.GA28073@nevyn.them.org> <20060509230123.GA19291@nevyn.them.org> X-IsSubscribed: yes Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org X-SW-Source: 2006-05/txt/msg00110.txt.bz2 > Date: Tue, 9 May 2006 19:01:23 -0400 > From: Daniel Jacobowitz > > Here are the new commands currently in my patch, and the updated > protocol documentation. If no one has any additional comments about > it, I'll submit the corresponding code soon. The text is okay with me, but please take care of these minor gotchas: > +@item qSupported @r{[}:@var{feature} @r{[};@var{feature}@r{]}... @r{]} ^^^ This should use @dots{}, not literal dots. > +@cindex supported packets, remote query > +@cindex features of the remote protocol > +@cindex @samp{qSupported} packet > +Tell the remote target about features supported by @value{GDBN}, and > +query it for features it supports. This packet allows @value{GDBN} > +and the remote target to take advantage of each others' features. > +It also can eliminate excessive round trips when connecting to > +a target; on high-latency links, a single @samp{qSupported} packet > +is faster than a series of probe packets for unsupported packets. > + > +No values of @var{feature} are defined yet. Is there any way to somehow mark this last sentence, so that we will remove it as soon as at least one feature is defined? I'm afraid we will forget. > +Currently, all remote packets which are not mentioned in the response > +will be probed individually, just as if the @samp{qSupported} query > +was not supported. In the future, some new packets may be added to Same here. > +@item @var{name}? > +The remote protocol packet @var{name} may be supported, and @value{GDBN} > +should attempt to detect the packet when it is needed. "attempt to detect the packet"? Perhaps it's better to say "attempt to detect whether the packet is supported". > +The name of a packet which can be marked as supported or unsupported > +is the text of the packet in this documentation, up to but not > +including the first punctuation character or variable. For example, a > +target which supports hardware watchpoints but not hardware > +breakpoints might report @samp{Z0-;Z1-;Z2+;Z3+;Z4+}. An exception is > +made for @samp{qPart:@var{object}} packets; the name of the packet > +includes the @var{object}, but not the @var{annex}. Individual > +@samp{qPart} objects types must be reported separately. Please add a cross-reference to the two places where the two example packets are described, so that the reader could consult them in case they don't remember the packets' formats by heart.