Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 1 | /* |
| 2 | * builtin-annotate.c |
| 3 | * |
| 4 | * Builtin annotate command: Analyze the perf.data input file, |
| 5 | * look up and read DSOs and symbol information and display |
| 6 | * a histogram of results, along various sorting keys. |
| 7 | */ |
| 8 | #include "builtin.h" |
| 9 | |
| 10 | #include "util/util.h" |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 11 | #include "util/color.h" |
Arnaldo Carvalho de Melo | 5da5025 | 2009-07-01 14:46:08 -0300 | [diff] [blame] | 12 | #include <linux/list.h> |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 13 | #include "util/cache.h" |
Arnaldo Carvalho de Melo | 43cbcd8 | 2009-07-01 12:28:37 -0300 | [diff] [blame] | 14 | #include <linux/rbtree.h> |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 15 | #include "util/symbol.h" |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 16 | |
| 17 | #include "perf.h" |
Frederic Weisbecker | 8f28827 | 2009-08-16 22:05:48 +0200 | [diff] [blame] | 18 | #include "util/debug.h" |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 19 | |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 20 | #include "util/evlist.h" |
| 21 | #include "util/evsel.h" |
Arnaldo Carvalho de Melo | 78f7def | 2011-02-04 09:45:46 -0200 | [diff] [blame] | 22 | #include "util/annotate.h" |
Arnaldo Carvalho de Melo | 62daacb | 2009-11-27 16:29:22 -0200 | [diff] [blame] | 23 | #include "util/event.h" |
Josh Poimboeuf | 4b6ab94 | 2015-12-15 09:39:39 -0600 | [diff] [blame] | 24 | #include <subcmd/parse-options.h> |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 25 | #include "util/parse-events.h" |
Frederic Weisbecker | 6baa0a5 | 2009-08-14 12:21:53 +0200 | [diff] [blame] | 26 | #include "util/thread.h" |
John Kacur | dd68ada | 2009-09-24 18:02:49 +0200 | [diff] [blame] | 27 | #include "util/sort.h" |
John Kacur | 3d1d07e | 2009-09-28 15:32:55 +0200 | [diff] [blame] | 28 | #include "util/hist.h" |
Arnaldo Carvalho de Melo | 94c744b | 2009-12-11 21:24:02 -0200 | [diff] [blame] | 29 | #include "util/session.h" |
Arnaldo Carvalho de Melo | 45694aa | 2011-11-28 08:30:20 -0200 | [diff] [blame] | 30 | #include "util/tool.h" |
Jiri Olsa | f5fc141 | 2013-10-15 16:27:32 +0200 | [diff] [blame] | 31 | #include "util/data.h" |
Irina Tirdea | 68e94f4 | 2012-10-16 02:33:38 +0300 | [diff] [blame] | 32 | #include "arch/common.h" |
Peter Zijlstra | 70fbe05 | 2016-09-05 16:08:12 -0300 | [diff] [blame] | 33 | #include "util/block-range.h" |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 34 | |
Namhyung Kim | fc67297 | 2013-09-13 15:27:43 +0900 | [diff] [blame] | 35 | #include <dlfcn.h> |
Anton Blanchard | 5d67be9 | 2011-07-04 21:57:50 +1000 | [diff] [blame] | 36 | #include <linux/bitmap.h> |
| 37 | |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 38 | struct perf_annotate { |
Arnaldo Carvalho de Melo | 45694aa | 2011-11-28 08:30:20 -0200 | [diff] [blame] | 39 | struct perf_tool tool; |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 40 | struct perf_session *session; |
| 41 | bool use_tui, use_stdio, use_gtk; |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 42 | bool full_paths; |
| 43 | bool print_line; |
Namhyung Kim | 18c9e5c | 2013-02-07 18:02:14 +0900 | [diff] [blame] | 44 | bool skip_missing; |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 45 | const char *sym_hist_filter; |
| 46 | const char *cpu_list; |
| 47 | DECLARE_BITMAP(cpu_bitmap, MAX_NR_CPUS); |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 48 | }; |
Anton Blanchard | 5d67be9 | 2011-07-04 21:57:50 +1000 | [diff] [blame] | 49 | |
Peter Zijlstra | 70fbe05 | 2016-09-05 16:08:12 -0300 | [diff] [blame] | 50 | /* |
| 51 | * Given one basic block: |
| 52 | * |
| 53 | * from to branch_i |
| 54 | * * ----> * |
| 55 | * | |
| 56 | * | block |
| 57 | * v |
| 58 | * * ----> * |
| 59 | * from to branch_i+1 |
| 60 | * |
| 61 | * where the horizontal are the branches and the vertical is the executed |
| 62 | * block of instructions. |
| 63 | * |
| 64 | * We count, for each 'instruction', the number of blocks that covered it as |
| 65 | * well as count the ratio each branch is taken. |
| 66 | * |
| 67 | * We can do this without knowing the actual instruction stream by keeping |
| 68 | * track of the address ranges. We break down ranges such that there is no |
| 69 | * overlap and iterate from the start until the end. |
| 70 | * |
| 71 | * @acme: once we parse the objdump output _before_ processing the samples, |
| 72 | * we can easily fold the branch.cycles IPC bits in. |
| 73 | */ |
| 74 | static void process_basic_block(struct addr_map_symbol *start, |
| 75 | struct addr_map_symbol *end, |
| 76 | struct branch_flags *flags) |
| 77 | { |
| 78 | struct symbol *sym = start->sym; |
| 79 | struct annotation *notes = sym ? symbol__annotation(sym) : NULL; |
| 80 | struct block_range_iter iter; |
| 81 | struct block_range *entry; |
| 82 | |
| 83 | /* |
| 84 | * Sanity; NULL isn't executable and the CPU cannot execute backwards |
| 85 | */ |
| 86 | if (!start->addr || start->addr > end->addr) |
| 87 | return; |
| 88 | |
| 89 | iter = block_range__create(start->addr, end->addr); |
| 90 | if (!block_range_iter__valid(&iter)) |
| 91 | return; |
| 92 | |
| 93 | /* |
| 94 | * First block in range is a branch target. |
| 95 | */ |
| 96 | entry = block_range_iter(&iter); |
| 97 | assert(entry->is_target); |
| 98 | entry->entry++; |
| 99 | |
| 100 | do { |
| 101 | entry = block_range_iter(&iter); |
| 102 | |
| 103 | entry->coverage++; |
| 104 | entry->sym = sym; |
| 105 | |
| 106 | if (notes) |
| 107 | notes->max_coverage = max(notes->max_coverage, entry->coverage); |
| 108 | |
| 109 | } while (block_range_iter__next(&iter)); |
| 110 | |
| 111 | /* |
| 112 | * Last block in rage is a branch. |
| 113 | */ |
| 114 | entry = block_range_iter(&iter); |
| 115 | assert(entry->is_branch); |
| 116 | entry->taken++; |
| 117 | if (flags->predicted) |
| 118 | entry->pred++; |
| 119 | } |
| 120 | |
| 121 | static void process_branch_stack(struct branch_stack *bs, struct addr_location *al, |
| 122 | struct perf_sample *sample) |
| 123 | { |
| 124 | struct addr_map_symbol *prev = NULL; |
| 125 | struct branch_info *bi; |
| 126 | int i; |
| 127 | |
| 128 | if (!bs || !bs->nr) |
| 129 | return; |
| 130 | |
| 131 | bi = sample__resolve_bstack(sample, al); |
| 132 | if (!bi) |
| 133 | return; |
| 134 | |
| 135 | for (i = bs->nr - 1; i >= 0; i--) { |
| 136 | /* |
| 137 | * XXX filter against symbol |
| 138 | */ |
| 139 | if (prev) |
| 140 | process_basic_block(prev, &bi[i].from, &bi[i].flags); |
| 141 | prev = &bi[i].to; |
| 142 | } |
| 143 | |
| 144 | free(bi); |
| 145 | } |
| 146 | |
Arnaldo Carvalho de Melo | d04b35f | 2011-11-11 22:17:32 -0200 | [diff] [blame] | 147 | static int perf_evsel__add_sample(struct perf_evsel *evsel, |
Namhyung Kim | fd36f3d | 2015-12-23 02:06:58 +0900 | [diff] [blame] | 148 | struct perf_sample *sample, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 149 | struct addr_location *al, |
| 150 | struct perf_annotate *ann) |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 151 | { |
Arnaldo Carvalho de Melo | 4ea062ed | 2014-10-09 13:13:41 -0300 | [diff] [blame] | 152 | struct hists *hists = evsel__hists(evsel); |
Arnaldo Carvalho de Melo | 628ada0 | 2010-02-25 12:57:40 -0300 | [diff] [blame] | 153 | struct hist_entry *he; |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 154 | int ret; |
Arnaldo Carvalho de Melo | 628ada0 | 2010-02-25 12:57:40 -0300 | [diff] [blame] | 155 | |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 156 | if (ann->sym_hist_filter != NULL && |
| 157 | (al->sym == NULL || |
| 158 | strcmp(ann->sym_hist_filter, al->sym->name) != 0)) { |
Arnaldo Carvalho de Melo | 628ada0 | 2010-02-25 12:57:40 -0300 | [diff] [blame] | 159 | /* We're only interested in a symbol named sym_hist_filter */ |
Arnaldo Carvalho de Melo | facf3f0 | 2015-05-25 15:30:09 -0300 | [diff] [blame] | 160 | /* |
| 161 | * FIXME: why isn't this done in the symbol_filter when loading |
| 162 | * the DSO? |
| 163 | */ |
Arnaldo Carvalho de Melo | 628ada0 | 2010-02-25 12:57:40 -0300 | [diff] [blame] | 164 | if (al->sym != NULL) { |
| 165 | rb_erase(&al->sym->rb_node, |
| 166 | &al->map->dso->symbols[al->map->type]); |
| 167 | symbol__delete(al->sym); |
Arnaldo Carvalho de Melo | c0b4dff | 2015-08-24 13:33:14 -0300 | [diff] [blame] | 168 | dso__reset_find_symbol_cache(al->map->dso); |
Arnaldo Carvalho de Melo | 628ada0 | 2010-02-25 12:57:40 -0300 | [diff] [blame] | 169 | } |
| 170 | return 0; |
| 171 | } |
| 172 | |
Peter Zijlstra | 70fbe05 | 2016-09-05 16:08:12 -0300 | [diff] [blame] | 173 | /* |
| 174 | * XXX filtered samples can still have branch entires pointing into our |
| 175 | * symbol and are missed. |
| 176 | */ |
| 177 | process_branch_stack(sample->branch_stack, al, sample); |
| 178 | |
Namhyung Kim | fd36f3d | 2015-12-23 02:06:58 +0900 | [diff] [blame] | 179 | sample->period = 1; |
| 180 | sample->weight = 1; |
| 181 | |
Jiri Olsa | 0102ef3 | 2016-06-14 20:19:21 +0200 | [diff] [blame] | 182 | he = hists__add_entry(hists, al, NULL, NULL, NULL, sample, true); |
Arnaldo Carvalho de Melo | 9735abf | 2009-10-03 10:42:45 -0300 | [diff] [blame] | 183 | if (he == NULL) |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 184 | return -ENOMEM; |
Arnaldo Carvalho de Melo | 628ada0 | 2010-02-25 12:57:40 -0300 | [diff] [blame] | 185 | |
Arnaldo Carvalho de Melo | 00e5521 | 2013-12-18 15:46:32 -0300 | [diff] [blame] | 186 | ret = hist_entry__inc_addr_samples(he, evsel->idx, al->addr); |
Arnaldo Carvalho de Melo | 4ea062ed | 2014-10-09 13:13:41 -0300 | [diff] [blame] | 187 | hists__inc_nr_samples(hists, true); |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 188 | return ret; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 189 | } |
| 190 | |
Arnaldo Carvalho de Melo | 45694aa | 2011-11-28 08:30:20 -0200 | [diff] [blame] | 191 | static int process_sample_event(struct perf_tool *tool, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 192 | union perf_event *event, |
Arnaldo Carvalho de Melo | 8115d60 | 2011-01-29 14:01:45 -0200 | [diff] [blame] | 193 | struct perf_sample *sample, |
Arnaldo Carvalho de Melo | 9e69c21 | 2011-03-15 15:44:01 -0300 | [diff] [blame] | 194 | struct perf_evsel *evsel, |
Arnaldo Carvalho de Melo | 743eb86 | 2011-11-28 07:56:39 -0200 | [diff] [blame] | 195 | struct machine *machine) |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 196 | { |
Arnaldo Carvalho de Melo | 45694aa | 2011-11-28 08:30:20 -0200 | [diff] [blame] | 197 | struct perf_annotate *ann = container_of(tool, struct perf_annotate, tool); |
Arnaldo Carvalho de Melo | 1ed091c | 2009-11-27 16:29:23 -0200 | [diff] [blame] | 198 | struct addr_location al; |
Arnaldo Carvalho de Melo | b91fc39 | 2015-04-06 20:43:22 -0300 | [diff] [blame] | 199 | int ret = 0; |
Frederic Weisbecker | 6baa0a5 | 2009-08-14 12:21:53 +0200 | [diff] [blame] | 200 | |
Arnaldo Carvalho de Melo | bb3eb56 | 2016-03-22 18:39:09 -0300 | [diff] [blame] | 201 | if (machine__resolve(machine, &al, sample) < 0) { |
Arnaldo Carvalho de Melo | 29a9f66 | 2010-02-03 16:52:06 -0200 | [diff] [blame] | 202 | pr_warning("problem processing %d event, skipping it.\n", |
| 203 | event->header.type); |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 204 | return -1; |
| 205 | } |
| 206 | |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 207 | if (ann->cpu_list && !test_bit(sample->cpu, ann->cpu_bitmap)) |
Arnaldo Carvalho de Melo | b91fc39 | 2015-04-06 20:43:22 -0300 | [diff] [blame] | 208 | goto out_put; |
Anton Blanchard | 5d67be9 | 2011-07-04 21:57:50 +1000 | [diff] [blame] | 209 | |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 210 | if (!al.filtered && perf_evsel__add_sample(evsel, sample, &al, ann)) { |
Arnaldo Carvalho de Melo | 29a9f66 | 2010-02-03 16:52:06 -0200 | [diff] [blame] | 211 | pr_warning("problem incrementing symbol count, " |
| 212 | "skipping event\n"); |
Arnaldo Carvalho de Melo | b91fc39 | 2015-04-06 20:43:22 -0300 | [diff] [blame] | 213 | ret = -1; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 214 | } |
Arnaldo Carvalho de Melo | b91fc39 | 2015-04-06 20:43:22 -0300 | [diff] [blame] | 215 | out_put: |
| 216 | addr_location__put(&al); |
| 217 | return ret; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 218 | } |
| 219 | |
Namhyung Kim | db8fd07 | 2013-03-05 14:53:21 +0900 | [diff] [blame] | 220 | static int hist_entry__tty_annotate(struct hist_entry *he, |
| 221 | struct perf_evsel *evsel, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 222 | struct perf_annotate *ann) |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 223 | { |
Namhyung Kim | db8fd07 | 2013-03-05 14:53:21 +0900 | [diff] [blame] | 224 | return symbol__tty_annotate(he->ms.sym, he->ms.map, evsel, |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 225 | ann->print_line, ann->full_paths, 0, 0); |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 226 | } |
| 227 | |
Arnaldo Carvalho de Melo | c824c43 | 2013-10-22 19:01:31 -0300 | [diff] [blame] | 228 | static void hists__find_annotations(struct hists *hists, |
Namhyung Kim | db8fd07 | 2013-03-05 14:53:21 +0900 | [diff] [blame] | 229 | struct perf_evsel *evsel, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 230 | struct perf_annotate *ann) |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 231 | { |
Arnaldo Carvalho de Melo | c824c43 | 2013-10-22 19:01:31 -0300 | [diff] [blame] | 232 | struct rb_node *nd = rb_first(&hists->entries), *next; |
Arnaldo Carvalho de Melo | cf95800 | 2011-10-20 16:59:15 -0200 | [diff] [blame] | 233 | int key = K_RIGHT; |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 234 | |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 235 | while (nd) { |
Arnaldo Carvalho de Melo | ed52ce2 | 2009-10-19 17:17:57 -0200 | [diff] [blame] | 236 | struct hist_entry *he = rb_entry(nd, struct hist_entry, rb_node); |
Arnaldo Carvalho de Melo | 78f7def | 2011-02-04 09:45:46 -0200 | [diff] [blame] | 237 | struct annotation *notes; |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 238 | |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 239 | if (he->ms.sym == NULL || he->ms.map->dso->annotate_warned) |
| 240 | goto find_next; |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 241 | |
Arnaldo Carvalho de Melo | 78f7def | 2011-02-04 09:45:46 -0200 | [diff] [blame] | 242 | notes = symbol__annotation(he->ms.sym); |
Arnaldo Carvalho de Melo | ce6f4fa | 2011-02-08 13:27:39 -0200 | [diff] [blame] | 243 | if (notes->src == NULL) { |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 244 | find_next: |
Arnaldo Carvalho de Melo | cf95800 | 2011-10-20 16:59:15 -0200 | [diff] [blame] | 245 | if (key == K_LEFT) |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 246 | nd = rb_prev(nd); |
| 247 | else |
| 248 | nd = rb_next(nd); |
Arnaldo Carvalho de Melo | e420499 | 2009-10-20 14:25:40 -0200 | [diff] [blame] | 249 | continue; |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 250 | } |
Arnaldo Carvalho de Melo | e420499 | 2009-10-20 14:25:40 -0200 | [diff] [blame] | 251 | |
Namhyung Kim | 2b676bf | 2013-02-07 18:02:08 +0900 | [diff] [blame] | 252 | if (use_browser == 2) { |
Namhyung Kim | 18c9e5c | 2013-02-07 18:02:14 +0900 | [diff] [blame] | 253 | int ret; |
Namhyung Kim | fc67297 | 2013-09-13 15:27:43 +0900 | [diff] [blame] | 254 | int (*annotate)(struct hist_entry *he, |
| 255 | struct perf_evsel *evsel, |
| 256 | struct hist_browser_timer *hbt); |
Namhyung Kim | 18c9e5c | 2013-02-07 18:02:14 +0900 | [diff] [blame] | 257 | |
Namhyung Kim | fc67297 | 2013-09-13 15:27:43 +0900 | [diff] [blame] | 258 | annotate = dlsym(perf_gtk_handle, |
| 259 | "hist_entry__gtk_annotate"); |
| 260 | if (annotate == NULL) { |
| 261 | ui__error("GTK browser not found!\n"); |
| 262 | return; |
| 263 | } |
| 264 | |
| 265 | ret = annotate(he, evsel, NULL); |
Namhyung Kim | 18c9e5c | 2013-02-07 18:02:14 +0900 | [diff] [blame] | 266 | if (!ret || !ann->skip_missing) |
| 267 | return; |
| 268 | |
| 269 | /* skip missing symbols */ |
| 270 | nd = rb_next(nd); |
Namhyung Kim | 2b676bf | 2013-02-07 18:02:08 +0900 | [diff] [blame] | 271 | } else if (use_browser == 1) { |
Namhyung Kim | db8fd07 | 2013-03-05 14:53:21 +0900 | [diff] [blame] | 272 | key = hist_entry__tui_annotate(he, evsel, NULL); |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 273 | switch (key) { |
Namhyung Kim | 18c9e5c | 2013-02-07 18:02:14 +0900 | [diff] [blame] | 274 | case -1: |
| 275 | if (!ann->skip_missing) |
| 276 | return; |
| 277 | /* fall through */ |
Arnaldo Carvalho de Melo | cf95800 | 2011-10-20 16:59:15 -0200 | [diff] [blame] | 278 | case K_RIGHT: |
Arnaldo Carvalho de Melo | b50e003 | 2010-08-11 10:07:43 -0300 | [diff] [blame] | 279 | next = rb_next(nd); |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 280 | break; |
Arnaldo Carvalho de Melo | cf95800 | 2011-10-20 16:59:15 -0200 | [diff] [blame] | 281 | case K_LEFT: |
Arnaldo Carvalho de Melo | b50e003 | 2010-08-11 10:07:43 -0300 | [diff] [blame] | 282 | next = rb_prev(nd); |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 283 | break; |
Arnaldo Carvalho de Melo | b50e003 | 2010-08-11 10:07:43 -0300 | [diff] [blame] | 284 | default: |
| 285 | return; |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 286 | } |
Arnaldo Carvalho de Melo | b50e003 | 2010-08-11 10:07:43 -0300 | [diff] [blame] | 287 | |
| 288 | if (next != NULL) |
| 289 | nd = next; |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 290 | } else { |
Namhyung Kim | db8fd07 | 2013-03-05 14:53:21 +0900 | [diff] [blame] | 291 | hist_entry__tty_annotate(he, evsel, ann); |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 292 | nd = rb_next(nd); |
| 293 | /* |
| 294 | * Since we have a hist_entry per IP for the same |
Arnaldo Carvalho de Melo | ce6f4fa | 2011-02-08 13:27:39 -0200 | [diff] [blame] | 295 | * symbol, free he->ms.sym->src to signal we already |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 296 | * processed this symbol. |
| 297 | */ |
Andi Kleen | d495763 | 2015-07-18 08:24:48 -0700 | [diff] [blame] | 298 | zfree(¬es->src->cycles_hist); |
Arnaldo Carvalho de Melo | 0466252 | 2013-12-26 17:41:15 -0300 | [diff] [blame] | 299 | zfree(¬es->src); |
Arnaldo Carvalho de Melo | 46e3e05 | 2010-05-22 11:25:40 -0300 | [diff] [blame] | 300 | } |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 301 | } |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 302 | } |
| 303 | |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 304 | static int __cmd_annotate(struct perf_annotate *ann) |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 305 | { |
Li Zefan | bab81b6 | 2009-12-01 14:04:49 +0800 | [diff] [blame] | 306 | int ret; |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 307 | struct perf_session *session = ann->session; |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 308 | struct perf_evsel *pos; |
| 309 | u64 total_nr_samples; |
Arnaldo Carvalho de Melo | 94c744b | 2009-12-11 21:24:02 -0200 | [diff] [blame] | 310 | |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 311 | if (ann->cpu_list) { |
| 312 | ret = perf_session__cpu_bitmap(session, ann->cpu_list, |
| 313 | ann->cpu_bitmap); |
Anton Blanchard | 5d67be9 | 2011-07-04 21:57:50 +1000 | [diff] [blame] | 314 | if (ret) |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 315 | goto out; |
Anton Blanchard | 5d67be9 | 2011-07-04 21:57:50 +1000 | [diff] [blame] | 316 | } |
| 317 | |
Irina Tirdea | 68e94f4 | 2012-10-16 02:33:38 +0300 | [diff] [blame] | 318 | if (!objdump_path) { |
Arnaldo Carvalho de Melo | eebd0bf | 2015-09-08 15:52:20 -0300 | [diff] [blame] | 319 | ret = perf_env__lookup_objdump(&session->header.env); |
Irina Tirdea | 68e94f4 | 2012-10-16 02:33:38 +0300 | [diff] [blame] | 320 | if (ret) |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 321 | goto out; |
Irina Tirdea | 68e94f4 | 2012-10-16 02:33:38 +0300 | [diff] [blame] | 322 | } |
| 323 | |
Arnaldo Carvalho de Melo | b7b61cb | 2015-03-03 11:58:45 -0300 | [diff] [blame] | 324 | ret = perf_session__process_events(session); |
Li Zefan | bab81b6 | 2009-12-01 14:04:49 +0800 | [diff] [blame] | 325 | if (ret) |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 326 | goto out; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 327 | |
Arnaldo Carvalho de Melo | 62daacb | 2009-11-27 16:29:22 -0200 | [diff] [blame] | 328 | if (dump_trace) { |
Arnaldo Carvalho de Melo | c8446b9 | 2010-05-14 10:36:42 -0300 | [diff] [blame] | 329 | perf_session__fprintf_nr_events(session, stdout); |
Arnaldo Carvalho de Melo | 2a1731f | 2014-10-10 15:49:21 -0300 | [diff] [blame] | 330 | perf_evlist__fprintf_nr_events(session->evlist, stdout); |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 331 | goto out; |
Arnaldo Carvalho de Melo | 62daacb | 2009-11-27 16:29:22 -0200 | [diff] [blame] | 332 | } |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 333 | |
Arnaldo Carvalho de Melo | da21d1b | 2009-10-07 10:49:00 -0300 | [diff] [blame] | 334 | if (verbose > 3) |
Arnaldo Carvalho de Melo | b3165f4 | 2009-12-13 19:50:28 -0200 | [diff] [blame] | 335 | perf_session__fprintf(session, stdout); |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 336 | |
Arnaldo Carvalho de Melo | da21d1b | 2009-10-07 10:49:00 -0300 | [diff] [blame] | 337 | if (verbose > 2) |
Arnaldo Carvalho de Melo | cbf6968 | 2010-04-27 21:22:44 -0300 | [diff] [blame] | 338 | perf_session__fprintf_dsos(session, stdout); |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 339 | |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 340 | total_nr_samples = 0; |
Arnaldo Carvalho de Melo | e5cadb9 | 2016-06-23 11:26:15 -0300 | [diff] [blame] | 341 | evlist__for_each_entry(session->evlist, pos) { |
Arnaldo Carvalho de Melo | 4ea062ed | 2014-10-09 13:13:41 -0300 | [diff] [blame] | 342 | struct hists *hists = evsel__hists(pos); |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 343 | u32 nr_samples = hists->stats.nr_events[PERF_RECORD_SAMPLE]; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 344 | |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 345 | if (nr_samples > 0) { |
| 346 | total_nr_samples += nr_samples; |
Namhyung Kim | c1fb565 | 2013-10-11 14:15:38 +0900 | [diff] [blame] | 347 | hists__collapse_resort(hists, NULL); |
Kan Liang | f9db0d0 | 2015-08-11 06:30:48 -0400 | [diff] [blame] | 348 | /* Don't sort callchain */ |
| 349 | perf_evsel__reset_sample_bit(pos, CALLCHAIN); |
Jiri Olsa | 452ce03 | 2016-01-18 10:24:00 +0100 | [diff] [blame] | 350 | perf_evsel__output_resort(pos, NULL); |
Namhyung Kim | b1dd443 | 2013-03-05 14:53:25 +0900 | [diff] [blame] | 351 | |
| 352 | if (symbol_conf.event_group && |
| 353 | !perf_evsel__is_group_leader(pos)) |
| 354 | continue; |
| 355 | |
Namhyung Kim | db8fd07 | 2013-03-05 14:53:21 +0900 | [diff] [blame] | 356 | hists__find_annotations(hists, pos, ann); |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 357 | } |
| 358 | } |
| 359 | |
| 360 | if (total_nr_samples == 0) { |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 361 | ui__error("The %s file has no samples!\n", session->file->path); |
| 362 | goto out; |
Arnaldo Carvalho de Melo | e248de3 | 2011-03-05 21:40:06 -0300 | [diff] [blame] | 363 | } |
Namhyung Kim | 7a60ba9 | 2013-02-07 18:02:09 +0900 | [diff] [blame] | 364 | |
Namhyung Kim | fc67297 | 2013-09-13 15:27:43 +0900 | [diff] [blame] | 365 | if (use_browser == 2) { |
| 366 | void (*show_annotations)(void); |
| 367 | |
| 368 | show_annotations = dlsym(perf_gtk_handle, |
| 369 | "perf_gtk__show_annotations"); |
| 370 | if (show_annotations == NULL) { |
| 371 | ui__error("GTK browser not found!\n"); |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 372 | goto out; |
Namhyung Kim | fc67297 | 2013-09-13 15:27:43 +0900 | [diff] [blame] | 373 | } |
| 374 | show_annotations(); |
| 375 | } |
Namhyung Kim | 7a60ba9 | 2013-02-07 18:02:09 +0900 | [diff] [blame] | 376 | |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 377 | out: |
Li Zefan | bab81b6 | 2009-12-01 14:04:49 +0800 | [diff] [blame] | 378 | return ret; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 379 | } |
| 380 | |
| 381 | static const char * const annotate_usage[] = { |
Namhyung Kim | 9934525 | 2012-01-08 02:25:30 +0900 | [diff] [blame] | 382 | "perf annotate [<options>]", |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 383 | NULL |
| 384 | }; |
| 385 | |
Irina Tirdea | 1d037ca | 2012-09-11 01:15:03 +0300 | [diff] [blame] | 386 | int cmd_annotate(int argc, const char **argv, const char *prefix __maybe_unused) |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 387 | { |
| 388 | struct perf_annotate annotate = { |
Arnaldo Carvalho de Melo | 45694aa | 2011-11-28 08:30:20 -0200 | [diff] [blame] | 389 | .tool = { |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 390 | .sample = process_sample_event, |
| 391 | .mmap = perf_event__process_mmap, |
Stephane Eranian | 5c5e854 | 2013-08-21 12:10:25 +0200 | [diff] [blame] | 392 | .mmap2 = perf_event__process_mmap2, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 393 | .comm = perf_event__process_comm, |
Arnaldo Carvalho de Melo | ec4622f | 2012-10-06 15:53:41 -0300 | [diff] [blame] | 394 | .exit = perf_event__process_exit, |
Arnaldo Carvalho de Melo | f62d3f0 | 2012-10-06 15:44:59 -0300 | [diff] [blame] | 395 | .fork = perf_event__process_fork, |
Jiri Olsa | 0a8cb85 | 2014-07-06 14:18:21 +0200 | [diff] [blame] | 396 | .ordered_events = true, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 397 | .ordering_requires_timestamps = true, |
| 398 | }, |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 399 | }; |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 400 | struct perf_data_file file = { |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 401 | .mode = PERF_DATA_MODE_READ, |
| 402 | }; |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 403 | const struct option options[] = { |
Feng Tang | 70cb4e9 | 2012-10-30 11:56:02 +0800 | [diff] [blame] | 404 | OPT_STRING('i', "input", &input_name, "file", |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 405 | "input file name"), |
Arnaldo Carvalho de Melo | ac73c5a | 2010-03-24 16:40:16 -0300 | [diff] [blame] | 406 | OPT_STRING('d', "dsos", &symbol_conf.dso_list_str, "dso[,dso...]", |
| 407 | "only consider symbols in these dsos"), |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 408 | OPT_STRING('s', "symbol", &annotate.sym_hist_filter, "symbol", |
Ingo Molnar | 0b73da3 | 2009-06-06 15:48:52 +0200 | [diff] [blame] | 409 | "symbol to annotate"), |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 410 | OPT_BOOLEAN('f', "force", &file.force, "don't complain, do it"), |
Ian Munsie | c055564 | 2010-04-13 18:37:33 +1000 | [diff] [blame] | 411 | OPT_INCR('v', "verbose", &verbose, |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 412 | "be more verbose (show symbol address, etc)"), |
| 413 | OPT_BOOLEAN('D', "dump-raw-trace", &dump_trace, |
| 414 | "dump raw trace in ASCII"), |
Namhyung Kim | 2b676bf | 2013-02-07 18:02:08 +0900 | [diff] [blame] | 415 | OPT_BOOLEAN(0, "gtk", &annotate.use_gtk, "Use the GTK interface"), |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 416 | OPT_BOOLEAN(0, "tui", &annotate.use_tui, "Use the TUI interface"), |
| 417 | OPT_BOOLEAN(0, "stdio", &annotate.use_stdio, "Use the stdio interface"), |
Arnaldo Carvalho de Melo | b32d133 | 2009-11-24 12:05:15 -0200 | [diff] [blame] | 418 | OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name, |
| 419 | "file", "vmlinux pathname"), |
| 420 | OPT_BOOLEAN('m', "modules", &symbol_conf.use_modules, |
Mike Galbraith | 4297648 | 2009-07-02 08:09:46 +0200 | [diff] [blame] | 421 | "load module symbols - WARNING: use only with -k and LIVE kernel"), |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 422 | OPT_BOOLEAN('l', "print-line", &annotate.print_line, |
Frederic Weisbecker | 301406b | 2009-06-13 00:11:21 +0200 | [diff] [blame] | 423 | "print matching source lines (may be slow)"), |
Arnaldo Carvalho de Melo | 7009cc3 | 2011-11-17 12:33:21 -0200 | [diff] [blame] | 424 | OPT_BOOLEAN('P', "full-paths", &annotate.full_paths, |
Mike Galbraith | 4297648 | 2009-07-02 08:09:46 +0200 | [diff] [blame] | 425 | "Don't shorten the displayed pathnames"), |
Namhyung Kim | 18c9e5c | 2013-02-07 18:02:14 +0900 | [diff] [blame] | 426 | OPT_BOOLEAN(0, "skip-missing", &annotate.skip_missing, |
| 427 | "Skip symbols that cannot be annotated"), |
David Ahern | c8e6672 | 2011-11-13 11:30:08 -0700 | [diff] [blame] | 428 | OPT_STRING('C', "cpu", &annotate.cpu_list, "cpu", "list of cpus to profile"), |
He Kuang | a706670 | 2016-05-19 11:47:37 +0000 | [diff] [blame] | 429 | OPT_CALLBACK(0, "symfs", NULL, "directory", |
| 430 | "Look for files with symbols relative to this directory", |
| 431 | symbol__config_symfs), |
Arnaldo Carvalho de Melo | 64c6f0c | 2011-10-06 12:48:31 -0300 | [diff] [blame] | 432 | OPT_BOOLEAN(0, "source", &symbol_conf.annotate_src, |
Stephane Eranian | 3e6a2a7 | 2011-05-17 17:32:07 +0200 | [diff] [blame] | 433 | "Interleave source code with assembly code (default)"), |
Arnaldo Carvalho de Melo | 64c6f0c | 2011-10-06 12:48:31 -0300 | [diff] [blame] | 434 | OPT_BOOLEAN(0, "asm-raw", &symbol_conf.annotate_asm_raw, |
Stephane Eranian | 3e6a2a7 | 2011-05-17 17:32:07 +0200 | [diff] [blame] | 435 | "Display raw encoding of assembly instructions (default)"), |
Andi Kleen | f69b64f | 2011-09-15 14:31:41 -0700 | [diff] [blame] | 436 | OPT_STRING('M', "disassembler-style", &disassembler_style, "disassembler style", |
| 437 | "Specify disassembler style (e.g. -M intel for intel syntax)"), |
Maciek Borzecki | 7a4ec93 | 2012-09-04 12:32:30 +0200 | [diff] [blame] | 438 | OPT_STRING(0, "objdump", &objdump_path, "path", |
| 439 | "objdump binary to use for disassembly and annotations"), |
Namhyung Kim | b1dd443 | 2013-03-05 14:53:25 +0900 | [diff] [blame] | 440 | OPT_BOOLEAN(0, "group", &symbol_conf.event_group, |
| 441 | "Show event group information together"), |
Martin Liška | 0c4a5bc | 2015-06-19 16:10:43 -0300 | [diff] [blame] | 442 | OPT_BOOLEAN(0, "show-total-period", &symbol_conf.show_total_period, |
| 443 | "Show a column with the sum of periods"), |
Arnaldo Carvalho de Melo | 53fe4ba | 2016-07-05 11:08:17 -0300 | [diff] [blame] | 444 | OPT_CALLBACK_DEFAULT(0, "stdio-color", NULL, "mode", |
| 445 | "'always' (default), 'never' or 'auto' only applicable to --stdio mode", |
| 446 | stdio__config_color, "always"), |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 447 | OPT_END() |
Arnaldo Carvalho de Melo | d20deb6 | 2011-11-25 08:19:45 -0200 | [diff] [blame] | 448 | }; |
Arnaldo Carvalho de Melo | a635fc5 | 2014-10-09 16:16:00 -0300 | [diff] [blame] | 449 | int ret = hists__init(); |
| 450 | |
| 451 | if (ret < 0) |
| 452 | return ret; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 453 | |
Arnaldo Carvalho de Melo | 655000e | 2009-12-15 20:04:40 -0200 | [diff] [blame] | 454 | argc = parse_options(argc, argv, options, annotate_usage, 0); |
Namhyung Kim | 50e19ef | 2015-12-10 12:00:53 +0900 | [diff] [blame] | 455 | if (argc) { |
| 456 | /* |
| 457 | * Special case: if there's an argument left then assume that |
| 458 | * it's a symbol filter: |
| 459 | */ |
| 460 | if (argc > 1) |
| 461 | usage_with_options(annotate_usage, options); |
| 462 | |
| 463 | annotate.sym_hist_filter = argv[0]; |
| 464 | } |
Arnaldo Carvalho de Melo | 655000e | 2009-12-15 20:04:40 -0200 | [diff] [blame] | 465 | |
Martin Liška | 44848cd | 2015-05-29 14:06:44 +0200 | [diff] [blame] | 466 | file.path = input_name; |
| 467 | |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 468 | annotate.session = perf_session__new(&file, false, &annotate.tool); |
| 469 | if (annotate.session == NULL) |
Taeung Song | 52e02834 | 2014-09-24 10:33:37 +0900 | [diff] [blame] | 470 | return -1; |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 471 | |
Arnaldo Carvalho de Melo | b01141f | 2016-08-25 16:09:21 -0300 | [diff] [blame] | 472 | ret = symbol__annotation_init(); |
| 473 | if (ret < 0) |
| 474 | goto out_delete; |
| 475 | |
Arnaldo Carvalho de Melo | 75be6cf | 2009-12-15 20:04:39 -0200 | [diff] [blame] | 476 | symbol_conf.try_vmlinux_path = true; |
| 477 | |
Namhyung Kim | 0a7e6d1 | 2014-08-12 15:40:45 +0900 | [diff] [blame] | 478 | ret = symbol__init(&annotate.session->header.env); |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 479 | if (ret < 0) |
| 480 | goto out_delete; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 481 | |
Namhyung Kim | 40184c4 | 2015-12-23 02:07:01 +0900 | [diff] [blame] | 482 | if (setup_sorting(NULL) < 0) |
Namhyung Kim | 5530998 | 2013-02-06 14:57:16 +0900 | [diff] [blame] | 483 | usage_with_options(annotate_usage, options); |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 484 | |
Namhyung Kim | 3df668e7 | 2015-12-10 12:00:54 +0900 | [diff] [blame] | 485 | if (annotate.use_stdio) |
| 486 | use_browser = 0; |
| 487 | else if (annotate.use_tui) |
| 488 | use_browser = 1; |
| 489 | else if (annotate.use_gtk) |
| 490 | use_browser = 2; |
| 491 | |
| 492 | setup_browser(true); |
| 493 | |
Namhyung Kim | fa10f31 | 2014-08-12 15:40:35 +0900 | [diff] [blame] | 494 | ret = __cmd_annotate(&annotate); |
| 495 | |
| 496 | out_delete: |
| 497 | /* |
| 498 | * Speed up the exit process, for large files this can |
| 499 | * take quite a while. |
| 500 | * |
| 501 | * XXX Enable this when using valgrind or if we ever |
| 502 | * librarize this command. |
| 503 | * |
| 504 | * Also experiment with obstacks to see how much speed |
| 505 | * up we'll get here. |
| 506 | * |
| 507 | * perf_session__delete(session); |
| 508 | */ |
| 509 | return ret; |
Ingo Molnar | 8035e42 | 2009-06-06 15:19:13 +0200 | [diff] [blame] | 510 | } |