From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lndn.lancelotsix.com (vps-42846194.vps.ovh.net [IPv6:2001:41d0:801:2000::2400]) by sourceware.org (Postfix) with ESMTPS id CCBAB3858C74 for ; Wed, 9 Mar 2022 22:29:12 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org CCBAB3858C74 Received: from Plymouth (unknown [IPv6:2a02:390:9086:0:7955:7ef5:70af:a502]) by lndn.lancelotsix.com (Postfix) with ESMTPSA id 6F4FE81859; Wed, 9 Mar 2022 22:29:11 +0000 (UTC) Date: Wed, 9 Mar 2022 22:29:06 +0000 From: Lancelot SIX To: Pedro Alves Cc: Simon Marchi , gdb-patches@sourceware.org Subject: Re: [PATCH 09/11] Ensure EXIT is last event, gdb/linux Message-ID: <20220309222906.c4twjzet6w6pkvuc@Plymouth> References: <20220303144020.3601082-1-pedro@palves.net> <20220303144020.3601082-10-pedro@palves.net> <308a1654-45a3-3886-0094-eff4300c15a8@simark.ca> <20220309002120.bbmp67zkiiwk4hpr@Plymouth> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.11 (lndn.lancelotsix.com [0.0.0.0]); Wed, 09 Mar 2022 22:29:11 +0000 (UTC) X-Spam-Status: No, score=-2.9 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_SBL_CSS, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 09 Mar 2022 22:29:14 -0000 > > So overall, I'm thinking of dropping this patch from the series for now, as it's not really > needed AFAICT, and is incomplete. > > I've dropped patches #9 and #10 locally, and left the testcase (patch #11) running in a loop > testing both native and gdbserver for over an hour and saw no failures. Any objections > to pushing the series without those two patches? WHIW, that would be fine by me. As you pointed out the real fix for the issues is provided by other patches of this series anyway. Lancelot.