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
arch/x86_64: Paging code - Enclose params of function-like macros in brackets #987
Conversation
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, @cppstef , please follow the guidelines on formatting your commit. Please also add a detailed description to motivate your changes.
✅ 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.
Nice work, @cppstef
Please sign off with your real email address in the commit message. For now, it is something default, provided by Github.
You can find more information here about how to do that.
Enclose parameters of function-like macros in brackets in x86_64 paging code to avoid potential unwanted operator precedence consequences after macro expansion. Signed-off-by: Cosmin Popa <cosmin7popa@gmail.com>
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.
All good from my side.
Reviewed-by: Radu Nichita radunichita99@gmail.com
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.
Looks good to me. Thank you!
Reviewed-by: Florin Postolache florin.postolache.of@gmail.com
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
Minor improvement in paging code: enclosed params of function-like macros in brackets to account for expressions that mess with operator precedence where the param is used within the macro.