From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-x332.google.com (mail-wm1-x332.google.com [IPv6:2a00:1450:4864:20::332]) by sourceware.org (Postfix) with ESMTPS id 5BB0D3858D1E for ; Tue, 31 Jan 2023 07:03:12 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 5BB0D3858D1E Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com Received: by mail-wm1-x332.google.com with SMTP id m16-20020a05600c3b1000b003dc4050c94aso7016842wms.4 for ; Mon, 30 Jan 2023 23:03:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=1XIbERNFHAXlllG/DJxUwH9gnnm1TXiToh0WRTh0EtE=; b=BGb6eGixseocVyafU2wanmkoH8Rj8uhzSbrCMkCBgiIW2SL7uik4b1l+vYZvLcnln1 vgM9m7ojcmql4wqfFFyEpK7dk3FXZngeHOJTf3aM7JajOyGgvSgD93G5pg7qO42CRR1x PAiouPcTT3j/lPnuoC7PSPMS1XR68j/5DOLnl1meTaTNdexGSVRkooy7UbWZVwmBdK+4 F7noBvydx8QyngId3aDWRbK8zrJsAobREI7Y0EASFQC/4vLiNISRSqUS5We3K3j50C/Z iDyxk1tpOnD/+ftJujYHxLjBU/1jD63pDvw12HrwY+rhfrD+s8DtCO9+VYlMqQEOWwL0 ajrA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=1XIbERNFHAXlllG/DJxUwH9gnnm1TXiToh0WRTh0EtE=; b=GBpdZCPBrGVB1LLDDfrYopIXsCqPQHTvp8eT5ScPxRRn6eYhmGSVwZnlkzoeeMIVBA HUYAM2imJjpCQwIrfC7B3RnCZljNnb6tTnnf/rZ8OK6TV2y+S3oDuVzifxO5QUBwczRW 0dHu/B3ry6WJw+PUYUB2b3KIPiwfEN9ZARY6LidQRdlRbELToetfeqFGz81gjZzLCfzC 2l4sEXafEh+ZfLVKV1C7l6vBKJ+KTeNZ82dVzTqMEpGgpMX/oEbs/HhqioxfljlKevsK EjLm27k2MnyjsurPtJpmD4ag6Otm9aoYTlGfYpxEI48BywKRfVxsuhwdITokWgpHtDXE HSPw== X-Gm-Message-State: AO0yUKXcxcq1DQRg+5+Lshy586K9mXGySUn2JBG4blWEfWIrve8gY82V O2mM1MyF7ZbyIuBdwISfG67F X-Google-Smtp-Source: AK7set+Qc7MvXSBXzyZl73N4PqP0+pTQpMB/DycJnIFntUnb2cZ5s7e2YlDxfBpza0UcWf9gApE5JQ== X-Received: by 2002:a1c:720c:0:b0:3dc:59e9:9285 with SMTP id n12-20020a1c720c000000b003dc59e99285mr6213601wmc.18.1675148591000; Mon, 30 Jan 2023 23:03:11 -0800 (PST) Received: from takamaka.gnat.com ([2a01:cb22:1d5:1100:1551:30d4:9a8e:30aa]) by smtp.gmail.com with ESMTPSA id o18-20020a05600c339200b003dc42d48defsm10882145wmp.6.2023.01.30.23.03.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Jan 2023 23:03:09 -0800 (PST) Received: by takamaka.gnat.com (Postfix, from userid 1000) id B020C8224B; Tue, 31 Jan 2023 11:03:07 +0400 (+04) Date: Tue, 31 Jan 2023 11:03:07 +0400 From: Joel Brobecker To: Luis Machado Cc: Joel Brobecker , Simon Marchi , Torbjorn SVENSSON , tom@tromey.com, gdb-patches@sourceware.org Subject: Re: GDB 13 release -- 2023-01-21 Update Message-ID: References: <95dc547e-59f8-95b3-903c-138d8842cea0@foss.st.com> <47dff98e-b714-2d72-bc3b-c14c4e0313d0@arm.com> <1689df81-6ab3-c167-24a4-415a86216b5a@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1689df81-6ab3-c167-24a4-415a86216b5a@arm.com> X-Spam-Status: No, score=-3.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hi Luis, On Mon, Jan 30, 2023 at 11:23:06AM +0000, Luis Machado wrote: > On 1/29/23 11:52, Joel Brobecker wrote: > > > > > > - With that said, the patch appears to simply add a cache, > > > > > > so the logic of it all doesn't appear to be extremely > > > > > > complicated. So I would rate the risk to be low. > > > > > > > > > > That's what it seems to me. > > > > > > > > > > I think another round of testing would be a good step to make sure there are no hidden bugs. > > > > My understanding is that the problem was making debugging borderline > > > > impossible. Or at least, really, really unpleasant. So I think it > > > > qualifies as a bug fix, and that the benefit is worth the risk. > > > > > > Right. Past a short number of frames, unwinding got really really slow. > > > > The part that I haven't been clear on is whether this affected everyone > > on Arm-32bit, or everyone on both Arm and AArch64, or just a subset > > of the users? > > Just a subset. > > It affects 32-bit m-profile Arm targets that report the additional stack pointers. This is either > the org.gnu.gdb.arm.m-system feature or the org.gnu.gdb.arm.secext feature. > > I'm fairly sure these features are only reported by emulators and bare-metal targets. > > Now, a 64-bit gdb can debug 32-bit Arm as well, so you could have a 64-bit gdb running into this too > if the target is a 32-bit m-profile Arm. But the target is still 32-bit Arm. > > This doesn't affect AArch64 at all. Thanks for the extra effort explaining the impact. This is much clear for me, now. With your permission, I'd like to copy the text above as a comment in the PR. Would that be OK? > > Regardless of the above, I agree we can backport. To avoid confusion, > > can I leave this to you, Luis, to confirm that the two patches I identified > > previously are the only two patches that need to be backported, and > > can you backport those for us, with a round of testing if you haven't > > done so already, just to double-check? > > Sure. I gave it a try with master and the test results look the same with or without the patches. > > Unfortunately the 32-bit Arm test results are not too clean as there is quite a bit of noise from > failing watchpoints tests. > > I'll try with GDB 13 just to be sure. Thank you. Once you've finished testing, whatever that might be, you can go ahead and push the patches. As disussed earlier, we all agree the risk of bad impact is very low. Let me know when this is done. It's starting to look like we might have a pre-release created this weekend! Maybe earlier, if I can carve some time out of my work schedule. > > For the record, those patches were: > > > > commit d72ba177c85f2ad18d0dcabdd8844532c9acb819 > > Author: Torbj�rn SVENSSON > > Date: Thu Nov 17 12:17:53 2022 +0100 > > Subject: gdb: dwarf2 generic implementation for caching function data > > > > ... and ... > > > > commit 5cf11483141a58314834653003e49709b47822d5 > > Author: Torbj�rn SVENSSON > > Date: Thu Nov 17 12:18:20 2022 +0100 > > Subject: gdb/arm: Use new dwarf2 function cache > > > > Thank you! > -- Joel