linux-kernel - Re: [PATCH 00/15] add zpdesc memory descriptor for zswap.zpool

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>] [day] [month] [year] [list]

Date: Tue, 25 Jun 2024 21:39:43 +0800
From: Alex Shi <seakeel@...il.com>
To: Hyeonggon Yoo <42.hyeyoo@...il.com>
Cc: Yosry Ahmed <yosryahmed@...gle.com>, alexs@...nel.org,
 Vitaly Wool <vitaly.wool@...sulko.com>, Miaohe Lin <linmiaohe@...wei.com>,
 Andrew Morton <akpm@...ux-foundation.org>, linux-kernel@...r.kernel.org,
 linux-mm@...ck.org, minchan@...nel.org, willy@...radead.org,
 senozhatsky@...omium.org, david@...hat.com
Subject: Re: [PATCH 00/15] add zpdesc memory descriptor for zswap.zpool



On 6/25/24 5:28 PM, Hyeonggon Yoo wrote:
> 
> 
> On Tue, Jun 25, 2024 at 5:11 PM Alex Shi <seakeel@...il.com <mailto:seakeel@...il.com>> wrote:
> 
> 
> 
>     On 6/25/24 5:46 AM, Yosry Ahmed wrote:
>     > On Thu, Jun 20, 2024 at 10:42 PM <alexs@...nel.org <mailto:alexs@...nel.org>> wrote:
>     >>
>     >> From: Alex Shi <alexs@...nel.org <mailto:alexs@...nel.org>>
>     >>
>     >> According to Metthew's plan, the page descriptor will be replace by a 8
>     >> bytes mem_desc on destination purpose.
>     >> https://lore.kernel.org/lkml/YvV1KTyzZ+Jrtj9x@casper.infradead.org/ <https://lore.kernel.org/lkml/YvV1KTyzZ+Jrtj9x@casper.infradead.org/>
>     >>
>     >> Here is a implement on z3fold to replace page descriptor by zpdesc,
>     >> which is still overlay on struct page now. but it's a step move forward
>     >> above destination.
>     >>
>     >> To name the struct zpdesc instead of z3fold_desc, since there are 3 zpool
>     >> usages under zswap, zbud, z3fold, zsmalloc. It looks like we may extend the
>     >> zpdesc to zbud and zsmalloc, combined their usage into one.
>     >
>     > For zsmalloc, there is already an ongoing effort to split zsdesc from
>     > struct page [2].
>     >
>     > [2]https://lore.kernel.org/lkml/20230713042037.980211-1-42.hyeyoo@gmail.com/ <https://lore.kernel.org/lkml/20230713042037.980211-1-42.hyeyoo@gmail.com/>
> 
>     David had pointed out this to me few weeks ago too. This patchset hasn't updated nearly a year. If Yoo don't object I'd like to pick up from his left and update it to latest zsmalloc.c.
> 
> 
> Hi Alex and Yosry,
> 
> Thank you for mentioning this! I still believe the work is worth pursuing, but recently I haven't had the capacity to push it further.
> 
> I'm on board with you taking it over and updating it with the latest source code. If you have any questions while bringing it up to date, don't hesitate please reach out to me.


Hi Yoo,

Thanks a lot for generous offer! I will update the patchset and send you for review ASAP. :)

Cheers!
Alex