Re: [PATCH 0/7] zram: switch to crypto api

2016-05-25 Thread Joonsoo Kim
On Thu, May 26, 2016 at 10:12:16AM +0900, Sergey Senozhatsky wrote: > On (05/26/16 09:43), Joonsoo Kim wrote: > [..] > > Hello, Sergey. > > > > I don't look at each patches deeply but nice work! I didn't notice that > > rececnt zram changes makes thing simpler. :) > > Hello Joonsoo, > > thanks.

Re: [PATCH 0/7] zram: switch to crypto api

2016-05-25 Thread Sergey Senozhatsky
On (05/26/16 09:43), Joonsoo Kim wrote: [..] > Hello, Sergey. > > I don't look at each patches deeply but nice work! I didn't notice that > rececnt zram changes makes thing simpler. :) Hello Joonsoo, thanks. I owe you a drink for pushing it in the context-less crypto API direction. sorry about

Re: [PATCH 0/7] zram: switch to crypto api

2016-05-25 Thread Sergey Senozhatsky
On (05/26/16 09:52), Minchan Kim wrote: > On Wed, May 25, 2016 at 11:29:59PM +0900, Sergey Senozhatsky wrote: > > Hello, > > > > This has started as a 'add zlib support' work, but after some > > thinking I saw no blockers for a bigger change -- a switch to > > crypto API. > > > > We don't hav

Re: [PATCH 0/7] zram: switch to crypto api

2016-05-25 Thread Minchan Kim
On Wed, May 25, 2016 at 11:29:59PM +0900, Sergey Senozhatsky wrote: > Hello, > > This has started as a 'add zlib support' work, but after some > thinking I saw no blockers for a bigger change -- a switch to > crypto API. > > We don't have an idle zstreams list anymore and our write path > now

Re: [PATCH 0/7] zram: switch to crypto api

2016-05-25 Thread Joonsoo Kim
On Wed, May 25, 2016 at 11:29:59PM +0900, Sergey Senozhatsky wrote: > Hello, > > This has started as a 'add zlib support' work, but after some > thinking I saw no blockers for a bigger change -- a switch to > crypto API. > > We don't have an idle zstreams list anymore and our write path > now