[PATCH linux-next] mm: memcontrol: Fix typos in comment Fix a spelling mistake in comment.

cgel.zte@gmail.com posted 1 patch 3 years, 7 months ago
mm/memcontrol.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH linux-next] mm: memcontrol: Fix typos in comment Fix a spelling mistake in comment.
Posted by cgel.zte@gmail.com 3 years, 7 months ago
From: xupanda <xu.panda@zte.com.cn>

Reported-by: Zeal Robot <zealic@zte.com.cn>
Signed-off-by: xupanda <xu.panda@zte.com.cn>
Thanks-to: Yang Yang <yang.yang29@zte.com.cn>

---
 mm/memcontrol.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index b69979c9ced5..4dddd8be320a 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -1143,7 +1143,7 @@ static void invalidate_reclaim_iterators(struct mem_cgroup *dead_memcg)
 	} while ((memcg = parent_mem_cgroup(memcg)));
 
 	/*
-	 * When cgruop1 non-hierarchy mode is used,
+	 * When cgroup1 non-hierarchy mode is used,
 	 * parent_mem_cgroup() does not walk all the way up to the
 	 * cgroup root (root_mem_cgroup). So we have to handle
 	 * dead_memcg from cgroup root separately.
-- 
2.25.1
Re: [PATCH linux-next] mm: memcontrol: Fix typos in comment Fix a spelling mistake in comment.
Posted by Muchun Song 3 years, 7 months ago

> On Aug 15, 2022, at 14:12, cgel.zte@gmail.com wrote:
> 
> From: xupanda <xu.panda@zte.com.cn>

1. Your subject seems to have redundant description.
   “Fix typos in comment” vs “Fix a spelling mistake in comment"
2. It is better to clarify what you have changed here. Like:

Fix a spelling mistake on “cgruop1”.

With this:

Reviewed-by: Muchun Song <songmuchun@bytedance.com>

Thanks.