From: "Jiayuan Chen" <jiayuan.chen@linux.dev>
To: "Michal Hocko" <mhocko@suse.com>
Cc: "Shakeel Butt" <shakeel.butt@linux.dev>,
linux-mm@kvack.org, "Jiayuan Chen" <jiayuan.chen@shopee.com>,
"Andrew Morton" <akpm@linux-foundation.org>,
"Johannes Weiner" <hannes@cmpxchg.org>,
"David Hildenbrand" <david@kernel.org>,
"Qi Zheng" <zhengqi.arch@bytedance.com>,
"Lorenzo Stoakes" <lorenzo.stoakes@oracle.com>,
"Axel Rasmussen" <axelrasmussen@google.com>,
"Yuanchu Xie" <yuanchu@google.com>, "Wei Xu" <weixugc@google.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1] mm/vmscan: mitigate spurious kswapd_failures reset from direct reclaim
Date: Tue, 06 Jan 2026 11:19:21 +0000 [thread overview]
Message-ID: <52cc0b2671b068903c6580b7431db0f22982ae86@linux.dev> (raw)
In-Reply-To: <aVzaqAZ5Wagh5cDQ@tiehlicka>
January 6, 2026 at 17:49, "Michal Hocko" <mhocko@suse.com mailto:mhocko@suse.com?to=%22Michal%20Hocko%22%20%3Cmhocko%40suse.com%3E > wrote:
>
> On Tue 06-01-26 05:25:42, Jiayuan Chen wrote:
>
> >
> > That said, I believe this patch is still a valid fix on its own - resetting kswapd_failures
> > when the node is not actually balanced doesn't seem like correct behavior regardless of the
> > broader context.
> >
> Originally I was more inclined to opt out memcg reclaim from reseting
> kswapd retry counter but the more I am thiking about that the more your
> patch makes sense to me.
>
> The reason being that it handles both memcg and global direct reclaims
> in the same way which makes the logic easier to follow. Afterall the
> primary purpose is to resurrect kswapd after we can see there is a
> better chance to reclaim something for kswapd. Until that moment direct
> reclaim is the only reclaim mechanism.
>
> Relying on pgdat_balanced might lead to re-enabling kswapd way much
> later while memory reclaim would be still mostly direct reclaim bound -
> thus increase allocation latencies.
> If we wanted to do better we would need to evaluate recent
> refaults/thrashing behavior but even then I am not sure we can make a
> good cut off.
>
> So in the end pgdat_balanced approach seems worth trying and see whether
> this could cause any corner cases.
Thanks Michal.
Regarding the allocation latency concern - we are already
in the direct reclaim slowpath, so a little extra overhead
from the pgdat_balanced check should be negligible.
> --
> Michal Hocko
> SUSE Labs
>
next prev parent reply other threads:[~2026-01-06 11:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-22 12:20 [PATCH v1] mm/vmscan: mitigate spurious kswapd_failures reset from direct reclaim Jiayuan Chen
2025-12-22 18:29 ` Andrew Morton
2025-12-23 1:51 ` Jiayuan Chen
2025-12-22 21:15 ` Shakeel Butt
2025-12-23 1:42 ` Jiayuan Chen
2025-12-23 6:11 ` Shakeel Butt
2025-12-23 8:22 ` Jiayuan Chen
2026-01-05 4:51 ` Shakeel Butt
2026-01-06 5:25 ` Jiayuan Chen
2026-01-06 9:49 ` Michal Hocko
2026-01-06 11:19 ` Jiayuan Chen [this message]
2026-01-06 12:59 ` Michal Hocko
2026-01-06 16:50 ` Shakeel Butt
2026-01-06 19:14 ` Michal Hocko
2026-01-06 17:45 ` Shakeel Butt
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=52cc0b2671b068903c6580b7431db0f22982ae86@linux.dev \
--to=jiayuan.chen@linux.dev \
--cc=akpm@linux-foundation.org \
--cc=axelrasmussen@google.com \
--cc=david@kernel.org \
--cc=hannes@cmpxchg.org \
--cc=jiayuan.chen@shopee.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lorenzo.stoakes@oracle.com \
--cc=mhocko@suse.com \
--cc=shakeel.butt@linux.dev \
--cc=weixugc@google.com \
--cc=yuanchu@google.com \
--cc=zhengqi.arch@bytedance.com \
/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.