[PATCH v2 07/24] Documentation for memory tagging remote packets
Eli Zaretskii
eliz@gnu.org
Fri Oct 23 06:25:28 GMT 2020
> Date: Thu, 22 Oct 2020 16:59:57 -0300
> From: Luis Machado via Gdb-patches <gdb-patches@sourceware.org>
> Cc: david.spickett@linaro.org
>
> gdb/doc/ChangeLog:
>
> YYYY-MM-DD Luis Machado <luis.machado@linaro.org>
>
> * gdb.texinfo (General Query Packets): Document qMemTags and
> QMemTags.
> Document the "memory-tagging" feature.
Since the last sentence belongs to the same node as the one before it,
please don't start it on a new line, but after the previous sentence
ends (with 2 spaces between them).
> +@item qMemTags:@var{start address},@var{length}:@var{type}
> +@cindex fetch memory tags
> +@cindex @samp{qMemTags} packet
> +Fetch memory tags of type @var{type} from the address range
> +@r{[}@var{start address}, @var{start address} + @var{length}@r{)}. The target
The expression in @r{...} should be wrapped with @w{..}, so that it
doesn't get split between two lines.
> +@var{type} is the type of tag, a signed integer, the request wants to fetch.
This is ambiguous: does "the request wants to fetch" refer to the tag
or to the "signed integer" part? Suggest to move the "a signed
integer" part to the end of the sentence.
> +tags found in the request memory range.
^^^^^^^
"requested"
> +Store memory tags of type @var{type} to the address range
> +@r{[}@var{start address}, @var{start address} + @var{length}@r{)}. The target
Same comment here about @r{..}.
> +If the number of memory tags, @var{nt}, is greater than or equal to the
> +number of memory tag granules, @var{ng}, only @var{ng} tags will be
> +stored.
It is not clear here how NT and NG are related to the parameters of
the packet. Can you add something that explains the relation?
> +@var{type} is the type of tag, a signed integer, the request wants to store.
Same comment as above regarding ambiguity of the sentence.
> +@item memory-tagging
> +The remote stub supports and implements the required memory tagging
> +functionality and understands the @samp{qMemTags} and @samp{QMemTags} packets.
This is far enough from the description of these packets to warrant a
cross-reference to that description. Please add a cross-reference.
Thanks.
More information about the Gdb-patches
mailing list