Skip to content

x.crypto.chacha20: expands xchacha20 construction to support for 64-bit counter#25377

Merged
spytheman merged 3 commits into
vlang:masterfrom
blackshirt:ext_chacha20
Sep 24, 2025
Merged

x.crypto.chacha20: expands xchacha20 construction to support for 64-bit counter#25377
spytheman merged 3 commits into
vlang:masterfrom
blackshirt:ext_chacha20

Conversation

@blackshirt

@blackshirt blackshirt commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

This patch expands eXtended ChaCha20 construction to support for 64-bit counter, completes support for the existing 32-bit counter construct.
@spytheman, @tankf33der: can you look into it ?

Cheers,
thanks

@huly-for-github

Copy link
Copy Markdown

Connected to Huly®: V_0.6-26066

@blackshirt blackshirt changed the title x.crypto.chacha20: expands xchacha20 construction support for 64-bit counter x.crypto.chacha20: expands xchacha20 construction to support for 64-bit counter Sep 23, 2025
@tankf33der

Copy link
Copy Markdown
Contributor

Extensively tested. All passed.

Comment thread vlib/x/crypto/chacha20/chacha.v Outdated
Comment thread vlib/x/crypto/chacha20/chacha.v Outdated

@spytheman spytheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @blackshirt .

@spytheman spytheman merged commit e0d20a0 into vlang:master Sep 24, 2025
72 checks passed
@blackshirt

blackshirt commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

Thank you @blackshirt .

Thank you @spytheman for suggestion and reviews. There are patch in queue related to apply precomputation technique for 64-bit counter variant, like we do on 32-bit counter variant, but, i dont know, is this should be sent or not?. In my test, it improves a small of bit, but not too much.

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.

4 participants