From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27105 invoked by alias); 8 Sep 2004 14:38:10 -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 27084 invoked from network); 8 Sep 2004 14:38:07 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sourceware.org with SMTP; 8 Sep 2004 14:38:07 -0000 Received: from drow by nevyn.them.org with local (Exim 4.34 #1 (Debian)) id 1C53aE-0007BU-Nl; Wed, 08 Sep 2004 10:38:02 -0400 Date: Wed, 08 Sep 2004 14:38:00 -0000 From: Daniel Jacobowitz To: "M.M. Kettenis" Cc: Michael Snyder , Andrew Cagney , gdb-patches@sources.redhat.com, kettenis@gnu.org Subject: Re: [rfa] Delete keep_thread_db Message-ID: <20040908143802.GA23406@nevyn.them.org> Mail-Followup-To: "M.M. Kettenis" , Michael Snyder , Andrew Cagney , gdb-patches@sources.redhat.com, kettenis@gnu.org References: <7318322520085180@weblx058.utsp.utwente.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7318322520085180@weblx058.utsp.utwente.nl> User-Agent: Mutt/1.5.5.1+cvs20040105i X-SW-Source: 2004-09/txt/msg00124.txt.bz2 On Wed, Sep 08, 2004 at 07:00:00AM +0000, M.M. Kettenis wrote: > Michael Snyder wrote: > > > Andrew Cagney wrote: > > > It would appear that, at one stage, this variable had something to do > > > with corefiles, static programs and threads. That's no longer the case > > > so this deletes the stray global. > > > > > > Tested on FC3 with no regressions, > > > > > > Ok? > > > Andrew > > > > I would have to defer to Mark Kettenis, since he added the > > variable, and from the ChangeLogs, seems to be the only one > > who's touched it. > > The code defenitely was there to deal with static binaries. Andrew, > how did you come to the conclusion that it is no longer useful? > AFAIK there is no static threads test in the testsuite. FYI, I believe this is what happened: keep_thread_db was added to deal with static binaries. It was necessary. Core file debugging did not work. Core file debugging was then fixed, which broke static binaries. keep_thread_db handles the fact that we only used the load-symfile event to initialize thread_db, not any inferior_created event also. If you use an observer after inferior creation for this, it won't be necessary. I don't see much point in removing it earlier than fixing the bug; it's not "stray" yet, just broken. But do whichever you want. > Anyway, I don't care about this code anymore. I've given up on > debugging-with threads on Linux. thread-db.c should renamed into > linux-thread.c or somesuch. It, and lin-lwp.c have become a mess of > workarounds around workarounds around workarounds that probably only > work if you run a specific combination of glibc and patched kernel. I do care about this code. It needs some major surgery, but it is still useful (and, while fragile, not quite as fragile as all that). If no one else is interested in maintaining it then I would like to. -- Daniel Jacobowitz