-
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: Align TLS sections to PAGE_SIZE #1050
Conversation
`mkbootinfo.py` takes each ELF segment and aligns it to PAGE_SIZE to convert it to a `struct ukplat_memregion_desc`. However, this ends up generating overlapping memregion descriptors in the case of the ELF segment corresponding to the TLS sections. To solve this, simply ensure that the TLS sections are already aligned by PAGE_SIZE. Signed-off-by: Sergiu Moga <sergiu@unikraft.io>
ede325e
to
db9814a
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.
Reviewed-by: Michalis Pappas michalis@unikraft.io
✅ 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: Marco Schlumpp marco@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
mkbootinfo.py
takes each ELF segment and aligns it to PAGE_SIZE to convert it to astruct ukplat_memregion_desc
. However, this ends up generating overlapping memregion descriptors in the case of the ELF segment corresponding to the TLS sections.To solve this, simply ensure that the TLS sections are already aligned by PAGE_SIZE.
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