* Wei Yang <richard.weiyang@xxxxxxxxx> [241018 22:37]: > Currently for an empty tree, it would print: > > maple_tree(0x7ffcd02c6ee0) flags 1, height 0 root (nil) > 0: (nil) > > This is a little misleading. > > Let's print (empty) for an empty tree. > > Signed-off-by: Wei Yang <richard.weiyang@xxxxxxxxx> > CC: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx> > CC: Sidhartha Kumar <sidhartha.kumar@xxxxxxxxxx> > CC: Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> Reviewed-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx> > --- > lib/maple_tree.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/lib/maple_tree.c b/lib/maple_tree.c > index 3619301dda2e..21e6895b7aef 100644 > --- a/lib/maple_tree.c > +++ b/lib/maple_tree.c > @@ -7287,10 +7287,12 @@ void mt_dump(const struct maple_tree *mt, enum mt_dump_format format) > > pr_info("maple_tree(%p) flags %X, height %u root %p\n", > mt, mt->ma_flags, mt_height(mt), entry); > - if (!xa_is_node(entry)) > - mt_dump_entry(entry, 0, 0, 0, format); > - else if (entry) > + if (xa_is_node(entry)) > mt_dump_node(mt, entry, 0, mt_node_max(entry), 0, format); > + else if (entry) > + mt_dump_entry(entry, 0, 0, 0, format); > + else > + pr_info("(empty)\n"); > } > EXPORT_SYMBOL_GPL(mt_dump); > > -- > 2.34.1 >