Skip to content

Update the uefi::allocator module to use the global system table #1343

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

Conversation

nicholasbishop
Copy link
Member

Explicitly initializing and shutting down the allocator is no longer needed, since all the internal operations use the global system table now. Mark allocator::init and allocator::exit_boot_services as deprecated, and update uefi::helper to stop calling those functions as well.

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-update-alloc-impl-2 branch 2 times, most recently from c569870 to 0c3424d Compare August 24, 2024 02:10
@phip1611 phip1611 added this pull request to the merge queue Aug 24, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Aug 24, 2024
The static memory type is now initialized at the time of the first allocation
rather than requiring a call to `init`.
These are no longer needed and will be deprecated in the following commit.
@nicholasbishop nicholasbishop force-pushed the bishop-update-alloc-impl-2 branch from 0c3424d to 85667fb Compare August 24, 2024 14:51
@nicholasbishop nicholasbishop added this pull request to the merge queue Aug 24, 2024
Merged via the queue into rust-osdev:main with commit 03dc5e9 Aug 24, 2024
14 checks passed
@nicholasbishop nicholasbishop deleted the bishop-update-alloc-impl-2 branch August 24, 2024 14:59
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