Re: [RFC PATCH 0/4] mm/damon: introduce perf event based access check

[SeongJae Park <sj@kernel.org>]

On Fri, 23 Jan 2026 11:10:10 +0900 Akinobu Mita <akinobu.mita@gmail.com> wrote:

> DAMON currently only provides PTE accessed-bit based access check, this
> patch series adds a new perf event based access check.

Very interesting patch series.  Thank you for sharing this Akinobu!

I only took a glance on the patches, but my understanding is that this series
modifies DAMON to be able to enable perf events of PERF_SAMPLE_ADDR type, and
utilize the sampled perf events in the perf events buffer as the source of
DAMON's access checks.  In more detail, I understand enabling PERF_SAMPLE_ADDR
type perf events makes the perf events buffer filled with memory access event
information with the access destination address.  The event will be sampled
based on time (e.g., one access event per X milliseconds).  And probably that
sample data could include more information includign the CPU and the process
that executing the sampled access?  Please correct me if I'm wrong and add more
details I'm missing, as my understanding of perf event is very poor.

And one quick question.  Can this work on virtual machines?  I'm asking this
question the for following reason.  I'm actuaally working on a similar project
that extends DAMON for page fault based access events sampling [1].  The
project aims to use page fault event rather than other h/w features such as AMD
IBS or Intel PEBS, because my understanding is that such h/w features are not
available on virtual machines.

> 
> Since perf event-based access checks do not require modifying the PTE
> accessed-bit for pages representing each damon region, this patch series
> also includes a feature that allows you to set upper and lower limits on
> the damon region size to enable access checks with finer granularity.

I was also thinking about extending DAMON with AMD IBS or Intel PEBS like h/w
features for this kind of sub-page granularity access monitoring.  So this
makes sense to me, and sounds useful!

> 
> Using these features also requires modifications to damo, but these are
> not included in this patch series and are currently under development in
> the following branch:
> 
> https://github.com/mita/damo/tree/damo-perf-for-v3.1.0
> 
> Any feedback or advice on the patch set would be greatly appreciated.
> 
> Akinobu Mita (4):
>   mm/damon/core: add common code for perf event based access check
>   mm/damon/vaddr: support perf event based access check
>   mm/damon/paddr: support perf event based access check

I find your patches are introducing new infra code for this extension.  It
seems bit specialized for perf event only, though.  I'm concerned if future
extension for another access check primitives cannot reuse the infra.

My DAMON extension project [1] is for page fault based access monitoring, but
it also introduces a framework for general multiple access sampling primitives.
I'm wondering if perf event based extension can be implemented using the
general acces ssampling primitives infra code, and if you already considered
that but found it is not feasible.

>   mm/damon: allow user to set min and max size of region

The min size setup makes sense.  I understand the max size is for disabling the
regions adjustment (merge/split) mechanism.  IOW, for fixed granularity
monitoring.  Users can do that by setting min_nr_regions and max_nr_regions
same [2], though.  So, max size setting seems not really needed.

Again, great RFC patch series, thank you for sharing!  I'm looking forward to
your answers to above high level questions and comments.

> 
>  .../ABI/testing/sysfs-kernel-mm-damon         |  11 +
>  include/linux/damon.h                         |  42 +-
>  mm/damon/core.c                               | 202 ++++-
>  mm/damon/ops-common.h                         |  39 +
>  mm/damon/paddr.c                              | 106 ++-
>  mm/damon/sysfs.c                              | 402 +++++++++-
>  mm/damon/tests/core-kunit.h                   |   2 +-
>  mm/damon/tests/sysfs-kunit.h                  |   2 +
>  mm/damon/tests/vaddr-kunit.h                  |   7 +-
>  mm/damon/vaddr.c                              | 690 ++++++++++++++++--
>  10 files changed, 1425 insertions(+), 78 deletions(-)
> 
> -- 
> 2.43.0

[1] https://lore.kernel.org/damon/20251208062943.68824-1-sj@kernel.org/
[2] https://origin.kernel.org/doc/html/latest/mm/damon/faq.html#can-i-simply-monitor-page-granularity


Thanks,
SJ