RE: [PATCH v4 0/4] supplement of slab allocator removal

From: Song, Xiongwei
Date: Wed Jan 10 2024 - 20:33:40 EST




> -----Original Message-----
> From: Vlastimil Babka <vbabka@xxxxxxx>
> Sent: Wednesday, January 10, 2024 11:12 PM
> To: sxwjean@xxxxxx; 42.hyeyoo@xxxxxxxxx; cl@xxxxxxxxx; linux-mm@xxxxxxxxx
> Cc: penberg@xxxxxxxxxx; rientjes@xxxxxxxxxx; iamjoonsoo.kim@xxxxxxx;
> roman.gushchin@xxxxxxxxx; corbet@xxxxxxx; keescook@xxxxxxxxxxxx; arnd@xxxxxxxx;
> akpm@xxxxxxxxxxxxxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; quic_jjohnson@xxxxxxxxxxx;
> linux-doc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Song, Xiongwei
> <Xiongwei.Song@xxxxxxxxxxxxx>
> Subject: Re: [PATCH v4 0/4] supplement of slab allocator removal
>
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content
> is safe.
>
> On 12/15/23 04:41, sxwjean@xxxxxx wrote:
> > From: Xiongwei Song <xiongwei.song@xxxxxxxxxxxxx>
> >
> > Hi,
> >
> > Patch 1 is to remove an unused parameter. It has a longer history, please
> > see the change history inside the patch.
> >
> > ---
> > Patch 2 is to replace slub_$params with slab_$params.
> > Vlastimil Babka pointed out we should use "slab_$param" as the primary
> > prefix for long-term plan. Please see [1] for more information.
> >
> > I did the basic tests with qemu, which passed values by sl[au]b_max_order,
> > sl[au]b_min_order, sl[au]b_min_objects and sl[au]b_debug in command line.
> > The values looks correct by printing them out before calculating orders.
> >
> > ---
> > Patch 3 is to replace slub_$params in Documentation/mm/slub.rst based on
> > the changes of patch 2.
> >
> > ---
> > Patch 4 is original patch 3. It is not related to slab allocator removal.
> > It's to correct the description of default value of slab_min_objects in
> > Documentation/mm/slub.rst.
> >
> > ---
> > This series is based on [2].
>
> I will queue this for 6.9 after the merge window. But since the changes are
> mostly in Documentation, maybe Jon would prefer his tree, then we would
> split away the code comments changes in patch 2 to go through slab tree.

Ok, got it.

Thanks,
Xiongwei