From: "Lorenzo Stoakes (ARM)" <ljs@kernel.org>
To: Michal Hocko <mhocko@suse.com>
Cc: deepakraog <gaikwad.dcg@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
David Hildenbrand <david@kernel.org>,
"Liam R. Howlett" <liam@infradead.org>,
Vlastimil Babka <vbabka@kernel.org>,
Mike Rapoport <rppt@kernel.org>,
Suren Baghdasaryan <surenb@google.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/4] mm: vmpressure: scale sampling window with machine size
Date: Wed, 22 Jul 2026 09:52:49 +0100 [thread overview]
Message-ID: <amCEULry4nsM_Uqn@lucifer> (raw)
In-Reply-To: <amB6WXIEKEeKudOh@tiehlicka>
On Wed, Jul 22, 2026 at 10:07:53AM +0200, Michal Hocko wrote:
> On Wed 15-07-26 20:06:41, deepakraog wrote:
> > The vmpressure window size was fixed at 512 pages (SWAP_CLUSTER_MAX * 16).
> > Scale it at boot from total memory and CPU count using the same fls-based
> > formula as vmstat per-zone thresholds, clamped between 128 and 2048 pages.
>
> This is not giving us any rationale why this is needed or desirable.
Agreed, the series are all somewhat corrupted here (seemingly some patches
missing + not sent to the same list of people, unrelated stuff in the same
series, etc.).
I replied to the v2 of this ([0]), in general this is essentially the same thing
submitted by a previous person ([1]), and Deepak acked that ([2]) and agreed
that this isn't worth pursuing.
Cheers, Lorenzo
[0]:https://lore.kernel.org/linux-mm/aljYxLusTzZkTfnH@lucifer/
[1]:https://lore.kernel.org/all/20260227221555.29969-1-mcq@disroot.org/
[2]:https://lore.kernel.org/all/20260716134938.42495-1-gaikwad.dcg@gmail.com/
>
> --
> Michal Hocko
> SUSE Labs
prev parent reply other threads:[~2026-07-22 8:53 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-15 14:36 [PATCH 1/4] mm: vmpressure: scale sampling window with machine size deepakraog
2026-07-15 14:36 ` [PATCH 2/4] selftests/proc: expand smaps field coverage and require TMPFS deepakraog
2026-07-15 14:36 ` [PATCH 3/4] mm: vrealloc: grow vm_area mappings in place deepakraog
2026-07-15 14:36 ` [PATCH 4/4] ceph: release unused cap reservation on readdir error deepakraog
2026-07-15 17:51 ` Viacheslav Dubeyko
2026-07-15 18:16 ` [PATCH] fix: release unused ceph " deepakroag
2026-07-16 17:24 ` Viacheslav Dubeyko
2026-07-15 18:16 ` [PATCH 4/4] " deepakroag
2026-07-15 18:26 ` deepakroag
2026-07-22 8:07 ` [PATCH 1/4] mm: vmpressure: scale sampling window with machine size Michal Hocko
2026-07-22 8:52 ` Lorenzo Stoakes (ARM) [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=amCEULry4nsM_Uqn@lucifer \
--to=ljs@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=david@kernel.org \
--cc=gaikwad.dcg@gmail.com \
--cc=liam@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.com \
--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.