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

Speck: added basic benchmarking. #60

Merged
merged 5 commits into from Jul 30, 2017
Merged

Speck: added basic benchmarking. #60

merged 5 commits into from Jul 30, 2017

Conversation

ghost
Copy link

@ghost ghost commented Jul 29, 2017

I added this when I noticed that decrypt performance was significantly slower than encrypt performance.

Cooper Paul EdenDay added 4 commits July 29, 2017 12:13
Signed-off-by: Cooper Paul EdenDay <cedenday@protonmail.com>
Signed-off-by: Cooper Paul EdenDay <cedenday@protonmail.com>
Signed-off-by: Cooper Paul EdenDay <cedenday@protonmail.com>
Signed-off-by: Cooper Paul EdenDay <cedenday@protonmail.com>
@ghost
Copy link
Author

ghost commented Jul 29, 2017

I am currently getting these results on my laptop (CPU: Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz, 16383212 kB DDR4 @ 2133 MHz):

test decrypt      ... bench:          32 ns/iter (+/- 1)
test encrypt      ... bench:          25 ns/iter (+/- 1)
test generate_key ... bench:          36 ns/iter (+/- 1)

Edit: this is with Gnome running, I don't know the effect of it.

Copy link
Contributor

@ticki ticki left a comment

Choose a reason for hiding this comment

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

Very cool stuff, but before I can merge please fix the comments :)

speck/.gitignore Outdated
@@ -0,0 +1,5 @@
/target/
Copy link
Contributor

Choose a reason for hiding this comment

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

You don't need this file as it already exists in the root of the repo.

Copy link
Author

Choose a reason for hiding this comment

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

I removed the .gitignore file, I hope that's what you wanted.

Signed-off-by: Cooper Paul EdenDay <cedenday@protonmail.com>
@ticki
Copy link
Contributor

ticki commented Jul 30, 2017

Thanks!

@ticki ticki merged commit 8f48225 into redox-os:master Jul 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant