Re: [PATCH crypto v3 0/2] reduce code size from blake2s on m68k and other small platforms

From: Jason A. Donenfeld
Date: Tue Jan 18 2022 - 07:51:15 EST


On Tue, Jan 18, 2022 at 1:45 PM David Laight <David.Laight@xxxxxxxxxx> wrote:
> I've rammed the code through godbolt... https://godbolt.org/z/Wv64z9zG8
>
> Some things I've noticed;

It seems like you've done a lot of work here but...

> But I've not got time to test the code.

But you're not going to take it all the way. So it unfortunately
amounts to mailing list armchair optimization. That's too bad because
it really seems like you might be onto something worth seeing through.
As I've mentioned a few times now, I've dropped the blake2s
optimization patch, and I won't be developing that further. But it
appears as though you've really been captured by it, so I urge you:
please send a real patch with benchmarks on various platforms! (And CC
me on the patch.) Faster reference code would really be terrific.

Jason