-
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/kvm: Check return code of all memory region inserting methods #1052
Conversation
f84dfa3
to
9f84e66
Compare
9f84e66
to
7eac436
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: Marco Schlumpp marco@unikraft.io
@michpappas Do not merge this until the prerequisite PR is in! |
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
That would be @razvand :) |
Make sure that we do check the return codes of all of the memory region inserting methods so that we can crash in case of failure, instead of letting the system run with a corrupted state. Signed-off-by: Sergiu Moga <sergiu@unikraft.io>
7eac436
to
4698612
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.
Approved-by: Razvan Deaconescu razvand@unikraft.io
✅ Checkpatch passed Beep boop! I ran Unikraft's
|
Make sure that we do check the return codes of all of the memory region inserting methods so that we can crash in case of failure, instead of letting the system run with a corrupted state.
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