linux-kernel - Re: [PATCH v4 0/4] supplement of slab allocator removal

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3f9e3cfa-c86d-4770-b531-87138803156f@suse.cz>
Date: Wed, 10 Jan 2024 16:12:26 +0100
From: Vlastimil Babka <vbabka@...e.cz>
To: sxwjean@...com, 42.hyeyoo@...il.com, cl@...ux.com, linux-mm@...ck.org
Cc: penberg@...nel.org, rientjes@...gle.com, iamjoonsoo.kim@....com,
 roman.gushchin@...ux.dev, corbet@....net, keescook@...omium.org,
 arnd@...db.de, akpm@...ux-foundation.org, gregkh@...uxfoundation.org,
 quic_jjohnson@...cinc.com, linux-doc@...r.kernel.org,
 linux-kernel@...r.kernel.org, Xiongwei Song <xiongwei.song@...driver.com>
Subject: Re: [PATCH v4 0/4] supplement of slab allocator removal

On 12/15/23 04:41, sxwjean@...com wrote:
> From: Xiongwei Song <xiongwei.song@...driver.com>
> 
> 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.

Vlastimil