From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18715 invoked by alias); 21 May 2007 12:00:04 -0000 Received: (qmail 18575 invoked by uid 22791); 21 May 2007 11:59:58 -0000 X-Spam-Check-By: sourceware.org Received: from return.false.org (HELO return.false.org) (66.207.162.98) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 21 May 2007 11:59:55 +0000 Received: from return.false.org (localhost [127.0.0.1]) by return.false.org (Postfix) with ESMTP id AAE6E4B267; Mon, 21 May 2007 06:59:50 -0500 (CDT) Received: from caradoc.them.org (dsl093-172-095.pit1.dsl.speakeasy.net [66.93.172.95]) by return.false.org (Postfix) with ESMTP id 6E7B84B262; Mon, 21 May 2007 06:59:48 -0500 (CDT) Received: from drow by caradoc.them.org with local (Exim 4.67) (envelope-from ) id 1Hq6YF-0007oN-Ss; Mon, 21 May 2007 07:59:47 -0400 Date: Mon, 21 May 2007 12:00:00 -0000 From: Daniel Jacobowitz To: Mark Kettenis , gdb-patches@sourceware.org, Kevin Buettner Subject: Re: [rfc / remote protocol] Remote shared library events Message-ID: <20070521115947.GA30013@caradoc.them.org> Mail-Followup-To: Mark Kettenis , gdb-patches@sourceware.org, Kevin Buettner References: <20070509201627.GA23422@caradoc.them.org> <200705102134.l4ALYpQU016401@brahms.sibelius.xs4all.nl> <20070515132826.GA18923@caradoc.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070515132826.GA18923@caradoc.them.org> User-Agent: Mutt/1.5.15 (2007-04-09) 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: 2007-05/txt/msg00328.txt.bz2 On Tue, May 15, 2007 at 09:28:26AM -0400, Daniel Jacobowitz wrote: > Here are just the infrun parts. They've gained a test case for the > trickiest part, a cleanup, a better changelog, and an explanation. > > First of all, I don't remember why I was testing > inferior_ignoring_startup_exec_events in the last version of this > patch. I discovered that now there are no references to it (there may > have been one when I wrote it). I didn't really need it, so I garbage > collected it. > > Secondly, I needed to make TARGET_WAITKIND_LOADED support work with > the current solib.c implementation. You can tell it hasn't been used > in a while since it was conditioned on "#ifdef SOLIB_ADD"; there are > almost no definitions of that left. > > Third, while debugging shared library support for SymbianOS I kept > encountering an unpleasant surprise. I would disconnect from the > remote target when it was stopped at a load event (usually because my > GDB crashed :-). When I reconnected, GDB would automatically resume > the program! That's where the ugly stop_soon test was in the last > version of this. For this version, I reviewed every setting and use > of stop_soon and decided that none of them quite matched the case in > stop_remote, so I added a new STOP_QUIETLY_REMOTE (and some comments > about it). This is just like STOP_QUIETLY_NO_SIGSTOP except that it > does not need to hide a SIGSTOP if it finds one. It means we're > connecting to the target for the first time, so we shouldn't decide to > resume it without giving the user a chance to look around. > > I hope this version is clearer; what do you think of it? Hi Mark, Kevin, Did either of you have a chance to look at this? You both had concerns about the infrun parts of the combined patch, so I'd like to make sure this version is better and clearer before I go on with the other pieces. -- Daniel Jacobowitz CodeSourcery