Re: [PATCH] mm/damon/reclaim: improve grammar and style

[SJ Park]

On Mon, 26 Oct 2026 18:51:40 +0100 Manuel Ebner <manuelebnerli@xxxxxxxxxxx> wrote:

> Hi,
> 
> took me some time to respond because of the procrastinations.

No worry!

> 
> On Sat, 2026-10-17 at 09:27 -0700, SJ Park wrote:
> > On Sat, 17 Oct 2026 11:18:34 +0200 Manuel Ebner <manuelebnerli@xxxxxxxxxxx> wrote:
> > 
> > > Hi SJ,
> > > 
> > > On Thu, 2026-10-15 at 07:01 -0700, SJ Park wrote:
> > > > On Thu, 15 Oct 2026 10:06:23 +0200 Manuel Ebner <manuelebnerli@xxxxxxxxxxx> wrote:
> > > > 
> > > > > Fix tenses. Replace 'set as' with 'set to'.
> > > > > Rewrite complex sentences.
> > > > 
> > > > Let's put two sentences in one line, while breaking it for the 75 column limit.
> > > 
> > > Will do.
> > > > 
> > > > [...]
> > > > > @@ -113,8 +113,8 @@ quota_sz
> > > > >  
> > > > >  Limit of size of memory for the reclamation in bytes.
> > > > >  
> > > > > -DAMON_RECLAIM charges amount of memory which it tried to reclaim within a time
> > > > > -window (quota_reset_interval_ms) and makes no more than this limit is tried.
> > > > > +DAMON_RECLAIM recordes the amount of memory which it tried to reclaim within a time
> > > > 
> > > > s/recordes/charges/  or s/recordes/records/ ?
> > > 
> > > For me charge is such a weird word exepct in 'charge a battery' or 'charge a fee'. And
> > > 'recordes' is a typo. Does this work:
> > 
> > 'charge' in this sentence is somewhat like 'charge a fee'.  I personally don't
> > feel it is weird.  But 'record' also work for me.
> 
> I will keep 'charge' at the later mention in the file.

Thank you for accepting my humble suggestion.

[...]
> > Sounds good.  But let's finish the discussion about use of the term, 'charge'
> > before posting v2.
> 
> Then I'll head to [v2] and make it a 2 part series.

Looking forward to v2!


Thanks,
SJ

[...]