From: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Subject: Re: [PATCH RFC 0/7] Introduce new async/sync compression APIs
Date: Wed, 13 Jan 2016 17:31:34 +0900
Message-ID: <20160113083134.GA6966@js1304-P5Q-DELUXE>
References: <1451882819-2810-1-git-send-email-iamjoonsoo.kim@lge.com>
 <5694B2B2.7050806@intel.com>
 <20160112124739.GA679@gondor.apana.org.au>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: "Li, Weigang" <weigang.li@intel.com>,
	"David S. Miller" <davem@davemloft.net>,
	Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>,
	Minchan Kim <minchan@kernel.org>, linux-crypto@vger.kernel.org,
	linux-kernel@vger.kernel.org
To: Herbert Xu <herbert@gondor.apana.org.au>
Content-Disposition: inline
In-Reply-To: <20160112124739.GA679@gondor.apana.org.au>
Sender: linux-crypto-owner@vger.kernel.org

On Tue, Jan 12, 2016 at 08:47:39PM +0800, Herbert Xu wrote:
> On Tue, Jan 12, 2016 at 04:00:50PM +0800, Li, Weigang wrote:
> > On 1/4/2016 12:46 PM, Joonsoo Kim wrote:
> > >Happy New Year!
> > >
> > >This patchset is to introduce new compression APIs. It supports async/sync
> > >compression although there is no async compression driver yet. But, async
> > >APIs can be used as front-end to sync compression algorithm. Major change
> > >is that now APIs are stateless. Instead of previous implementation, tfm
> > >objects doesn't embedded any context so we can de/compress concurrently
> > >with one tfm object. Instead, this de/compression context is coupled with
> > >the request. This architecture change will make APIs more flexible.
> > >
> > >This is just a RFC so commit description isn't sufficient. If Herbert
> > >confirms that this is what he expect, I will respin the series with proper
> > >commit description and adding more algorithm support.
> > >
> > >Thanks.
> > >
> > >Joonsoo Kim (6):
> > >   crypto/compress: remove unused pcomp interface
> > >   crypto/compress: introduce sychronuous compression API
> > >   crypto/lzo: support SCOMPRESS alg type
> > >   crypto: testmgr: add scompress/acompress test
> > >   crypto/compress: add algorithm type specific flag, DECOMP_NOCTX
> > >   crypto/lzo: add CRYPTO_ALG_SCOMPRESS_DECOMP_NOCTX cra_flags
> > >
> > >Weigang Li (1):
> > >   crypto: add asynchronous compression support
> > >
> > >  crypto/Kconfig                     |  19 +-
> > >  crypto/Makefile                    |   4 +-
> > >  crypto/acompress.c                 | 164 ++++++++++++++++
> > >  crypto/lzo.c                       |  95 +++++++--
> > >  crypto/pcompress.c                 | 115 -----------
> > >  crypto/scompress.c                 | 283 +++++++++++++++++++++++++++
> > >  crypto/testmgr.c                   | 376 +++++++++++++++---------------------
> > >  crypto/testmgr.h                   | 142 --------------
> > >  crypto/zlib.c                      | 381 -------------------------------------
> > >  include/crypto/compress.h          | 354 +++++++++++++++++++++++++---------
> > >  include/crypto/internal/compress.h |  32 +---
> > >  include/linux/crypto.h             |  10 +-
> > >  12 files changed, 969 insertions(+), 1006 deletions(-)
> > >  create mode 100644 crypto/acompress.c
> > >  delete mode 100644 crypto/pcompress.c
> > >  create mode 100644 crypto/scompress.c
> > >  delete mode 100644 crypto/zlib.c
> > >
> > Hello Herbert,
> > Any comments on this patchset?
> 
> The interfaces look good to me.

Thanks for confirmation.
Then, I will add more algorithm support and respin next version
with minor fixes.

Herbert, for you, is it better to send it after merge window?

Thanks.