From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id tFGBBuR/NmTWjykAWB0awg (envelope-from ) for ; Wed, 12 Apr 2023 05:54:44 -0400 Received: by simark.ca (Postfix, from userid 112) id 081D81E221; Wed, 12 Apr 2023 05:54:44 -0400 (EDT) Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=lH5JgI2z; dkim-atps=neutral X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-8.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id A31B71E0D3 for ; Wed, 12 Apr 2023 05:54:43 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 222D53857025 for ; Wed, 12 Apr 2023 09:54:43 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 222D53857025 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1681293283; bh=xpwUZpN/uYbyD6C9doY0vd6qQO594SPJLWytMxnL6KE=; h=Date:To:Cc:In-Reply-To:Subject:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=lH5JgI2zZq59DCHhBTkt7AhRg4o+xmATBHCr+mScxtLeq1LiPDlK127Xm38SLr3Wm MgPBPEl+dvrT0qzVp8We7Pqcn4b8Dhp0slcF+wq3gUUcRJ8XeF76ZW2VAZApUkxU8R N3HDUQwzrpIqY2y4mTISgV+kpexX9uJmXaxYVCu4= Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) by sourceware.org (Postfix) with ESMTPS id 49F393857012 for ; Wed, 12 Apr 2023 09:54:18 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 49F393857012 Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pmXB9-0002yj-8h; Wed, 12 Apr 2023 05:54:15 -0400 Received: from [87.69.77.57] (helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pmXB8-0004xe-Gv; Wed, 12 Apr 2023 05:54:14 -0400 Date: Wed, 12 Apr 2023 12:54:59 +0300 Message-Id: <83v8i177d8.fsf@gnu.org> To: Luis Machado Cc: gdb-patches@sourceware.org, alex.bennee@linaro.org, richard.henderson@linaro.org, peter.maydell@linaro.org In-Reply-To: <9ae7c1bd-64d3-7444-fbc7-e536f8b2f971@arm.com> (message from Luis Machado on Wed, 12 Apr 2023 10:25:56 +0100) Subject: Re: [PATCH] [gdb/aarch64] pauth: Create new feature string for pauth to prevent crashing older gdb's References: <20230411224327.2142994-1-luis.machado@arm.com> <83mt3d8wkf.fsf@gnu.org> <63c2cf2c-692c-7737-d0ee-b8cb30439b4c@arm.com> <831qkp8nin.fsf@gnu.org> <9ae7c1bd-64d3-7444-fbc7-e536f8b2f971@arm.com> 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: , From: Eli Zaretskii via Gdb-patches Reply-To: Eli Zaretskii Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" > Date: Wed, 12 Apr 2023 10:25:56 +0100 > Cc: gdb-patches@sourceware.org, alex.bennee@linaro.org, > richard.henderson@linaro.org, peter.maydell@linaro.org > From: Luis Machado > > > The flow is OK, but the last paragraph is too long. The main reason > > of this text is to tell people to use the v2 feature, not to explain > > why it exists. So the explanation of the reason should be shorter > > like > > Though it is documentation for users, it is also one of the only documents providing guidance for developers of debugging stubs (like QEMU's, openOCD etc). > > So I feel like it should include a bit more information rather than the bare minimum. > > I don't think this particular section (XML target descriptions) is geared towards the common debugger users. > > > > > We recommend always using the @samp{org.gnu.gdb.aarch64.pauth_v2} > > feature because using @samp{org.gnu.gdb.aarch64.pauth} could crash > > older versions of @value{GDBN} due to a known bug in those versions. > > I think the above is sane. It just has less detail, which, as explained above, might not be too useful for > developers of debugging stubs. > > > > > Is it really important to say that the bug happens only when > > additional system registers are reported? > > We could make it shorter by only mentioning this is only an issue when using emulators. Fine by me, thanks.