Skip to content

runtime: Add freestanding functions to get/set/delete UEFI variables #1250

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

Merged
merged 3 commits into from
Jul 21, 2024

Conversation

nicholasbishop
Copy link
Member

The new version of get_variable returns the required size in the error data if the input buffer is too small. This allows get_variable_boxed to use make_boxed, and also makes get_variable_size unnecessary.

Also:

  • Added more info about errors to the docstrings.
  • impl Align for [u8] to allow make_boxed to work here.

(Note: variable_keys function to come in a later PR.)

Checklist

  • Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
  • Update the changelog (if necessary)

@nicholasbishop nicholasbishop force-pushed the bishop-runtime-mod-vars branch from 698b597 to 5cbdb12 Compare July 19, 2024 22:44
This will allow `make_boxed` to be used in `get_variable_boxed`.
@nicholasbishop nicholasbishop force-pushed the bishop-runtime-mod-vars branch from 5cbdb12 to 800af86 Compare July 19, 2024 22:45
The new version of `get_variable` returns the required size in the error data if
the input buffer is too small. This allows `get_variable_boxed` to use
`make_boxed`, and also makes `get_variable_size` unnecessary.

Also added more info about errors to the docstrings.

(Note: `variable_keys` function to come in a later commit.)
@nicholasbishop nicholasbishop force-pushed the bishop-runtime-mod-vars branch from 800af86 to 3fe8b6b Compare July 19, 2024 22:48
@phip1611 phip1611 added this pull request to the merge queue Jul 21, 2024
Merged via the queue into main with commit 23eb0e8 Jul 21, 2024
15 checks passed
@phip1611 phip1611 deleted the bishop-runtime-mod-vars branch July 21, 2024 07:17
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