Re: [PATCH, v2] [AArch64] MTE corefile support

[Eli Zaretskii <eliz@gnu.org>]

> Date: Fri, 22 Apr 2022 09:12:44 +0100
> Cc: gdb-patches@sourceware.org
> From: Luis Machado <luis.machado@arm.com>
> 
> >>   *** Changes since GDB 12
> >>   
> >> +* GDB now supports dumping memory tag data for AArch64 MTE.  It also supports
> >> +  reading memory tag data for AArch64 MTE from core files generated by
> >> +  the gcore command or the Linux kernel.
> >> +
> > 
> > I think this should say something about the significance of this
> > feature.  Otherwise it is completely unclear why would GDB want to
> > support that.
> > 
> > Same comment for the addition to the manual.
> > 
> > Thanks.
> 
> It is a bit short. How about the following additional paragraph?
> 
> "When a process uses memory-mapped pages protected by memory tags (for 
> example, AArch64 MTE), this additional information will be recorded in 
> the core file in the event of a crash or if GDB generates a core file 
> from the current process state.
> 
> The memory tag data will be used so developers can display the memory 
> tags from a particular memory region, and will also be used to show a 
> detailed message about a crash that happened due to a memory tag violation."

This is fine, but can what's described in the last paragraph be done
by GDB commands, or does it require external tools?  In the former
case, we should mention those GDB commands; in the latter case I'd
settle for mentioning the fact that external tools are required.

Thanks.