Draft
Conversation
The CI script requires it to be in our source tree, so we should ignore it.
The split virtqueue implementation previously still used a Box around the token ring items. The array the token ring is based on is howevery already on the heap (in another Box). To avoid having to allocate for individual items, only allocating the ring array up-front, remove the Box around the items. To successfully recover memory from the balloon device in an OOM situation, virtqueue submission for a deflate request must not allocate. During OOM handling the global allocator is already locked, so this would deadlock.
Talc previously did not support allocations without attempting to use the OOM handler to recover. For the balloon device to allocate pages for the balloon however we want to just probe for and then allocate a chunk of pages. If the allocation fails we don't want to recover old memory, we just want to allocate a smaller chunk (since recovery with the balloon will be shrinking the balloon, causing us to uselessly shuffle around pages, possibly even failing). See SFBdragon/talc#43 which is the PR for merging this modification upstream. Another API function may be added for querying the current largest possible new allocation in the future. In that case the balloon driver code should be changed to use that instead of probing allocatable size in powers of two.
Talc can optionally collect and expose some basic statistics about allocations. Provide a feature that regularly prints these statistics via another executor tassk.
TODO: explanatory commit message
hcsch
force-pushed
the
add-virtio-balloon
branch
from
97d40e1 to
9ae5b2c
Compare
This was referenced Oct 27, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description and more explanatory comments are to follow