diff options
author | Jiri Olsa <jolsa@redhat.com> | 2013-10-15 10:27:32 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2013-10-21 16:33:24 -0400 |
commit | f5fc14124c5cefdd052a2b2a6a3f0ed531540113 (patch) | |
tree | 9cef9b2a75334478486869ff002749679a9e488d /tools/perf/builtin-top.c | |
parent | 09600e0f9ebb06235b852a646a3644b7d4a71aca (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.c | 6 |
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 | ||