Re: [PATCH v2] mm/damon/tests/core-kunit: test damon_nr_accesses_mvsum()
From: SJ Park
Date: Mon Jul 13 2026 - 10:22:38 EST
On Mon, 13 Jul 2026 17:46:48 +0800 Song Hu <husong@xxxxxxxxxx> wrote:
> damon_nr_accesses_mvsum() wraps damon_mvsum() with the monitoring
> intervals of the context to compute the pseudo moving sum of a region's
> access frequency, with a special case for when the whole aggregation
> window remains. damon_mvsum() itself is already covered by
> damon_test_mvsum(), but the wrapper is not.
>
> Add a table-driven KUnit test that exercises the full-window-remaining
> boundary (with both reset and not-yet-reset nr_accesses), partially
> elapsed windows, and the no-window-remaining case.
Thank you for this patch!
>
> Signed-off-by: Song Hu <husong@xxxxxxxxxx>
Reviewed-by: SJ Park <sj@xxxxxxxxxx>
> ---
> Changes in v2:
> - Drop the confusing 'bp=...:' annotation from each test-case comment.
> - Wrap the function documentation comment to the 80-column limit.
And the v1 is available [1] at mailing list.
Thank you for adding this changelog. Please add links [2] to the previous
revisions too, from next time. For other readers, v1
This patch is applied to damon/next [3] tree. If this patch 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
directly ask that to Andrew.
[1] https://lore.kernel.org/20260710041530.557455-1-husong@xxxxxxxxxx
[2] https://docs.kernel.org/process/submitting-patches.html#commentary
[3] https://origin.kernel.org/doc/html/latest/mm/damon/maintainer-profile.html#scm-trees
Thanks,
SJ
[...]