From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.baldwin.cx (bigwig.baldwin.cx [66.216.25.90]) by sourceware.org (Postfix) with ESMTPS id DD5243857B8C for ; Fri, 8 Jul 2022 01:03:38 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org DD5243857B8C Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=FreeBSD.org Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=FreeBSD.org Received: from ralph.com (ralph.baldwin.cx [66.234.199.215]) by mail.baldwin.cx (Postfix) with ESMTPSA id EBF9F1A84C54 for ; Thu, 7 Jul 2022 21:03:32 -0400 (EDT) From: John Baldwin To: gdb-patches@sourceware.org Subject: [PATCH 1/5] regcache: Add collect/supply_regset variants that accept a register base. Date: Thu, 7 Jul 2022 17:58:12 -0700 Message-Id: <20220708005816.9408-2-jhb@FreeBSD.org> X-Mailer: git-send-email 2.36.1 In-Reply-To: <20220708005816.9408-1-jhb@FreeBSD.org> References: <20220708005816.9408-1-jhb@FreeBSD.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.6.4 (mail.baldwin.cx [0.0.0.0]); Thu, 07 Jul 2022 21:03:33 -0400 (EDT) X-Virus-Scanned: clamav-milter 0.103.1 at mail.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-12.2 required=5.0 tests=BAYES_00, FORGED_SPF_HELO, GIT_PATCH_0, KAM_DMARC_STATUS, KHOP_HELO_FCRDNS, SPF_HELO_PASS, SPF_SOFTFAIL, TXREP, T_SCC_BODY_TEXT_LINE 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, 08 Jul 2022 01:03:40 -0000 Some register sets described by an array of regcache_map_entry structures do not have fixed register numbers in their associated architecture but do describe a block of registers whose numbers are at fixed offsets relative to some base register value. An example of this are the TLS register sets for the ARM and AArch64 architectures. Currently OS-specific architectures create register maps and register sets dynamically using the register base number. However, this requires duplicating the code to create the register map and register set. To reduce duplication, add variants of the collect_regset and supply_regset regcache methods which accept a base register number. For valid register map entries (i.e. not REGCACHE_MAP_SKIP), add this base register number to the value from the map entry to determine the final register number. --- gdb/regcache.c | 28 +++++++++++++++++++++++++--- gdb/regcache.h | 24 ++++++++++++++++++++++-- 2 files changed, 47 insertions(+), 5 deletions(-) diff --git a/gdb/regcache.c b/gdb/regcache.c index 037659ef8fa..1db3d972ef8 100644 --- a/gdb/regcache.c +++ b/gdb/regcache.c @@ -1180,7 +1180,7 @@ regcache::transfer_regset_register (struct regcache *out_regcache, int regnum, /* See regcache.h. */ void -regcache::transfer_regset (const struct regset *regset, +regcache::transfer_regset (const struct regset *regset, int regbase, struct regcache *out_regcache, int regnum, const gdb_byte *in_buf, gdb_byte *out_buf, size_t size) const @@ -1195,6 +1195,9 @@ regcache::transfer_regset (const struct regset *regset, int regno = map->regno; int slot_size = map->size; + if (regno != REGCACHE_MAP_SKIP) + regno += regbase; + if (slot_size == 0 && regno != REGCACHE_MAP_SKIP) slot_size = m_descr->sizeof_register[regno]; @@ -1242,7 +1245,18 @@ void regcache::supply_regset (const struct regset *regset, int regnum, const void *buf, size_t size) { - transfer_regset (regset, this, regnum, (const gdb_byte *) buf, nullptr, size); + transfer_regset (regset, 0, this, regnum, (const gdb_byte *) buf, nullptr, + size); +} + +/* See regcache.h. */ + +void +regcache::supply_regset (const struct regset *regset, int regbase, + int regnum, const void *buf, size_t size) +{ + transfer_regset (regset, regbase, this, regnum, (const gdb_byte *) buf, + nullptr, size); } /* Collect register REGNUM from REGCACHE to BUF, using the register @@ -1261,11 +1275,19 @@ void regcache::collect_regset (const struct regset *regset, int regnum, void *buf, size_t size) const { - transfer_regset (regset, nullptr, regnum, nullptr, (gdb_byte *) buf, size); + transfer_regset (regset, 0, nullptr, regnum, nullptr, (gdb_byte *) buf, size); } /* See regcache.h */ +void +regcache::collect_regset (const struct regset *regset, int regbase, + int regnum, void *buf, size_t size) const +{ + transfer_regset (regset, regbase, nullptr, regnum, nullptr, (gdb_byte *) buf, + size); +} + bool regcache_map_supplies (const struct regcache_map_entry *map, int regnum, struct gdbarch *gdbarch, size_t size) diff --git a/gdb/regcache.h b/gdb/regcache.h index 1dbba5ce9af..f01127b20fb 100644 --- a/gdb/regcache.h +++ b/gdb/regcache.h @@ -150,6 +150,14 @@ extern void regcache_collect_regset (const struct regset *regset, int regnum, void *buf, size_t size); +extern void regcache_supply_regset (const struct regset *regset, int regbase, + struct regcache *regcache, + int regnum, const void *buf, + size_t size); +extern void regcache_collect_regset (const struct regset *regset, int regbase, + const struct regcache *regcache, + int regnum, void *buf, size_t size); + /* Return true if a set of registers contains the value of the register numbered REGNUM. The size of the set of registers is given in SIZE, and the layout of the set of registers is described @@ -375,10 +383,22 @@ class regcache : public detached_regcache void cooked_write_part (int regnum, int offset, int len, const gdb_byte *buf); + /* Transfer a set of registers (as described by REGSET) between + REGCACHE and BUF. If REGNUM == -1, transfer all registers + belonging to the regset, otherwise just the register numbered + REGNUM. The REGSET's 'regmap' field must point to an array of + 'struct regcache_map_entry'. The valid register numbers in each + entry in 'struct regcache_map_entry' are offset by REGBASE. */ + + void supply_regset (const struct regset *regset, int regbase, + int regnum, const void *buf, size_t size); + + void collect_regset (const struct regset *regset, int regbase, int regnum, + void *buf, size_t size) const; + void supply_regset (const struct regset *regset, int regnum, const void *buf, size_t size); - void collect_regset (const struct regset *regset, int regnum, void *buf, size_t size) const; @@ -419,7 +439,7 @@ class regcache : public detached_regcache /* Transfer a single or all registers belonging to a certain register set to or from a buffer. This is the main worker function for regcache_supply_regset and regcache_collect_regset. */ - void transfer_regset (const struct regset *regset, + void transfer_regset (const struct regset *regset, int regbase, struct regcache *out_regcache, int regnum, const gdb_byte *in_buf, gdb_byte *out_buf, size_t size) const; -- 2.36.1