* [PATCH] Docs/mm/damon: Fix spelling and grammar in monitoring_intervals_tuning_example.rst
@ 2025-02-17 21:54 Marcelo Moreira
2025-02-17 22:32 ` SeongJae Park
0 siblings, 1 reply; 2+ messages in thread
From: Marcelo Moreira @ 2025-02-17 21:54 UTC (permalink / raw)
To: linux-doc, linux-mm, damon, skhan, linux-kernel,
linux-kernel-mentees, ~lkcamp/patches
This patch fixes some spelling and grammar mistakes in the documentation,
improving the readability.
- multipled -> multiplied
- idential -> identical
- minuts -> minutes
- efficieny -> efficiency
Signed-off-by: Marcelo Moreira <marcelomoreira1905@gmail.com>
---
.../mm/damon/monitoring_intervals_tuning_example.rst | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/mm/damon/monitoring_intervals_tuning_example.rst b/Documentation/mm/damon/monitoring_intervals_tuning_example.rst
index 334a854efb40..7207cbed591f 100644
--- a/Documentation/mm/damon/monitoring_intervals_tuning_example.rst
+++ b/Documentation/mm/damon/monitoring_intervals_tuning_example.rst
@@ -36,7 +36,7 @@ Then, list the DAMON-found regions of different access patterns, sorted by the
"access temperature". "Access temperature" is a metric representing the
access-hotness of a region. It is calculated as a weighted sum of the access
frequency and the age of the region. If the access frequency is 0 %, the
-temperature is multipled by minus one. That is, if a region is not accessed,
+temperature is multiplied by minus one. That is, if a region is not accessed,
it gets minus temperature and it gets lower as not accessed for longer time.
The sorting is in temperature-ascendint order, so the region at the top of the
list is the coldest, and the one at the bottom is the hottest one. ::
@@ -58,11 +58,11 @@ list is the coldest, and the one at the bottom is the hottest one. ::
The list shows not seemingly hot regions, and only minimum access pattern
diversity. Every region has zero access frequency. The number of region is
10, which is the default ``min_nr_regions value``. Size of each region is also
-nearly idential. We can suspect this is because “adaptive regions adjustment”
+nearly identical. We can suspect this is because “adaptive regions adjustment”
mechanism was not well working. As the guide suggested, we can get relative
hotness of regions using ``age`` as the recency information. That would be
better than nothing, but given the fact that the longest age is only about 6
-seconds while we waited about ten minuts, it is unclear how useful this will
+seconds while we waited about ten minutes, it is unclear how useful this will
be.
The temperature ranges to total size of regions of each range histogram
@@ -190,7 +190,7 @@ for sampling and aggregation intervals, respectively). ::
The number of regions having different access patterns has significantly
increased. Size of each region is also more varied. Total size of non-zero
access frequency regions is also significantly increased. Maybe this is already
-good enough to make some meaningful memory management efficieny changes.
+good enough to make some meaningful memory management efficiency changes.
800ms/16s intervals: Another bias
=================================
--
2.48.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Docs/mm/damon: Fix spelling and grammar in monitoring_intervals_tuning_example.rst
2025-02-17 21:54 [PATCH] Docs/mm/damon: Fix spelling and grammar in monitoring_intervals_tuning_example.rst Marcelo Moreira
@ 2025-02-17 22:32 ` SeongJae Park
0 siblings, 0 replies; 2+ messages in thread
From: SeongJae Park @ 2025-02-17 22:32 UTC (permalink / raw)
To: Marcelo Moreira
Cc: SeongJae Park, Andrew Morton, linux-doc, linux-mm, damon, skhan,
linux-kernel, linux-kernel-mentees, ~lkcamp/patches
On Mon, 17 Feb 2025 18:54:31 -0300 Marcelo Moreira <marcelomoreira1905@gmail.com> wrote:
> This patch fixes some spelling and grammar mistakes in the documentation,
> improving the readability.
>
> - multipled -> multiplied
> - idential -> identical
> - minuts -> minutes
> - efficieny -> efficiency
Nice catch! Thank you for fixing these!
>
> Signed-off-by: Marcelo Moreira <marcelomoreira1905@gmail.com>
Reviewed-by: SeongJae Park <sj@kernel.org>
Thanks,
SJ
[...]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-02-17 22:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-02-17 21:54 [PATCH] Docs/mm/damon: Fix spelling and grammar in monitoring_intervals_tuning_example.rst Marcelo Moreira
2025-02-17 22:32 ` SeongJae Park
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox