linux-kernel - Re: [PATCH] z3fold: add shrinker

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

Message-ID: <CAMJBoFPWVGfwPK5gC+MJOXHQ=d4D+MM+6yBNVXEiVmCnKW_44Q@mail.gmail.com>
Date:   Tue, 11 Oct 2016 23:51:46 +0200
From:   Vitaly Wool <vitalywool@...il.com>
To:     Dave Chinner <david@...morbit.com>
Cc:     Linux-MM <linux-mm@...ck.org>, LKML <linux-kernel@...r.kernel.org>,
        Seth Jennings <sjenning@...hat.com>,
        Dan Streetman <ddstreet@...e.org>,
        Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] z3fold: add shrinker

On Tue, Oct 11, 2016 at 11:36 PM, Dave Chinner <david@...morbit.com> wrote:
> On Tue, Oct 11, 2016 at 11:14:08PM +0200, Vitaly Wool wrote:
>> This patch implements shrinker for z3fold. This shrinker
>> implementation does not free up any pages directly but it allows
>> for a denser placement of compressed objects which results in
>> less actual pages consumed and higher compression ratio therefore.
>>
>> Signed-off-by: Vitaly Wool <vitalywool@...il.com>
>
> This seems to implement the shrinker API we removed a ~3 years ago
> (commit a0b02131c5fc ("shrinker: Kill old ->shrink API.")). Forward
> porting and testing required, perhaps?

Bah, right. That's the wrong patch I submitted (for the 3.10-stable).

Thanks for pointing out, I'll come up with the right patch shortly.

~vitaly