Thread (5 messages) 5 messages, 4 authors, 2018-09-04

Re: [PATCH] crypto: x86 - remove SHA multibuffer routines and mcryptd

From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2018-09-04 03:18:08
Also in: linux-s390, lkml

On Mon, Aug 27, 2018 at 04:19:53PM -0700, Megha Dey wrote:
On Mon, 2018-08-27 at 15:28 -0700, Tim Chen wrote:
quoted
On 08/22/2018 01:51 AM, Ard Biesheuvel wrote:
quoted
As it turns out, the AVX2 multibuffer SHA routines are currently
broken [0], in a way that would have likely been noticed if this
code were in wide use. Since the code is too complicated to be
maintained by anyone except the original authors, and since the
performance benefits for real-world use cases are debatable to
begin with, it is better to drop it entirely for the moment.

[0] https://marc.info/?l=linux-crypto-vger&m=153476243825350&w=2
Sorry I was out of the loop for a while and haven't been following
the code too closely.

Megha is maintaining the code now.  Before we pull the code,
please give us a chance to fix it first.

Thanks.

Tim
Hi,

I am working to find a fix for these corner cases. If possible, we would
like to fix the issues instead of removing the code altogether.
I think it has taken way too long to fix these issues.  The fact
that these issues have existed for so long also means that hardly
anyone uses these mb algorithms.

So I think it is best if we remove everything and then add them
back after a proper review process.

Thanks,
-- 
Email: Herbert Xu [off-list ref]
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help