* [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks @ 2026-10-28 2:36 Jinjie Ruan 2026-10-28 2:39 ` sashiko-bot 2026-10-28 5:24 ` SJ Park 0 siblings, 2 replies; 6+ messages in thread From: Jinjie Ruan @ 2026-10-28 2:36 UTC (permalink / raw) To: sj, corbet, skhan, rdunlap, damon, linux-mm, linux-doc, linux-kernel Cc: ruanjinjie DAMON_RECLAIM runs its scheme only while the free memory rate is between `wmarks_low` and `wmarks_mid`. But some users want to reclaim cold pages even while the system has plenty of free memory, and it is not obvious that the existing watermarks parameters can already do that. So Document the setup. Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com> --- Documentation/admin-guide/mm/damon/reclaim.rst | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Documentation/admin-guide/mm/damon/reclaim.rst b/Documentation/admin-guide/mm/damon/reclaim.rst index ec7e3e32b4ac..23fede7d1cdb 100644 --- a/Documentation/admin-guide/mm/damon/reclaim.rst +++ b/Documentation/admin-guide/mm/damon/reclaim.rst @@ -193,6 +193,13 @@ DAMON_RECLAIM becomes inactive, so it does nothing but periodically checks the watermarks. In the case, the system falls back to the LRU-list based page granularity reclamation logic. +To keep DAMON_RECLAIM always active regardless of the free memory rate, set +``wmarks_high``, ``wmarks_mid`` and ``wmarks_low`` to ``1000``, ``1000`` and +``0``, respectively. Then the reclamation is limited only by ``min_age`` and +the quotas. This is useful for reclaiming cold pages even while the system +has plenty of free memory, e.g., right after an idle virtual machine is +paused. + sample_interval --------------- -- 2.53.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks 2026-10-28 2:36 [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks Jinjie Ruan @ 2026-10-28 2:39 ` sashiko-bot 2026-10-28 5:24 ` SJ Park 1 sibling, 0 replies; 6+ messages in thread From: sashiko-bot @ 2026-10-28 2:39 UTC (permalink / raw) To: Jinjie Ruan, damon > DAMON_RECLAIM runs its scheme only while the free memory rate is > between `wmarks_low` and `wmarks_mid`. > > But some users want to reclaim cold pages even while the system has > plenty of free memory, and it is not obvious that the existing > watermarks parameters can already do that. So Document the setup. > > Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com> Sashiko has reviewed this patch and found no issues. -- Sashiko AI review · https://sashiko.dev/#/patchset/20261028023628.795637-1-ruanjinjie@huawei.com?part=1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks 2026-10-28 2:36 [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks Jinjie Ruan 2026-10-28 2:39 ` sashiko-bot @ 2026-10-28 5:24 ` SJ Park 2026-10-28 5:35 ` SJ Park 1 sibling, 1 reply; 6+ messages in thread From: SJ Park @ 2026-10-28 5:24 UTC (permalink / raw) To: Jinjie Ruan Cc: SJ Park, corbet, skhan, rdunlap, damon, linux-mm, linux-doc, linux-kernel Hello Jinjie, On Wed, 28 Oct 2026 10:36:28 +0800 Jinjie Ruan <ruanjinjie@huawei.com> wrote: > DAMON_RECLAIM runs its scheme only while the free memory rate is > between `wmarks_low` and `wmarks_mid`. > > But some users want to reclaim cold pages even while the system has > plenty of free memory, and it is not obvious that the existing > watermarks parameters can already do that. So Document the setup. Thank you for this patch! > > Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com> > --- > Documentation/admin-guide/mm/damon/reclaim.rst | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/admin-guide/mm/damon/reclaim.rst b/Documentation/admin-guide/mm/damon/reclaim.rst > index ec7e3e32b4ac..23fede7d1cdb 100644 > --- a/Documentation/admin-guide/mm/damon/reclaim.rst > +++ b/Documentation/admin-guide/mm/damon/reclaim.rst > @@ -193,6 +193,13 @@ DAMON_RECLAIM becomes inactive, so it does nothing but periodically checks the > watermarks. In the case, the system falls back to the LRU-list based page > granularity reclamation logic. > > +To keep DAMON_RECLAIM always active regardless of the free memory rate, set > +``wmarks_high``, ``wmarks_mid`` and ``wmarks_low`` to ``1000``, ``1000`` and > +``0``, respectively. Then the reclamation is limited only by ``min_age`` and > +the quotas. This is useful for reclaiming cold pages even while the system > +has plenty of free memory, e.g., right after an idle virtual machine is > +paused. > + I feel like above might be too detailed. Particularly, the second sentence looks too detailed. In a sense, it is not technically correct because of 'skip_anon' parameter. We might later need to add more such controls, and I concern if this kind of detailed documentation will limit such extensions. What about keeping only the first sentence and dropping later two sentences? Thanks, SJ [...] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks 2026-10-28 5:24 ` SJ Park @ 2026-10-28 5:35 ` SJ Park 2026-10-28 6:09 ` Jinjie Ruan 0 siblings, 1 reply; 6+ messages in thread From: SJ Park @ 2026-10-28 5:35 UTC (permalink / raw) To: SJ Park Cc: Jinjie Ruan, corbet, skhan, rdunlap, damon, linux-mm, linux-doc, linux-kernel On Tue, 27 Oct 2026 22:24:24 -0700 SJ Park <sj@kernel.org> wrote: > Hello Jinjie, > > On Wed, 28 Oct 2026 10:36:28 +0800 Jinjie Ruan <ruanjinjie@huawei.com> wrote: > > > DAMON_RECLAIM runs its scheme only while the free memory rate is > > between `wmarks_low` and `wmarks_mid`. > > > > But some users want to reclaim cold pages even while the system has > > plenty of free memory, and it is not obvious that the existing > > watermarks parameters can already do that. So Document the setup. > > Thank you for this patch! > > > > > Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com> > > --- > > Documentation/admin-guide/mm/damon/reclaim.rst | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/Documentation/admin-guide/mm/damon/reclaim.rst b/Documentation/admin-guide/mm/damon/reclaim.rst > > index ec7e3e32b4ac..23fede7d1cdb 100644 > > --- a/Documentation/admin-guide/mm/damon/reclaim.rst > > +++ b/Documentation/admin-guide/mm/damon/reclaim.rst > > @@ -193,6 +193,13 @@ DAMON_RECLAIM becomes inactive, so it does nothing but periodically checks the > > watermarks. In the case, the system falls back to the LRU-list based page > > granularity reclamation logic. > > > > +To keep DAMON_RECLAIM always active regardless of the free memory rate, set > > +``wmarks_high``, ``wmarks_mid`` and ``wmarks_low`` to ``1000``, ``1000`` and > > +``0``, respectively. Then the reclamation is limited only by ``min_age`` and > > +the quotas. This is useful for reclaiming cold pages even while the system > > +has plenty of free memory, e.g., right after an idle virtual machine is > > +paused. > > + > > I feel like above might be too detailed. Particularly, the second sentence > looks too detailed. In a sense, it is not technically correct because of > 'skip_anon' parameter. We might later need to add more such controls, and I > concern if this kind of detailed documentation will limit such extensions. > > What about keeping only the first sentence and dropping later two sentences? If you don't mind, I could add this patch to damon/next after dropping the last two sentences on my own. Thanks, SJ [...] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks 2026-10-28 5:35 ` SJ Park @ 2026-10-28 6:09 ` Jinjie Ruan 2026-10-28 14:08 ` SJ Park 0 siblings, 1 reply; 6+ messages in thread From: Jinjie Ruan @ 2026-10-28 6:09 UTC (permalink / raw) To: SJ Park; +Cc: corbet, skhan, rdunlap, damon, linux-mm, linux-doc, linux-kernel 在 2026/10/28 13:35, SJ Park 写道: > On Tue, 27 Oct 2026 22:24:24 -0700 SJ Park <sj@kernel.org> wrote: > >> Hello Jinjie, >> >> On Wed, 28 Oct 2026 10:36:28 +0800 Jinjie Ruan <ruanjinjie@huawei.com> wrote: >> >>> DAMON_RECLAIM runs its scheme only while the free memory rate is >>> between `wmarks_low` and `wmarks_mid`. >>> >>> But some users want to reclaim cold pages even while the system has >>> plenty of free memory, and it is not obvious that the existing >>> watermarks parameters can already do that. So Document the setup. >> >> Thank you for this patch! >> >>> >>> Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com> >>> --- >>> Documentation/admin-guide/mm/damon/reclaim.rst | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/Documentation/admin-guide/mm/damon/reclaim.rst b/Documentation/admin-guide/mm/damon/reclaim.rst >>> index ec7e3e32b4ac..23fede7d1cdb 100644 >>> --- a/Documentation/admin-guide/mm/damon/reclaim.rst >>> +++ b/Documentation/admin-guide/mm/damon/reclaim.rst >>> @@ -193,6 +193,13 @@ DAMON_RECLAIM becomes inactive, so it does nothing but periodically checks the >>> watermarks. In the case, the system falls back to the LRU-list based page >>> granularity reclamation logic. >>> >>> +To keep DAMON_RECLAIM always active regardless of the free memory rate, set >>> +``wmarks_high``, ``wmarks_mid`` and ``wmarks_low`` to ``1000``, ``1000`` and >>> +``0``, respectively. Then the reclamation is limited only by ``min_age`` and >>> +the quotas. This is useful for reclaiming cold pages even while the system >>> +has plenty of free memory, e.g., right after an idle virtual machine is >>> +paused. >>> + >> >> I feel like above might be too detailed. Particularly, the second sentence >> looks too detailed. In a sense, it is not technically correct because of >> 'skip_anon' parameter. We might later need to add more such controls, and I >> concern if this kind of detailed documentation will limit such extensions. >> >> What about keeping only the first sentence and dropping later two sentences? > > If you don't mind, I could add this patch to damon/next after dropping the last > two sentences on my own. Hi SJ, It is fine with me, thanks. Please go ahead. > > > Thanks, > SJ > > [...] -- Best regards, Jinjie ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks 2026-10-28 6:09 ` Jinjie Ruan @ 2026-10-28 14:08 ` SJ Park 0 siblings, 0 replies; 6+ messages in thread From: SJ Park @ 2026-10-28 14:08 UTC (permalink / raw) To: Jinjie Ruan Cc: SJ Park, corbet, skhan, rdunlap, damon, linux-mm, linux-doc, linux-kernel On Wed, 28 Oct 2026 14:09:39 +0800 Jinjie Ruan <ruanjinjie@huawei.com> wrote: > > > 在 2026/10/28 13:35, SJ Park 写道: > > On Tue, 27 Oct 2026 22:24:24 -0700 SJ Park <sj@kernel.org> wrote: > > > >> Hello Jinjie, > >> > >> On Wed, 28 Oct 2026 10:36:28 +0800 Jinjie Ruan <ruanjinjie@huawei.com> wrote: > >> > >>> DAMON_RECLAIM runs its scheme only while the free memory rate is > >>> between `wmarks_low` and `wmarks_mid`. > >>> > >>> But some users want to reclaim cold pages even while the system has > >>> plenty of free memory, and it is not obvious that the existing > >>> watermarks parameters can already do that. So Document the setup. > >> > >> Thank you for this patch! > >> > >>> > >>> Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com> Reviewed-by: SJ Park <sj@kernel.org> > >>> --- > >>> Documentation/admin-guide/mm/damon/reclaim.rst | 7 +++++++ > >>> 1 file changed, 7 insertions(+) > >>> > >>> diff --git a/Documentation/admin-guide/mm/damon/reclaim.rst b/Documentation/admin-guide/mm/damon/reclaim.rst > >>> index ec7e3e32b4ac..23fede7d1cdb 100644 > >>> --- a/Documentation/admin-guide/mm/damon/reclaim.rst > >>> +++ b/Documentation/admin-guide/mm/damon/reclaim.rst > >>> @@ -193,6 +193,13 @@ DAMON_RECLAIM becomes inactive, so it does nothing but periodically checks the > >>> watermarks. In the case, the system falls back to the LRU-list based page > >>> granularity reclamation logic. > >>> > >>> +To keep DAMON_RECLAIM always active regardless of the free memory rate, set > >>> +``wmarks_high``, ``wmarks_mid`` and ``wmarks_low`` to ``1000``, ``1000`` and > >>> +``0``, respectively. Then the reclamation is limited only by ``min_age`` and > >>> +the quotas. This is useful for reclaiming cold pages even while the system > >>> +has plenty of free memory, e.g., right after an idle virtual machine is > >>> +paused. > >>> + > >> > >> I feel like above might be too detailed. Particularly, the second sentence > >> looks too detailed. In a sense, it is not technically correct because of > >> 'skip_anon' parameter. We might later need to add more such controls, and I > >> concern if this kind of detailed documentation will limit such extensions. > >> > >> What about keeping only the first sentence and dropping later two sentences? > > > > If you don't mind, I could add this patch to damon/next after dropping the last > > two sentences on my own. > > Hi SJ, > > It is fine with me, thanks. Please go ahead. Thank you for accepting my suggestion. I applied this patch to damon/next [1] with my suggested change. I will repost it for mm.git merge next week. 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 ping me or Andrew. [1] https://origin.kernel.org/doc/html/latest/mm/damon/maintainer-profile.html#scm-trees Thanks, SJ [...] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-10-28 14:08 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-10-28 2:36 [PATCH] Docs/admin-guide/mm/damon/reclaim: document how to disable watermarks Jinjie Ruan 2026-10-28 2:39 ` sashiko-bot 2026-10-28 5:24 ` SJ Park 2026-10-28 5:35 ` SJ Park 2026-10-28 6:09 ` Jinjie Ruan 2026-10-28 14:08 ` SJ Park
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox