[rom_ext] Perform ECDSA ans SPX verification in parallel #26160
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cfrantz
force-pushed
the
spx-in-parallel
branch
from
075a63e to
7a8f676
Compare
timothytrippel
approved these changes
Feb 7, 2025
|
You will need to update these two files as well: |
cfrantz
force-pushed
the
spx-in-parallel
branch
from
7a8f676 to
c58ddcb
Compare
Split ECDSA sigverify into start and finish operations so Ibex and execute code while OTBN is verifying the signature. Signed-off-by: Chris Frantz <[email protected]>
Signed-off-by: Chris Frantz <[email protected]>
cfrantz
force-pushed
the
spx-in-parallel
branch
from
c58ddcb to
953db71
Compare
cfrantz
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
By running the ECDSA and SPX operations in parallel, the 4.2ms ECDSA verification time is subsumed into the ~9ms SPX verification time.