public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Document the memory_tagged argument for memory region callbacks
@ 2022-11-23 9:45 Luis Machado
0 siblings, 0 replies; only message in thread
From: Luis Machado @ 2022-11-23 9:45 UTC (permalink / raw)
To: gdb-cvs
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=8181f0045df765a7f28db745f4a8bd3c5a1da82d
commit 8181f0045df765a7f28db745f4a8bd3c5a1da82d
Author: Luis Machado <luis.machado@arm.com>
Date: Thu Nov 17 10:19:30 2022 +0000
Document the memory_tagged argument for memory region callbacks
There were no comments in some instances (gdb/defs.h, gdb/core.c and
gdb/linux-tdep.c), so address that by adding comments where those are missing.
Diff:
---
gdb/defs.h | 8 ++++++--
gdb/gcore.c | 12 ++++++++++--
gdb/linux-tdep.c | 5 ++++-
3 files changed, 20 insertions(+), 5 deletions(-)
diff --git a/gdb/defs.h b/gdb/defs.h
index 0d9a39a59d0..f51ab9e5c0c 100644
--- a/gdb/defs.h
+++ b/gdb/defs.h
@@ -340,8 +340,12 @@ extern const char *pc_prefix (CORE_ADDR);
/* * Process memory area starting at ADDR with length SIZE. Area is
readable iff READ is non-zero, writable if WRITE is non-zero,
executable if EXEC is non-zero. Area is possibly changed against
- its original file based copy if MODIFIED is non-zero. DATA is
- passed without changes from a caller. */
+ its original file based copy if MODIFIED is non-zero.
+
+ MEMORY_TAGGED is true if the memory region contains memory tags, false
+ otherwise.
+
+ DATA is passed without changes from a caller. */
typedef int (*find_memory_region_ftype) (CORE_ADDR addr, unsigned long size,
int read, int write, int exec,
diff --git a/gdb/gcore.c b/gdb/gcore.c
index ede78534bd8..672bdf78736 100644
--- a/gdb/gcore.c
+++ b/gdb/gcore.c
@@ -372,8 +372,12 @@ make_output_phdrs (bfd *obfd, asection *osec)
bfd_record_phdr (obfd, p_type, 1, p_flags, 0, 0, 0, 0, 1, &osec);
}
-/* find_memory_region_ftype implementation. DATA is 'bfd *' for the core file
- GDB is creating. */
+/* find_memory_region_ftype implementation.
+
+ MEMORY_TAGGED is true if the memory region contains memory tags, false
+ otherwise.
+
+ DATA is 'bfd *' for the core file GDB is creating. */
static int
gcore_create_callback (CORE_ADDR vaddr, unsigned long size, int read,
@@ -462,6 +466,10 @@ gcore_create_callback (CORE_ADDR vaddr, unsigned long size, int read,
}
/* gdbarch_find_memory_region callback for creating a memory tag section.
+
+ MEMORY_TAGGED is true if the memory region contains memory tags, false
+ otherwise.
+
DATA is 'bfd *' for the core file GDB is creating. */
static int
diff --git a/gdb/linux-tdep.c b/gdb/linux-tdep.c
index 0a2fced7804..c30d9fb13f8 100644
--- a/gdb/linux-tdep.c
+++ b/gdb/linux-tdep.c
@@ -1695,7 +1695,10 @@ struct linux_make_mappings_data
static linux_find_memory_region_ftype linux_make_mappings_callback;
/* A callback for linux_find_memory_regions_full that updates the
- mappings data for linux_make_mappings_corefile_notes. */
+ mappings data for linux_make_mappings_corefile_notes.
+
+ MEMORY_TAGGED is true if the memory region contains memory tags, false
+ otherwise. */
static int
linux_make_mappings_callback (ULONGEST vaddr, ULONGEST size,
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-11-23 9:45 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-23 9:45 [binutils-gdb] Document the memory_tagged argument for memory region callbacks Luis Machado
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).