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

Add BoringSSL and CIRCL kyber768 benchmarks. #75

Merged
merged 8 commits into from
Sep 13, 2023
Merged

Conversation

xvzcf
Copy link
Contributor

@xvzcf xvzcf commented Sep 11, 2023

No description provided.

Copy link
Member

@franziskuskiefer franziskuskiefer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add readmes with instructions no how run it and how to interpret the results.

I get these numbers.

CIRCL

BenchmarkKeyGeneration-10          28485             40193 ns/op
BenchmarkEncapsulation-10          26870             44287 ns/op
BenchmarkDecapsulation-10          24258             49165 ns/op

BoringSSL

Each Key-generation operation took 0.032474 milliseconds.
Each Encapsulation operation took 0.028555 milliseconds.
Each Decapsulation operation took 0.035148 milliseconds.

If I'm reading this right it means that boring is faster than circl?

Copy link
Member

@franziskuskiefer franziskuskiefer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the release and then merge it.
We still need to figure out what's going on though.

benches/boringssl/README.md Outdated Show resolved Hide resolved
@xvzcf xvzcf enabled auto-merge (squash) September 13, 2023 15:20
@xvzcf xvzcf merged commit 903dd25 into dev Sep 13, 2023
6 checks passed
@xvzcf xvzcf deleted the add-more-kyber-benches branch September 13, 2023 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants