-
Notifications
You must be signed in to change notification settings - Fork 480
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
configure: Check /proc/sys/kernel/perf_event_paranoid #1847
base: master
Are you sure you want to change the base?
Conversation
If there's any better idea, please let me know. |
In order to get kernel information with uftrace without any problem, I think /proc/sys/kernel/perf_event_paranoid should be 2 or less. But many distros nowadays set this value to more than 3 as default. I wanted to inform users about this in the docs, but it's so general that it's unclear which docs to leave this information in. So this commit suggests checking this information when configuring uftrace. Closes: namhyung#1424 Signed-off-by: Kang Minchul <[email protected]>
f39af38
to
7e16413
Compare
Showing the value when I think it'd be better if
|
I think I would like it to have compile time and run time feature tests separately. |
Thank you for your detailed feedbacks, |
Maybe |
In order to get kernel information with uftrace without any problem, I think /proc/sys/kernel/perf_event_paranoid should be 2 or less. But many distros nowadays set this value to more than 3 as default.
I wanted to inform users about this in the docs, but it's so general that it's unclear which docs to leave this information in.
So this commit suggests checking this information
when configuring uftrace.
Closes: #1424