Skip to content

RFC mm/damon: improve readability, clarity and test coverage

SJ Park damon linux-doc linux-kselftest linux-mm lkml 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 RFC Latest10 patches 0 repliesNew
  2. v1 RFC9 patches 7 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10mm/damon/api: remove NR_DAMOS_FILTER_TYPESinclude/linux/damon.h+0 -20 replies
2/10mm/damon/core: use abs_diff() in damon_feed_loop_next_input()mm/damon/core.c+1 -40 replies
3/10mm/damon/core: use mult_frac() in damon_feed_loop_next_input()mm/damon/core.c+1 -50 replies
4/10mm/damon/core: set damon_ctx->walk_control_obsolete in damon_new_ctx()mm/damon/core.c+1 -40 replies
5/10mm/damon/core: document damon_call()/damon_start() race hang issuemm/damon/core.c+3 -00 replies
6/10mm/damon/paddr: remove pa parameter from damon_pa_filter_pass()mm/damon/paddr.c+2 -30 replies
7/10mm/damon/tests/core-kunit: test eligible_mem_bp commitmentmm/damon/tests/core-kunit.h+10 -00 replies
8/10mm/damon/tests/core-kunit: add probe_hits_wsum damos filter commit testmm/damon/tests/core-kunit.h+20 -00 replies
9/10selftests/damon/sysfs_memcg_path_leak: fail only for real DAMON leaktools/testing/selftests/damon/sysfs_memcg_path_leak.sh+7 -00 replies
10/10Docs/mm/damon/design: clarify bp is basis pointDocumentation/mm/damon/design.rst+3 -20 replies

Reviews trailers per patch

1 of 10 patches has a Reviewed-by or Acked-by.

PatchRandy Dunlap
1/10
2/10
3/10
4/10
5/10
6/10
7/10
8/10
9/10
10/10A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [RFC PATCH v2 00/10] mm/damon: improve readability, clarity and test coverage

Yet another batch of miscellaneous DAMON minor improvements. Mostly focused on readability and clarity of code and document, and unit/self test coverage. No user-visible behavioral change is intended.

Changes from RFC
- RFC: https://lore.kernel.org/[email protected]
- Fix a typo in the selftest warning message.
- Collect A-b: from Randy.
- Split out damos_adjust_quota() fix to an individual hotfix patch.
- Add a patch for initializing walk_control_obsolete in damon_new_ctx().
- Add a patch for probe_hits_wsum damos filter commit kunit test.
- Trivial patch sequence update.
- Rebase to the latest mm-new.

SJ Park (10):
mm/damon/api: remove NR_DAMOS_FILTER_TYPES
mm/damon/core: use abs_diff() in damon_feed_loop_next_input() mm/damon/core: use mult_frac() in damon_feed_loop_next_input() mm/damon/core: set damon_ctx->walk_control_obsolete in damon_new_ctx() mm/damon/core: document damon_call()/damon_start() race hang issue mm/damon/paddr: remove pa parameter from damon_pa_filter_pass() mm/damon/tests/core-kunit: test eligible_mem_bp commitment mm/damon/tests/core-kunit: add probe_hits_wsum damos filter commit
test
selftests/damon/sysfs_memcg_path_leak: fail only for real DAMON leak Docs/mm/damon/design: clarify bp is basis point

 Documentation/mm/damon/design.rst             |  5 ++--
 include/linux/damon.h                         |  2 --
 mm/damon/core.c                               | 19 ++++--------
 mm/damon/paddr.c                              |  5 ++--
 mm/damon/tests/core-kunit.h                   | 30 +++++++++++++++++++
 .../selftests/damon/sysfs_memcg_path_leak.sh  |  7 +++++
 6 files changed, 48 insertions(+), 20 deletions(-)

base-commit: b7c76a308c2afda7cfc2ce98be3eed466be71337

Signature · 1 line
2.47.3