diff options
author | Adrian Hunter <adrian.hunter@intel.com> | 2013-12-09 08:18:40 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-12-09 13:17:23 -0500 |
commit | 53653d70a0784a997748cc9e315ddf19d310e812 (patch) | |
tree | 64e52db9c8a062a34dde7bd2315aa5c4f4691a1b | |
parent | cc8fae1d81648e85587f5d18b4f93e0b771fb02d (diff) |
perf record: Fix display of incorrect mmap pages
'mmap_pages' is 'unsigned int' not 'int' e.g.
perf record -m2147483648 uname
Permission error mapping pages.
Consider increasing /proc/sys/kernel/perf_event_mlock_kb,
or try again with a smaller value of -m/--mmap_pages.
(current value: -2147483648)
Fixed:
perf record -m2147483648 uname
Permission error mapping pages.
Consider increasing /proc/sys/kernel/perf_event_mlock_kb,
or try again with a smaller value of -m/--mmap_pages.
(current value: 2147483648)
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Acked-by: Jiri Olsa <jolsa@redhat.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Namhyung Kim <namhyung@gmail.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1386595120-22978-5-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/builtin-record.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index d93e2eef0979..c1c1200d2f0a 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c | |||
@@ -224,7 +224,7 @@ try_again: | |||
224 | "Consider increasing " | 224 | "Consider increasing " |
225 | "/proc/sys/kernel/perf_event_mlock_kb,\n" | 225 | "/proc/sys/kernel/perf_event_mlock_kb,\n" |
226 | "or try again with a smaller value of -m/--mmap_pages.\n" | 226 | "or try again with a smaller value of -m/--mmap_pages.\n" |
227 | "(current value: %d)\n", opts->mmap_pages); | 227 | "(current value: %u)\n", opts->mmap_pages); |
228 | rc = -errno; | 228 | rc = -errno; |
229 | } else { | 229 | } else { |
230 | pr_err("failed to mmap with %d (%s)\n", errno, strerror(errno)); | 230 | pr_err("failed to mmap with %d (%s)\n", errno, strerror(errno)); |