From 8181f0045df765a7f28db745f4a8bd3c5a1da82d Mon Sep 17 00:00:00 2001
From: Luis Machado <luis.machado@arm.com>
Date: Thu, 17 Nov 2022 10:19:30 +0000
Subject: 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.
---
 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 0d9a39a..f51ab9e 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 ede7853..672bdf7 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 0a2fced..c30d9fb 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,
-- 
cgit v1.1