Re: [PATCH] mm: fix minor spelling mistakes in comments

[Kevin Lourenco <klourencodev@gmail.com>]

Hi SJ,

I think you’re right — the wording there is probably not correct. If
anything, I would be more inclined to use “the next step is”
(singular), since there is only one alignment step following the start
gap adjustment.

That said, I intentionally limited this patch to clear typos. While
going through mm/, I did notice a number of grammatical issues in
comments, but I chose to ignore them to keep the scope focused cause
some typos actually made me wonder whether they were kernel-specific
terminology or just misspellings (e.g. “sinze boot”).

So I’d prefer to keep this patch dedicated to obvious typos only, and
leave grammar cleanups for future patches.

Thanks a lot for the review!

Best regards,
Kevin


Le sam. 20 déc. 2025 à 03:27, SeongJae Park <sj@kernel.org> a écrit :
>
> On Thu, 18 Dec 2025 16:09:06 +0100 klourencodev@gmail.com wrote:
>
> > From: Kevin Lourenco <klourencodev@gmail.com>
> >
> > Correct several typos in comments across files in mm/
> >
> > Signed-off-by: Kevin Lourenco <klourencodev@gmail.com>
> > ---
> [...]
> > diff --git a/mm/vma.c b/mm/vma.c
> > index fc90befd162f..bf62ac1c52ad 100644
> > --- a/mm/vma.c
> > +++ b/mm/vma.c
> > @@ -2909,8 +2909,8 @@ unsigned long unmapped_area(struct vm_unmapped_area_info *info)
> >       /*
> >        * Adjust for the gap first so it doesn't interfere with the
> >        * later alignment. The first step is the minimum needed to
> > -      * fulill the start gap, the next steps is the minimum to align
> > -      * that. It is the minimum needed to fulill both.
> > +      * fulfill the start gap, the next steps is the minimum to align
> > +      * that. It is the minimum needed to fulfill both.
>
> Nit.  s/next steps is/next steps are/ ?
>
> I'm not sure if that is really grammartically correct.  I'm just asking.  Even
> if I'm correct, I wouldn't argue you have to fix it right now.  So please feel
> free to ignore my comment.
>
> Reviewed-by: SeongJae Park <sj@kernel.org>
>
>
> Thanks,
> SJ
>
> [...]