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.

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

Update SHA3 Dependency #7

Merged
merged 3 commits into from
Aug 15, 2023
Merged

Update SHA3 Dependency #7

merged 3 commits into from
Aug 15, 2023

Conversation

itzmeanjan
Copy link
Owner

Update SHA3 dependency to latest commit itzmeanjan/sha3@2228e43.

This change increases time required for performing keygen, encaps and decaps. Considering Saber KEM implementation at commit 3628f5a as baseline while commit 1bf5a74 is contender implementation, we get 2-4% performance reduction.

Benchmark                           Time             CPU      Time Old      Time New       CPU Old       CPU New
----------------------------------------------------------------------------------------------------------------
lightsaber/keygen                +0.0453         +0.0453         40799         42647         40797         42644
lightsaber/encaps                +0.0313         +0.0313         60504         62400         60499         62395
lightsaber/decaps                +0.0228         +0.0228         77636         79408         77632         79402
saber/keygen                     +0.0314         +0.0314         87429         90178         87423         90172
saber/encaps                     +0.0346         +0.0347        116823        120867        116809        120857
saber/decaps                     +0.0223         +0.0223        142016        145187        142005        145178
firesaber/keygen                 +0.0281         +0.0281        154801        159151        154789        159137
firesaber/encaps                 +0.0239         +0.0239        193369        197989        193358        197978
firesaber/decaps                 +0.0201         +0.0201        227555        232126        227537        232109
OVERALL_GEOMEAN                  +0.0289         +0.0289             0             0             0             0

@itzmeanjan itzmeanjan merged commit 95f5926 into main Aug 15, 2023
1 check passed
@itzmeanjan itzmeanjan deleted the update-sha3 branch August 15, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant