From: John Baldwin <jhb@FreeBSD.org>
To: gdb-patches@sourceware.org
Subject: [PATCH v2 2/5] fbsd-nat: Add helper functions to fetch and store register sets.
Date: Mon, 12 Jul 2021 08:53:50 -0700 [thread overview]
Message-ID: <20210712155353.75907-3-jhb@FreeBSD.org> (raw)
In-Reply-To: <20210712155353.75907-1-jhb@FreeBSD.org>
In particular, this supports register sets described by a regcache_map
which are fetched and stored with dedicated ptrace operations. These
functions are intended to be used in architecture-specific
fetch_registers and store_registers target methods.
---
gdb/fbsd-nat.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++
gdb/fbsd-nat.h | 43 +++++++++++++++++++++++++++++++++++++++++++
2 files changed, 89 insertions(+)
diff --git a/gdb/fbsd-nat.c b/gdb/fbsd-nat.c
index 234e74fcfd..33eddb5f22 100644
--- a/gdb/fbsd-nat.c
+++ b/gdb/fbsd-nat.c
@@ -1601,6 +1601,52 @@ fbsd_nat_target::supports_disable_randomization ()
#endif
}
+/* See fbsd-nat.h. */
+
+void
+fbsd_nat_target::fetch_register_set (struct regcache *regcache, int regnum,
+ int fetch_op, const struct regset *regset,
+ void *regs, size_t size)
+{
+ const struct regcache_map_entry *map
+ = (const struct regcache_map_entry *) regset->regmap;
+ pid_t pid = get_ptrace_pid (regcache->ptid ());
+
+ if (regnum == -1 || regcache_map_supplies (map, regnum, regcache->arch(),
+ size))
+ {
+ if (ptrace (fetch_op, pid, (PTRACE_TYPE_ARG3) regs, 0) == -1)
+ perror_with_name (_("Couldn't get registers"));
+
+ regcache->supply_regset (regset, regnum, regs, size);
+ }
+}
+
+/* See fbsd-nat.h. */
+
+void
+fbsd_nat_target::store_register_set (struct regcache *regcache, int regnum,
+ int fetch_op, int store_op,
+ const struct regset *regset, void *regs,
+ size_t size)
+{
+ const struct regcache_map_entry *map
+ = (const struct regcache_map_entry *) regset->regmap;
+ pid_t pid = get_ptrace_pid (regcache->ptid ());
+
+ if (regnum == -1 || regcache_map_supplies (map, regnum, regcache->arch(),
+ size))
+ {
+ if (ptrace (fetch_op, pid, (PTRACE_TYPE_ARG3) regs, 0) == -1)
+ perror_with_name (_("Couldn't get registers"));
+
+ regcache->collect_regset (regset, regnum, regs, size);
+
+ if (ptrace (store_op, pid, (PTRACE_TYPE_ARG3) regs, 0) == -1)
+ perror_with_name (_("Couldn't write registers"));
+ }
+}
+
void _initialize_fbsd_nat ();
void
_initialize_fbsd_nat ()
diff --git a/gdb/fbsd-nat.h b/gdb/fbsd-nat.h
index 1fdb939935..a59065415b 100644
--- a/gdb/fbsd-nat.h
+++ b/gdb/fbsd-nat.h
@@ -21,6 +21,8 @@
#define FBSD_NAT_H
#include "inf-ptrace.h"
+#include "regcache.h"
+#include "regset.h"
#include <osreldate.h>
#include <sys/proc.h>
@@ -103,6 +105,47 @@ class fbsd_nat_target : public inf_ptrace_target
bool supports_multi_process () override;
bool supports_disable_randomization () override;
+
+private:
+ /* Helper routines for use in fetch_registers and store_registers in
+ subclasses. These routines fetch and store a single set of
+ registers described by REGSET. The REGSET's 'regmap' field must
+ point to an array of 'struct regcache_map_entry'.
+
+ FETCH_OP is a ptrace operation to fetch the set of registers from
+ a native thread. STORE_OP is a ptrace operation to store the set
+ of registers to a native thread.
+
+ The caller must provide storage for the set of registers in REGS,
+ and SIZE is the size of the storage. */
+
+ void fetch_register_set (struct regcache *regcache, int regnum, int fetch_op,
+ const struct regset *regset, void *regs, size_t size);
+
+ void store_register_set (struct regcache *regcache, int regnum, int fetch_op,
+ int store_op, const struct regset *regset,
+ void *regs, size_t size);
+protected:
+ /* Wrapper versions of the above helpers which accept a register set
+ type such as 'struct reg' or 'struct fpreg'. */
+
+ template <class Regset>
+ void fetch_register_set (struct regcache *regcache, int regnum, int fetch_op,
+ const struct regset *regset)
+ {
+ Regset regs;
+ fetch_register_set (regcache, regnum, fetch_op, regset, ®s,
+ sizeof (regs));
+ }
+
+ template <class Regset>
+ void store_register_set (struct regcache *regcache, int regnum, int fetch_op,
+ int store_op, const struct regset *regset)
+ {
+ Regset regs;
+ store_register_set (regcache, regnum, fetch_op, store_op, regset, ®s,
+ sizeof (regs));
+ }
};
#endif /* fbsd-nat.h */
--
2.31.1
next prev parent reply other threads:[~2021-07-12 15:56 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-12 15:53 [PATCH v2 0/5] Add helper functions for FreeBSD native targets John Baldwin
2021-07-12 15:53 ` [PATCH v2 1/5] Add regcache_map_supplies helper routine John Baldwin
2021-07-12 15:53 ` John Baldwin [this message]
2021-07-12 15:53 ` [PATCH v2 3/5] riscv-fbsd-nat: Use fetch_register_set and store_register_set John Baldwin
2021-07-12 15:53 ` [PATCH v2 4/5] aarch64-fbsd-nat: " John Baldwin
2021-07-12 15:53 ` [PATCH v2 5/5] arm-fbsd-nat: " John Baldwin
2021-07-13 12:45 ` [PATCH v2 0/5] Add helper functions for FreeBSD native targets Pedro Alves
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210712155353.75907-3-jhb@FreeBSD.org \
--to=jhb@freebsd.org \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).