From: Herbert Xu Subject: Re: [PATCH] crypto: compress - Add pcomp interface Date: Fri, 16 Jan 2009 15:27:49 +1100 Message-ID: <20090116042749.GA11435@gondor.apana.org.au> References: <1231862386-11128-1-git-send-email-Geert.Uytterhoeven@sonycom.com> <1231862386-11128-2-git-send-email-Geert.Uytterhoeven@sonycom.com> <20090114031613.GA7429@gondor.apana.org.au> <20090115030526.GA27726@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org To: Geert Uytterhoeven Return-path: Received: from rhun.apana.org.au ([64.62.148.172]:44919 "EHLO arnor.apana.org.au" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754679AbZAPE1y (ORCPT ); Thu, 15 Jan 2009 23:27:54 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-crypto-owner@vger.kernel.org List-ID: On Thu, Jan 15, 2009 at 11:54:53AM +0100, Geert Uytterhoeven wrote: > > But there are pr_debug()s ;-) Well they must've turned invisible in the copy I received :) > By "both alloc function", do you mean the .setup_comp() and .setup_decomp() > functions? If yes, then I understand. Yes. > (Yesterday, I thought you meant to have separate alloc functions instead of the > one crypto_alloc_pcomp()) No we want a single tfm for this. If we ever get hardware that can handle only one of the two operations we can always implement software fallbacks as we do for hashing and encryption. > I doubt whether it's a good idea to drop the compatibility soon, as the > underlying LZO library in the kernel doesn't support partial (de)compression, > and needs significant rework to do so. Yes but LZO has exactly one user (ubifs) in the kernel and it can stick to the current interface until we convert it over. It's just not worth the extra work to help a single user. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt