feat: add hv_end address output and clarify hvisor memory layout #224
Conversation
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
Contributor
Author
|
Based on our discussion today, I plan to add a I have converted this PR to Draft status for now. I will mark it as Ready for review once the implementation is complete. |
This commit enhances the memory management of hvisor by adding hv_end address calculation and output, providing a clear definition of the hvisor memory layout and reserved memory regions. Key changes: - Enhanced documentation with comments on all memory boundary functions - Introduced `HV_EXTENDED_SIZE` constant representing the total additional memory required beyond the core static area - Exposed `HV_EXTENDED_SIZE` as a global symbol for linker accessibility - Added compilation output of (START_ADDR END_ADDR) to clearly indicate the reserved memory region for hvisor This implementation provides a clear and comprehensive view of the hvisor's memory requirements, making it easier to configure memory reservations for hvisor in different deployment scenarios.
enkerewpo
approved these changes
Dec 11, 2025
Member
|
LGTM |
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.
This commit enhances the memory management of hvisor by adding hv_end address calculation and output, providing a clear definition of the hvisor memory layout and reserved memory regions.
Key changes:
HV_EXTENDED_SIZEconstant representing the total additional memory required beyond the core static areaHV_EXTENDED_SIZEas a global symbol for linker accessibilityThis implementation provides a clear and comprehensive view of the hvisor's memory requirements, making it easier to configure memory reservations for hvisor in different deployment scenarios.
Closes: #223