Skip to content

Commit

Permalink
Suggest /etc/sysctl.d instead of /etc/sysctl.conf
Browse files Browse the repository at this point in the history
On systemd systems, the latter is silently and confusingly ignored (see,
approximately, systemd/systemd#12791). The
former is compatible with non-systemd systems according to man 5
sysctl.conf, so we should suggest it instead as it works everywhere.
  • Loading branch information
lf- authored and rocallahan committed Apr 17, 2021
1 parent 165f85a commit 7cf2f0f
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/RecordSession.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2142,8 +2142,9 @@ static string lookup_by_path(const string& name) {
fprintf(stderr,
"rr needs /proc/sys/kernel/perf_event_paranoid <= 1, but it is %d.\n"
"Change it to 1, or use 'rr record -n' (slow).\n"
"Consider putting 'kernel.perf_event_paranoid = 1' in /etc/sysctl.conf.\n"
"See 'man 8 sysctl', 'man 5 sysctl.d' and 'man 5 sysctl.conf' for more details.\n",
"Consider putting 'kernel.perf_event_paranoid = 1' in /etc/sysctl.d/10-rr.conf.\n"
"See 'man 8 sysctl', 'man 5 sysctl.d' (systemd systems)\n"
"and 'man 5 sysctl.conf' (non-systemd systems) for more details.\n",
val);
exit(1);
}
Expand Down

0 comments on commit 7cf2f0f

Please sign in to comment.