Re: [PATCH 7/7] lib/lzo: separate lzo-rle from lzo

From: Andrew Morton
Date: Thu Nov 29 2018 - 15:32:41 EST


On Thu, 29 Nov 2018 10:21:53 +0000 Dave Rodgman <dave.rodgman@xxxxxxx> wrote:

> >> @@ -41,7 +41,7 @@ static DEFINE_IDR(zram_index_idr);
> >> static DEFINE_MUTEX(zram_index_mutex);
> >>
> >> static int zram_major;
> >> -static const char *default_compressor = "lzo";
> >> +static const char *default_compressor = "lzo-rle";
> >
> > OK, so it's not just "separate lzo-rle", it's also "switch zram to
> > a new compression algorithm by default". I'd say that usually I'd
> > expect this to be separate patches.
>
> Yes, fair point. akpm has picked this up now though, so probably a bit
> late to break it out into a separate patch?

That's fine. akpm is flexible ;) Whatever produces the best overall
result in the permanent record, please.