| Message ID | 20230110152216.1712120-1-vernon2gm@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | maple_tree: fix comment of mte_destroy_walk | expand |
On Tue, Jan 10, 2023 at 11:22:16PM +0800, Vernon Yang wrote: > /* > * mte_destroy_walk() - Free a tree or sub-tree. > * @enode - the encoded maple node (maple_enode) to start > - * @mn - the tree to free - needed for node types. > + * @mt - the tree to free - needed for node types. Should probably also fix the separator between the parameter name and the description to be : instead of -.
On Tue, Jan 10, 2023 at 06:43:22PM +0000, Matthew Wilcox wrote: > On Tue, Jan 10, 2023 at 11:22:16PM +0800, Vernon Yang wrote: > > /* > > * mte_destroy_walk() - Free a tree or sub-tree. > > * @enode - the encoded maple node (maple_enode) to start > > - * @mn - the tree to free - needed for node types. > > + * @mt - the tree to free - needed for node types. > > Should probably also fix the separator between the parameter name and > the description to be : instead of -. Okay, thanks you for review.
diff --git a/lib/maple_tree.c b/lib/maple_tree.c index 26e2045d3cda..69be9d3db0c8 100644 --- a/lib/maple_tree.c +++ b/lib/maple_tree.c @@ -5591,7 +5591,7 @@ static void mt_destroy_walk(struct maple_enode *enode, unsigned char ma_flags, /* * mte_destroy_walk() - Free a tree or sub-tree. * @enode - the encoded maple node (maple_enode) to start - * @mn - the tree to free - needed for node types. + * @mt - the tree to free - needed for node types. * * Must hold the write lock. */
The parameter name of maple tree is mt, so make the comment to correct. Fixes: 54a611b60590 ("Maple Tree: add new data structure") Signed-off-by: Vernon Yang <vernon2gm@gmail.com> --- lib/maple_tree.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)