-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add kernel user dtb extraction #126
Conversation
9a78e33
to
17be369
Compare
4474c47
to
45c5d35
Compare
// Check if kernel is recent enough to have PTI support (backports for LTS releases are currently ignored) | ||
if (auto [major, minor, _patch] = extractKernelVersion(); major > 4 || (major == 4 && minor >= 15)) | ||
{ | ||
// Check if kernel page table isolation is enabled |
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.
This will fail poorly on ARM
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.
The project is not yet ARM aware, but I am open to alternative suggestions.
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.
Merging that functionality could be a good idea
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.
IIRC That PR still had open review comments. If you finish that PR and rebase it onto the newest version, I'd be happy to merge it.
91ab0e8
to
8bf67c5
Compare
8bf67c5
to
541e11e
Compare
[InMemoryScanner Plugin] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
[ApiTracing Plugin] Kudos, SonarCloud Quality Gate passed! |
[VMICore] Kudos, SonarCloud Quality Gate passed! |
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.
LGTM
No description provided.