From: Herbert Xu Subject: Re: Bug of dm-crypt? Date: Fri, 27 Feb 2009 19:39:27 +0800 Message-ID: <20090227113927.GA24024@gondor.apana.org.au> References: <1235712716.6204.100.camel@yhuang-dev.sh.intel.com> <20090227054147.GA22159@gondor.apana.org.au> <49A7A71E.50702@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Huang Ying , linux-kernel@vger.kernel.org, linux-crypto@vger.kernel.org To: Milan Broz Return-path: Received: from rhun.apana.org.au ([64.62.148.172]:44035 "EHLO arnor.apana.org.au" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754696AbZB0Lje (ORCPT ); Fri, 27 Feb 2009 06:39:34 -0500 Content-Disposition: inline In-Reply-To: <49A7A71E.50702@redhat.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Fri, Feb 27, 2009 at 09:41:02AM +0100, Milan Broz wrote: > > Can you send me backtrace from that panic? > > What's wrong in async callback now - mempool_free using async_req directly? The req you get in the completion function is not necessarily the req that you started with. The only thing you can rely on is req->data. > (Btw in some previous discussuion I asked if cryptd can be used > for some kind of parallel speedup for dm-crypt in async mode and response > was that cryptd "is only meant to be a demo showing how an async > implementation should be written". > > So this changed to real working thread for AES-NI?) Yes we changed it :) cryptd is no longer for fun only. 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