> From: Herbert Xu [mailto:herb...@gondor.apana.org.au] > Sent: Tuesday, April 05, 2016 5:04 AM > To: megha....@linux.intel.com > Cc: da...@davemloft.net; linux-crypto@vger.kernel.org; linux- > ker...@vger.kernel.org; tim.c.c...@linux.intel.com; Yu, Fenghua > <fenghua...@intel.com>; Dey, Megha <megha....@intel.com> > Subject: Re: [PATCH 2/7] crypto: sha256-mb - SHA256 multibuffer job > manager and glue code > > On Thu, Mar 24, 2016 at 01:25:58PM -0700, megha....@linux.intel.com > wrote: > > From: Megha Dey <megha....@intel.com> > > > > This patch introduces the multi-buffer job manager which is > > responsible for submitting scatter-gather buffers from several SHA256 > > jobs to the multi-buffer algorithm. It also contains the flush routine > > to that's called by the crypto daemon to complete the job when no new > > jobs arrive before the deadline of maximum latency of a SHA256 crypto job. > > > > The SHA256 multi-buffer crypto algorithm is defined and initialized in > > this patch. > > > > Signed-off-by: Megha Dey <megha....@linux.intel.com> > > Reviewed-by: Fenghua Yu <fenghua...@intel.com> > > Reviewed-by: Tim Chen <tim.c.c...@linux.intel.com> > > sha1-mb still has the same issues that I complained about for the aes-mb > submission. In particular, I don't like the use of shash to handle what is > really > an async implmentation. > > So I'd like to see this fixed first before we add any more copies of this > code.
We will update the patches and re-send out sha1/256/512 AVX2 mb algos using async implementation. Thanks. -Fenghua -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html