Re: [PATCH] mm/slab: rename CONFIG_SLAB to CONFIG_SLAB_DEPRECATED

Vlastimil Babka <[email protected]>
Newsgroups gmane.linux.ports.m68k,gmane.linux.kernel.arc,gmane.linux.ports.arm.kernel,gmane.linux.ports.arm.omap,gmane.linux.kernel.renesas-soc,gmane.linux.ports.tegra,gmane.linux.ports.mips,gmane.linux.ports.parisc,gmane.linux.ports.ppc64.devel,gmane.linux.ports.sh.devel,gmane.linux.ports.sparc,gmane.linux.uml.devel,gmane.linux.kernel.mm
Message-ID <[email protected]>
On 5/23/23 11:22, Geert Uytterhoeven wrote:
> Hi Vlastimil,
> 
> Thanks for your patch!
> 
> On Tue, May 23, 2023 at 11:12 AM Vlastimil Babka <[email protected]> wrote:
>> As discussed at LSF/MM [1] [2] and with no objections raised there,
>> deprecate the SLAB allocator. Rename the user-visible option so that
>> users with CONFIG_SLAB=y get a new prompt with explanation during make
>> oldconfig, while make olddefconfig will just switch to SLUB.
>>
>> In all defconfigs with CONFIG_SLAB=y remove the line so those also
>> switch to SLUB. Regressions due to the switch should be reported to
>> linux-mm and slab maintainers.
> 
> Technically, removing these lines from the defconfig files does not
> have any impact, right?

Well, it doesn't, but I thought it's at least a useful heads-up for the
maintainers in case some have specific reasons for keeping SLAB there.

> And it removes one more sync point indicating the last time some
> defconfig files were (not) updated by their maintainers ;-)

Sure, I can exclude yours (and anyone else who'd prefer that), the ack on
the deprecation itself is sufficient.

>> [1] https://lore.kernel.org/all/[email protected]/
>> [2] https://lwn.net/Articles/932201/
>>
>> Signed-off-by: Vlastimil Babka <[email protected]>
> 
>>  arch/m68k/configs/amiga_defconfig               |  1 -
>>  arch/m68k/configs/apollo_defconfig              |  1 -
>>  arch/m68k/configs/atari_defconfig               |  1 -
>>  arch/m68k/configs/bvme6000_defconfig            |  1 -
>>  arch/m68k/configs/hp300_defconfig               |  1 -
>>  arch/m68k/configs/mac_defconfig                 |  1 -
>>  arch/m68k/configs/multi_defconfig               |  1 -
>>  arch/m68k/configs/mvme147_defconfig             |  1 -
>>  arch/m68k/configs/mvme16x_defconfig             |  1 -
>>  arch/m68k/configs/q40_defconfig                 |  1 -
>>  arch/m68k/configs/sun3_defconfig                |  1 -
>>  arch/m68k/configs/sun3x_defconfig               |  1 -
> 
> Regardless,
> Acked-by: Geert Uytterhoeven <[email protected]>

Thanks!

> Gr{oetje,eeting}s,
> 
>                         Geert
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.