Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Armv7M support for Cortex-M4 and Cortex-M7 #61

Draft
wants to merge 374 commits into
base: main
Choose a base branch
from
Draft

Add Armv7M support for Cortex-M4 and Cortex-M7 #61

wants to merge 374 commits into from

Conversation

mkannwischer
Copy link
Collaborator

Continuation of #55

@jnk0le
Copy link

jnk0le commented Jul 4, 2024

regarding the pipeline models, I did some analysis on those already.
M7 (it was even cited by paper linked by one keccak file): https://github.com/jnk0le/random/tree/master/pipeline%20cycle%20test#cortex-m7
and M4: https://github.com/jnk0le/random/tree/master/pipeline%20cycle%20test#cortex-m3-and-m4

@jnk0le jnk0le mentioned this pull request Aug 11, 2024
Open
mkannwischer and others added 19 commits October 17, 2024 20:42
@mkannwischer
switch some kyber examples to sw pipelining
6888a7b
@dop-amin
optimize_loop for Dilithium examples
c69ad9c 
* Introduces loop class for loops that restore the compare-to pointer using a vmov
* Modifies mode of operation for loop extraction: The loop instructions need to appear in sequence
@dop-amin
Fix frombytes_mul_acc_32_16_kyber loop
f8550a0 
* Slight code modification to make loop parsable
@dop-amin
Slight tweaks to example config
5d34476
@mkannwischer
make .equ regex more general
3c00fc8
@mkannwischer
symbolic Kyber NTT: switch stack locations from #define to .equ
4bbf1a1
@mkannwischer
debug symbolic NTT
8df6444
@mkannwischer
re-add ldrd splitting for symbolic NTT
f75fc14
@mkannwischer
switch ntt_kyber to sw pipelining
a29d5f5
@mkannwischer
Kyber intt: add sw pipelining
631bca2
@mkannwischer
change config for Kyber intt
f49e7ce
@mkannwischer
model bank conflicts using execution units
65b37d6
@mkannwischer
fix kyber ntt configs
24f09c2
@dop-amin
Add some optimized code
e51f423
@mkannwischer
fix symbolic NTT
606251b
@mkannwischer
loop opt: do not count empty lines for instruction count
fc8a963
@mkannwischer
Merge branch 'main' into armv7m
0dfdfd3
@mkannwischer
fix Dilithium NTT
9310140
@mkannwischer
fix typo
6a2898f
dop-amin and others added 10 commits December 17, 2024 10:10
@dop-amin
Allow to force loop type
ce74da9 
* Helps in case of ambigous loop types and makes the operation more predictable.
@dop-amin
Arch armv7m: fix & enable BranchLoop
122c823 
* Fix issue with wrong detection of FPR for storing end-pointer
* Move to the end of loop classes (i.e., lowest priority)
* Apply BranchLoop to ntt_dilithium example to get deprecate of boundary_reserved_regs
@dop-amin
Armv7m Keccak: Add missing .type directive to pqm4 implementation
a0900e7
@dop-amin
Armv7m Keccak: Re-optimize M7 hand-optimized code with more liberal p…
ca54a08 
…arams
@dop-amin
Armv7m Dilithium NTT: Slight alignment tweak for CM4
709a383
@dop-amin
Armv7m: Add .size directive to all Kyber, Dilithium, and Keccak examples
b22e78b
@dop-amin
Armv7m: Fix .size directive in frombytes_mul
d2a33ca
@mkannwischer
Merge remote-tracking branch 'origin' into armv7m
fb6fe0d
@mkannwischer
Merge branch 'main' into armv7m
47560ea
@mkannwischer
Merge remote-tracking branch 'origin/main' into armv7m
8a6ff35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mkannwischer @jnk0le @hanno-becker @dop-amin