slab: switch away from the legacy param parser

[PATCH 0/3] slab: switch away from the legacy param parser

Posted by Petr Tesarik 3 months, 2 weeks ago

The handling of legacy __setup() parameters has some confusing quirks.
Instead of fixing them, convert the code to struct kerenel_param, which
is a saner API.

Note that parameters defined with core_param() and __core_param_cb() are
parsed early in start_kernel(). Do not confuse them with core_param_cb(),
which are processed at a later stage.

Petr Tesarik (3):
  slab: constify slab debug strings
  slab: convert setup_slub_debug() to use __core_param_cb()
  slab: use core_param for remaining command line parameters

 mm/slub.c | 87 ++++++++++++++++++++++++++++++++-----------------------
 1 file changed, 51 insertions(+), 36 deletions(-)

-- 
2.51.0

Re: [PATCH 0/3] slab: switch away from the legacy param parser

Posted by Vlastimil Babka 3 months, 1 week ago

On 10/24/25 19:06, Petr Tesarik wrote:
> The handling of legacy __setup() parameters has some confusing quirks.
> Instead of fixing them, convert the code to struct kerenel_param, which
> is a saner API.
> 
> Note that parameters defined with core_param() and __core_param_cb() are
> parsed early in start_kernel(). Do not confuse them with core_param_cb(),
> which are processed at a later stage.

Thanks! Added to slab/for-next

> Petr Tesarik (3):
>   slab: constify slab debug strings
>   slab: convert setup_slub_debug() to use __core_param_cb()
>   slab: use core_param for remaining command line parameters
> 
>  mm/slub.c | 87 ++++++++++++++++++++++++++++++++-----------------------
>  1 file changed, 51 insertions(+), 36 deletions(-)
>

Re: [PATCH 0/3] slab: switch away from the legacy param parser

Posted by Petr Tesarik 3 months, 1 week ago

On Mon, 27 Oct 2025 16:06:21 +0100
Vlastimil Babka <vbabka@suse.cz> wrote:

> On 10/24/25 19:06, Petr Tesarik wrote:
> > The handling of legacy __setup() parameters has some confusing quirks.
> > Instead of fixing them, convert the code to struct kerenel_param, which
> > is a saner API.
> > 
> > Note that parameters defined with core_param() and __core_param_cb() are
> > parsed early in start_kernel(). Do not confuse them with core_param_cb(),
> > which are processed at a later stage.  
> 
> Thanks! Added to slab/for-next

Thank you! Appreciated.

Petr T