* [PATCH] maple_tree: fix comment of mte_destroy_walk
@ 2023-01-10 15:22 Vernon Yang
2023-01-10 18:43 ` Matthew Wilcox
0 siblings, 1 reply; 3+ messages in thread
From: Vernon Yang @ 2023-01-10 15:22 UTC (permalink / raw)
To: Liam.Howlett, akpm, willy; +Cc: linux-mm, linux-kernel, Vernon Yang
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(-)
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.
*/
--
2.34.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] maple_tree: fix comment of mte_destroy_walk
2023-01-10 15:22 [PATCH] maple_tree: fix comment of mte_destroy_walk Vernon Yang
@ 2023-01-10 18:43 ` Matthew Wilcox
2023-01-11 12:52 ` Vernon Yang
0 siblings, 1 reply; 3+ messages in thread
From: Matthew Wilcox @ 2023-01-10 18:43 UTC (permalink / raw)
To: Vernon Yang; +Cc: Liam.Howlett, akpm, linux-mm, linux-kernel
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 -.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] maple_tree: fix comment of mte_destroy_walk
2023-01-10 18:43 ` Matthew Wilcox
@ 2023-01-11 12:52 ` Vernon Yang
0 siblings, 0 replies; 3+ messages in thread
From: Vernon Yang @ 2023-01-11 12:52 UTC (permalink / raw)
To: Matthew Wilcox; +Cc: Liam.Howlett, akpm, linux-mm, linux-kernel
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.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-01-11 12:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-10 15:22 [PATCH] maple_tree: fix comment of mte_destroy_walk Vernon Yang
2023-01-10 18:43 ` Matthew Wilcox
2023-01-11 12:52 ` Vernon Yang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox