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

[csrng/doc] Clarify description section of README.md #26005

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

vogelpi
Copy link
Contributor

@vogelpi vogelpi commented Jan 24, 2025

Previously, this section:

  • was referring to a derivation function which our implementation doesn't use,
  • mentioned and undefined CSRNG_MAX_GENERATE constant,
  • mentioned the support of a prediction resistance flag for generate calls (only relevant for instantiate and reseed commands)
  • mentioned the update routine only in the context of generate commands (it's used by any single command call and can be called explicitly).

Previously, this section:
- was referring to a derivation function which our implementation
  doesn't use,
- mentioned and undefined CSRNG_MAX_GENERATE constant,
- mentioned the support of a prediction resistance flag for generate
  calls (only relevant for instantiate and reseed commands)
- mentioned the update routine only in the context of generate commands
  (it's used by any single command call and can be called explicitly).

Signed-off-by: Pirmin Vogel <vogelpi@lowrisc.org>
@vogelpi vogelpi added the CherryPick:earlgrey_1.0.0 This PR should be cherry-picked to earlgrey_1.0.0 label Jan 24, 2025
@vogelpi vogelpi merged commit 3107903 into lowRISC:master Jan 24, 2025
40 checks passed
Copy link

Successfully created backport PR for earlgrey_1.0.0:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPick:earlgrey_1.0.0 This PR should be cherry-picked to earlgrey_1.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants