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

2016-05-27 Thread Sergey Senozhatsky
On (05/27/16 13:22), Minchan Kim wrote: [..] > > static void zcomp_strm_free(struct zcomp *comp, struct zcomp_strm *zstrm) > > { > > - if (zstrm->private) > > - comp->backend->destroy(zstrm->private); > > + if (!IS_ERR_OR_NULL(zstrm->private)) > > Let's change private with tfm. ok

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

2016-05-26 Thread Minchan Kim
On Wed, May 25, 2016 at 11:30:01PM +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

[PATCH 2/7] zram: switch to crypto compress API

2016-05-25 Thread Sergey Senozhatsky
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 could badly affect the performance of both paths) and at the same time opens t