Re: [PATCH 2/2] zram: remove zlib from the list of recommended algorithms

From: Sergey Senozhatsky
Date: Tue Sep 12 2017 - 03:01:11 EST


On (09/12/17 15:49), Minchan Kim wrote:
> On Tue, Sep 12, 2017 at 02:00:05PM +0900, Sergey Senozhatsky wrote:
> > ZSTD tends to outperform deflate/inflate, thus we remove
> > zlib from the list of recommended algorithms and recommend
> > zstd instead.
> >
> > Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@xxxxxxxxx>
> > Suggested-by: Minchan Kim <minchan@xxxxxxxxxx>
>
>
> One of my colleague have tested zstd compared zlib with a some
> workload 4K data and he said it was not much better compared to
> zlib so please give me a time to test it by myself.
> I will test it with my sample data which dumped from my desktop.

ok, no prob.

this patch is not mandatory, and I don't see any problems if we
will keep zlib in the list.

-ss