Re: [PATCH] mm/slub: disable slab merging in the default configuration

From: Julian Pidancet
Date: Wed Jun 28 2023 - 11:06:31 EST


On Tue Jun 27, 2023 at 21:32, David Rientjes wrote:
> On Tue, 27 Jun 2023, Julian Pidancet wrote:
>
> > Make CONFIG_SLAB_MERGE_DEFAULT default to n unless CONFIG_SLUB_TINY is
> > enabled. Benefits of slab merging is limited on systems that are not
> > memory constrained: the overhead is negligible and evidence of its
> > effect on cache hotness is hard to come by.
> >
>
> I don't have an objection to this, I think it makes sense.
>
> When you say overhead here, I assume you're referring to memory footprint?
> Did you happen to have some system-wide numbers for what that looks like
> when running some benchmarks, or even what the slab usage looks like after
> a fresh boot?
>

Thank you David for the quick review. I'll re-run the benchmark and
measure slab usage when the system is under pressure.

Regards,

--
Julian

Attachment: signature.asc
Description: PGP signature