From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9452 invoked by alias); 20 Mar 2012 17:36:45 -0000 Received: (qmail 9161 invoked by uid 22791); 20 Mar 2012 17:36:42 -0000 X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout21.012.net.il (HELO mtaout21.012.net.il) (80.179.55.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 20 Mar 2012 17:36:24 +0000 Received: from conversion-daemon.a-mtaout21.012.net.il by a-mtaout21.012.net.il (HyperSendmail v2007.08) id <0M17003001YA3B00@a-mtaout21.012.net.il> for gdb-patches@sourceware.org; Tue, 20 Mar 2012 19:35:52 +0200 (IST) Received: from HOME-C4E4A596F7 ([77.124.29.76]) by a-mtaout21.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0M17002TN27PDUM0@a-mtaout21.012.net.il>; Tue, 20 Mar 2012 19:35:50 +0200 (IST) Date: Tue, 20 Mar 2012 17:36:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH] Add autoload-breakpoints [7/7] autoload-breakpoints doc In-reply-to: <4F68A53F.6040103@mentor.com> To: Hui Zhu Cc: gdb-patches@sourceware.org, stan_shebs@mentor.com Reply-to: Eli Zaretskii Message-id: <83mx7b18g5.fsf@gnu.org> References: <4F6450FF.1070103@mentor.com> <83wr6j33u1.fsf@gnu.org> <4F68A53F.6040103@mentor.com> 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/msg00748.txt.bz2 > Date: Tue, 20 Mar 2012 23:41:51 +0800 > From: Hui Zhu > CC: , > > > @value{GDBN} can receive the information about autoload-breakpoints > > from the remote stub, if the stub supports that. > > > > Also, as we don't mention autoload-breakpoints anywhere else in the > > manual, we need to explain here what these breakpoints are and why > > controlling them is useful. > > What about: > Some stub support set breakpoints with itself, with this function, GDB > can handle the breakpoint that set by the stub better. I think we need to explain more. How about using what Yao wrote yesterday: Prior to GDB connects to remote stub, some breakpoints might have been already set in target, either by a prior GDB session or by the program itself through some special system API. When GDB connects to target, it should handle these existing breakpoints from remote stub. We call these breakpoints as "auotload breakpoints". The other use case we want to support in GDB is that inferior may delete/clear breakpoints on its own, and remote stub gets GDB notified, and GDB can take some actions accordingly. We believe such reliable report is useful to GDB, and generalize it, called "ReportAsync". The connection between "ReportAsync" and "autoload breakpoint" is when inferior/stub delete breakpoints, sends async report to GDB about removed breakpoints. If this is accurate, it just needs some rewording for it to be a good introduction to this topic. > >> +@item set breakpoint autoload merge > >> +If this option is merge, the autoload-breakpoints of @value{GDBN} > > ^^^^^ > > @samp{merge} > > > >> +and stub will merge together when @value{GDBN} connect to stub. > > > > I don't understand what it means to "merge" breakpoints. > > It means that merge together. Keep both the autoload-breakpoints in the > GDB and the stub. Then please don't use "merge", its normal meaning gets in the way. Instead, say this: the autoload-breakpoints of both @value{GDBN} and of the stub will be kept. > > Please don't add subsections without a node (and a menu in the parent > > section). Such subsections cannot be navigated with on-line Info > > commands. > > Should I change it to: > @node Autoload-breakpoints base format > @subsection Autoload-breakpoints base format > @cindex Autoload-breakpoints base format Yes, but (a) you also need a menu in the parent node, and (b) @cindex entries should always begin with a lower-case letter. > OK. I will do it according to your comments. Thank you.