From: Harry Yoo <harry@kernel.org>
To: "Vlastimil Babka (SUSE)" <vbabka@kernel.org>
Cc: Hao Li <hao.li@linux.dev>, Christoph Lameter <cl@gentwo.org>,
David Rientjes <rientjes@google.com>,
Roman Gushchin <roman.gushchin@linux.dev>,
Geert Uytterhoeven <geert@linux-m68k.org>,
Conor Dooley <conor@kernel.org>,
Damien Le Moal <damien.lemoal@opensource.wdc.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH RFC 0/8] replace CONFIG_SLUB_TINY with a slab_tiny boot parameter
Date: Thu, 24 Sep 2026 16:18:56 +0100 [thread overview]
Message-ID: <arU956kfc1QpA7N-@thinkstation> (raw)
In-Reply-To: <20260923-slub_tiny_rework-v1-0-a0e66d536eb5@kernel.org>
On Wed, Sep 23, 2026 at 05:45:05PM +0200, Vlastimil Babka (SUSE) wrote:
> To sum up, after this series:
>
> - CONFIG_SLUB_TINY is a deprecated option that has no interaction with
> other options and only controls the default behavior.
>
> - New slab_tiny boot parameter controls the functionality (and can
> override CONFIG_SLUB_TINY in both directions).
>
> - slab_debug=N can effectively achieve most of slab_tiny and on a
> per-cache basic
Allowing "slab_tiny=0" doesn't make much sense given that
CONFIG_SLUB_TINY is deprecated anyway. I was assuming specifying
"slab_tiny" would enable it and not specifying it would disable it
unless CONFIG_SLUB_TINY is set.
Also it's bit confusing to have multiple ways to achieve the same thing.
Is there a reason why we can't interpret "slab_debug=N" as
"slab_tiny_enabled = 1" internally and drop "slab_tiny" boot parameter?
> - the implementation is simpler with many ifdefs removed
>
> [1] https://lore.kernel.org/all/CAHk-=wh-k04MsoEC0SGKff2Snm6bBF_e+0pHOKwaWv4umZ_SnQ@mail.gmail.com/
>
> Signed-off-by: Vlastimil Babka (SUSE) <vbabka@kernel.org>
> ---
> Vlastimil Babka (SUSE) (8):
> mm, slab: refactor slab_min/max_order handling
> mm, slab: introduce slab_debug=N
> mm, slab: rework KMALLOC_RECLAIM handling of SLUB_TINY
> mm, slab: make SLUB_TINY handling dynamic for sizing decisions
> mm, slab: convert CONFIG_SLUB_TINY checks to kmem_cache_debug()
> mm, slab: remove remaining compile-time checks for CONFIG_SLUB_TINY
> mm, slab: add slab_tiny boot param
> mm, slab: deprecate CONFIG_SLUB_TINY and reduce its Kconfig effects
>
> Documentation/admin-guide/kernel-parameters.txt | 4 +
> Documentation/admin-guide/mm/slab.rst | 18 ++-
> include/linux/slab.h | 17 +--
> lib/Kconfig.kasan | 2 +-
> mm/Kconfig | 16 ++-
> mm/Kconfig.debug | 2 +-
> mm/slab.h | 32 +++--
> mm/slab_common.c | 12 +-
> mm/slub.c | 149 +++++++++++-------------
> 9 files changed, 120 insertions(+), 132 deletions(-)
> ---
> base-commit: b4218126d498275444f315b0ed4cef6add6beabc
> change-id: 20260911-slub_tiny_rework-8f2634dd17a7
>
--
Cheers,
Harry / Hyeonggon
prev parent reply other threads:[~2026-09-24 15:19 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-23 15:45 [PATCH RFC 0/8] replace CONFIG_SLUB_TINY with a slab_tiny boot parameter Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 1/8] mm, slab: refactor slab_min/max_order handling Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 2/8] mm, slab: introduce slab_debug=N Vlastimil Babka (SUSE)
2026-09-24 14:48 ` Harry Yoo
2026-09-24 15:39 ` Harry Yoo
2026-09-23 15:45 ` [PATCH RFC 3/8] mm, slab: rework KMALLOC_RECLAIM handling of SLUB_TINY Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 4/8] mm, slab: make SLUB_TINY handling dynamic for sizing decisions Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 5/8] mm, slab: convert CONFIG_SLUB_TINY checks to kmem_cache_debug() Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 6/8] mm, slab: remove remaining compile-time checks for CONFIG_SLUB_TINY Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 7/8] mm, slab: add slab_tiny boot param Vlastimil Babka (SUSE)
2026-09-24 7:26 ` Vlastimil Babka (SUSE)
2026-09-23 15:45 ` [PATCH RFC 8/8] mm, slab: deprecate CONFIG_SLUB_TINY and reduce its Kconfig effects Vlastimil Babka (SUSE)
2026-09-24 15:18 ` Harry Yoo [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=arU956kfc1QpA7N-@thinkstation \
--to=harry@kernel.org \
--cc=cl@gentwo.org \
--cc=conor@kernel.org \
--cc=damien.lemoal@opensource.wdc.com \
--cc=geert@linux-m68k.org \
--cc=hao.li@linux.dev \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=rientjes@google.com \
--cc=roman.gushchin@linux.dev \
--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.