From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30594 invoked by alias); 20 Mar 2012 13:25:47 -0000 Received: (qmail 30579 invoked by uid 22791); 20 Mar 2012 13:25:45 -0000 X-SWARE-Spam-Status: No, hits=-1.6 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 20 Mar 2012 13:25:12 +0000 Received: from svr-orw-fem-01.mgc.mentorg.com ([147.34.98.93]) by relay1.mentorg.com with esmtp id 1S9z3n-0004s8-C9 from Hui_Zhu@mentor.com ; Tue, 20 Mar 2012 06:25:11 -0700 Received: from SVR-ORW-FEM-05.mgc.mentorg.com ([147.34.97.43]) by svr-orw-fem-01.mgc.mentorg.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675); Tue, 20 Mar 2012 06:25:11 -0700 Received: from [127.0.0.1] (147.34.91.1) by svr-orw-fem-05.mgc.mentorg.com (147.34.97.43) with Microsoft SMTP Server id 14.1.289.1; Tue, 20 Mar 2012 06:25:10 -0700 Message-ID: <4F688533.1050705@mentor.com> Date: Tue, 20 Mar 2012 13:25:00 -0000 From: Hui Zhu User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120310 Thunderbird/11.0 MIME-Version: 1.0 To: Eli Zaretskii CC: , Subject: Re: [PATCH] Add autoload-breakpoints [3/7] ReportAsync-doc References: <4F6450D2.9030109@mentor.com> <83y5qz359r.fsf@gnu.org> In-Reply-To: <83y5qz359r.fsf@gnu.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-03/txt/msg00739.txt.bz2 Hi Eli, Thank you for your help. On 03/17/12 18:12, Eli Zaretskii wrote: >> Date: Sat, 17 Mar 2012 16:52:34 +0800 >> From: Hui Zhu >> CC: Stan Shebs >> >> This is patch for the doc about ReportAsync. > > Thanks. > >> +The @value{GDBN} remote serial protocol includes @dfn{reportasync}, >> +packets that the stub can report status or do an operation in any time. > > I don't understand the "do an operation in any time" part. What does > it mean in practice? It is means that after gdb connected with the stub, the stub can report when the inferior is running and GDB is waiting it, or inferior is stoped and GDB just control it. It make the stub can report the status for any time. I didn't have good words on this part to make it more clear. Could you help me with it? > > When the stub wants to send a @samp{reportasync} packet, it first sends > a handshake packet @samp{^} to @value{GDBN}. > >> +If @value{GDBN} reply a @samp{^}, the shake hands is successful. > > If @value{GDBN} replies with a @samp{^}, the handshake is successful. > >> +If not, the shake hands is fail and this packet need to be handle >> +as a simple packet. > > If not, the handshake fails, and the @samp{reportasync} packet needs > to be handled as a simple packet. I will fix them. > > And I don't understand what you mean by "needs to be handled as a > simple packet". If that's a reference to something discussed earlier > in the manual, please add here a cross-reference to that part. That means when the stub send a shake hands package ^ and wait the reply, if it got a package that is not ^. Then the shake hands is fail. It need give up this shake hands and begin to handle this packet that it just get. For example: Stub send a ^ to GDB. It readchar, and got a $. It should not drop this $, After that, it will got "c#csum". It need handle the package "$c#csum". My words in the doc is not very clear, could you help me with it? > >> +After shake hands, the stub can send packet to @value{GDBN}, the packet >> +format is same with simple remote serial protocol. > > After the handshake, the stub can send @samp{reportasync} packets to > @value{GDBN}, using the same packet format as in simple remote serial > protocol. > > Is this only for the failed handshake? If so, perhaps we need to tell > what happens when the handshake succeeds. No, this is for success. What about I change it to: After the handshake success, the stub can send @samp{reportasync} packets to @value{GDBN}, using the same packet format as in simple remote serial protocol. Best, Hui