From: Herbert Xu Subject: Re: [PATCH RFC 0/7] Introduce new async/sync compression APIs Date: Tue, 12 Jan 2016 20:47:39 +0800 Message-ID: <20160112124739.GA679@gondor.apana.org.au> References: <1451882819-2810-1-git-send-email-iamjoonsoo.kim@lge.com> <5694B2B2.7050806@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Joonsoo Kim , "David S. Miller" , Sergey Senozhatsky , Minchan Kim , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, Joonsoo Kim To: "Li, Weigang" Return-path: Content-Disposition: inline In-Reply-To: <5694B2B2.7050806@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org 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! -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt