Messages in this thread | | | Date | Mon, 02 Dec 2013 11:30:00 -0700 | From | David Ahern <> | Subject | Re: [PATCH] perf tools: Make trace command to follow -m option |
| |
On 11/28/13, 9:57 AM, Jiri Olsa wrote: > Currently trace command supports '-m' option, but > does not follow its value and keeps the default. > > Changing the perf_evlist__mmap function call to use > the '-m' configured value. > > Signed-off-by: Jiri Olsa <jolsa@redhat.com> > Cc: Corey Ashford <cjashfor@linux.vnet.ibm.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> > Cc: Arnaldo Carvalho de Melo <acme@redhat.com> > Cc: David Ahern <dsahern@gmail.com> > --- > tools/perf/builtin-trace.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c > index e9f345e2..9f2a242 100644 > --- a/tools/perf/builtin-trace.c > +++ b/tools/perf/builtin-trace.c > @@ -1890,7 +1890,7 @@ static int trace__run(struct trace *trace, int argc, const char **argv) > if (err < 0) > goto out_error_open; > > - err = perf_evlist__mmap(evlist, UINT_MAX, false); > + err = perf_evlist__mmap(evlist, trace->opts.mmap_pages, false); > if (err < 0) { > fprintf(trace->output, "Couldn't mmap the events: %s\n", strerror(errno)); > goto out_close_evlist; >
Acked-by: David Ahern <dsahern@gmail.com>
| |