On Wed, Dec 14, 2022 at 4:28 PM lijiang <lijiang(a)redhat.com> wrote:
On Tue, Dec 6, 2022 at 4:40 PM <crash-utility-request(a)redhat.com> wrote:
> Date: Tue, 6 Dec 2022 16:40:21 +0800
> From: Tao Liu <ltao(a)redhat.com>
> To: crash-utility(a)redhat.com
> Subject: [Crash-utility] [PATCH v3 5/6] Update the maple tree help
> info for tree cmd
> Message-ID: <20221206084022.58693-6-ltao(a)redhat.com>
> Content-Type: text/plain; charset="US-ASCII"; x-default=true
>
> Signed-off-by: Tao Liu <ltao(a)redhat.com>
> ---
> help.c | 46 ++++++++++++++++++++++++----------------------
> 1 file changed, 24 insertions(+), 22 deletions(-)
>
> diff --git a/help.c b/help.c
> index 99214c1..f764d08 100644
> --- a/help.c
> +++ b/help.c
> @@ -6306,19 +6306,20 @@ NULL
>
> char *help_tree[] = {
> "tree",
> -"display radix tree, XArray or red-black tree",
> -"[-t [radix|xarray|rbtree]] [-r offset] [-[s|S] struct[.member[,member]]]\n
-[x|d] [-o offset] [-l] [-p] [-N] start",
> -" This command dumps the contents of a radix tree, an XAarray, or a
red-black",
> -" tree. The arguments are as follows:\n",
> +"display radix tree, XArray, red-black tree or maple tree",
> +"[-t [radix|xarray|rbtree|maple]] [-r offset] [-[s|S]
struct[.member[,member]]]\n"
> +" -[x|d] [-o offset] [-l] [-p] [-v] [-N] start",
> +" This command dumps the contents of a radix tree, an XAarray, a
red-black",
> +" tree, or a maple tree. The arguments are as follows:\n",
> " -t type The type of tree to dump; the type string can be one of ",
> -" \"radix\", \"rbtree\", or
\"xarray\", or alternatively, \"ra\",",
> -" \"rb\" or \"x\" are acceptable. If not
specified, rbtree is the",
> -" default type.",
> +" \"radix\", \"rbtree\", \"xarray\",
or \"maple\", or alternatively,",
> +" \"ra\", \"rb\", \"x\" or
\"m\" are acceptable. If not specified,",
> +" rbtree is the default type.",
> " -r offset If the \"start\" argument is the address of a data
structure that",
> -" contains an radix_tree_root, xarray or rb_root structure, then
this",
> -" is the offset to that structure member. If the offset is
non-zero,",
> -" then this option is required. The offset may be entered in
either",
> -" of two manners:",
> +" contains an radix_tree_root, maple_tree, xarray or
rb_root",
> +" structure, then this is the offset to that structure member.
If",
> +" the offset is non-zero, then this option is required. The
offset",
> +" may be entered in either of two manners:",
> " 1. In \"structure.member\" format.",
> " 2. A number of bytes.",
> " -o offset For red-black trees only, the offset of the rb_node within its
",
> @@ -6347,25 +6348,26 @@ char *help_tree[] = {
> " -p Display the node's position information, showing the
relationship",
> " between it and the root. For red-black trees, a position
that",
> " indicates \"root/l/r\" means that the node is the
right child",
> -" of the left child of the root node. For radix trees and
xarrays,",
> -" the index, the height, and the slot index values are shown
with",
> -" respect to the root.",
> +" of the left child of the root node. For radix trees, xarrays
and",
> +" maple trees, the index, the height, and the slot index values
are",
> +" shown with respect to the root.",
> " -x Override default output format with hexadecimal format.",
> " -d Override default output format with decimal format.",
> +" -v For maple trees only, showing the contents of each maple tree
node.",
> " ",
> " The meaning of the \"start\" argument, which can be expressed
either in",
> " hexadecimal format or symbolically, depends upon whether the -N
option",
> " is prepended:",
> " ",
> -" start The address of a radix_tree_root, xarray or rb_root structure,
or",
> -" the address of a structure containing the radix_tree_root,
xarray",
> -" or rb_root structure; if the latter, then the \"-r
offset\" option",
> -" must be used if the member offset of the root structure is
",
> -" non-zero.",
> +" start The address of a radix_tree_root, maple_tree, xarray or
rb_root",
> +" structure, or the address of a structure containing the",
> +" radix_tree_root, maple_tree, xarray or rb_root structure; if
the",
> +" latter, then the \"-r offset\" option must be used if
the member",
> +" offset of the root structure is non-zero.",
> " ",
> -" -N start The address of a radix_tree_node, xa_node or rb_node
structure,",
> -" bypassing the radix_tree_root, xarray, or rb_root that
points",
> -" to it.",
> +" -N start The address of a radix_tree_node, maple_node, xa_node or
rb_node",
> +" structure, bypassing the radix_tree_root, maple_tree, xarray,
or",
> +" rb_root that points to it.",
> "",
> "\nEXAMPLES",
> " The vmap_area_root is a standalone rb_root structure. Display the ",
Could you please also add a simple example for the maple tree option
to the above EXAMPLES section? That looks more helpful.
BTW: I noticed that you had put several examples to patch[2] log.
Good suggestion! I have added the patch[2] examples to the EXAMPLES
section. It is more convenient for people to find it from the help
message instead of the commit log.
Thanks,
Tao Liu
Thanks.
Lianbo
> --
> 2.33.1