Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Minchan Kim
On Tue, Feb 18, 2014 at 10:38:18PM +0300, Sergey Senozhatsky wrote: > On (02/18/14 15:04), Minchan Kim wrote: > [..] > > 1.8.5.3 > > > > As you can see, your patch made zram too much regressed when it > > used one stream buffer. The reason I guessed is overhead of > > scheduling by sleep/wakeup

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Minchan Kim
Hello Sergey, On Tue, Feb 18, 2014 at 12:20:17PM +0300, Sergey Senozhatsky wrote: > Hello, > > On (02/18/14 15:04), Minchan Kim wrote: > > Hello Sergey, > > > > On Thu, Feb 13, 2014 at 08:43:18PM +0300, Sergey Senozhatsky wrote: > > > This patchset introduces zcomp compression backend

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Sergey Senozhatsky
On (02/18/14 15:04), Minchan Kim wrote: [..] > 1.8.5.3 > > As you can see, your patch made zram too much regressed when it > used one stream buffer. The reason I guessed is overhead of > scheduling by sleep/wakeup when others couldn't find a idle stream > so I had an experiment with below simple

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Sergey Senozhatsky
Hello, On (02/18/14 15:04), Minchan Kim wrote: > Hello Sergey, > > On Thu, Feb 13, 2014 at 08:43:18PM +0300, Sergey Senozhatsky wrote: > > This patchset introduces zcomp compression backend abstraction > > adding ability to support compression algorithms other than LZO; > > support for multi

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Sergey Senozhatsky
Hello, On (02/18/14 15:04), Minchan Kim wrote: Hello Sergey, On Thu, Feb 13, 2014 at 08:43:18PM +0300, Sergey Senozhatsky wrote: This patchset introduces zcomp compression backend abstraction adding ability to support compression algorithms other than LZO; support for multi compression

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Sergey Senozhatsky
On (02/18/14 15:04), Minchan Kim wrote: [..] 1.8.5.3 As you can see, your patch made zram too much regressed when it used one stream buffer. The reason I guessed is overhead of scheduling by sleep/wakeup when others couldn't find a idle stream so I had an experiment with below simple patch

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Minchan Kim
Hello Sergey, On Tue, Feb 18, 2014 at 12:20:17PM +0300, Sergey Senozhatsky wrote: Hello, On (02/18/14 15:04), Minchan Kim wrote: Hello Sergey, On Thu, Feb 13, 2014 at 08:43:18PM +0300, Sergey Senozhatsky wrote: This patchset introduces zcomp compression backend abstraction adding

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-18 Thread Minchan Kim
On Tue, Feb 18, 2014 at 10:38:18PM +0300, Sergey Senozhatsky wrote: On (02/18/14 15:04), Minchan Kim wrote: [..] 1.8.5.3 As you can see, your patch made zram too much regressed when it used one stream buffer. The reason I guessed is overhead of scheduling by sleep/wakeup when others

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-17 Thread Minchan Kim
Hello Sergey, On Thu, Feb 13, 2014 at 08:43:18PM +0300, Sergey Senozhatsky wrote: > This patchset introduces zcomp compression backend abstraction > adding ability to support compression algorithms other than LZO; > support for multi compression streams, making parallel compressions > possible. >

Re: [PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-17 Thread Minchan Kim
Hello Sergey, On Thu, Feb 13, 2014 at 08:43:18PM +0300, Sergey Senozhatsky wrote: This patchset introduces zcomp compression backend abstraction adding ability to support compression algorithms other than LZO; support for multi compression streams, making parallel compressions possible.

[PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-13 Thread Sergey Senozhatsky
This patchset introduces zcomp compression backend abstraction adding ability to support compression algorithms other than LZO; support for multi compression streams, making parallel compressions possible. Diff from v4 (reviewed by Minchan Kim): -- renamed zcomp buffer_lock; removed src len and

[PATCHv5 0/4] add compressing abstraction and multi stream support

2014-02-13 Thread Sergey Senozhatsky
This patchset introduces zcomp compression backend abstraction adding ability to support compression algorithms other than LZO; support for multi compression streams, making parallel compressions possible. Diff from v4 (reviewed by Minchan Kim): -- renamed zcomp buffer_lock; removed src len and