From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70088.outbound.protection.outlook.com [40.107.7.88]) by sourceware.org (Postfix) with ESMTPS id 269CF3857B8D for ; Fri, 23 Sep 2022 13:47:27 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 269CF3857B8D ARC-Seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=igE6G6iiPsEhzGxOQYOl5ZY1UV3UR2wAcmkZ4BeGDVv953o8DIojanF8zK0lSZYUbAaLAA7ybLFeD3+sPuzSR215+rr/CDqKos1714mPPx7pBYLQT/rdZCM7OGpjxHrw4RqxL1VDhQzei0/NuNbI7ioYyOHZkgZ8YhC21ZPtKNCfhWKP90hifZEVeef0E46OQdJ5fp0Qy4TXDbkFz9p3LW1sV98TlozJ5+T66gIe4iomrpPxzE4Et/ZicUUcERDajBOhg029f1J0BHEmUkAwJ4UBq/aabC4CRU4JaT5hCJfimZN9YFa2hIDeyG/HgVkaoBkDaCTZxameKmY0uWsZDw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Lz2b0+VgTLO5LAnJx0nbLPy22UvYhtli6zz51KFBass=; b=RSfJlkxMLBFw72TG4I9VBga2Nm8kq9h/V5jWW1UYqfNNQaNtW+maOl6Fj3v7cJYkBzqrP0JSpTW5bAbFqX99wGewGl15hYjhRfWmPg+pSDhUseoG+oFTpUR6j4CA45keWVmfh3yA5IGeUa09NT5F3wuGax93vOR6TSq2R+aCrxqAPEbrFZ2R5WAhPWcaSwAHkI9hLMCSASQYGROWiiuCSWaipB+aPOASjRYX+ljHtF+s/CcT8cCHkYEPHBY7DlzraYycwu6h0M8Sp21PBDKTZfVzpgc0Tso998TnQVpgzJ0MUyAWqbs5QnXqZ8H0Lowaulu+k+BoLvK+M/oWECnYCQ== ARC-Authentication-Results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=sourceware.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1, 1, smtp.mailfrom=arm.com] dmarc=[1, 1, header.from=arm.com]) Received: from DB8PR06CA0047.eurprd06.prod.outlook.com (2603:10a6:10:120::21) by AM8PR08MB6467.eurprd08.prod.outlook.com (2603:10a6:20b:316::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.16; Fri, 23 Sep 2022 13:47:23 +0000 Received: from DBAEUR03FT061.eop-EUR03.prod.protection.outlook.com (2603:10a6:10:120:cafe::bf) by DB8PR06CA0047.outlook.office365.com (2603:10a6:10:120::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.19 via Frontend Transport; Fri, 23 Sep 2022 13:47:23 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; pr=C Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by DBAEUR03FT061.mail.protection.outlook.com (100.127.143.28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.14 via Frontend Transport; Fri, 23 Sep 2022 13:47:23 +0000 Received: ("Tessian outbound fc2405f9ecaf:v124"); Fri, 23 Sep 2022 13:47:23 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: 1a60c359478a1ae8 X-CR-MTA-TID: 64aa7808 Received: from 015533c63610.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id AA1DB8B0-D314-4A6C-98A3-875C09E85B40.1; Fri, 23 Sep 2022 13:47:16 +0000 Received: from EUR04-VI1-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id 015533c63610.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Fri, 23 Sep 2022 13:47:16 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ls5PKSXQj+GLEDrFCSIhRN7MS7AL+LBefTCLTCy1UT/QTUN0F4Yq9xxK0ml5ZgFKKy7iPzqxk7CWCtUR7C1GEG9eu4MU/MacRZQrCk0SYBBy+ioI36PjjwFvtcpmZTXD99vFrxFF9rCpzbyIaXfugWvsEiHmOcCPjy542xRv4mL/MnCPZsuzTpHt0k2pSEygT1ILHWuTfRMZZHRLbNWEUAo9hMbVqs/USZqwi546x3Xnyp822OlBLpNVRNYM1bySojH2xAfYLx7jtaGn5ZwpYWZq2i3//r+qukyZ+EZ1t+LKXSX+YKTg/nxAr8v4uldhtKnYYKBFZeR71Twor6nPVg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Lz2b0+VgTLO5LAnJx0nbLPy22UvYhtli6zz51KFBass=; b=ZoAfA+d90WibpPMB3Vxalk/Bl1aRiBvOlIZ0EibSRX278QGBCDI4UwlncSvMJwHOlEPWPsssSB07PTr7aJpWdLUKOOAZyBDzjWMg8jW7x+VNAcNdS9nsB0kk3KpNrgygKzXsf4NrnFn01izfKLyG5NMR1P4pzPUwwp+j+TG6u+knIgfN7aqNGG0j5DdFeohhhKtfruv67NTyXmBqeZnz55nVxeisuveTMcV5UWbZJWGXXsqZXewFuQoBsMlC6o5yqCD8nS9Z6RQjk244aHQhkYqMY1ZXO82O90GMNqG3CYzzvHtGA1HS2NnoNn0D0SfbV/+Qk70VY90UOSxE/Ljh7Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 40.67.248.234) smtp.rcpttodomain=sourceware.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=none (message not signed); arc=none Received: from AS9PR06CA0482.eurprd06.prod.outlook.com (2603:10a6:20b:49b::7) by AS8PR08MB5912.eurprd08.prod.outlook.com (2603:10a6:20b:29f::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.20; Fri, 23 Sep 2022 13:47:13 +0000 Received: from VE1EUR03FT055.eop-EUR03.prod.protection.outlook.com (2603:10a6:20b:49b:cafe::25) by AS9PR06CA0482.outlook.office365.com (2603:10a6:20b:49b::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.20 via Frontend Transport; Fri, 23 Sep 2022 13:47:13 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 40.67.248.234) smtp.mailfrom=arm.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 40.67.248.234 as permitted sender) receiver=protection.outlook.com; client-ip=40.67.248.234; helo=nebula.arm.com; pr=C Received: from nebula.arm.com (40.67.248.234) by VE1EUR03FT055.mail.protection.outlook.com (10.152.19.158) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.5654.14 via Frontend Transport; Fri, 23 Sep 2022 13:47:13 +0000 Received: from AZ-NEU-EX04.Arm.com (10.251.24.32) by AZ-NEU-EX04.Arm.com (10.251.24.32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.9; Fri, 23 Sep 2022 13:46:51 +0000 Received: from e129171.cambridge.arm.com (10.1.34.22) by mail.arm.com (10.251.24.32) with Microsoft SMTP Server id 15.1.2507.9 via Frontend Transport; Fri, 23 Sep 2022 13:46:51 +0000 From: Luis Machado To: Subject: [PATCH] [AArch64] Add TPIDR2 register support for Linux Date: Fri, 23 Sep 2022 14:46:48 +0100 Message-ID: <20220923134648.114930-1-luis.machado@arm.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-EOPAttributedMessage: 1 X-MS-TrafficTypeDiagnostic: VE1EUR03FT055:EE_|AS8PR08MB5912:EE_|DBAEUR03FT061:EE_|AM8PR08MB6467:EE_ X-MS-Office365-Filtering-Correlation-Id: e6525e7a-4536-4c80-b3ad-08da9d6a249f x-checkrecipientrouted: true NoDisclaimer: true X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: r4clKstwZvvnrOXTIXJtm+8YoDZl1vW6VT/wSFbBdDzDgYvqUSVlQ78A1v4vcwY8F+LqmXNcGeTDPc4n0qA2htt010nl59cpJ0P/uPtSEK1POSNrclg/CmFobe24AotQA4pmxp9SnF9o0wa+jz0th1M0AxPlrpw6aoFsMkAIQAwRvdDL+Gv2v8ARqcUvh7XXsutTD71tg8ucMVOGmYq+KLT1bHmNtXixg0InOP6zHShd5i0hbKaaAM4YN2BK+SodYjo8EWx0VjZeWpHfJTxasDGjLGVm6KyqZq2W53Gqf4kvx7lPYnNAoVaMM+S0zjlaE8jZk/gxN+pfKu8I/a3MfwvttMLYv9PzSnAVNy0lT8zsuMirEf4X0m7XGQ9PZUrJwtwg2aUb2vLQdSu3hVdAOe08KNsugnN7dVWSEMPpi6Kpni9A/Fr+GlBPJ33T1ouy0KZasYeYTO1m1mFqn5F4XN1Mqyw7q2vx5o84Yq5CWgUcHEexIy8LdODkX/lCBld9Y8h1I5cmYclmwLbaP/+PmjfquAz1IzNMGRIw0wyATKCDPI9GpmfgX25mjrmOIFmC7C1cstcSYJfS8zMhYa6vx8NNFKlECxS8ys+UoFj/meCXkyrmA9Ok4kPPlw4VuAz+qLA9oe/lN2FkBRX8ldCqq8T5LUpgioWYwFOFCeWX+jZWedYGIvcl/ewp7r3zkN+2m/UVWdLUkIEWFf8xrqPOxh7/FO0Ly8CQrDCGMvqk4qGsVS4w3lYEqWbNsZwmpOwXwZJYp32vesSTjj3JFRsvmVI1MpxqH1R/cSgYw4jbVwfgsDtGjGQ8gAFwXVMyNoJ+DbG3aobeM9pFvy74R1XMTKJXuRVSHE1NDjcYgvUjRRLD6pUlZK78c2FWaV0ud/xs X-Forefront-Antispam-Report-Untrusted: CIP:40.67.248.234; CTRY:IE; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:nebula.arm.com; PTR:InfoDomainNonexistent; CAT:NONE; SFS:(13230022)(4636009)(396003)(136003)(39860400002)(346002)(376002)(451199015)(40470700004)(36840700001)(46966006)(36860700001)(30864003)(44832011)(81166007)(82310400005)(82740400003)(356005)(478600001)(40460700003)(2616005)(1076003)(86362001)(186003)(70206006)(83380400001)(70586007)(21480400003)(4326008)(2906002)(26005)(966005)(6916009)(7696005)(6666004)(47076005)(36756003)(5660300002)(8936002)(8676002)(336012)(426003)(41300700001)(40480700001)(316002)(2004002)(36900700001); DIR:OUT; SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB5912 X-MS-Exchange-Transport-CrossTenantHeadersStripped: DBAEUR03FT061.eop-EUR03.prod.protection.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: 5aa93cbc-fb0e-478e-eb48-08da9d6a1ea8 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: nK+ppX8YWiqxHIYGIKY2NHYEH+qCJhzeICJ35S2vjLwyvZugujKtR8w4i7iX9D/qqrb8xptWxB2r6QrHThEwSY90WmMQbo4gfqc7GGQ+/WdBfgU+pmVnOS0y3M5Oyx98xKPDlkNozeOhzVbVOuEf0h0zQsCBGYova5AbOuzLfVYwdLl4mEH1EiswyZ7A8tS2hz71RsuhsyryOQjuCRSweAsXd3BbC8PR582KGCBH/5wp7NbLfHfnaAtUlA0oQMO0muwCsmi/1fyGZ/WeE4xOcPJpzFaCbw+8lbZ2ZR0lB82uE4gMk3HrRQ7pwjTNd3ABYk8a2Dd8trsS9p1FCbq/SkKQFa5T69HFCTktiUJy4B7OCJizHU6q5z6hDxpU0i6gvB+6J6A2S+v7oVTG7s9hJN7UAqc64DA80OxfZwId6MjAYDDtfXT2Xe2iczggnhRaN+I5XEW9QdIVda8reM78Z3FlSMm68wRdFwCpT02t9JZxV3B+K3av/CNuPdDhYyPUfBtM8xzSCGbXCwD6+CIyPg6ZDI7tRKBWFNiSxVKhgIPJ2ecYD2pHi43TxNRu7Q6LNugV8o321fBSahN74oKi3pxcoFZRZVNpP0qOGP7jT/jzx+5YFBBHTK8a4UTOSzrnVBLNUebiL2/kU/zlITdiuN7Dqkifzi2W0GZGAUjxXFWYuMHOdHXImf3E9mIiF2fn/ro+QweJHaqluVvRrGUYqOM/gXz/rdpFpgxH918eAjwCD+cWNVBmW+okoAIdtkIVNlq4hjLsn+wWIKXHL1FFc8dI1TnLWiiOZNfTU4MOFAwJYeR1zU1XYHkRxNmEaBal5HJPcx35IhPOlOeKoCYxSHGhgy/V8wgdNzlaKvoRxLU= X-Forefront-Antispam-Report: CIP:63.35.35.123; CTRY:IE; LANG:en; SCL:1; SRV:; IPV:CAL; SFV:NSPM; H:64aa7808-outbound-1.mta.getcheckrecipient.com; PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com; CAT:NONE; SFS:(13230022)(4636009)(376002)(346002)(39860400002)(396003)(136003)(451199015)(46966006)(36840700001)(40470700004)(107886003)(478600001)(44832011)(41300700001)(5660300002)(40480700001)(8936002)(966005)(30864003)(26005)(82310400005)(36756003)(2906002)(6916009)(8676002)(7696005)(6666004)(70206006)(316002)(81166007)(86362001)(70586007)(40460700003)(426003)(47076005)(21480400003)(83380400001)(4326008)(36860700001)(1076003)(336012)(82740400003)(2616005)(186003)(2004002); DIR:OUT; SFP:1101; X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Sep 2022 13:47:23.2730 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: e6525e7a-4536-4c80-b3ad-08da9d6a249f X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d; Ip=[63.35.35.123]; Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: DBAEUR03FT061.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR08MB6467 X-Spam-Status: No, score=-12.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, FORGED_SPF_HELO, GIT_PATCH_0, KAM_DMARC_NONE, KAM_SHORT, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_PASS, SPF_NONE, TXREP, UNPARSEABLE_RELAY autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) 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: Fri, 23 Sep 2022 13:47:31 -0000 With the AArch64 Scalable Matrix Extension we have a new TPIDR2 register, and it will be added to the existing NT_ARM_TLS register set. Kernel patches are being reviewed here: https://lore.kernel.org/linux-arm-kernel/20220818170111.351889-1-broonie@kernel.org/ >From GDB's perspective, we handle it in a similar way to the existing TPIDR register. But we need to consider cases of systems that only have TPIDR and systems that have both TPIDR and TPIDR2. With that in mind, the following patch adds the required code to support TPIDR2 and turns the org.gnu.gdb.aarch64.tls feature into a dynamically-generated target description as opposed to a static target description containing only TPIDR. That means we can remove the gdb/features/aarch64-tls.xml file and replace the existing gdb/features/aarch64-tls.c auto-generated file with a new file that dynamically generates the target description containing either TPIDR alone or TPIDR and TPIDR2. In the future, when *BSD's start to support this register, they can just enable it as is being done for the AArch64 Linux target. The core file read/write code has been updated to support TPIDR2 as well. On GDBserver's side, there is a small change to the find_regno function to expose a non-throwing version of it. It always seemed strange to me how find_regno causes the whole operation to abort if it doesn't find a particular register name. The patch moves code from find_regno into find_regno_no_throw and makes find_regno call find_regno_no_throw instead. This allows us to do register name lookups to find a particular register number without risking erroring out if nothing is found. The patch also adjusts the feature detection code for aarch64-fbsd, since the infrastructure is shared amongst all aarch64 targets. I haven't added code to support TPIDR2 in aarch64-fbsd though, as I'm not sure when/if that will happen. --- gdb/aarch64-fbsd-nat.c | 2 +- gdb/aarch64-fbsd-tdep.c | 2 +- gdb/aarch64-linux-nat.c | 38 +++++++++++++++------------- gdb/aarch64-linux-tdep.c | 25 ++++++++++++++----- gdb/aarch64-linux-tdep.h | 3 --- gdb/aarch64-tdep.c | 45 ++++++++++++++++++++++++++-------- gdb/aarch64-tdep.h | 3 ++- gdb/arch/aarch64.c | 5 ++-- gdb/arch/aarch64.h | 14 +++++++---- gdb/features/Makefile | 1 - gdb/features/aarch64-tls.c | 35 +++++++++++++++++++++++--- gdb/features/aarch64-tls.xml | 11 --------- gdb/nat/aarch64-linux.c | 17 +++++++++++++ gdb/nat/aarch64-linux.h | 4 +++ gdbserver/linux-aarch64-low.cc | 24 ++++++++++++++---- gdbserver/regcache.cc | 17 +++++++++++-- gdbserver/regcache.h | 5 ++++ 17 files changed, 183 insertions(+), 68 deletions(-) delete mode 100644 gdb/features/aarch64-tls.xml diff --git a/gdb/aarch64-fbsd-nat.c b/gdb/aarch64-fbsd-nat.c index 708ddc40d58..40941f138d9 100644 --- a/gdb/aarch64-fbsd-nat.c +++ b/gdb/aarch64-fbsd-nat.c @@ -149,7 +149,7 @@ const struct target_desc * aarch64_fbsd_nat_target::read_description () { aarch64_features features; - features.tls = have_regset (inferior_ptid, NT_ARM_TLS) != 0; + features.tls = have_regset (inferior_ptid, NT_ARM_TLS)? 1 : 0; return aarch64_read_description (features); } diff --git a/gdb/aarch64-fbsd-tdep.c b/gdb/aarch64-fbsd-tdep.c index 4a6b4115234..c9971434777 100644 --- a/gdb/aarch64-fbsd-tdep.c +++ b/gdb/aarch64-fbsd-tdep.c @@ -179,7 +179,7 @@ aarch64_fbsd_core_read_description (struct gdbarch *gdbarch, asection *tls = bfd_get_section_by_name (abfd, ".reg-aarch-tls"); aarch64_features features; - features.tls = tls != nullptr; + features.tls = tls != nullptr? 1 : 0; return aarch64_read_description (features); } diff --git a/gdb/aarch64-linux-nat.c b/gdb/aarch64-linux-nat.c index eda79ec6d35..03eec37a6c4 100644 --- a/gdb/aarch64-linux-nat.c +++ b/gdb/aarch64-linux-nat.c @@ -442,19 +442,19 @@ fetch_tlsregs_from_thread (struct regcache *regcache) = gdbarch_tdep (regcache->arch ()); int regno = tdep->tls_regnum; - gdb_assert (regno != -1); + gdb_assert (regno != -1 && tdep->tls_register_count > 0); - uint64_t tpidr = 0; + uint64_t tpidrs[tdep->tls_register_count] = { 0 }; struct iovec iovec; - - iovec.iov_base = &tpidr; - iovec.iov_len = sizeof (tpidr); + iovec.iov_base = tpidrs; + iovec.iov_len = sizeof (tpidrs); int tid = get_ptrace_pid (regcache->ptid ()); if (ptrace (PTRACE_GETREGSET, tid, NT_ARM_TLS, &iovec) != 0) - perror_with_name (_("unable to fetch TLS register")); + perror_with_name (_("unable to fetch TLS registers")); - regcache->raw_supply (regno, &tpidr); + for (int i = 0; i < tdep->tls_register_count; i++) + regcache->raw_supply (regno + i, &tpidrs[i]); } /* Store to the current thread the valid TLS register set in GDB's @@ -467,19 +467,21 @@ store_tlsregs_to_thread (struct regcache *regcache) = gdbarch_tdep (regcache->arch ()); int regno = tdep->tls_regnum; - gdb_assert (regno != -1); + gdb_assert (regno != -1 && tdep->tls_register_count > 0); - uint64_t tpidr = 0; + uint64_t tpidrs[tdep->tls_register_count] = { 0 }; - if (REG_VALID != regcache->get_register_status (regno)) - return; + for (int i = 0; i < tdep->tls_register_count; i++) + { + if (REG_VALID != regcache->get_register_status (regno + i)) + continue; - regcache->raw_collect (regno, (char *) &tpidr); + regcache->raw_collect (regno + i, (char *) &tpidrs[i]); + } struct iovec iovec; - - iovec.iov_base = &tpidr; - iovec.iov_len = sizeof (tpidr); + iovec.iov_base = &tpidrs; + iovec.iov_len = sizeof (tpidrs); int tid = get_ptrace_pid (regcache->ptid ()); if (ptrace (PTRACE_SETREGSET, tid, NT_ARM_TLS, &iovec) != 0) @@ -607,7 +609,9 @@ aarch64_store_registers (struct regcache *regcache, int regno) && (regno == tdep->mte_reg_base)) store_mteregs_to_thread (regcache); - if (tdep->has_tls () && regno == tdep->tls_regnum) + if (tdep->has_tls () + && regno >= tdep->tls_regnum + && regno < tdep->tls_regnum + tdep->tls_register_count) store_tlsregs_to_thread (regcache); } @@ -788,7 +792,7 @@ aarch64_linux_nat_target::read_description () features.vq = aarch64_sve_get_vq (tid); features.pauth = hwcap & AARCH64_HWCAP_PACA; features.mte = hwcap2 & HWCAP2_MTE; - features.tls = true; + features.tls = aarch64_tls_register_count (tid); return aarch64_read_description (features); } diff --git a/gdb/aarch64-linux-tdep.c b/gdb/aarch64-linux-tdep.c index 15773c75da8..d6ca3ae7228 100644 --- a/gdb/aarch64-linux-tdep.c +++ b/gdb/aarch64-linux-tdep.c @@ -753,11 +753,17 @@ aarch64_linux_iterate_over_regset_sections (struct gdbarch *gdbarch, "MTE registers", cb_data); } + /* Handle the TLS registers. */ if (tdep->has_tls ()) { + gdb_assert (tdep->tls_regnum != -1 && tdep->tls_register_count > 0); + int sizeof_tls_regset + = AARCH64_TLS_REGISTER_SIZE * tdep->tls_register_count; + const struct regcache_map_entry tls_regmap[] = { - { 1, tdep->tls_regnum, 8 }, + { tdep->tls_register_count, tdep->tls_regnum, + AARCH64_TLS_REGISTER_SIZE }, { 0 } }; @@ -766,9 +772,8 @@ aarch64_linux_iterate_over_regset_sections (struct gdbarch *gdbarch, tls_regmap, regcache_supply_regset, regcache_collect_regset }; - cb (".reg-aarch-tls", AARCH64_LINUX_SIZEOF_TLSREGSET, - AARCH64_LINUX_SIZEOF_TLSREGSET, &aarch64_linux_tls_regset, - "TLS register", cb_data); + cb (".reg-aarch-tls", sizeof_tls_regset, sizeof_tls_regset, + &aarch64_linux_tls_regset, "TLS register", cb_data); } } @@ -778,7 +783,6 @@ static const struct target_desc * aarch64_linux_core_read_description (struct gdbarch *gdbarch, struct target_ops *target, bfd *abfd) { - asection *tls = bfd_get_section_by_name (abfd, ".reg-aarch-tls"); CORE_ADDR hwcap = linux_get_hwcap (target); CORE_ADDR hwcap2 = linux_get_hwcap2 (target); @@ -786,7 +790,16 @@ aarch64_linux_core_read_description (struct gdbarch *gdbarch, features.vq = aarch64_linux_core_read_vq (gdbarch, abfd); features.pauth = hwcap & AARCH64_HWCAP_PACA; features.mte = hwcap2 & HWCAP2_MTE; - features.tls = tls != nullptr; + + /* Handle the TLS section. */ + asection *tls = bfd_get_section_by_name (abfd, ".reg-aarch-tls"); + if (tls != nullptr) + { + size_t size = bfd_section_size (tls); + /* Convert the size to the number of actual registers, by + dividing by 8. */ + features.tls = size >> 3; + } return aarch64_read_description (features); } diff --git a/gdb/aarch64-linux-tdep.h b/gdb/aarch64-linux-tdep.h index 9a7e4339dba..8ae33efc605 100644 --- a/gdb/aarch64-linux-tdep.h +++ b/gdb/aarch64-linux-tdep.h @@ -39,9 +39,6 @@ /* The MTE regset consists of a 64-bit register. */ #define AARCH64_LINUX_SIZEOF_MTE_REGSET (8) -/* The TLS regset consists of a single register. */ -#define AARCH64_LINUX_SIZEOF_TLSREGSET (X_REGISTER_SIZE) - extern const struct regset aarch64_linux_gregset; extern const struct regset aarch64_linux_fpregset; diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c index d0387044934..5d82f1993a4 100644 --- a/gdb/aarch64-tdep.c +++ b/gdb/aarch64-tdep.c @@ -3399,8 +3399,18 @@ aarch64_features_from_target_desc (const struct target_desc *tdesc) = (tdesc_find_feature (tdesc, "org.gnu.gdb.aarch64.pauth") != nullptr); features.mte = (tdesc_find_feature (tdesc, "org.gnu.gdb.aarch64.mte") != nullptr); - features.tls - = (tdesc_find_feature (tdesc, "org.gnu.gdb.aarch64.tls") != nullptr); + + const struct tdesc_feature *tls_feature + = tdesc_find_feature (tdesc, "org.gnu.gdb.aarch64.tls"); + + if (tls_feature != nullptr) + { + /* We have TLS registers. Find out how many. */ + if (tdesc_unnumbered_register (tls_feature, "tpidr2")) + features.tls = 2; + else + features.tls = 1; + } return features; } @@ -3454,7 +3464,8 @@ aarch64_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) bool valid_p = true; int i, num_regs = 0, num_pseudo_regs = 0; int first_pauth_regnum = -1, ra_sign_state_offset = -1; - int first_mte_regnum = -1, tls_regnum = -1; + int first_mte_regnum = -1, first_tls_regnum = -1; + int tls_register_count = 0; uint64_t vq = aarch64_get_tdesc_vq (info.target_desc); if (vq > AARCH64_MAX_SVE_VQ) @@ -3544,13 +3555,26 @@ aarch64_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) /* Add the TLS register. */ if (feature_tls != nullptr) { - tls_regnum = num_regs; - /* Validate the descriptor provides the mandatory TLS register - and allocate its number. */ - valid_p = tdesc_numbered_register (feature_tls, tdesc_data.get (), - tls_regnum, "tpidr"); + const char *tls_register_names[2] = { "tpidr", "tpidr2" }; + first_tls_regnum = num_regs; + + /* Look for the TLS registers. */ + for (i = 0; i < ARRAY_SIZE (tls_register_names); i++) + { + valid_p + = tdesc_numbered_register (feature_tls, tdesc_data.get (), + first_tls_regnum + tls_register_count, + tls_register_names[i]); + if (valid_p) + tls_register_count++; + } + valid_p = true; + + if (tls_register_count == 0) + warning (_("Provided TLS register feature doesn't contain " + "any registers.")); - num_regs++; + num_regs += tls_register_count; } /* Add the pauth registers. */ @@ -3601,7 +3625,8 @@ aarch64_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) tdep->ra_sign_state_regnum = (feature_pauth == NULL) ? -1 : ra_sign_state_offset + num_regs; tdep->mte_reg_base = first_mte_regnum; - tdep->tls_regnum = tls_regnum; + tdep->tls_regnum = first_tls_regnum; + tdep->tls_register_count = tls_register_count; set_gdbarch_push_dummy_call (gdbarch, aarch64_push_dummy_call); set_gdbarch_frame_align (gdbarch, aarch64_frame_align); diff --git a/gdb/aarch64-tdep.h b/gdb/aarch64-tdep.h index d8513023c37..8166df0ada8 100644 --- a/gdb/aarch64-tdep.h +++ b/gdb/aarch64-tdep.h @@ -111,8 +111,9 @@ struct aarch64_gdbarch_tdep : gdbarch_tdep_base return mte_reg_base != -1; } - /* TLS register. This is -1 if the TLS register is not available. */ + /* TLS registers. This is -1 if the TLS registers are not available. */ int tls_regnum = 0; + int tls_register_count = 0; bool has_tls() const { diff --git a/gdb/arch/aarch64.c b/gdb/arch/aarch64.c index 0f73286f145..565c5e7a81c 100644 --- a/gdb/arch/aarch64.c +++ b/gdb/arch/aarch64.c @@ -53,8 +53,9 @@ aarch64_create_target_description (const aarch64_features &features) if (features.mte) regnum = create_feature_aarch64_mte (tdesc.get (), regnum); - if (features.tls) - regnum = create_feature_aarch64_tls (tdesc.get (), regnum); + /* TLS registers. */ + if (features.tls > 0) + regnum = create_feature_aarch64_tls (tdesc.get (), regnum, features.tls); return tdesc.release (); } diff --git a/gdb/arch/aarch64.h b/gdb/arch/aarch64.h index 8e3fd36726a..b1a6ce3ef0e 100644 --- a/gdb/arch/aarch64.h +++ b/gdb/arch/aarch64.h @@ -33,7 +33,9 @@ struct aarch64_features bool pauth = false; bool mte = false; - bool tls = false; + + /* A positive TLS value indicates the number of TLS registers available. */ + uint8_t tls = 0; }; inline bool operator==(const aarch64_features &lhs, const aarch64_features &rhs) @@ -56,7 +58,9 @@ namespace std h = features.vq; h = h << 1 | features.pauth; h = h << 1 | features.mte; - h = h << 1 | features.tls; + /* Shift by two bits for now. We may need to increase this in the future + if more TLS registers get added. */ + h = h << 2 | features.tls; return h; } }; @@ -96,7 +100,9 @@ enum aarch64_regnum AARCH64_LAST_V_ARG_REGNUM = AARCH64_V0_REGNUM + 7 }; -#define V_REGISTER_SIZE 16 +/* Sizes of various AArch64 registers. */ +#define AARCH64_TLS_REGISTER_SIZE 8 +#define V_REGISTER_SIZE 16 /* Pseudo register base numbers. */ #define AARCH64_Q0_REGNUM 0 @@ -117,8 +123,6 @@ enum aarch64_regnum #define AARCH64_NUM_REGS AARCH64_FPCR_REGNUM + 1 #define AARCH64_SVE_NUM_REGS AARCH64_SVE_VG_REGNUM + 1 -#define AARCH64_TLS_REGS_SIZE (8) - /* There are a number of ways of expressing the current SVE vector size: VL : Vector Length. diff --git a/gdb/features/Makefile b/gdb/features/Makefile index 4bc85962dfe..037f900b58b 100644 --- a/gdb/features/Makefile +++ b/gdb/features/Makefile @@ -198,7 +198,6 @@ FEATURE_XMLFILES = aarch64-core.xml \ aarch64-fpu.xml \ aarch64-pauth.xml \ aarch64-mte.xml \ - aarch64-tls.xml \ arc/v1-core.xml \ arc/v1-aux.xml \ arc/v2-core.xml \ diff --git a/gdb/features/aarch64-tls.c b/gdb/features/aarch64-tls.c index 30d730dffba..8a59d26b353 100644 --- a/gdb/features/aarch64-tls.c +++ b/gdb/features/aarch64-tls.c @@ -1,14 +1,43 @@ -/* THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi:set ro: - Original: aarch64-tls.xml */ +/* Copyright (C) 2022 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ #include "gdbsupport/tdesc.h" +/* This function is NOT auto generated from xml. + + Create the aarch64 description containing the TLS registers. TPIDR is + always available, but TPIDR2 is only available on some systems. + + COUNT is the number of registers in this set. The minimum is 1. */ + static int -create_feature_aarch64_tls (struct target_desc *result, long regnum) +create_feature_aarch64_tls (struct target_desc *result, long regnum, int count) { + /* TPIDR is always present. */ + gdb_assert (count >= 1); + struct tdesc_feature *feature; feature = tdesc_create_feature (result, "org.gnu.gdb.aarch64.tls"); tdesc_create_reg (feature, "tpidr", regnum++, 1, NULL, 64, "data_ptr"); + + /* Add TPIDR2. */ + if (count > 1) + tdesc_create_reg (feature, "tpidr2", regnum++, 1, NULL, 64, "data_ptr"); + return regnum; } diff --git a/gdb/features/aarch64-tls.xml b/gdb/features/aarch64-tls.xml deleted file mode 100644 index f6437785f71..00000000000 --- a/gdb/features/aarch64-tls.xml +++ /dev/null @@ -1,11 +0,0 @@ - - - - - - - diff --git a/gdb/nat/aarch64-linux.c b/gdb/nat/aarch64-linux.c index 421d1ecb53c..017c30948a9 100644 --- a/gdb/nat/aarch64-linux.c +++ b/gdb/nat/aarch64-linux.c @@ -250,3 +250,20 @@ aarch64_ps_get_thread_area (struct ps_prochandle *ph, return PS_OK; } + +/* See nat/aarch64-linux.h */ + +int +aarch64_tls_register_count (int tid) +{ + uint64_t tls_regs[2]; + struct iovec iovec; + iovec.iov_base = tls_regs; + iovec.iov_len = sizeof (tls_regs); + + if (ptrace (PTRACE_GETREGSET, tid, NT_ARM_TLS, &iovec) != 0) + return 1; + + /* TPIDR2 is available. */ + return 2; +} diff --git a/gdb/nat/aarch64-linux.h b/gdb/nat/aarch64-linux.h index 1777ce32522..517cbb33963 100644 --- a/gdb/nat/aarch64-linux.h +++ b/gdb/nat/aarch64-linux.h @@ -129,4 +129,8 @@ ps_err_e aarch64_ps_get_thread_area (struct ps_prochandle *ph, lwpid_t lwpid, int idx, void **base, int is_64bit_p); +/* Return the number of TLS registers in the NT_ARM_TLS set. This is only + used for aarch64 state. */ +int aarch64_tls_register_count (int tid); + #endif /* NAT_AARCH64_LINUX_H */ diff --git a/gdbserver/linux-aarch64-low.cc b/gdbserver/linux-aarch64-low.cc index db508696261..b657a265ee7 100644 --- a/gdbserver/linux-aarch64-low.cc +++ b/gdbserver/linux-aarch64-low.cc @@ -292,9 +292,16 @@ aarch64_store_mteregset (struct regcache *regcache, const void *buf) static void aarch64_fill_tlsregset (struct regcache *regcache, void *buf) { + gdb_byte *tls_buf = (gdb_byte *) buf; int tls_regnum = find_regno (regcache->tdesc, "tpidr"); - collect_register (regcache, tls_regnum, buf); + collect_register (regcache, tls_regnum, tls_buf); + + /* Read TPIDR2, if it exists. */ + gdb::optional regnum = find_regno_no_throw (regcache->tdesc, "tpidr2"); + + if (regnum.has_value ()) + collect_register (regcache, *regnum, tls_buf + sizeof (uint64_t)); } /* Store TLS register to regcache. */ @@ -302,9 +309,16 @@ aarch64_fill_tlsregset (struct regcache *regcache, void *buf) static void aarch64_store_tlsregset (struct regcache *regcache, const void *buf) { + gdb_byte *tls_buf = (gdb_byte *) buf; int tls_regnum = find_regno (regcache->tdesc, "tpidr"); - supply_register (regcache, tls_regnum, buf); + supply_register (regcache, tls_regnum, tls_buf); + + /* Write TPIDR2, if it exists. */ + gdb::optional regnum = find_regno_no_throw (regcache->tdesc, "tpidr2"); + + if (regnum.has_value ()) + supply_register (regcache, *regnum, tls_buf + sizeof (uint64_t)); } bool @@ -795,8 +809,8 @@ aarch64_adjust_register_sets (const struct aarch64_features &features) regset->size = AARCH64_LINUX_SIZEOF_MTE; break; case NT_ARM_TLS: - if (features.tls) - regset->size = AARCH64_TLS_REGS_SIZE; + if (features.tls > 0) + regset->size = AARCH64_TLS_REGISTER_SIZE * features.tls; break; default: gdb_assert_not_reached ("Unknown register set found."); @@ -829,7 +843,7 @@ aarch64_target::low_arch_setup () features.pauth = linux_get_hwcap (8) & AARCH64_HWCAP_PACA; /* A-profile MTE is 64-bit only. */ features.mte = linux_get_hwcap2 (8) & HWCAP2_MTE; - features.tls = true; + features.tls = aarch64_tls_register_count (tid); current_process ()->tdesc = aarch64_linux_read_description (features); diff --git a/gdbserver/regcache.cc b/gdbserver/regcache.cc index 27491efc52d..0b0d0855db3 100644 --- a/gdbserver/regcache.cc +++ b/gdbserver/regcache.cc @@ -244,14 +244,27 @@ registers_from_string (struct regcache *regcache, char *buf) hex2bin (buf, registers, len / 2); } -int -find_regno (const struct target_desc *tdesc, const char *name) +/* See regcache.h */ + +gdb::optional +find_regno_no_throw (const struct target_desc *tdesc, const char *name) { for (int i = 0; i < tdesc->reg_defs.size (); ++i) { if (strcmp (name, find_register_by_number (tdesc, i).name) == 0) return i; } + return {}; +} + +int +find_regno (const struct target_desc *tdesc, const char *name) +{ + gdb::optional regnum = find_regno_no_throw (tdesc, name); + + if (regnum.has_value ()) + return *regnum; + internal_error (__FILE__, __LINE__, "Unknown register %s requested", name); } diff --git a/gdbserver/regcache.h b/gdbserver/regcache.h index 731c18d16e2..b67eefbe04b 100644 --- a/gdbserver/regcache.h +++ b/gdbserver/regcache.h @@ -110,6 +110,11 @@ int register_cache_size (const struct target_desc *tdesc); int register_size (const struct target_desc *tdesc, int n); +/* No throw version of find_regno. If NAME is not a known register, return + an empty value. */ +gdb::optional find_regno_no_throw (const struct target_desc *tdesc, + const char *name); + int find_regno (const struct target_desc *tdesc, const char *name); void supply_register (struct regcache *regcache, int n, const void *buf); -- 2.25.1