Re: [PATCH v2 0/2] mm/damon: preserve quota state when constructing schemes

[SJ Park <sj@kernel.org>]

On Thu, 24 Sep 2026 22:16:12 +0200 Karl Mehltretter <kmehltretter@gmail.com> wrote:

> damon_commit_ctx() first commits the running context's parameters to a
> temporary context for validating proposed updates. Constructing the
> temporary schemes clears the running schemes' quota state because
> damon_new_scheme() initializes the quota passed as a parameter before
> copying it to the new scheme. Even an update later rejected with -EINVAL
> loses the running quota state.
> 
> Initialize the new scheme's copy instead, and add KUnit tests for the
> constructor and for accepted and rejected context updates.

Looks good to me.  Thank you for this series, Karl.

This series is applied to damon/next [1] tree, with an addition of user impact
detail on patch 1.  If there is no more change requests and this series is not
added to mm.git in short term (~1 week?), I will ask mm.git maintainer (Andrew
Morton) to pick this.  So, no action from your side is needed for now.  If it
seems I also forgot doing that or you cannot wait for my action, please feel
free to ping me or Andrew.

[1] https://origin.kernel.org/doc/html/latest/mm/damon/maintainer-profile.html#scm-trees


Thanks,
SJ