Skip to content

Replace Xoshiro with ChaCha in kickmix simulator - #1978

Merged
tanujkhattar merged 2 commits into
quantumlib:mainfrom
tanujkhattar:chacha8_csprng
Oct 16, 2026
Merged

tanujkhattar merged 2 commits into
quantumlib:mainfrom
tanujkhattar:chacha8_csprng

Conversation

@tanujkhattar

Copy link
Copy Markdown
Collaborator

Use a CSPRNG in kickmix simulator.

@tanujkhattar
tanujkhattar requested a review from Strilanc October 15, 2026 21:23
@tanujkhattar tanujkhattar changed the title Replace Xoshiro with ChaCha Replace Xoshiro with ChaCha in kickmix simulator Oct 15, 2026

@Strilanc Strilanc 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.

LGTM % add references for where the test vectors come from

}
rng.buffer_idx = 16;

const uint32_t expected_block[16] = {

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.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done

Comment thread kickmix/src/kickmix/util/chacha.test.cc Outdated
rng.buffer_idx = 16;

const uint32_t expected_block[16] = {
0xfb9dadee,

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.

Where do these come from?

Comment thread kickmix/src/kickmix/util/chacha.test.cc Outdated

namespace {
constexpr uint32_t RFC8439_INIT_WORDS[16] = {
0x61707865,

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.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done

@tanujkhattar
tanujkhattar added this pull request to the merge queue Oct 16, 2026
Merged via the queue into quantumlib:main with commit ae4cb0e Oct 16, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants