Re: [PATCH v3 1/2] mm/slub: limit number of node partial slabs only in cache creation

From: Roman Gushchin
Date: Mon Mar 07 2022 - 23:48:25 EST


On Mon, Mar 07, 2022 at 07:40:55AM +0000, Hyeonggon Yoo wrote:
> SLUB sets number of minimum partial slabs for node (min_partial)
> using set_min_partial(). SLUB holds at least min_partial slabs even if
> they're empty to avoid excessive use of page allocator.
>
> set_min_partial() limits value of min_partial limits value of
> min_partial MIN_PARTIAL and MAX_PARTIAL. As set_min_partial() can be
> called by min_partial_store() too, Only limit value of min_partial
> in kmem_cache_open() so that it can be changed to value that a user wants.
>
> [ rientjes@xxxxxxxxxx: Fold set_min_partial() into its callers ]
>
> Signed-off-by: Hyeonggon Yoo <42.hyeyoo@xxxxxxxxx>
> Reviewed-by: Vlastimil Babka <vbabka@xxxxxxx>

LGTM!

Reviewed-by: Roman Gushchin <roman.gushchin@xxxxxxxxx>