Thread (67 messages) 67 messages, 8 authors, 2019-10-07

Re: [PATCH v2 05/20] crypto: mips/chacha - import accelerated 32r2 code from Zinc

From: "Jason A. Donenfeld" <Jason@zx2c4.com>
Date: 2019-10-04 14:59:18

On Fri, Oct 4, 2019 at 4:44 PM Ard Biesheuvel [off-list ref] wrote:
The round count is passed via the fifth function parameter, so it is
already on the stack. Reloading it for every block doesn't sound like
a huge deal to me.
Please benchmark it to indicate that, if it really isn't a big deal. I
recall finding that memory accesses on common mips32r2 commodity
router hardware was extremely inefficient. The whole thing is designed
to minimize memory accesses, which are the primary bottleneck on that
platform.

Seems like this thing might be best deferred for after this all lands.
IOW, let's get this in with the 20 round original now, and later you
can submit a change for the 12 round and René and I can spend time
dusting off our test rigs and seeing which strategy works best. I very
nearly tossed out a bunch of old router hardware last night when
cleaning up. Glad I saved it!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help