8000 Benchmark Ecrecover and BLS verify in parallel by blukat29 · Pull Request #1867 · klaytn/klaytn · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 19, 2024. It is now read-only.

Benchmark Ecrecover and BLS verify in parallel #1867

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Benchmark Ecrecover and BLS verify in parallel #1867

merged 1 commit into from
Jul 17, 2023

Conversation

blukat29
Copy link
Contributor

Proposed changes

Add multi-threaded benchmarks for ECDSA (Ecrecover) and BLS (Verify)

  • crypto.BenchmarkEcrecover: single-threaded Ecrecover
  • crypto.BenchmarkParallelEcrecver: multi-threaded Ecrecover
  • blst.BenchmarkParallelVerify: multi-threaded BLS verify

Types of changes

  • Bugfix
  • New feature or enhancement
  • Others

Checklist

  • I have read the CONTRIBUTING GUIDELINES doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes ($ make test)
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Related issues

Further comments

@blukat29 blukat29 requested a review from ian0371 June 19, 2023 06:55
ian0371
ian0371 reviewed Jun 20, 2023
View reviewed changes
@blukat29 blukat29 force-pushed the benchmark-parallel-crypto branch from 3a1819d to c52cb7b Compare June 20, 2023 03:41
hyunsooda
hyunsooda reviewed Jun 21, 2023
View reviewed changes
@blukat29 blukat29 merged commit 74b5389 into klaytn:dev Jul 17, 2023
@blukat29 blukat29 deleted the benchmark-parallel-crypto branch July 17, 2023 03:45
@JayChoi1736 JayChoi1736 mentioned this pull request Jul 24, 2023
Closed
20 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ian0371 ian0371 ian0371 approved these changes

@hyunsooda hyunsooda hyunsooda approved these changes

@jiseongnoh jiseongnoh jiseongnoh approved these changes

@aidan-kwon aidan-kwon Awaiting requested review from aidan-kwon

Assignees

@blukat29 blukat29

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@blukat29 @ian0371 @hyunsooda @jiseongnoh
0