diff options
author | Jiri Olsa <jolsa@redhat.com> | 2013-09-01 06:36:12 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-10-09 10:24:10 -0400 |
commit | 994a1f78b191df0c9d6caca3f3afb03e247aff26 (patch) | |
tree | 387ca6391f946bbbca73a579105ee0bc2ba76ad8 /tools/perf/builtin-trace.c | |
parent | f37376cd721a539ac398cbb7718b72fce83cd49b (diff) |
perf tools: Check mmap pages value early
Move the check of the mmap_pages value to the options parsing time, so
we could rely on this value on other parts of code.
Related changes come in the next patches.
Also changes perf_evlist::mmap_len to proper size_t type.
Signed-off-by: Jiri Olsa <jolsa@redhat.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1378031796-17892-2-git-send-email-jolsa@redhat.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-trace.c')
-rw-r--r-- | tools/perf/builtin-trace.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c index be658433d3bb..f3ddbb008fb4 100644 --- a/tools/perf/builtin-trace.c +++ b/tools/perf/builtin-trace.c | |||
@@ -1528,8 +1528,9 @@ int cmd_trace(int argc, const char **argv, const char *prefix __maybe_unused) | |||
1528 | "list of cpus to monitor"), | 1528 | "list of cpus to monitor"), |
1529 | OPT_BOOLEAN(0, "no-inherit", &trace.opts.no_inherit, | 1529 | OPT_BOOLEAN(0, "no-inherit", &trace.opts.no_inherit, |
1530 | "child tasks do not inherit counters"), | 1530 | "child tasks do not inherit counters"), |
1531 | OPT_UINTEGER('m', "mmap-pages", &trace.opts.mmap_pages, | 1531 | OPT_CALLBACK('m', "mmap-pages", &trace.opts.mmap_pages, "pages", |
1532 | "number of mmap data pages"), | 1532 | "number of mmap data pages", |
1533 | perf_evlist__parse_mmap_pages), | ||
1533 | OPT_STRING('u', "uid", &trace.opts.target.uid_str, "user", | 1534 | OPT_STRING('u', "uid", &trace.opts.target.uid_str, "user", |
1534 | "user to profile"), | 1535 | "user to profile"), |
1535 | OPT_CALLBACK(0, "duration", &trace, "float", | 1536 | OPT_CALLBACK(0, "duration", &trace, "float", |