Received-SPF: pass (google.com: domain of linux-kernel+bounces-213395-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: SeongJae Park <sj@kernel.org>
To: Honggyu Kim <honggyu.kim@sk.com>
Cc: SeongJae Park <sj@kernel.org>,
	damon@lists.linux.dev,
	Andrew Morton <akpm@linux-foundation.org>,
	Masami Hiramatsu <mhiramat@kernel.org>,
	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	Gregory Price <gregory.price@memverge.com>,
	linux-mm@kvack.org,
	linux-kernel@vger.kernel.org,
	linux-trace-kernel@vger.kernel.org,
	42.hyeyoo@gmail.com,
	art.jeongseob@gmail.com,
	kernel_team@skhynix.com
Subject: Re: [PATCH v5 7/8] Docs/admin-guide/mm/damon/usage: add missing actions
Date: Thu, 13 Jun 2024 07:03:19 -0700
Message-Id: <20240613140319.27075-1-sj@kernel.org>
In-Reply-To: <20240613132056.608-8-honggyu.kim@sk.com>
Precedence: bulk

Hi Honggyu,

On Thu, 13 Jun 2024 22:20:54 +0900 Honggyu Kim <honggyu.kim@sk.com> wrote:

> "lru_prio" and "lru_deprio" are missing in the damos action list so they
> have to be added properly at damon usage document.
> 
> Signed-off-by: Honggyu Kim <honggyu.kim@sk.com>
> ---
>  Documentation/admin-guide/mm/damon/usage.rst | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/admin-guide/mm/damon/usage.rst b/Documentation/admin-guide/mm/damon/usage.rst
> index e58ceb89ea2a..7bff54963975 100644
> --- a/Documentation/admin-guide/mm/damon/usage.rst
> +++ b/Documentation/admin-guide/mm/damon/usage.rst
> @@ -757,7 +757,9 @@ list on :ref:`design doc <damon_design_damos_action>`.
>   - 2: ``pageout``
>   - 3: ``hugepage``
>   - 4: ``nohugepage``
> - - 5: ``stat``
> + - 5: ``lru_prio``
> + - 6: ``lru_deprio``
> + - 7: ``stat``

This section is for DAMON debugfs interface, which is deprecated.  After the
deprecation, we stopped supporting new DAMOS actions.  This section is not
documenting lru_[de]prio for the reason.  I think this patch is not needed.

Please let me know if I'm missing something, though.


Thanks,
SJ

[...]