Re: [PATCH] zram: add zpool support

From	Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] zram: add zpool support
Date	2016-06-09 03:50 +0200
Message-ID	<rHXgB-7Yw-1@gated-at.bofh.it> (permalink)
References	<rHI7U-6Pu-31@gated-at.bofh.it> <rHN7z-1ts-27@gated-at.bofh.it> <rHX6V-7Ur-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hello,

On (06/09/16 10:34), Minchan Kim wrote:
> On Wed, Jun 08, 2016 at 10:51:28AM -0400, Dan Streetman wrote:
> > On Wed, Jun 8, 2016 at 5:39 AM, Geliang Tang <geliangtang@gmail.com> wrote:
> > > This patch adds zpool support for zram, it will allow us to use both
> > > the zpool api and directly zsmalloc api in zram.
> > 
> > besides the problems below, this was discussed a while ago and I
> > believe Minchan is still against it, as nobody has so far shown what
> > the benefit to zram would be; zram doesn't need the predictability, or
> > evictability, of zbud or z3fold.
> 
> Right.
> 
> Geliang, I cannot ack without any *detail* that what's the problem of
> zram/zsmalloc, why we can't fix it in zsmalloc itself.
> The zbud and zsmalloc is otally different design to aim different goal
> determinism vs efficiency so you can choose what you want between zswap
> and zram rather than mixing the features.

I'd also probably Cc Vitaly Wool on this

(http://marc.info/?l=linux-mm&m=146537877415982&w=2)

	-ss

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] zram: add zpool support Geliang Tang <geliangtang@gmail.com> - 2016-06-08 11:40 +0200
  Re: [PATCH] zram: add zpool support Dan Streetman <ddstreet@ieee.org> - 2016-06-08 17:00 +0200
    Re: [PATCH] zram: add zpool support Minchan Kim <minchan@kernel.org> - 2016-06-09 03:40 +0200
      Re: [PATCH] zram: add zpool support Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2016-06-09 03:50 +0200

csiph-web