Re: [PATCH net-next v6 21/23] crypto: port ChaCha20 to Zinc

From: Jason A. Donenfeld
Date: Mon Oct 01 2018 - 23:31:56 EST


Hi Herbert,

On Tue, Oct 2, 2018 at 5:26 AM Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> wrote:
> Oh nice, so you did the conversion of the existing crypto code.
>
> I presume someone has done the numbers and verified that there
> is no performance regression?
>
> If so it would be good to include those numbers somewhere in this
> submission (within a patch description so that it goes into git).

Indeed I've done a pretty ridiculous amount of benchmarking, and the
x86_64 numbers are included in those commits because they're pretty
straightforward to show. The performance is also better on
arm1176,cortex-a7,8,9,15,53,73. On mips, as well, performance is
better, since there wasn't any optimized code there at all. Early
versions of this patchset included a regression on cortex-a7, which
was particularly problematic because Eric Biggers recent fscrypt work
requires fast chacha on that hardware, but this v6 (and the future v7)
contain code that is faster on all platforms across the board.

Regards,
Jason