-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
plat/common: Increase granularity of hardcoded legacy high memory #1051
Conversation
aa58500
to
d23ea00
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work in all configurations I tested (QEMU/Firecracker).
Reviewed-by: Marco Schlumpp marco@unikraft.io
To ensure compatibility with boot protocols that do report the BIOS System Memory region as a reserved region, split the memory region inserted by `ukplat_memregion_list_insert_legacy_hi_mem` into two memory region descriptors: one to contain known memory holes (e.g. VGA Text Mode framebuffer) and one to contain the previously mentioned BIOS System Memory region. The former will have read/write permissions to ensure that zones like the VGA framebuffer or certain PCI BARs mapped by the BIOS are writable, while the latter will only have read permissions to ensure compatibility with other boot protocols. Signed-off-by: Sergiu Moga <sergiu@unikraft.io>
d23ea00
to
b77dbd9
Compare
✅ Checkpatch passed Beep boop! I ran Unikraft's
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed-by: Michalis Pappas michalis@unikraft.io
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved-by: Razvan Deaconescu razvand@unikraft.io
To ensure compatibility with boot protocols that do report the BIOS System Memory region as a reserved region, split the memory region inserted by
ukplat_memregion_list_insert_legacy_hi_mem
into two memory region descriptors: one to contain known memory holes (e.g. VGA Text Mode framebuffer) and one to contain the previously mentioned BIOS System Memory region. The former will have read/write permissions to ensure that zones like the VGA framebuffer or certain PCI BARs mapped by the BIOS are writable, while the latter will only have read permissions to ensure compatibility with other boot protocols.Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;Base target
x86_64
or N/A]kvm
,xen
or N/A]app-python3
or N/A]Additional configuration
Description of changes