From: SJ Park <sj@kernel.org>
To: Zhijian Han <hanzhijian1991@gmail.com>
Cc: SJ Park <sj@kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
David Hildenbrand <david@kernel.org>,
Lorenzo Stoakes <ljs@kernel.org>,
"Liam R. Howlett" <liam@infradead.org>,
Vlastimil Babka <vbabka@kernel.org>,
Mike Rapoport <rppt@kernel.org>,
Suren Baghdasaryan <surenb@google.com>,
Michal Hocko <mhocko@suse.com>,
Muchun Song <muchun.song@linux.dev>,
Oscar Salvador <osalvador@suse.de>,
Alexander Potapenko <glider@google.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mm: fix typos in various comments
Date: Tue, 22 Sep 2026 05:31:49 -0700 [thread overview]
Message-ID: <20260922123150.2763-1-sj@kernel.org> (raw)
In-Reply-To: <20260922031843.2857104-1-hanzhijian1991@gmail.com>
On Tue, 22 Sep 2026 11:18:43 +0800 Zhijian Han <hanzhijian1991@gmail.com> wrote:
> Fix spelling errors found by codespell in several mm source files:
[...]
> --- a/mm/damon/core.c
> +++ b/mm/damon/core.c
> @@ -2328,7 +2328,7 @@ static bool damos_valid_target(struct damon_ctx *c, struct damon_region *r,
> * This function checks if a given region should be skipped or not for the
> * reason. If only the starting part of the region has previously charged,
> * this function splits the region into two so that the second one covers the
> - * area that not charged in the previous charge widnow, and return true. The
> + * area that not charged in the previous charge window, and return true. The
> * caller can see the second one on the next iteration of the region walk.
> * Note that this means the caller should use damon_for_each_region() instead
> * of damon_for_each_region_safe(). If damon_for_each_region_safe() is used,
> @@ -2486,7 +2486,7 @@ static void damos_walk_call_walk(struct damon_ctx *ctx, struct damon_target *t,
> * This function is called when kdamond finished applying the action of a DAMOS
> * scheme to all regions that eligible for the given &damos->apply_interval_us.
> * If every scheme of @ctx including @s now finished walking for at least one
> - * &damos->apply_interval_us, this function makrs the handling of the given
> + * &damos->apply_interval_us, this function marks the handling of the given
> * DAMOS walk request is done, so that damos_walk() can wake up and return.
> */
> static void damos_walk_complete(struct damon_ctx *ctx, struct damos *s)
As I replied [1] to your another patch, the above DAMON side change cannot be
accepted since the same fix is already suggested and accepted.
[1] https://lore.kernel.org/20260922064216.43053-1-sj@kernel.org
Thanks,
SJ
[...]
prev parent reply other threads:[~2026-09-22 12:32 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-22 3:18 [PATCH] mm: fix typos in various comments Zhijian Han
2026-09-22 11:43 ` David Hildenbrand (Arm)
2026-09-22 12:31 ` SJ Park [this message]
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=20260922123150.2763-1-sj@kernel.org \
--to=sj@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=david@kernel.org \
--cc=glider@google.com \
--cc=hanzhijian1991@gmail.com \
--cc=liam@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=ljs@kernel.org \
--cc=mhocko@suse.com \
--cc=muchun.song@linux.dev \
--cc=osalvador@suse.de \
--cc=rppt@kernel.org \
--cc=surenb@google.com \
--cc=vbabka@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.