Skip to content

mm/damon: cleanup kdamond, damon_call(), damos filter and DAMON_MIN_REGION

SeongJae Park damon linux-mm lkml 1 version Applied

Status from this message: [merged mm-stable] mm-damon-rename-min_sz_region-of-damon_ctx-to-min_region_sz.patch removed from -mm tree

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 0 repliesApplied 8/8

Patches in v1 All 8 applied

#Subject and filesSizeRepliesApplied
1/8mm/damon: remove damon_operations->cleanup()include/linux/damon.h mm/damon/core.c mm/damon/paddr.c +1 more+0 -70 repliesApplied
2/8mm/damon/core: cleanup targets and regions at once on kdamond terminationmm/damon/core.c+1 -70 repliesApplied
3/8mm/damon/core: cancel damos_walk() before damon_ctx->kdamond resetmm/damon/core.c+1 -20 repliesApplied
4/8mm/damon/core: process damon_call_control requests on a local listmm/damon/core.c+21 -380 repliesApplied
5/8mm/damon: document damon_call_control->dealloc_on_cancel repeat behaviorinclude/linux/damon.h+1 -10 repliesApplied
6/8mm/damon/core: rename damos_filter_out() to damos_core_filter_out()mm/damon/core.c+2 -20 repliesApplied
7/8mm/damon: rename DAMON_MIN_REGION to DAMON_MIN_REGION_SZinclude/linux/damon.h mm/damon/core.c mm/damon/lru_sort.c +4 more+18 -180 repliesApplied
8/8mm/damon: rename min_sz_region of damon_ctx to min_region_szinclude/linux/damon.h mm/damon/core.c mm/damon/lru_sort.c +3 more+49 -470 repliesApplied

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/8] mm/damon: cleanup kdamond, damon_call(), damos filter and DAMON_MIN_REGION

Do miscellaneous code cleanups for improving readability. First three patches cleanup kdamond termination process, by removing unused operation set cleanup callback (patch 1) and moving damon_ctx specific resource cleanups on kdamond termination to synchronization-easy place (patches 2 and 3). Next two patches touch damon_call() infrastructure, by refactoring kdamond_call() function to do less and simpler locking operations (patch 4), and documenting when dealloc_on_free does work (patch 5). Final three patches rename things for clear uses of those. Those rename damos_filter_out() to be more explicit about the fact that it is only for core-handled filters (patch 6), DAMON_MIN_REGION macro to be more explicit it is not about number of regions but size of each region (patch 7), and damon_ctx->min_sz_region to be different from damos_access_patern->min_sz_region (patch 8), so that those are not confusing and easy to grep.

SeongJae Park (8):
mm/damon: remove damon_operations->cleanup() mm/damon/core: cleanup targets and regions at once on kdamond
termination
mm/damon/core: cancel damos_walk() before damon_ctx->kdamond reset mm/damon/core: process damon_call_control requests on a local list mm/damon: document damon_call_control->dealloc_on_cancel repeat
behavior
mm/damon/core: rename damos_filter_out() to damos_core_filter_out() mm/damon: rename DAMON_MIN_REGION to DAMON_MIN_REGION_SZ mm/damon: rename min_sz_region of damon_ctx to min_region_sz

 include/linux/damon.h        |  15 ++--
 mm/damon/core.c              | 143 +++++++++++++++--------------------
 mm/damon/lru_sort.c          |   4 +-
 mm/damon/paddr.c             |   1 -
 mm/damon/reclaim.c           |   4 +-
 mm/damon/stat.c              |   2 +-
 mm/damon/sysfs.c             |  11 +--
 mm/damon/tests/vaddr-kunit.h |   2 +-
 mm/damon/vaddr.c             |  25 +++---
 9 files changed, 89 insertions(+), 118 deletions(-)

base-commit: 886ef5515d08f05191b61a347fc18f371627bdd4

Signature · 1 line
2.47.3