> -----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