Add native implementations for Keccak on Armv8.1-m #511
Conversation
There was a problem hiding this comment.
Thanks @bremoran! We'll need to discuss this with the others. Here are some initial comments.
I don't see this test being exercised in the MVE CI so far. That definitely should be changed.
We should align the test names with mlkem-native.
My biggest concern right now is uniform licensing. At least everything in mldsa/ needs to be Apache-2.0 OR ISC OR MIT. For test/ we may be slightly more flexible, but just having Apache-2.0 OR ISC OR MIT for everything makes it less confusing. For code that is public domain/CC0 relicensing should be easy.
mldsa/fips202/keccakf1600.c
Outdated
| unsigned i; | ||
| #if defined(MLD_SYS_LITTLE_ENDIAN) | ||
| #if defined(MLD_USE_FIPS202_X1_NATIVE) | ||
| (void) i; |
There was a problem hiding this comment.
remove this and move the unsigned i inside of the other block instead.
Makefile
Outdated
| # SPDX-License-Identifier: Apache-2.0 OR ISC OR MIT | ||
|
|
||
| .PHONY: func kat acvp stack \ | ||
| .PHONY: func kat acvp stack keccak \ |
There was a problem hiding this comment.
Can we use the same naming as in mlkem-native and call this unit: https://github.com/pq-code-package/mlkem-native/blob/9ae2223e835c1abcc8d6857dd3b7f8ce00a05216/Makefile#L84?
If we can keep the diff to mlkem-native minimal that would be appreciated.
There was a problem hiding this comment.
Yes, I will attempt to set this up more like the unit test framework in mlkem-native.
bremoran
force-pushed
the
armv8.1-m-native
branch
2 times, most recently
from
cb94d1c to
d45104c
Compare
Add unit tests for FIPS202 APIs Add unit tests for x4 keccak Signed-off-by: Brendan Moran <[email protected]>
bremoran
force-pushed
the
armv8.1-m-native
branch
from
d45104c to
46ca703
Compare
Signed-off-by: Brendan Moran <[email protected]>
Signed-off-by: Brendan Moran <[email protected]>
Signed-off-by: Brendan Moran <[email protected]>
Signed-off-by: Brendan Moran <[email protected]>
Signed-off-by: Brendan Moran <[email protected]>
2 participants
Also add tests for Keccak implementations on Armv8.1-m