From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27155 invoked by alias); 9 Sep 2008 20:43:58 -0000 Received: (qmail 27147 invoked by uid 22791); 9 Sep 2008 20:43:57 -0000 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 09 Sep 2008 20:43:23 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 2B9702A9696; Tue, 9 Sep 2008 16:43:21 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id x0AtJT-B7bsd; Tue, 9 Sep 2008 16:43:21 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id D9C522A96A2; Tue, 9 Sep 2008 16:43:20 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id 720FBE7ACD; Tue, 9 Sep 2008 22:43:18 +0200 (CEST) Date: Tue, 09 Sep 2008 20:43:00 -0000 From: Joel Brobecker To: Ulrich Weigand Cc: gdb-patches@sourceware.org Subject: Re: [rfc] Fix removing breakpoint from shared library race Message-ID: <20080909204318.GG12222@adacore.com> References: <20080818114617.GH16894@adacore.com> <200808181414.m7IEEZtO010275@d12av02.megacenter.de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200808181414.m7IEEZtO010275@d12av02.megacenter.de.ibm.com> User-Agent: Mutt/1.4.2.2i 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: 2008-09/txt/msg00197.txt.bz2 Ulrich, Sorry for the late reply, but: > This is the situation that my patch attempts to address, by having the > remove_breakpoints not fail. At some later point, either when GDB > actually gets to process the solib event, or when GDB tries to > re-insert the breakpoints, GDB will properly recognize that the > library has been unloaded and sets the state to solib_disabled. Just wanted to let you know that you patch makes complete sense to me. As you pointed out in your first message, this is in fact very similar to what we do in insert_bp_location. -- Joel