Re: [PATCH v3 7/9] zram: pass zstrm down to decompression path

2015-09-20 Thread Minchan Kim
com> > Signed-off-by: Joonsoo Kim <iamjoonsoo@lge.com> Acked-by: Minchan Kim <minc...@kernel.org> There is a trivial comment below. > --- > drivers/block/zram/zcomp.c| 3 ++- > drivers/block/zram/zcomp.h| 3 ++- > drivers/block/zram/zram_drv.c | 20

Re: [PATCH v3 6/9] zram: make stream find and release functions static

2015-09-20 Thread Minchan Kim
tion to make crypto API-powered zcomp > possible. The reasoning is that some crypto compression backends > require zstrm for decompression. > > Signed-off-by: Sergey Senozhatsky <sergey.senozhat...@gmail.com> > Signed-off-by: Joonsoo Kim <iamjoonsoo@lge.com> Acked-by: Minchan

Re: [PATCH v3 8/9] zram: use crypto API for compression

2015-09-20 Thread Minchan Kim
Hi Joonsoo, On Fri, Sep 18, 2015 at 02:19:23PM +0900, Joonsoo Kim wrote: > Until now, zram uses compression algorithm through direct call > to core algorithm function, but, it has drawback that we need to add > compression algorithm manually to zram if needed. Without this work, > we cannot

Re: [PATCH v3 9/9] zram: use crypto decompress_noctx API

2015-09-20 Thread Minchan Kim
On Fri, Sep 18, 2015 at 02:19:24PM +0900, Joonsoo Kim wrote: > Crypto subsystem now supports decompress_noctx API that requires > special tfm_noctx. This tfm can be shared by multiple concurrent > decompress user because this API doesn't rely on this tfm object > except to fetch decompress

Re: [PATCH v3 0/9] zram: introduce crypto decompress noctx API and use it on zram

2015-09-20 Thread Minchan Kim
On Fri, Sep 18, 2015 at 02:19:15PM +0900, Joonsoo Kim wrote: > This patchset makes zram to use crypto API in order to support > more compression algorithm. > > The reason we need to support vairous compression algorithms is that > zram's performance is highly depend on workload and compression

Re: [PATCH v7 0/5] Update LZ4 compressor module

2017-02-08 Thread Minchan Kim
Hello Sven, On Sun, Feb 05, 2017 at 08:09:03PM +0100, Sven Schmidt wrote: > > This patchset is for updating the LZ4 compression module to a version based > on LZ4 v1.7.3 allowing to use the fast compression algorithm aka LZ4 fast > which provides an "acceleration" parameter as a tradeoff between

Re: [PATCH v7 0/5] Update LZ4 compressor module

2017-02-09 Thread Minchan Kim
Hello Sven, On Thu, Feb 09, 2017 at 11:56:17AM +0100, Sven Schmidt wrote: > Hey Minchan, > > On Thu, Feb 09, 2017 at 08:31:21AM +0900, Minchan Kim wrote: > > Hello Sven, > > > > On Sun, Feb 05, 2017 at 08:09:03PM +0100, Sven Schmidt wrote: > > > > &g

Re: [PATCH v7 0/5] Update LZ4 compressor module

2017-02-14 Thread Minchan Kim
Hi Sven, On Mon, Feb 13, 2017 at 01:08:41PM +0100, Sven Schmidt wrote: > On Mon, Feb 13, 2017 at 09:03:24AM +0900, Minchan Kim wrote: > > Hi Sven, > > > > On Sun, Feb 12, 2017 at 12:16:17PM +0100, Sven Schmidt wrote: > > > > > > > > >

Re: [PATCH v7 0/5] Update LZ4 compressor module

2017-02-12 Thread Minchan Kim
Hi Sven, On Sun, Feb 12, 2017 at 12:16:17PM +0100, Sven Schmidt wrote: > > > > On 02/10/2017 01:13 AM, Minchan Kim wrote: > > Hello Sven, > > > > On Thu, Feb 09, 2017 at 11:56:17AM +0100, Sven Schmidt wrote: > >> Hey Minchan, > >> > >&

Re: [PATCH v7 0/5] add compression algorithm zBeWalgo

2018-04-20 Thread Minchan Kim
Hi Benjamin, Today I tried your new patchset but I couldn't go further due to below problem. Unfortunately, I don't have the time to look into. Could you check on it? Thanks. [ 169.597064] zram0: detected capacity change from 1073741824 to 0 [ 177.523268] zram0: detected capacity change from

Re: [PATCH v6 0/5] add compression algorithm zBeWalgo

2018-03-26 Thread Minchan Kim
Hi Benjamin, Thanks for the nice present and good testing! I hope to grab a chance to test this shiny new algorithm but is busy this week. Hopefully, I will get that soon and feedback to you asap. Thanks. On Mon, Mar 26, 2018 at 10:31:40AM +0200, Benjamin Warnke wrote: > This patch series adds