-
Notifications
You must be signed in to change notification settings - Fork 266
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 tracefs mount point select function #471
Conversation
✅ Deploy Preview for aya-rs-docs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
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 working on this! Just a few minor comments
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!
@alessandrod are you happy with the current state of this PR?
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.
Can you please rebase on main so we can get CI green?
348e255
to
48fdf5a
Compare
done |
This PR would to improve the access to
tracefs
.This is implemented with a single check to determine if the
tracefs
is mounted or not. It checks in two known locations, in order of priority:/sys/kernel/tracing
/sys/kernel/debug/tracing
Most of the systems have both
tracefs
anddebugfs
mounted, but in some hardened systems the kernel is compiled withoutdebugfs
or simply not mounted. Android documentation for example has a warning for debugfs in user builds.