From: Herbert Xu Subject: Re: [RFC PATCH crypto 4/4] AES-NI: Add support to Intel AES-NI instructions for x86_64 platform Date: Tue, 13 Jan 2009 13:39:38 +1100 Message-ID: <20090113023937.GA14460@gondor.apana.org.au> References: <1231120947.5937.31.camel@yhuang-dev.sh.intel.com> <20090109070144.GA7358@gondor.apana.org.au> <1231743310.5937.107.camel@yhuang-dev.sh.intel.com> <20090112104335.GA4942@gondor.apana.org.au> <1231814053.5937.120.camel@yhuang-dev.sh.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Sebastian Siewior , "linux-kernel@vger.kernel.org" , "linux-crypto@vger.kernel.org" To: Huang Ying Return-path: Received: from rhun.apana.org.au ([64.62.148.172]:50852 "EHLO arnor.apana.org.au" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753829AbZAMCjo (ORCPT ); Mon, 12 Jan 2009 21:39:44 -0500 Content-Disposition: inline In-Reply-To: <1231814053.5937.120.camel@yhuang-dev.sh.intel.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Tue, Jan 13, 2009 at 10:34:13AM +0800, Huang Ying wrote: > > static void ablk_complete(struct crypto_async_request *req, int err) > { > struct ablkcipher_request *ablk_req = ablkcipher_request_cast(req); > struct async_aes_req_ctx *req_ctx = > ablk_aes_req_ctx(ablk_req, crypto_ablkcipher_reqtfm(ablk_req)); This is not guaranteed to work. Your completion caller may be called with a request pointer other than the one that you passed to it. The only thing we guarantee is that req->data has the value that you set at the beginning. 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