Skip to content

bench: add cksum performance benchmarks with SHAKE algorithm support#9074

Closed
naoNao89 wants to merge 1 commit into
uutils:mainfrom
naoNao89:bench/cksum-performance-fixed
Closed

bench: add cksum performance benchmarks with SHAKE algorithm support#9074
naoNao89 wants to merge 1 commit into
uutils:mainfrom
naoNao89:bench/cksum-performance-fixed

Conversation

@naoNao89

@naoNao89 naoNao89 commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

cksum lacks performance benchmarks while other utilities have them.

This PR adds comprehensive performance benchmarks for all 17 SUPPORTED_ALGORITHMS including:

Resolves merge conflicts from previous PR #9064 by using clean main branch base.

Key Features:

  • Benchmarks for all 17 algorithms: sysv, bsd, crc, crc32b, md5, sha1, sha2, sha3, blake2b, sm3, sha224, sha256, sha384, sha512, blake3, shake128, shake256
  • Custom SHAKE algorithm support with required --length parameter
  • Direct digest calculation for meaningful SHAKE performance measurements
  • Proper error handling with LengthRequiredForShake error type
  • CI integration with CodSpeed benchmark workflow

…ithm support

- Add benchmarks for all 17 SUPPORTED_ALGORITHMS
- Implement custom SHAKE algorithm benchmarks with direct digest calculation
- Add SHAKE128/SHAKE256 algorithm support requiring --length parameter
- Add LengthRequiredForShake error type for proper validation
- Include benchmark workflow integration in CI

Resolves merge conflicts by using clean main branch base.
@sylvestre

Copy link
Copy Markdown
Contributor

still conflicts ;)

@naoNao89

Copy link
Copy Markdown
Contributor Author

🗿

@naoNao89

Copy link
Copy Markdown
Contributor Author

new pr #9075

@naoNao89 naoNao89 closed this Oct 29, 2025
@naoNao89
naoNao89 deleted the bench/cksum-performance-fixed branch November 7, 2025 10:58
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