On Thu, 29 Nov 2018 10:21:53 +0000 Dave Rodgman <dave.rodg...@arm.com> 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.