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 benchs #13

Merged
merged 2 commits into from
Dec 22, 2014
Merged

Add benchs #13

merged 2 commits into from
Dec 22, 2014

Conversation

Kagami
Copy link
Contributor

@Kagami Kagami commented Dec 22, 2014

Hi. I added ecdsa and secp256k1 to the benchmark.

Results on my system:

Benchmarking: sign
elliptic#sign x 292 ops/sec ±0.73% (177 runs sampled)
sjcl#sign x 74.50 ops/sec ±0.40% (153 runs sampled)
openssl#sign x 2,145 ops/sec ±0.22% (198 runs sampled)
ecdsa#sign x 45.28 ops/sec ±0.15% (117 runs sampled)
secp256k1#sign x 9,598 ops/sec ±1.65% (191 runs sampled)
------------------------
Fastest is secp256k1#sign
========================
Benchmarking: verify
elliptic#verify x 133 ops/sec ±0.69% (170 runs sampled)
sjcl#verify x 63.09 ops/sec ±0.37% (130 runs sampled)
openssl#verify x 1,955 ops/sec ±0.06% (199 runs sampled)
ecdsa#verify x 32.84 ops/sec ±0.42% (114 runs sampled)
secp256k1#verify x 10,370 ops/sec ±0.03% (202 runs sampled)
------------------------
Fastest is secp256k1#verify
========================
Benchmarking: gen
elliptic#gen x 321 ops/sec ±0.45% (182 runs sampled)
sjcl#gen x 82.28 ops/sec ±0.07% (141 runs sampled)
------------------------
Fastest is elliptic#gen
========================
Benchmarking: ecdh
elliptic#ecdh x 165 ops/sec ±0.80% (169 runs sampled)
------------------------
Fastest is elliptic#ecdh
========================
Benchmarking: curve25519
elliptic#curve25519 x 75.02 ops/sec ±2.64% (115 runs sampled)
jodid#curve25519 x 107 ops/sec ±0.30% (156 runs sampled)
------------------------
Fastest is jodid#curve25519
========================

indutny added a commit that referenced this pull request Dec 22, 2014
@indutny indutny merged commit 5e1794d into indutny:master Dec 22, 2014
@indutny
Copy link
Owner

indutny commented Dec 22, 2014

LGTM, thanks.

@Kagami Kagami deleted the add-benchs branch December 22, 2014 14:00
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.

2 participants