Skip to content

Benchmarking#6

Open
sooryan wants to merge 7 commits intomasterfrom
benchmarking
Open

Benchmarking#6
sooryan wants to merge 7 commits intomasterfrom
benchmarking

Conversation

@sooryan
Copy link
Collaborator

@sooryan sooryan commented Dec 22, 2016

Addresses #5

Soorya Narayan added 7 commits December 23, 2016 03:07
A driver program to run blake2b or any other hash function and measure the
cycles/hash for various input lengths
Output can be plotted with gnuplot
Use openssl's MD5 for benchmarking comparison with blake2b
Add a #ifdef to call blake2b while benchmarking
Add benchmarking output
Add build and bin folders
@@ -0,0 +1,10 @@
#include <openssl/md5.h>

Choose a reason for hiding this comment

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

Given that MD5 is not exactly used anymore, how about comparing with SHA1/SHA256? (I know that blake is supposed to compete with the speed of MD5, so this test is cool, but another test to compare performance with algorithms that are actually used would be great.)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The present structure makes it pretty easy to add other functions. I'll add in SHA1/SHA256 comparisons too.

@@ -0,0 +1,10 @@
#include <openssl/md5.h>

Choose a reason for hiding this comment

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

Comparing to openssl is great but comparing to NSS as well would be even greater 😄 . This will also help you getting used to NSS' "interesting" interfaces. You can check [1] and [2] on how to use NSS for hashing.

[1] https://searchfox.org/nss/source/cmd/digest/digest.c#91
[2] https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/nss_sample_code/NSS_Sample_Code_sample3

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Haha, I'll add in comparisons from other NSS implementations too

@@ -0,0 +1,19 @@
#maxx can hold values [0,4096]

Choose a reason for hiding this comment

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

👍 for using gplot

@Sachin-A Sachin-A closed this in 36dcdfe Jan 3, 2017
@Sachin-A Sachin-A reopened this Jan 8, 2017
Copy link
Owner

@Sachin-A Sachin-A left a comment

Choose a reason for hiding this comment

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

Add in comparisons with NSS

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.

3 participants