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

benchmark: refactor crypto oneshot #48267

Merged
merged 2 commits into from
Jun 5, 2023

Conversation

panva
Copy link
Member

@panva panva commented May 31, 2023

  • splits a combined sign+verify bench into one for sign, one for verify
  • adds config for other asymmetric key types (rsa, ec, ed25519)
  • adds config for jwk and der key formats
  • skips keyObject.unique configs unless the tested mode is async-parallel

This is the result of my shallow look at nodejs/performance#72

@panva panva added crypto Issues and PRs related to the crypto subsystem. benchmark Issues and PRs related to the benchmark subsystem. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels May 31, 2023
@panva panva added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 1, 2023
@panva
Copy link
Member Author

panva commented Jun 1, 2023

cc @nodejs/crypto, also see nodejs/performance#72 (comment)

@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 1, 2023
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@panva panva added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 1, 2023
@nodejs-github-bot
Copy link
Collaborator

@panva panva removed the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Jun 5, 2023
@panva panva requested a review from tniessen June 5, 2023 09:58
@panva
Copy link
Member Author

panva commented Jun 5, 2023

cc @nodejs/performance

benchmark/crypto/oneshot-sign.js Outdated Show resolved Hide resolved
benchmark/crypto/oneshot-sign.js Outdated Show resolved Hide resolved
benchmark/crypto/oneshot-verify.js Outdated Show resolved Hide resolved
@panva panva added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 5, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 5, 2023
@nodejs-github-bot nodejs-github-bot merged commit 0e92bf1 into nodejs:main Jun 5, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 0e92bf1

@panva panva deleted the bench-crypto-oneshots branch June 5, 2023 19:24
RafaelGSS pushed a commit that referenced this pull request Jul 3, 2023
PR-URL: #48267
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
@RafaelGSS RafaelGSS mentioned this pull request Jul 3, 2023
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48267
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48267
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
ruyadorno pushed a commit that referenced this pull request Aug 29, 2023
PR-URL: #48267
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
@ruyadorno ruyadorno mentioned this pull request Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. benchmark Issues and PRs related to the benchmark subsystem. crypto Issues and PRs related to the crypto subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants