Re: [PATCH v2 2/8] zram: switch to crypto compress API

2016-05-31 Thread Sergey Senozhatsky
On (06/01/16 08:44), Minchan Kim wrote: > > > trivial: > > One thing I got missed in review. > > > -int zcomp_compress(struct zcomp *comp, struct zcomp_strm *zstrm, > > - const unsigned char *src, size_t *dst_len); > > +int zcomp_compress(struct zcomp_strm *zstrm, > > + co

Re: [PATCH v2 2/8] zram: switch to crypto compress API

2016-05-31 Thread Minchan Kim
On Tue, May 31, 2016 at 09:20:11PM +0900, Sergey Senozhatsky wrote: trivial: One thing I got missed in review. > -int zcomp_compress(struct zcomp *comp, struct zcomp_strm *zstrm, > - const unsigned char *src, size_t *dst_len); > +int zcomp_compress(struct zcomp_strm *zstrm, > +

Re: [PATCH v2 2/8] zram: switch to crypto compress API

2016-05-31 Thread Minchan Kim
On Tue, May 31, 2016 at 09:20:11PM +0900, Sergey Senozhatsky wrote: > We don't have an idle zstreams list anymore and our write path > now works absolutely differently, preventing preemption during > compression. This removes possibilities of read paths preempting > writes at wrong places (which co