Re: [PATCH v2 1/3] mm/damon/core: prevent size quota overflow in the temporal goal tuner

From: Donggeun Yoo

Date: Sun Sep 20 2026 - 09:14:45 EST


On Sun, 20 Sep 2026 05:39:58 -0700 SJ Park <sj@xxxxxxxxxx> wrote:
> Don't post a new version without ensuring the discussion on the current version
> is completed. Also give time for others to chime in. As a rule of thumbs,
> ensure all questions on the thread are answered, and give at least ~1 day since
> the last comment on the thread before posting a new version.
>
> I will not review the v3.

I did not know. I will keep it in mind for the next patches. Thanks.

> So 256 MiB means nothing special, isn't it? Why you mention it?

I mentioned it because it is a value users are likely to set, and it wraps
to exactly zero, which I thought could be the worst case. But the problem
is explained well enough without it, so I am dropping it.

> Let's put "The scheme makes no progress ..." before "so it is unlikely ...".

Will do.

> Seems unnecessarily verbose. I'd suggest keeping only the first sentence.

Will do.

Thanks,
Donggeun