mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes
@ 2026-11-02 15:28 SJ Park
  2026-11-02 15:28 ` [PATCH v2 1/3] Docs/admin-guide/mm/damon: fix various typos SJ Park
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: SJ Park @ 2026-11-02 15:28 UTC (permalink / raw)
  To: Andrew Morton
  Cc: SJ Park, Jonathan Corbet, Randy Dunlap, Shuah Khan, damon,
	linux-doc, linux-kernel, linux-mm

From: Adrian Huang (Lenovo) <adrianhuang0701@gmail.com>

Docs/admin-guide/mm/damon: minor fixes

This small patch series fixes various typos, copy-paste mistakes, and an
incorrect command syntax in the DAMON documentation files.

Changes from v1
- v1: https://lore.kernel.org/20261006061549.627-1-adrianhuang0701@gmail.com
- Collect R-b: from SJ.
- Rebase to latest mm-new.

Adrian Huang (Lenovo) (3):
  Docs/admin-guide/mm/damon: fix various typos
  Docs/admin-guide/mm/damon/lru_sort: fix copy-paste mistakes
  Docs/admin-guide/mm/damon/usage: fix incorrect option syntax in perf
    example

 Documentation/admin-guide/mm/damon/lru_sort.rst | 6 +++---
 Documentation/admin-guide/mm/damon/reclaim.rst  | 4 ++--
 Documentation/admin-guide/mm/damon/start.rst    | 6 +++---
 Documentation/admin-guide/mm/damon/stat.rst     | 2 +-
 Documentation/admin-guide/mm/damon/usage.rst    | 2 +-
 5 files changed, 10 insertions(+), 10 deletions(-)


base-commit: 7fa9b9f1350aa38c32315a6de022c770e78dbfae
-- 
2.47.3

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2 1/3] Docs/admin-guide/mm/damon: fix various typos
  2026-11-02 15:28 [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
@ 2026-11-02 15:28 ` SJ Park
  2026-11-02 15:28 ` [PATCH v2 2/3] Docs/admin-guide/mm/damon/lru_sort: fix copy-paste mistakes SJ Park
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: SJ Park @ 2026-11-02 15:28 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Adrian Huang (Lenovo),
	Jonathan Corbet, Randy Dunlap, SJ Park, Shuah Khan, damon,
	linux-doc, linux-kernel, linux-mm

From: "Adrian Huang (Lenovo)" <adrianhuang0701@gmail.com>

Fix several spelling mistakes found while reading the DAMON
documentation:
  * trusworthy -> trustworthy
  * DAMON_RCLAIM -> DAMON_RECLAIM
  * charget -> charge
  * hetmap -> heatmap
  * colun -> column
  * evelenth -> eleventh
  * idlee -> idle

Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Shuah Khan <skhan@linuxfoundation.org>
Cc: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Adrian Huang (Lenovo) <adrianhuang0701@gmail.com>
Reviewed-by: SJ Park <sj@kernel.org>
Signed-off-by: SJ Park <sj@kernel.org>
---
 Documentation/admin-guide/mm/damon/lru_sort.rst | 2 +-
 Documentation/admin-guide/mm/damon/reclaim.rst  | 4 ++--
 Documentation/admin-guide/mm/damon/start.rst    | 6 +++---
 Documentation/admin-guide/mm/damon/stat.rst     | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/Documentation/admin-guide/mm/damon/lru_sort.rst b/Documentation/admin-guide/mm/damon/lru_sort.rst
index f1eeb6389c377..02ea694426b2c 100644
--- a/Documentation/admin-guide/mm/damon/lru_sort.rst
+++ b/Documentation/admin-guide/mm/damon/lru_sort.rst
@@ -7,7 +7,7 @@ DAMON-based LRU-lists Sorting
 DAMON-based LRU-lists Sorting (DAMON_LRU_SORT) is a static kernel module that
 aims to be used for proactive and lightweight data access pattern based
 (de)prioritization of pages on their LRU-lists for making LRU-lists a more
-trusworthy data access pattern source.
+trustworthy data access pattern source.
 
 Where Proactive LRU-lists Sorting is Required?
 ==============================================
diff --git a/Documentation/admin-guide/mm/damon/reclaim.rst b/Documentation/admin-guide/mm/damon/reclaim.rst
index 14f9b82b38ed9..028947f5da8ec 100644
--- a/Documentation/admin-guide/mm/damon/reclaim.rst
+++ b/Documentation/admin-guide/mm/damon/reclaim.rst
@@ -55,7 +55,7 @@ enabled
 
 Enable or disable DAMON_RECLAIM.
 
-You can enable DAMON_RCLAIM by setting the value of this parameter as ``Y``.
+You can enable DAMON_RECLAIM by setting the value of this parameter as ``Y``.
 Setting it as ``N`` disables DAMON_RECLAIM.  Note that DAMON_RECLAIM could do
 no real monitoring and reclamation due to the watermarks-based activation
 condition.  Refer to below descriptions for the watermarks parameter for this.
@@ -125,7 +125,7 @@ quota_reset_interval_ms
 
 The time/size quota charge reset interval in milliseconds.
 
-The charget reset interval for the quota of time (quota_ms) and size
+The charge reset interval for the quota of time (quota_ms) and size
 (quota_sz).  That is, DAMON_RECLAIM does not try reclamation for more than
 quota_ms milliseconds or quota_sz bytes within quota_reset_interval_ms
 milliseconds.
diff --git a/Documentation/admin-guide/mm/damon/start.rst b/Documentation/admin-guide/mm/damon/start.rst
index ec8c34b2d32f3..d98b0eb8e7713 100644
--- a/Documentation/admin-guide/mm/damon/start.rst
+++ b/Documentation/admin-guide/mm/damon/start.rst
@@ -69,16 +69,16 @@ monitored access pattern of the process from DAMON and shows the pattern in a
 human readable format.
 
 The first line of the output shows the relative access temperature (hotness) of
-the regions in a single row hetmap format.  Each column on the heatmap
+the regions in a single row heatmap format.  Each column on the heatmap
 represents regions of same size on the monitored virtual address space.  The
-position of the colun on the row and the number on the column represents the
+position of the column on the row and the number on the column represents the
 relative location and access temperature of the region.  ``[...]`` means
 unmapped huge regions on the virtual address spaces.  The second line shows
 additional information for better understanding the heatmap.
 
 Each line of the output from the third line shows which virtual address range
 (``addr XX size XX``) of the process is how frequently (``access XX %``)
-accessed for how long time (``age XX``).  For example, the evelenth region of
+accessed for how long time (``age XX``).  For example, the eleventh region of
 ~9.5 MiB size is being most frequently accessed for last 3.7 seconds.  Finally,
 the fourth command stops DAMON.
 
diff --git a/Documentation/admin-guide/mm/damon/stat.rst b/Documentation/admin-guide/mm/damon/stat.rst
index 46c5dd96aa2ed..4a9db755a5265 100644
--- a/Documentation/admin-guide/mm/damon/stat.rst
+++ b/Documentation/admin-guide/mm/damon/stat.rst
@@ -81,7 +81,7 @@ Per-byte idle time (milliseconds) percentiles of the system.
 DAMON_STAT calculates how long each byte of the memory was not accessed until
 now (idle time), based on the current DAMON results snapshot.  For regions
 having access frequency (nr_accesses) larger than zero, how long the current
-access frequency level was kept multiplied by ``-1`` becomes the idlee time of
+access frequency level was kept multiplied by ``-1`` becomes the idle time of
 every byte of the region.  If a region has zero access frequency (nr_accesses),
 how long the region was keeping the zero access frequency (age) becomes the
 idle time of every byte of the region.  Then, DAMON_STAT exposes the
-- 
2.47.3

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2 2/3] Docs/admin-guide/mm/damon/lru_sort: fix copy-paste mistakes
  2026-11-02 15:28 [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
  2026-11-02 15:28 ` [PATCH v2 1/3] Docs/admin-guide/mm/damon: fix various typos SJ Park
@ 2026-11-02 15:28 ` SJ Park
  2026-11-02 15:28 ` [PATCH v2 3/3] Docs/admin-guide/mm/damon/usage: fix incorrect option syntax in perf example SJ Park
  2026-11-02 15:49 ` [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
  3 siblings, 0 replies; 5+ messages in thread
From: SJ Park @ 2026-11-02 15:28 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Adrian Huang (Lenovo),
	Jonathan Corbet, Randy Dunlap, SJ Park, Shuah Khan, damon,
	linux-doc, linux-kernel, linux-mm

From: "Adrian Huang (Lenovo)" <adrianhuang0701@gmail.com>

The DAMON_LRU_SORT documentation mistakenly refers to DAMON_RECLAIM
in a few places.

Fix these copy-paste mistakes to correctly refer to DAMON_LRU_SORT.

Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Shuah Khan <skhan@linuxfoundation.org>
Cc: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Adrian Huang (Lenovo) <adrianhuang0701@gmail.com>
Reviewed-by: SJ Park <sj@kernel.org>
Signed-off-by: SJ Park <sj@kernel.org>
---
 Documentation/admin-guide/mm/damon/lru_sort.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/admin-guide/mm/damon/lru_sort.rst b/Documentation/admin-guide/mm/damon/lru_sort.rst
index 02ea694426b2c..aa4cc90ca57c6 100644
--- a/Documentation/admin-guide/mm/damon/lru_sort.rst
+++ b/Documentation/admin-guide/mm/damon/lru_sort.rst
@@ -263,7 +263,7 @@ addr_unit
 A scale factor for memory addresses and bytes.
 
 This parameter is for setting and getting the :ref:`address unit
-<damon_design_addr_unit>` parameter of the DAMON instance for DAMON_RECLAIM.
+<damon_design_addr_unit>` parameter of the DAMON instance for DAMON_LRU_SORT.
 
 ``monitor_region_start`` and ``monitor_region_end`` should be provided in this
 unit.  For example, let's suppose ``addr_unit``, ``monitor_region_start`` and
@@ -346,7 +346,7 @@ memory regions that not accessed for 120 seconds.  The prioritization and
 deprioritization is limited to be done using only up to 1% CPU time to avoid
 DAMON_LRU_SORT consuming too much CPU time for the (de)prioritization.  It also
 asks DAMON_LRU_SORT to do nothing if the system's free memory rate is more than
-50%, but start the real works if it becomes lower than 40%.  If DAMON_RECLAIM
+50%, but start the real works if it becomes lower than 40%.  If DAMON_LRU_SORT
 doesn't make progress and therefore the free memory rate becomes lower than
 20%, it asks DAMON_LRU_SORT to do nothing again, so that we can fall back to
 the LRU-list based page granularity reclamation. ::
-- 
2.47.3

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2 3/3] Docs/admin-guide/mm/damon/usage: fix incorrect option syntax in perf example
  2026-11-02 15:28 [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
  2026-11-02 15:28 ` [PATCH v2 1/3] Docs/admin-guide/mm/damon: fix various typos SJ Park
  2026-11-02 15:28 ` [PATCH v2 2/3] Docs/admin-guide/mm/damon/lru_sort: fix copy-paste mistakes SJ Park
@ 2026-11-02 15:28 ` SJ Park
  2026-11-02 15:49 ` [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
  3 siblings, 0 replies; 5+ messages in thread
From: SJ Park @ 2026-11-02 15:28 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Adrian Huang (Lenovo),
	Jonathan Corbet, Randy Dunlap, SJ Park, Shuah Khan, damon,
	linux-doc, linux-kernel, linux-mm

From: "Adrian Huang (Lenovo)" <adrianhuang0701@gmail.com>

The kill command in the DAMON documentation uses an incorrect
option syntax in the perf example.

Use kill -9 instead of kill 9 to correctly send SIGKILL to
the perf process.

Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Shuah Khan <skhan@linuxfoundation.org>
Cc: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Adrian Huang (Lenovo) <adrianhuang0701@gmail.com>
Reviewed-by: SJ Park <sj@kernel.org>
Signed-off-by: SJ Park <sj@kernel.org>
---
 Documentation/admin-guide/mm/damon/usage.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/admin-guide/mm/damon/usage.rst b/Documentation/admin-guide/mm/damon/usage.rst
index 3d43954c7046c..bd687d89ea4c0 100644
--- a/Documentation/admin-guide/mm/damon/usage.rst
+++ b/Documentation/admin-guide/mm/damon/usage.rst
@@ -749,7 +749,7 @@ show results using tracepoint supporting tools like ``perf``.  For example::
     # echo on > kdamonds/0/state
     # perf record -e damon:damon_aggregated &
     # sleep 5
-    # kill 9 $(pidof perf)
+    # kill -9 $(pidof perf)
     # echo off > kdamonds/0/state
     # perf script
     kdamond.0 46568 [027] 79357.842179: damon:damon_aggregated: target_id=0 nr_regions=11 122509119488-135708762112: 0 864
-- 
2.47.3

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes
  2026-11-02 15:28 [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
                   ` (2 preceding siblings ...)
  2026-11-02 15:28 ` [PATCH v2 3/3] Docs/admin-guide/mm/damon/usage: fix incorrect option syntax in perf example SJ Park
@ 2026-11-02 15:49 ` SJ Park
  3 siblings, 0 replies; 5+ messages in thread
From: SJ Park @ 2026-11-02 15:49 UTC (permalink / raw)
  To: SJ Park
  Cc: Andrew Morton, Jonathan Corbet, Randy Dunlap, Shuah Khan, damon,
	linux-doc, linux-kernel, linux-mm

Hi Andrew,

On Mon,  2 Nov 2026 07:28:44 -0800 SJ Park <sj@kernel.org> wrote:

> From: Adrian Huang (Lenovo) <adrianhuang0701@gmail.com>
> 
> Docs/admin-guide/mm/damon: minor fixes

I just found applying this series on mm-new fails, because this series depends
on another series [1].  Pleae apply after the other series, or let me know if
you prefer rebase of this series.  Sorry for inconvenience.

[1] https://lore.kernel.org/20261102150757.93058-1-sj@kernel.org


Thanks,
SJ

[...]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2026-11-02 15:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-02 15:28 [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park
2026-11-02 15:28 ` [PATCH v2 1/3] Docs/admin-guide/mm/damon: fix various typos SJ Park
2026-11-02 15:28 ` [PATCH v2 2/3] Docs/admin-guide/mm/damon/lru_sort: fix copy-paste mistakes SJ Park
2026-11-02 15:28 ` [PATCH v2 3/3] Docs/admin-guide/mm/damon/usage: fix incorrect option syntax in perf example SJ Park
2026-11-02 15:49 ` [PATCH v2 0/3] Docs/admin-guide/mm/damon: minor fixes SJ Park

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®