diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2009-12-14 11:23:00 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-12-14 11:34:56 -0500 |
commit | c019879bcc5692ec9267c1cedad91f1794d0b693 (patch) | |
tree | ec499400b1ee0b1d93e961cc67b9dead9cd8c9a9 /tools/perf/builtin-kmem.c | |
parent | a328626b61aeda1a7d00a80c475c76ca1b815e0d (diff) |
perf session: Adopt the sample_type variable
All tools had copies, and perf diff would have to specify a
sample_type_check method just for copying it.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1260807780-19377-2-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/builtin-kmem.c')
-rw-r--r-- | tools/perf/builtin-kmem.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/tools/perf/builtin-kmem.c b/tools/perf/builtin-kmem.c index 3b329c66b505..dda60869faad 100644 --- a/tools/perf/builtin-kmem.c +++ b/tools/perf/builtin-kmem.c | |||
@@ -20,8 +20,6 @@ typedef int (*sort_fn_t)(struct alloc_stat *, struct alloc_stat *); | |||
20 | 20 | ||
21 | static char const *input_name = "perf.data"; | 21 | static char const *input_name = "perf.data"; |
22 | 22 | ||
23 | static u64 sample_type; | ||
24 | |||
25 | static int alloc_flag; | 23 | static int alloc_flag; |
26 | static int caller_flag; | 24 | static int caller_flag; |
27 | 25 | ||
@@ -321,7 +319,7 @@ static int process_sample_event(event_t *event, struct perf_session *session) | |||
321 | data.cpu = -1; | 319 | data.cpu = -1; |
322 | data.period = 1; | 320 | data.period = 1; |
323 | 321 | ||
324 | event__parse_sample(event, sample_type, &data); | 322 | event__parse_sample(event, session->sample_type, &data); |
325 | 323 | ||
326 | dump_printf("(IP, %d): %d/%d: %p period: %Ld\n", | 324 | dump_printf("(IP, %d): %d/%d: %p period: %Ld\n", |
327 | event->header.misc, | 325 | event->header.misc, |
@@ -344,11 +342,9 @@ static int process_sample_event(event_t *event, struct perf_session *session) | |||
344 | return 0; | 342 | return 0; |
345 | } | 343 | } |
346 | 344 | ||
347 | static int sample_type_check(u64 type, struct perf_session *session __used) | 345 | static int sample_type_check(struct perf_session *session) |
348 | { | 346 | { |
349 | sample_type = type; | 347 | if (!(session->sample_type & PERF_SAMPLE_RAW)) { |
350 | |||
351 | if (!(sample_type & PERF_SAMPLE_RAW)) { | ||
352 | fprintf(stderr, | 348 | fprintf(stderr, |
353 | "No trace sample to read. Did you call perf record " | 349 | "No trace sample to read. Did you call perf record " |
354 | "without -R?"); | 350 | "without -R?"); |