From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14043 invoked by alias); 28 Oct 2004 19:52:12 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 14026 invoked from network); 28 Oct 2004 19:52:11 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sourceware.org with SMTP; 28 Oct 2004 19:52:11 -0000 Received: from drow by nevyn.them.org with local (Exim 4.34 #1 (Debian)) id 1CNGJc-0001Yo-NJ; Thu, 28 Oct 2004 15:52:08 -0400 Date: Thu, 28 Oct 2004 19:52:00 -0000 From: Daniel Jacobowitz To: Jeff Johnston Cc: Andrew Cagney , gdb-patches@sources.redhat.com Subject: Re: [RFA]: Watchpoints per thread patch Message-ID: <20041028195208.GA5882@nevyn.them.org> Mail-Followup-To: Jeff Johnston , Andrew Cagney , gdb-patches@sources.redhat.com References: <4175A9C9.8040300@redhat.com> <41769FF3.7010801@gnu.org> <20041020173035.GA26622@nevyn.them.org> <418022DE.204@redhat.com> <20041027224155.GA3485@nevyn.them.org> <41802C79.7040103@redhat.com> <20041028133333.GA32744@nevyn.them.org> <41814CCA.7080805@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <41814CCA.7080805@redhat.com> User-Agent: Mutt/1.5.5.1+cvs20040105i X-SW-Source: 2004-10/txt/msg00473.txt.bz2 On Thu, Oct 28, 2004 at 03:47:22PM -0400, Jeff Johnston wrote: > Daniel Jacobowitz wrote: > >On Wed, Oct 27, 2004 at 07:17:13PM -0400, Jeff Johnston wrote: > > > >>Were you thinking of add_thread()? If so, we would have to move the > >>calls to add_thread so they never occur before an attach because the > >>low-level observers will need the thread already attached. > > > > > >Oh, that's a good point. Do you think that's a reasonable change to > >make? > > > > It is a can of worms. I can move the add_thread call in attach_thread > easily enough, but there are other calls to add_thread strewn about. For > example, corelow.c calls add_thread as does infrun.c when it finds a new > process. I certainly don't see it being valid for either of these > scenarios to insert/remove all watchpoints. My personal preference would > be to leave it where it is for now. There are two separate questions here: - When do we need to be adding and removing watchpoints from threads on GNU/Linux? - When should an observer named "new_thread" be called? If it's not valid to do the former action at all the latter points, then it's not the right observer to be using. The code in infrun is never reached for native GNU/Linux threads, btw; I'm not sure which targets if any do reach it. I don't believe remote GNU/Linux threads do either. -- Daniel Jacobowitz