diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2011-01-29 11:01:45 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2011-01-29 13:25:37 -0500 |
commit | 8115d60c323dd9931b95221c0a392aeddc1d6ef3 (patch) | |
tree | 13e75d71c5163c78efc9d8ab33988cc6c7be74a1 /tools/perf/builtin-top.c | |
parent | 8d50e5b4171a69cf48ca94a1e7c14033d0b4771d (diff) |
perf tools: Kill event_t typedef, use 'union perf_event' instead
And move the event_t methods to the perf_event__ too.
No code changes, just namespace consistency.
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Tom Zanussi <tzanussi@gmail.com>
LKML-Reference: <new-submission>
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 | 33 |
1 files changed, 17 insertions, 16 deletions
diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c index d923127b41b6..2f4d1f244be1 100644 --- a/tools/perf/builtin-top.c +++ b/tools/perf/builtin-top.c | |||
@@ -401,7 +401,7 @@ static void show_details(struct sym_entry *syme) | |||
401 | } | 401 | } |
402 | 402 | ||
403 | /* | 403 | /* |
404 | * Symbols will be added here in event__process_sample and will get out | 404 | * Symbols will be added here in perf_event__process_sample and will get out |
405 | * after decayed. | 405 | * after decayed. |
406 | */ | 406 | */ |
407 | static LIST_HEAD(active_symbols); | 407 | static LIST_HEAD(active_symbols); |
@@ -996,15 +996,15 @@ static int symbol_filter(struct map *map, struct symbol *sym) | |||
996 | return 0; | 996 | return 0; |
997 | } | 997 | } |
998 | 998 | ||
999 | static void event__process_sample(const event_t *self, | 999 | static void perf_event__process_sample(const union perf_event *event, |
1000 | struct perf_sample *sample, | 1000 | struct perf_sample *sample, |
1001 | struct perf_session *session) | 1001 | struct perf_session *session) |
1002 | { | 1002 | { |
1003 | u64 ip = self->ip.ip; | 1003 | u64 ip = event->ip.ip; |
1004 | struct sym_entry *syme; | 1004 | struct sym_entry *syme; |
1005 | struct addr_location al; | 1005 | struct addr_location al; |
1006 | struct machine *machine; | 1006 | struct machine *machine; |
1007 | u8 origin = self->header.misc & PERF_RECORD_MISC_CPUMODE_MASK; | 1007 | u8 origin = event->header.misc & PERF_RECORD_MISC_CPUMODE_MASK; |
1008 | 1008 | ||
1009 | ++samples; | 1009 | ++samples; |
1010 | 1010 | ||
@@ -1023,7 +1023,7 @@ static void event__process_sample(const event_t *self, | |||
1023 | break; | 1023 | break; |
1024 | case PERF_RECORD_MISC_GUEST_KERNEL: | 1024 | case PERF_RECORD_MISC_GUEST_KERNEL: |
1025 | ++guest_kernel_samples; | 1025 | ++guest_kernel_samples; |
1026 | machine = perf_session__find_machine(session, self->ip.pid); | 1026 | machine = perf_session__find_machine(session, event->ip.pid); |
1027 | break; | 1027 | break; |
1028 | case PERF_RECORD_MISC_GUEST_USER: | 1028 | case PERF_RECORD_MISC_GUEST_USER: |
1029 | ++guest_us_samples; | 1029 | ++guest_us_samples; |
@@ -1038,15 +1038,15 @@ static void event__process_sample(const event_t *self, | |||
1038 | 1038 | ||
1039 | if (!machine && perf_guest) { | 1039 | if (!machine && perf_guest) { |
1040 | pr_err("Can't find guest [%d]'s kernel information\n", | 1040 | pr_err("Can't find guest [%d]'s kernel information\n", |
1041 | self->ip.pid); | 1041 | event->ip.pid); |
1042 | return; | 1042 | return; |
1043 | } | 1043 | } |
1044 | 1044 | ||
1045 | if (self->header.misc & PERF_RECORD_MISC_EXACT_IP) | 1045 | if (event->header.misc & PERF_RECORD_MISC_EXACT_IP) |
1046 | exact_samples++; | 1046 | exact_samples++; |
1047 | 1047 | ||
1048 | if (event__preprocess_sample(self, session, &al, sample, | 1048 | if (perf_event__preprocess_sample(event, session, &al, sample, |
1049 | symbol_filter) < 0 || | 1049 | symbol_filter) < 0 || |
1050 | al.filtered) | 1050 | al.filtered) |
1051 | return; | 1051 | return; |
1052 | 1052 | ||
@@ -1108,15 +1108,15 @@ static void event__process_sample(const event_t *self, | |||
1108 | static void perf_session__mmap_read_cpu(struct perf_session *self, int cpu) | 1108 | static void perf_session__mmap_read_cpu(struct perf_session *self, int cpu) |
1109 | { | 1109 | { |
1110 | struct perf_sample sample; | 1110 | struct perf_sample sample; |
1111 | event_t *event; | 1111 | union perf_event *event; |
1112 | 1112 | ||
1113 | while ((event = perf_evlist__read_on_cpu(evsel_list, cpu)) != NULL) { | 1113 | while ((event = perf_evlist__read_on_cpu(evsel_list, cpu)) != NULL) { |
1114 | perf_session__parse_sample(self, event, &sample); | 1114 | perf_session__parse_sample(self, event, &sample); |
1115 | 1115 | ||
1116 | if (event->header.type == PERF_RECORD_SAMPLE) | 1116 | if (event->header.type == PERF_RECORD_SAMPLE) |
1117 | event__process_sample(event, &sample, self); | 1117 | perf_event__process_sample(event, &sample, self); |
1118 | else | 1118 | else |
1119 | event__process(event, &sample, self); | 1119 | perf_event__process(event, &sample, self); |
1120 | } | 1120 | } |
1121 | } | 1121 | } |
1122 | 1122 | ||
@@ -1199,9 +1199,10 @@ static int __cmd_top(void) | |||
1199 | return -ENOMEM; | 1199 | return -ENOMEM; |
1200 | 1200 | ||
1201 | if (target_tid != -1) | 1201 | if (target_tid != -1) |
1202 | event__synthesize_thread(target_tid, event__process, session); | 1202 | perf_event__synthesize_thread(target_tid, perf_event__process, |
1203 | session); | ||
1203 | else | 1204 | else |
1204 | event__synthesize_threads(event__process, session); | 1205 | perf_event__synthesize_threads(perf_event__process, session); |
1205 | 1206 | ||
1206 | start_counters(evsel_list); | 1207 | start_counters(evsel_list); |
1207 | first = list_entry(evsel_list->entries.next, struct perf_evsel, node); | 1208 | first = list_entry(evsel_list->entries.next, struct perf_evsel, node); |