linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: SeongJae Park <sj@kernel.org>
To: Enze Li <lienze@kylinos.cn>
Cc: SeongJae Park <sj@kernel.org>,
	akpm@linux-foundation.org, damon@lists.linux.dev,
	linux-mm@kvack.org, enze.li@gmx.com
Subject: Re: [PATCH 0/2] mm/damon: export symbols and introduce prdm module
Date: Mon, 15 Dec 2025 15:16:38 -0800	[thread overview]
Message-ID: <20251215231639.56448-1-sj@kernel.org> (raw)
In-Reply-To: <20251215142057.588500-1-lienze@kylinos.cn>

Hello Enze,

On Mon, 15 Dec 2025 22:20:55 +0800 Enze Li <lienze@kylinos.cn> wrote:

> This patchset makes DAMON's advanced memory management accessible to a
> broader range of users through two complementary enhancements.  First,
> it exports the necessary DAMON core symbols to enable building loadable
> kernel modules.  Building on this foundation, it then introduces the
> 'prdm' module, which extends the concept from the DAMON sample code
> (prcl.c) into a production-ready, user-friendly tool.
> 
> The key strength of this module is its out-of-the-box simplicity: by
> simply specifying the PIDs of target processes and enabling the module,
> users can concurrently monitor memory access patterns across multiple
> processes and trigger proactive reclamation automatically.  This
> significantly lowers the barrier to using DAMON, allowing system
> administrators and regular users without deep kernel expertise to easily
> apply sophisticated memory optimization.

I agree your points about the problems and the direction for improving the
situation.  However, we have a module for such use case, namely DAMON_RECLAIM.
It was designed for a reason that very same to what you described above.  One
key difference between DAMON_RECLAIM and prdm is, to my understanding,
DAMON_RECLAIM works for the physical address space, while prdm works for
virtual address spaces.  Is the virtual address spaces proactive reclamation is
what you really need?  If so, could you please further describe your expected
or planned usage of it?

If the virtual address spaces proactive reclamation is really what you need, I
think extending DAMON_RECLAIM for the purpose might be doable and even simpler
than introducing a new module.  How about doing so?

And, even more importantly, why it should be a loadable module rather than a
static module?  Do you have a specific use case that requires it to be a
loadable module?


Thanks,
SJ

[...]


  parent reply	other threads:[~2025-12-15 23:16 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-15 14:20 Enze Li
2025-12-15 14:20 ` [PATCH 1/2] mm/damon/core: export necessary symbols Enze Li
2025-12-15 14:20 ` [PATCH 2/2] mm/damon/modules: introduce prdm module for DAMON Enze Li
2025-12-15 23:16 ` SeongJae Park [this message]
2025-12-18 13:46   ` [PATCH 0/2] mm/damon: export symbols and introduce prdm module Enze Li
2025-12-19 11:46     ` SeongJae Park
2025-12-21  2:42       ` JaeJoon Jung
2025-12-21  8:12         ` SeongJae Park
2025-12-21 11:04           ` JaeJoon Jung
2025-12-21 19:27             ` SeongJae Park
2025-12-22 20:57               ` JaeJoon Jung
2025-12-22 21:33                 ` SeongJae Park
2025-12-22  7:08       ` Enze Li
2025-12-22 15:21         ` SeongJae Park

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20251215231639.56448-1-sj@kernel.org \
    --to=sj@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=damon@lists.linux.dev \
    --cc=enze.li@gmx.com \
    --cc=lienze@kylinos.cn \
    --cc=linux-mm@kvack.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox