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: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b371c436-1af9-41f1-811f-0425bcc468d3@suse.cz>
Date: Fri, 5 Sep 2025 09:03:30 +0200
From: Vlastimil Babka <vbabka@...e.cz>
To: Vitaly Wool <vitaly.wool@...sulko.se>, hannes@...xchg.org
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org,
 Andrew Morton <akpm@...ux-foundation.org>,
 Christoph Hellwig <hch@...radead.org>
Subject: Re: [PATCH 0/3] mm: remove zpool

On 9/4/25 16:11, Vitaly Wool wrote:
> On 9/4/25 12:13, Vlastimil Babka wrote:
>> On 9/4/25 11:33, Vitaly Wool wrote:
>>>> With zswap using zsmalloc directly, there are no more in-tree users of
>>>> this code. Remove it.
>>>>
>>>> Signed-off-by: Johannes Weiner <hannes@...xchg.org>
>>>
> 
> Things keep changing, and some of the proven solutions may not be a good 
> fit moving forward. While not suggesting that we should have a handful 
> of zpool backends just for the sake of variety, I'd like to emphasize 
> that there are good reasons to have zblock (especially the Rust one), 
> and there are good reasons to keep zsmalloc. That leads to the 
> conclusion that zpool should stay.

Johannes already suggested it's possible to do that by reimplementing
zsmalloc APIs without the runtime switch layer and choosing at config time.
It would also mean zram would be able to switch.

Your argument was "zpool API is neutral and well-defined" but we don't
really care about API/KABI/etc stability inside the kernel:
Documentation/process/stable-api-nonsense.rst
So that's not a sufficient argument.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ