From: "Li, Weigang" Subject: Re: [PATCH RFC 0/7] Introduce new async/sync compression APIs Date: Tue, 12 Jan 2016 16:00:50 +0800 Message-ID: <5694B2B2.7050806@intel.com> References: <1451882819-2810-1-git-send-email-iamjoonsoo.kim@lge.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Cc: "David S. Miller" , Sergey Senozhatsky , Minchan Kim , , , Joonsoo Kim To: Joonsoo Kim , Herbert Xu Return-path: Received: from mga02.intel.com ([134.134.136.20]:31406 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752340AbcALIBH (ORCPT ); Tue, 12 Jan 2016 03:01:07 -0500 In-Reply-To: <1451882819-2810-1-git-send-email-iamjoonsoo.kim@lge.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: 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?