Re: [PATCH] mm: fix repeated words in comments

From: Nadav Amit
Date: Sun Sep 18 2022 - 13:43:48 EST


On Sep 18, 2022, at 8:08 AM, Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:

> ⚠ External Email
>
> On 9/18/22 03:01, Jilin Yuan wrote:
>> Delete the redundant word 'on'.
>>
>> Signed-off-by: Jilin Yuan <yuanjilin@xxxxxxxxxx>
>> ---
>> drivers/misc/vmw_balloon.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/misc/vmw_balloon.c b/drivers/misc/vmw_balloon.c
>> index 61a2be712bf7..c2e774f68133 100644
>> --- a/drivers/misc/vmw_balloon.c
>> +++ b/drivers/misc/vmw_balloon.c
>> @@ -736,7 +736,7 @@ static int vmballoon_handle_one_result(struct vmballoon *b, struct page *page,
>> * @p: pointer to where the page struct is returned.
>> *
>> * Following a lock or unlock operation, returns the status of the operation for
>> - * an individual page. Provides the page that the operation was performed on on
>> + * an individual page. Provides the page that the operation was performed on
>
> This would make more sense to me: (s/on on/on in/)

Yes, your version, Randy, is the right one. Thanks Randy and Jilin.

Jilin, while your work is highly appreciated, looking at lkml shows that you
put quite an effort (100s of patches) into the duplicate words problem.

Since 2020 checkpatch warns if you try to push a patch with repeated words.
So all of these mistakes will eventually go away as people make
adjacent/overlapping changes. Understanding the meaning of text with
repeated words is not that hard. Some might argue that polluting the git
history for this purpose is not helping.

Now it is hard to get into the kernel, and making trivial patches is a good
way to start. However, I think it is more helpful if you take one or few
subsystems and focus on fixing them, and thereby gaining deeper
understanding of their code. It would also allow you to introduce yourself
to the maintainers. There are other tasks you can consider, such as more
significant checkpatch warnings, there are bug reports of syzbot you can
look into, etc. As it is right now, it seems that the main benefit from
these trivial patches is increasing the number of patches you submitted to
the kernel.

Again, thank you Jilin.