[PATCH] ARC:mm:Fix syntax errors in comments

[PATCH] ARC:mm:Fix syntax errors in comments

[Jilin Yuan]

Delete the redundant word 'to'.

Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com>
---
 arch/arc/mm/cache.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arc/mm/cache.c b/arch/arc/mm/cache.c
index 5446967ea98d..84fe78c41c37 100644
--- a/arch/arc/mm/cache.c
+++ b/arch/arc/mm/cache.c
@@ -750,7 +750,7 @@ static inline void arc_slc_enable(void)
  *  -In SMP, if hardware caches are coherent
  *
  * There's a corollary case, where kernel READs from a userspace mapped page.
- * If the U-mapping is not congruent to to K-mapping, former needs flushing.
+ * If the U-mapping is not congruent to K-mapping, former needs flushing.
  */
 void flush_dcache_page(struct page *page)
 {
-- 
2.36.1

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Bagas Sanjaya]

On Wed, Jun 22, 2022 at 04:04:23PM +0800, Jilin Yuan wrote:
> Delete the redundant word 'to'.
> 
> Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com>
> ---
>  arch/arc/mm/cache.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arc/mm/cache.c b/arch/arc/mm/cache.c
> index 5446967ea98d..84fe78c41c37 100644
> --- a/arch/arc/mm/cache.c
> +++ b/arch/arc/mm/cache.c
> @@ -750,7 +750,7 @@ static inline void arc_slc_enable(void)
>   *  -In SMP, if hardware caches are coherent
>   *
>   * There's a corollary case, where kernel READs from a userspace mapped page.
> - * If the U-mapping is not congruent to to K-mapping, former needs flushing.
> + * If the U-mapping is not congruent to K-mapping, former needs flushing.
>   */
>  void flush_dcache_page(struct page *page)
>  {

The patch is OK, but its subject is wrong.

The patch above isn't fixing any syntax errors, but rather minor cleanup.
The subject should have been "Remove duplicate 'to' in the
flush_dcache_page() comment".

I think that this kind of cleanup patches (typofixes) are best done
as part of **actual** work on the code in question (for example
refactoring or fixing build errors).

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Shahab Vahedi]

On 6/22/22 10:30, Bagas Sanjaya wrote:

> I think that this kind of cleanup patches (typofixes) are best done
> as part of **actual** work on the code in question (for example
> refactoring or fixing build errors).

Amen!

-- 
Shahab

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Julia Lawall]

On Wed, 22 Jun 2022, Shahab Vahedi wrote:

> On 6/22/22 10:30, Bagas Sanjaya wrote:
>
> > I think that this kind of cleanup patches (typofixes) are best done
> > as part of **actual** work on the code in question (for example
> > refactoring or fixing build errors).
>
> Amen!

So code that is fine will have typos forever?  Fixing typos in comments
doesn't break git blame for the following code.  And typos in comments
give a bad impression about the state of the code in general.

But that is a general statement.  Perhaps this code does particularly need
work.

julia

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Shahab Vahedi]

On 6/22/22 11:16, Julia Lawall wrote:
> 
> 
> On Wed, 22 Jun 2022, Shahab Vahedi wrote:
> 
>> On 6/22/22 10:30, Bagas Sanjaya wrote:
>>
>>> I think that this kind of cleanup patches (typofixes) are best done
>>> as part of **actual** work on the code in question (for example
>>> refactoring or fixing build errors).
>>
>> Amen!
> 
> So code that is fine will have typos forever?  Fixing typos in comments
> doesn't break git blame for the following code.  And typos in comments
> give a bad impression about the state of the code in general.

Of course not! Documentation is as important as the code, if not even more.
However, fixing typos to increase your commit counts to a reputable project
is not fine either. For instance, many of these proposed fixes are targeting
one single typo at a time. Couldn't they just be sent altogether!?!

-- 
Shahab

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Julia Lawall]

On Wed, 22 Jun 2022, Shahab Vahedi wrote:

> On 6/22/22 11:16, Julia Lawall wrote:
> >
> >
> > On Wed, 22 Jun 2022, Shahab Vahedi wrote:
> >
> >> On 6/22/22 10:30, Bagas Sanjaya wrote:
> >>
> >>> I think that this kind of cleanup patches (typofixes) are best done
> >>> as part of **actual** work on the code in question (for example
> >>> refactoring or fixing build errors).
> >>
> >> Amen!
> >
> > So code that is fine will have typos forever?  Fixing typos in comments
> > doesn't break git blame for the following code.  And typos in comments
> > give a bad impression about the state of the code in general.
>
> Of course not! Documentation is as important as the code, if not even more.
> However, fixing typos to increase your commit counts to a reputable project
> is not fine either. For instance, many of these proposed fixes are targeting
> one single typo at a time. Couldn't they just be sent altogether!?!

I have the impression that the person is just trying to figure out the
patch submission process.  For example, the subject lines are not
formatter in the standard way (I sent the person a private email about
that).  Perhaps just let him know about how you would rather have received
the patches.

julia

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Bagas Sanjaya]

On 6/22/22 16:38, Julia Lawall wrote:
>>> So code that is fine will have typos forever?  Fixing typos in comments
>>> doesn't break git blame for the following code.  And typos in comments
>>> give a bad impression about the state of the code in general.
>>
>> Of course not! Documentation is as important as the code, if not even more.
>> However, fixing typos to increase your commit counts to a reputable project
>> is not fine either. For instance, many of these proposed fixes are targeting
>> one single typo at a time. Couldn't they just be sent altogether!?!
> 
> I have the impression that the person is just trying to figure out the
> patch submission process.  For example, the subject lines are not
> formatter in the standard way (I sent the person a private email about
> that).  Perhaps just let him know about how you would rather have received
> the patches.

In recent times I had seen many typofix patches sent to LKML. You can see most
of them by querying `s:"fix typo"` on lore.kernel.org. Some of these patches
have been merged, though.

What I say as starter thread is "ideal" scenario as described in
Documentation/process/2.Process.rst; that is we prefer to see these minor
fixes as part of real patches work (say refactoring), rather than just being
trivial patches.

But what most reviewers here missed is how these typos are found? I guess
these can be from codespell or some other tools, or even manual review,
then send the fixes en mass.

Take a look at "fix typo in a comment" aka "delete redundant word" patches
at [1], [2], [3]. and [4].

[1]: https://lore.kernel.org/lkml/20220618132659.17100-1-wangxiang@cdjrlc.com/
[2]: https://lore.kernel.org/lkml/20220618130349.11507-1-wangxiang@cdjrlc.com/
[3]: https://lore.kernel.org/lkml/20220616163830.11366-1-wangxiang@cdjrlc.com/
[4]: https://lore.kernel.org/lkml/20220606123419.29109-1-wangxiang@cdjrlc.com/

IMHO, these patches should have been in a single, consolidated patch, since
these strip duplicate (hence redundant) word (single logical change).

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Julia Lawall]

On Wed, 22 Jun 2022, Bagas Sanjaya wrote:

> On 6/22/22 16:38, Julia Lawall wrote:
> >>> So code that is fine will have typos forever?  Fixing typos in comments
> >>> doesn't break git blame for the following code.  And typos in comments
> >>> give a bad impression about the state of the code in general.
> >>
> >> Of course not! Documentation is as important as the code, if not even more.
> >> However, fixing typos to increase your commit counts to a reputable project
> >> is not fine either. For instance, many of these proposed fixes are targeting
> >> one single typo at a time. Couldn't they just be sent altogether!?!
> >
> > I have the impression that the person is just trying to figure out the
> > patch submission process.  For example, the subject lines are not
> > formatter in the standard way (I sent the person a private email about
> > that).  Perhaps just let him know about how you would rather have received
> > the patches.
>
> In recent times I had seen many typofix patches sent to LKML. You can see most
> of them by querying `s:"fix typo"` on lore.kernel.org. Some of these patches
> have been merged, though.
>
> What I say as starter thread is "ideal" scenario as described in
> Documentation/process/2.Process.rst; that is we prefer to see these minor
> fixes as part of real patches work (say refactoring), rather than just being
> trivial patches.
>
> But what most reviewers here missed is how these typos are found? I guess
> these can be from codespell or some other tools, or even manual review,
> then send the fixes en mass.
>
> Take a look at "fix typo in a comment" aka "delete redundant word" patches
> at [1], [2], [3]. and [4].
>
> [1]: https://lore.kernel.org/lkml/20220618132659.17100-1-wangxiang@cdjrlc.com/
> [2]: https://lore.kernel.org/lkml/20220618130349.11507-1-wangxiang@cdjrlc.com/
> [3]: https://lore.kernel.org/lkml/20220616163830.11366-1-wangxiang@cdjrlc.com/
> [4]: https://lore.kernel.org/lkml/20220606123419.29109-1-wangxiang@cdjrlc.com/
>
> IMHO, these patches should have been in a single, consolidated patch, since
> these strip duplicate (hence redundant) word (single logical change).

They go to different maintainers, so perhaps each one would prefer to get
their own patches?  There was a trivial tree, but it is apparently not
very active.

It would indeed be good to indicate how the problems were found.  That
could suggest whether the problem has been addressed comprehensively, or
whether just some random issues have been detected.

julia

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Conor.Dooley]

On 22/06/2022 14:46, Julia Lawall wrote:
> 
> 
> On Wed, 22 Jun 2022, Bagas Sanjaya wrote:
> 
>> On 6/22/22 16:38, Julia Lawall wrote:
>>>>> So code that is fine will have typos forever?  Fixing typos in comments
>>>>> doesn't break git blame for the following code.  And typos in comments
>>>>> give a bad impression about the state of the code in general.
>>>>
>>>> Of course not! Documentation is as important as the code, if not even more.
>>>> However, fixing typos to increase your commit counts to a reputable project
>>>> is not fine either. For instance, many of these proposed fixes are targeting
>>>> one single typo at a time. Couldn't they just be sent altogether!?!
>>>
>>> I have the impression that the person is just trying to figure out the
>>> patch submission process.  For example, the subject lines are not
>>> formatter in the standard way (I sent the person a private email about
>>> that).  Perhaps just let him know about how you would rather have received
>>> the patches.
>>
>> In recent times I had seen many typofix patches sent to LKML. You can see most
>> of them by querying `s:"fix typo"` on lore.kernel.org. Some of these patches
>> have been merged, though.
>>
>> What I say as starter thread is "ideal" scenario as described in
>> Documentation/process/2.Process.rst; that is we prefer to see these minor
>> fixes as part of real patches work (say refactoring), rather than just being
>> trivial patches.
>>
>> But what most reviewers here missed is how these typos are found? I guess
>> these can be from codespell or some other tools, or even manual review,
>> then send the fixes en mass.
>>
>> Take a look at "fix typo in a comment" aka "delete redundant word" patches
>> at [1], [2], [3]. and [4].
>>
>> [1]: https://lore.kernel.org/lkml/20220618132659.17100-1-wangxiang@cdjrlc.com/
>> [2]: https://lore.kernel.org/lkml/20220618130349.11507-1-wangxiang@cdjrlc.com/
>> [3]: https://lore.kernel.org/lkml/20220616163830.11366-1-wangxiang@cdjrlc.com/
>> [4]: https://lore.kernel.org/lkml/20220606123419.29109-1-wangxiang@cdjrlc.com/
>>
>> IMHO, these patches should have been in a single, consolidated patch, since
>> these strip duplicate (hence redundant) word (single logical change).
> 
> They go to different maintainers, so perhaps each one would prefer to get
> their own patches?  There was a trivial tree, but it is apparently not
> very active.
> 
> It would indeed be good to indicate how the problems were found.  That
> could suggest whether the problem has been addressed comprehensively, or
> whether just some random issues have been detected.

It would be interesting to see what tool it is, because there appear to be
a bunch of false positives (although maybe that is on the user to read and
understand the tool's output)

Thanks,
Conor.

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Vineet Gupta]

On 6/22/22 01:30, Bagas Sanjaya wrote:
>> - * If the U-mapping is not congruent to to K-mapping, former needs flushing.
>> + * If the U-mapping is not congruent to K-mapping, former needs flushing.
>>    */
>>   void flush_dcache_page(struct page *page)
>>   {
> The patch is OK, but its subject is wrong.

Right.

> The patch above isn't fixing any syntax errors, but rather minor cleanup.
> The subject should have been "Remove duplicate 'to' in the
> flush_dcache_page() comment".

I'd just say "ARC: mm: fix typos"

> I think that this kind of cleanup patches (typofixes) are best done
> as part of **actual** work on the code in question (for example
> refactoring or fixing build errors).

In an ideal world yes. But sometimes maintainer complain to break 
whitespacxe fixes and such into independent fix. Also as someone said 
later in the thread, for somebody just getting into kernel and figuring 
out patch submission etc this could be a perfect dry run and helps 
improve the code anyways.

-Vineet

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Bagas Sanjaya]

On 6/23/22 09:50, Vineet Gupta wrote:
>> The patch is OK, but its subject is wrong.
> 
> Right.
> 
>> The patch above isn't fixing any syntax errors, but rather minor cleanup.
>> The subject should have been "Remove duplicate 'to' in the
>> flush_dcache_page() comment".
> 
> I'd just say "ARC: mm: fix typos"
> 

OK.

> In an ideal world yes. But sometimes maintainer complain to break whitespacxe fixes and such into independent fix. Also as someone said later in the thread, for somebody just getting into kernel and figuring out patch submission etc this could be a perfect dry run and helps improve the code anyways.
> 

Seems like you missed the point that it's OK to have typofixes while
doing other real changes (like refactoring) in the same patch.

Quoting from [1]:

> My opinion is that trivial patches like this are fine as a starting
> point for new contributors, which is why I acked the previous patch from
> you guys. However, if we start getting two of these every week it just
> adds more maintenance burden than it's worth.

I tend to agree with the last sentence of above quote. Let's pretend that
I'm the tree maintainer. Besides reviewing real change patches, I get
flooded by these similar minor cleanup patches that I need to review.
Some (but not all) these patches have issues (say subject or description
error) that are repeated. Lazily speaking, I'd like to privately notice the
submitter about the situation, and I withhold these for now.

Thanks.

[1]: https://lore.kernel.org/lkml/0308c92a-0e10-35a4-928b-8f715a7bae44@linbit.com/

-- 
An old man doll... just what I always wanted! - Clara

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Vineet Gupta]

On 6/23/22 00:56, Bagas Sanjaya wrote:
>> In an ideal world yes. But sometimes maintainer complain to break whitespacxe fixes and such into independent fix. Also as someone said later in the thread, for somebody just getting into kernel and figuring out patch submission etc this could be a perfect dry run and helps improve the code anyways.
>>
> Seems like you missed the point that it's OK to have typofixes while
> doing other real changes (like refactoring) in the same patch.

No I did not.

> Quoting from [1]:
>
>> My opinion is that trivial patches like this are fine as a starting
>> point for new contributors, which is why I acked the previous patch from
>> you guys. However, if we start getting two of these every week it just
>> adds more maintenance burden than it's worth.
> I tend to agree with the last sentence of above quote. Let's pretend that
> I'm the tree maintainer. Besides reviewing real change patches, I get
> flooded by these similar minor cleanup patches that I need to review.
> Some (but not all) these patches have issues (say subject or description
> error) that are repeated.

"Repeated" is the key - First time I'd politely tell them to DTRT but 
will just ignore if things continue.

> Lazily speaking, I'd like to privately notice the
> submitter about the situation, and I withhold these for now.

Funny that you say this: ever since this got posted I now see 3 patches 
for typo fixes :-)
But it is something in maintainers purview and for now i'm ok.

Thx,
-Vineet

Re: [PATCH] ARC:mm:Fix syntax errors in comments

[Bagas Sanjaya]

On 6/24/22 08:57, Vineet Gupta wrote:
> "Repeated" is the key - First time I'd politely tell them to DTRT but will just ignore if things continue.
> 
>> Lazily speaking, I'd like to privately notice the
>> submitter about the situation, and I withhold these for now.
> 
> Funny that you say this: ever since this got posted I now see 3 patches for typo fixes :-)
> But it is something in maintainers purview and for now i'm ok.
> 

Thanks.

-- 
An old man doll... just what I always wanted! - Clara