aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf/builtin-top.c
diff options
context:
space:
mode:
authorJiri Olsa <jolsa@redhat.com>2013-10-15 10:27:32 -0400
committerArnaldo Carvalho de Melo <acme@redhat.com>2013-10-21 16:33:24 -0400
commitf5fc14124c5cefdd052a2b2a6a3f0ed531540113 (patch)
tree9cef9b2a75334478486869ff002749679a9e488d /tools/perf/builtin-top.c
parent09600e0f9ebb06235b852a646a3644b7d4a71aca (diff)
perf tools: Add data object to handle perf data file
This patch is adding 'struct perf_data_file' object as a placeholder for all attributes regarding perf.data file handling. Changing perf_session__new to take it as an argument. The rest of the functionality will be added later to keep this change simple enough, because all the places using perf_session are changed now. Signed-off-by: Jiri Olsa <jolsa@redhat.com> Acked-by: Namhyung Kim <namhyung@kernel.org> Cc: Adrian Hunter <adrian.hunter@intel.com> Cc: Andi Kleen <andi@firstfloor.org> 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/1381847254-28809-2-git-send-email-jolsa@redhat.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-top.c')
-rw-r--r--tools/perf/builtin-top.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c
index 65c49b2f51c1..752bebeac3aa 100644
--- a/tools/perf/builtin-top.c
+++ b/tools/perf/builtin-top.c
@@ -929,11 +929,15 @@ static int __cmd_top(struct perf_top *top)
929 struct perf_record_opts *opts = &top->record_opts; 929 struct perf_record_opts *opts = &top->record_opts;
930 pthread_t thread; 930 pthread_t thread;
931 int ret; 931 int ret;
932 struct perf_data_file file = {
933 .mode = PERF_DATA_MODE_WRITE,
934 };
935
932 /* 936 /*
933 * FIXME: perf_session__new should allow passing a O_MMAP, so that all this 937 * FIXME: perf_session__new should allow passing a O_MMAP, so that all this
934 * mmap reading, etc is encapsulated in it. Use O_WRONLY for now. 938 * mmap reading, etc is encapsulated in it. Use O_WRONLY for now.
935 */ 939 */
936 top->session = perf_session__new(NULL, O_WRONLY, false, false, NULL); 940 top->session = perf_session__new(&file, false, NULL);
937 if (top->session == NULL) 941 if (top->session == NULL)
938 return -ENOMEM; 942 return -ENOMEM;
939 943