Re: [PATCH] xfs: clean up spelling and grammar errors in comments

[caina <caina@uniontech.com>]

On Tue, Jun 16, 2026 at 06:08:47AM -0700, Christoph Hellwig wrote:
> On Tue, Jun 16, 2026 at 10:09:12AM +0800, caina wrote:
> > -    * If we need to re-sort the free extents by length, do so so that we
> > +    * If we need to re-sort the free extents by length, do so that we
> >      * can put the records into the cntbt in the correct order.
> 
> This looks incorrect to me, but I'm not a native speaker.

Hi Christoph,

Thank you for the review.

I agree with you – the original "do so so that" is indeed correct here.
"do so" means "perform the re-sorting", and "so that" introduces the purpose.
Removing one "so" would make it ungrammatical. I will keep this line as "do so so that" in the next version.

> > @@ -342,7 +342,7 @@ xchk_btree_block_check_siblings(
> >     /*
> >      * Does the left & right sibling pointers match the adjacent
> >      * parent level pointers?
> 
> Should the does here be a do?

You are right – the subject "pointers" is plural, so the auxiliary verb should be "Do".
I will change it to "Do the left & right sibling pointers..." accordingly.

I will send a v2 patch with both fixes.

Please let me know if you have any other suggestions.

Thanks,
Caina