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/xen/x86_64: Add missing segment alignment for xen
#1166
plat/xen/x86_64: Add missing segment alignment for xen
#1166
Conversation
xen
on x86_84xen
错误报告 阻止 |
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.
Hi, thanks for looking into this! Looks good! Just a couple of things 🤓:
- s/alignement/alignment/
- s/This fixes a memory overlap/This fixes a memory region overlap
- no "Merge" commits! You probably got this from an attempt to rebase 😃. In the future you can simply
git remote update; git pull origin staging --rebase
. Now you will have to force push the branch your PR is based on ☝️ .
b53e211
to
ecc0682
Compare
Make the second segment in a `xen/x86-64` ELF aligned to the page size. This fixes a memory region overlap issue that triggers an assert during memory coalescing. Signed-off-by: Andrei Stan <andreistan2003@gmail.com>
ecc0682
to
165b024
Compare
xen
xen
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.
Thanks for your work.
Reviewed-by: Sergiu Moga sergiu@unikraft.io
Approved-by: Sergiu Moga sergiu@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.
Reviewed-by: Razvan Deaconescu razvand@unikraft.io
Make the second segment in a `xen/x86-64` ELF aligned to the page size. This fixes a memory region overlap issue that triggers an assert during memory coalescing. Signed-off-by: Andrei Stan <andreistan2003@gmail.com> Approved-by: Sergiu Moga <sergiu@unikraft.io> Reviewed-by: Sergiu Moga <sergiu@unikraft.io> Reviewed-by: Razvan Deaconescu <razvand@unikraft.io> GitHub-Closes: #1166
Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;Base target
x86_64
xen
Additional configuration
Description of changes
Make the second segment in a
xen/x86-64
ELF aligned to the page size.This fixes a memory overlap issue that triggers an assert during
memory coalescing.