Re: [PATCH v2] mm/damon/tests/core-kunit: test damon_nr_accesses_mvsum()
SJ Park <[email protected]>
| Newsgroups | dev.linux.lists.damon,org.kernel.vger.linux-kernel,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On Mon, 13 Jul 2026 17:46:48 +0800 Song Hu <[email protected]> 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 <[email protected]> Reviewed-by: SJ Park <[email protected]> > --- > 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/[email protected] [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 [...]