]> nv-tegra.nvidia Code Review - linux-3.10.git/blobdiff - tools/perf/util/annotate.c
perf annotate browser: Use disasm__calc_percent()
[linux-3.10.git] / tools / perf / util / annotate.c
index e5a462f1d07c0c0ab6cbcc08bd1e4a317bac6693..d102716c43a1b30beb08ed89f1f18edc60d66dd0 100644 (file)
 #include "symbol.h"
 #include "debug.h"
 #include "annotate.h"
+#include "evsel.h"
 #include <pthread.h>
+#include <linux/bitops.h>
 
 const char     *disassembler_style;
+const char     *objdump_path;
 
-int symbol__annotate_init(struct map *map __used, struct symbol *sym)
+static struct ins *ins__find(const char *name);
+static int disasm_line__parse(char *line, char **namep, char **rawp);
+
+static void ins__delete(struct ins_operands *ops)
+{
+       free(ops->source.raw);
+       free(ops->source.name);
+       free(ops->target.raw);
+       free(ops->target.name);
+}
+
+static int ins__raw_scnprintf(struct ins *ins, char *bf, size_t size,
+                             struct ins_operands *ops)
+{
+       return scnprintf(bf, size, "%-6.6s %s", ins->name, ops->raw);
+}
+
+int ins__scnprintf(struct ins *ins, char *bf, size_t size,
+                 struct ins_operands *ops)
+{
+       if (ins->ops->scnprintf)
+               return ins->ops->scnprintf(ins, bf, size, ops);
+
+       return ins__raw_scnprintf(ins, bf, size, ops);
+}
+
+static int call__parse(struct ins_operands *ops)
+{
+       char *endptr, *tok, *name;
+
+       ops->target.addr = strtoull(ops->raw, &endptr, 16);
+
+       name = strchr(endptr, '<');
+       if (name == NULL)
+               goto indirect_call;
+
+       name++;
+
+       tok = strchr(name, '>');
+       if (tok == NULL)
+               return -1;
+
+       *tok = '\0';
+       ops->target.name = strdup(name);
+       *tok = '>';
+
+       return ops->target.name == NULL ? -1 : 0;
+
+indirect_call:
+       tok = strchr(endptr, '(');
+       if (tok != NULL) {
+               ops->target.addr = 0;
+               return 0;
+       }
+
+       tok = strchr(endptr, '*');
+       if (tok == NULL)
+               return -1;
+
+       ops->target.addr = strtoull(tok + 1, NULL, 16);
+       return 0;
+}
+
+static int call__scnprintf(struct ins *ins, char *bf, size_t size,
+                          struct ins_operands *ops)
+{
+       if (ops->target.name)
+               return scnprintf(bf, size, "%-6.6s %s", ins->name, ops->target.name);
+
+       if (ops->target.addr == 0)
+               return ins__raw_scnprintf(ins, bf, size, ops);
+
+       return scnprintf(bf, size, "%-6.6s *%" PRIx64, ins->name, ops->target.addr);
+}
+
+static struct ins_ops call_ops = {
+       .parse     = call__parse,
+       .scnprintf = call__scnprintf,
+};
+
+bool ins__is_call(const struct ins *ins)
+{
+       return ins->ops == &call_ops;
+}
+
+static int jump__parse(struct ins_operands *ops)
+{
+       const char *s = strchr(ops->raw, '+');
+
+       ops->target.addr = strtoll(ops->raw, NULL, 16);
+
+       if (s++ != NULL)
+               ops->target.offset = strtoll(s, NULL, 16);
+       else
+               ops->target.offset = UINT64_MAX;
+
+       return 0;
+}
+
+static int jump__scnprintf(struct ins *ins, char *bf, size_t size,
+                          struct ins_operands *ops)
+{
+       return scnprintf(bf, size, "%-6.6s %" PRIx64, ins->name, ops->target.offset);
+}
+
+static struct ins_ops jump_ops = {
+       .parse     = jump__parse,
+       .scnprintf = jump__scnprintf,
+};
+
+bool ins__is_jump(const struct ins *ins)
+{
+       return ins->ops == &jump_ops;
+}
+
+static int comment__symbol(char *raw, char *comment, u64 *addrp, char **namep)
+{
+       char *endptr, *name, *t;
+
+       if (strstr(raw, "(%rip)") == NULL)
+               return 0;
+
+       *addrp = strtoull(comment, &endptr, 16);
+       name = strchr(endptr, '<');
+       if (name == NULL)
+               return -1;
+
+       name++;
+
+       t = strchr(name, '>');
+       if (t == NULL)
+               return 0;
+
+       *t = '\0';
+       *namep = strdup(name);
+       *t = '>';
+
+       return 0;
+}
+
+static int lock__parse(struct ins_operands *ops)
+{
+       char *name;
+
+       ops->locked.ops = zalloc(sizeof(*ops->locked.ops));
+       if (ops->locked.ops == NULL)
+               return 0;
+
+       if (disasm_line__parse(ops->raw, &name, &ops->locked.ops->raw) < 0)
+               goto out_free_ops;
+
+       ops->locked.ins = ins__find(name);
+       if (ops->locked.ins == NULL)
+               goto out_free_ops;
+
+       if (!ops->locked.ins->ops)
+               return 0;
+
+       if (ops->locked.ins->ops->parse)
+               ops->locked.ins->ops->parse(ops->locked.ops);
+
+       return 0;
+
+out_free_ops:
+       free(ops->locked.ops);
+       ops->locked.ops = NULL;
+       return 0;
+}
+
+static int lock__scnprintf(struct ins *ins, char *bf, size_t size,
+                          struct ins_operands *ops)
+{
+       int printed;
+
+       if (ops->locked.ins == NULL)
+               return ins__raw_scnprintf(ins, bf, size, ops);
+
+       printed = scnprintf(bf, size, "%-6.6s ", ins->name);
+       return printed + ins__scnprintf(ops->locked.ins, bf + printed,
+                                       size - printed, ops->locked.ops);
+}
+
+static void lock__delete(struct ins_operands *ops)
+{
+       free(ops->locked.ops);
+       free(ops->target.raw);
+       free(ops->target.name);
+}
+
+static struct ins_ops lock_ops = {
+       .free      = lock__delete,
+       .parse     = lock__parse,
+       .scnprintf = lock__scnprintf,
+};
+
+static int mov__parse(struct ins_operands *ops)
+{
+       char *s = strchr(ops->raw, ','), *target, *comment, prev;
+
+       if (s == NULL)
+               return -1;
+
+       *s = '\0';
+       ops->source.raw = strdup(ops->raw);
+       *s = ',';
+       
+       if (ops->source.raw == NULL)
+               return -1;
+
+       target = ++s;
+
+       while (s[0] != '\0' && !isspace(s[0]))
+               ++s;
+       prev = *s;
+       *s = '\0';
+
+       ops->target.raw = strdup(target);
+       *s = prev;
+
+       if (ops->target.raw == NULL)
+               goto out_free_source;
+
+       comment = strchr(s, '#');
+       if (comment == NULL)
+               return 0;
+
+       while (comment[0] != '\0' && isspace(comment[0]))
+               ++comment;
+
+       comment__symbol(ops->source.raw, comment, &ops->source.addr, &ops->source.name);
+       comment__symbol(ops->target.raw, comment, &ops->target.addr, &ops->target.name);
+
+       return 0;
+
+out_free_source:
+       free(ops->source.raw);
+       ops->source.raw = NULL;
+       return -1;
+}
+
+static int mov__scnprintf(struct ins *ins, char *bf, size_t size,
+                          struct ins_operands *ops)
+{
+       return scnprintf(bf, size, "%-6.6s %s,%s", ins->name,
+                        ops->source.name ?: ops->source.raw,
+                        ops->target.name ?: ops->target.raw);
+}
+
+static struct ins_ops mov_ops = {
+       .parse     = mov__parse,
+       .scnprintf = mov__scnprintf,
+};
+
+static int dec__parse(struct ins_operands *ops)
+{
+       char *target, *comment, *s, prev;
+
+       target = s = ops->raw;
+
+       while (s[0] != '\0' && !isspace(s[0]))
+               ++s;
+       prev = *s;
+       *s = '\0';
+
+       ops->target.raw = strdup(target);
+       *s = prev;
+
+       if (ops->target.raw == NULL)
+               return -1;
+
+       comment = strchr(s, '#');
+       if (comment == NULL)
+               return 0;
+
+       while (comment[0] != '\0' && isspace(comment[0]))
+               ++comment;
+
+       comment__symbol(ops->target.raw, comment, &ops->target.addr, &ops->target.name);
+
+       return 0;
+}
+
+static int dec__scnprintf(struct ins *ins, char *bf, size_t size,
+                          struct ins_operands *ops)
+{
+       return scnprintf(bf, size, "%-6.6s %s", ins->name,
+                        ops->target.name ?: ops->target.raw);
+}
+
+static struct ins_ops dec_ops = {
+       .parse     = dec__parse,
+       .scnprintf = dec__scnprintf,
+};
+
+static int nop__scnprintf(struct ins *ins __maybe_unused, char *bf, size_t size,
+                         struct ins_operands *ops __maybe_unused)
+{
+       return scnprintf(bf, size, "%-6.6s", "nop");
+}
+
+static struct ins_ops nop_ops = {
+       .scnprintf = nop__scnprintf,
+};
+
+/*
+ * Must be sorted by name!
+ */
+static struct ins instructions[] = {
+       { .name = "add",   .ops  = &mov_ops, },
+       { .name = "addl",  .ops  = &mov_ops, },
+       { .name = "addq",  .ops  = &mov_ops, },
+       { .name = "addw",  .ops  = &mov_ops, },
+       { .name = "and",   .ops  = &mov_ops, },
+       { .name = "bts",   .ops  = &mov_ops, },
+       { .name = "call",  .ops  = &call_ops, },
+       { .name = "callq", .ops  = &call_ops, },
+       { .name = "cmp",   .ops  = &mov_ops, },
+       { .name = "cmpb",  .ops  = &mov_ops, },
+       { .name = "cmpl",  .ops  = &mov_ops, },
+       { .name = "cmpq",  .ops  = &mov_ops, },
+       { .name = "cmpw",  .ops  = &mov_ops, },
+       { .name = "cmpxch", .ops  = &mov_ops, },
+       { .name = "dec",   .ops  = &dec_ops, },
+       { .name = "decl",  .ops  = &dec_ops, },
+       { .name = "imul",  .ops  = &mov_ops, },
+       { .name = "inc",   .ops  = &dec_ops, },
+       { .name = "incl",  .ops  = &dec_ops, },
+       { .name = "ja",    .ops  = &jump_ops, },
+       { .name = "jae",   .ops  = &jump_ops, },
+       { .name = "jb",    .ops  = &jump_ops, },
+       { .name = "jbe",   .ops  = &jump_ops, },
+       { .name = "jc",    .ops  = &jump_ops, },
+       { .name = "jcxz",  .ops  = &jump_ops, },
+       { .name = "je",    .ops  = &jump_ops, },
+       { .name = "jecxz", .ops  = &jump_ops, },
+       { .name = "jg",    .ops  = &jump_ops, },
+       { .name = "jge",   .ops  = &jump_ops, },
+       { .name = "jl",    .ops  = &jump_ops, },
+       { .name = "jle",   .ops  = &jump_ops, },
+       { .name = "jmp",   .ops  = &jump_ops, },
+       { .name = "jmpq",  .ops  = &jump_ops, },
+       { .name = "jna",   .ops  = &jump_ops, },
+       { .name = "jnae",  .ops  = &jump_ops, },
+       { .name = "jnb",   .ops  = &jump_ops, },
+       { .name = "jnbe",  .ops  = &jump_ops, },
+       { .name = "jnc",   .ops  = &jump_ops, },
+       { .name = "jne",   .ops  = &jump_ops, },
+       { .name = "jng",   .ops  = &jump_ops, },
+       { .name = "jnge",  .ops  = &jump_ops, },
+       { .name = "jnl",   .ops  = &jump_ops, },
+       { .name = "jnle",  .ops  = &jump_ops, },
+       { .name = "jno",   .ops  = &jump_ops, },
+       { .name = "jnp",   .ops  = &jump_ops, },
+       { .name = "jns",   .ops  = &jump_ops, },
+       { .name = "jnz",   .ops  = &jump_ops, },
+       { .name = "jo",    .ops  = &jump_ops, },
+       { .name = "jp",    .ops  = &jump_ops, },
+       { .name = "jpe",   .ops  = &jump_ops, },
+       { .name = "jpo",   .ops  = &jump_ops, },
+       { .name = "jrcxz", .ops  = &jump_ops, },
+       { .name = "js",    .ops  = &jump_ops, },
+       { .name = "jz",    .ops  = &jump_ops, },
+       { .name = "lea",   .ops  = &mov_ops, },
+       { .name = "lock",  .ops  = &lock_ops, },
+       { .name = "mov",   .ops  = &mov_ops, },
+       { .name = "movb",  .ops  = &mov_ops, },
+       { .name = "movdqa",.ops  = &mov_ops, },
+       { .name = "movl",  .ops  = &mov_ops, },
+       { .name = "movq",  .ops  = &mov_ops, },
+       { .name = "movslq", .ops  = &mov_ops, },
+       { .name = "movzbl", .ops  = &mov_ops, },
+       { .name = "movzwl", .ops  = &mov_ops, },
+       { .name = "nop",   .ops  = &nop_ops, },
+       { .name = "nopl",  .ops  = &nop_ops, },
+       { .name = "nopw",  .ops  = &nop_ops, },
+       { .name = "or",    .ops  = &mov_ops, },
+       { .name = "orl",   .ops  = &mov_ops, },
+       { .name = "test",  .ops  = &mov_ops, },
+       { .name = "testb", .ops  = &mov_ops, },
+       { .name = "testl", .ops  = &mov_ops, },
+       { .name = "xadd",  .ops  = &mov_ops, },
+       { .name = "xbeginl", .ops  = &jump_ops, },
+       { .name = "xbeginq", .ops  = &jump_ops, },
+};
+
+static int ins__cmp(const void *name, const void *insp)
+{
+       const struct ins *ins = insp;
+
+       return strcmp(name, ins->name);
+}
+
+static struct ins *ins__find(const char *name)
+{
+       const int nmemb = ARRAY_SIZE(instructions);
+
+       return bsearch(name, instructions, nmemb, sizeof(struct ins), ins__cmp);
+}
+
+int symbol__annotate_init(struct map *map __maybe_unused, struct symbol *sym)
 {
        struct annotation *notes = symbol__annotation(sym);
        pthread_mutex_init(&notes->lock, NULL);
@@ -28,8 +430,19 @@ int symbol__annotate_init(struct map *map __used, struct symbol *sym)
 int symbol__alloc_hist(struct symbol *sym)
 {
        struct annotation *notes = symbol__annotation(sym);
-       size_t sizeof_sym_hist = (sizeof(struct sym_hist) +
-                                 (sym->end - sym->start) * sizeof(u64));
+       const size_t size = symbol__size(sym);
+       size_t sizeof_sym_hist;
+
+       /* Check for overflow when calculating sizeof_sym_hist */
+       if (size > (SIZE_MAX - sizeof(struct sym_hist)) / sizeof(u64))
+               return -1;
+
+       sizeof_sym_hist = (sizeof(struct sym_hist) + size * sizeof(u64));
+
+       /* Check for overflow in zalloc argument */
+       if (sizeof_sym_hist > (SIZE_MAX - sizeof(*notes->src))
+                               / symbol_conf.nr_events)
+               return -1;
 
        notes->src = zalloc(sizeof(*notes->src) + symbol_conf.nr_events * sizeof_sym_hist);
        if (notes->src == NULL)
@@ -64,8 +477,8 @@ int symbol__inc_addr_samples(struct symbol *sym, struct map *map,
 
        pr_debug3("%s: addr=%#" PRIx64 "\n", __func__, map->unmap_ip(map, addr));
 
-       if (addr >= sym->end)
-               return 0;
+       if (addr < sym->start || addr > sym->end)
+               return -ERANGE;
 
        offset = addr - sym->start;
        h = annotation__histogram(notes, evidx);
@@ -78,31 +491,110 @@ int symbol__inc_addr_samples(struct symbol *sym, struct map *map,
        return 0;
 }
 
-static struct objdump_line *objdump_line__new(s64 offset, char *line, size_t privsize)
+static void disasm_line__init_ins(struct disasm_line *dl)
+{
+       dl->ins = ins__find(dl->name);
+
+       if (dl->ins == NULL)
+               return;
+
+       if (!dl->ins->ops)
+               return;
+
+       if (dl->ins->ops->parse)
+               dl->ins->ops->parse(&dl->ops);
+}
+
+static int disasm_line__parse(char *line, char **namep, char **rawp)
+{
+       char *name = line, tmp;
+
+       while (isspace(name[0]))
+               ++name;
+
+       if (name[0] == '\0')
+               return -1;
+
+       *rawp = name + 1;
+
+       while ((*rawp)[0] != '\0' && !isspace((*rawp)[0]))
+               ++*rawp;
+
+       tmp = (*rawp)[0];
+       (*rawp)[0] = '\0';
+       *namep = strdup(name);
+
+       if (*namep == NULL)
+               goto out_free_name;
+
+       (*rawp)[0] = tmp;
+
+       if ((*rawp)[0] != '\0') {
+               (*rawp)++;
+               while (isspace((*rawp)[0]))
+                       ++(*rawp);
+       }
+
+       return 0;
+
+out_free_name:
+       free(*namep);
+       *namep = NULL;
+       return -1;
+}
+
+static struct disasm_line *disasm_line__new(s64 offset, char *line, size_t privsize)
 {
-       struct objdump_line *self = malloc(sizeof(*self) + privsize);
+       struct disasm_line *dl = zalloc(sizeof(*dl) + privsize);
+
+       if (dl != NULL) {
+               dl->offset = offset;
+               dl->line = strdup(line);
+               if (dl->line == NULL)
+                       goto out_delete;
 
-       if (self != NULL) {
-               self->offset = offset;
-               self->line = line;
+               if (offset != -1) {
+                       if (disasm_line__parse(dl->line, &dl->name, &dl->ops.raw) < 0)
+                               goto out_free_line;
+
+                       disasm_line__init_ins(dl);
+               }
        }
 
-       return self;
+       return dl;
+
+out_free_line:
+       free(dl->line);
+out_delete:
+       free(dl);
+       return NULL;
 }
 
-void objdump_line__free(struct objdump_line *self)
+void disasm_line__free(struct disasm_line *dl)
 {
-       free(self->line);
-       free(self);
+       free(dl->line);
+       free(dl->name);
+       if (dl->ins && dl->ins->ops->free)
+               dl->ins->ops->free(&dl->ops);
+       else
+               ins__delete(&dl->ops);
+       free(dl);
 }
 
-static void objdump__add_line(struct list_head *head, struct objdump_line *line)
+int disasm_line__scnprintf(struct disasm_line *dl, char *bf, size_t size, bool raw)
+{
+       if (raw || !dl->ins)
+               return scnprintf(bf, size, "%-6.6s %s", dl->name, dl->ops.raw);
+
+       return ins__scnprintf(dl->ins, bf, size, &dl->ops);
+}
+
+static void disasm__add(struct list_head *head, struct disasm_line *line)
 {
        list_add_tail(&line->node, head);
 }
 
-struct objdump_line *objdump__get_next_ip_line(struct list_head *head,
-                                              struct objdump_line *pos)
+struct disasm_line *disasm__get_next_ip_line(struct list_head *head, struct disasm_line *pos)
 {
        list_for_each_entry_continue(pos, head, node)
                if (pos->offset >= 0)
@@ -111,43 +603,80 @@ struct objdump_line *objdump__get_next_ip_line(struct list_head *head,
        return NULL;
 }
 
-static int objdump_line__print(struct objdump_line *oline, struct symbol *sym,
-                              int evidx, u64 len, int min_pcnt,
-                              int printed, int max_lines,
-                              struct objdump_line *queue)
+double disasm__calc_percent(struct annotation *notes, int evidx, s64 offset,
+                           s64 end, const char **path)
+{
+       struct source_line *src_line = notes->src->lines;
+       double percent = 0.0;
+
+       if (src_line) {
+               size_t sizeof_src_line = sizeof(*src_line) +
+                               sizeof(src_line->p) * (src_line->nr_pcnt - 1);
+
+               while (offset < end) {
+                       src_line = (void *)notes->src->lines +
+                                       (sizeof_src_line * offset);
+
+                       if (*path == NULL)
+                               *path = src_line->path;
+
+                       percent += src_line->p[evidx].percent;
+                       offset++;
+               }
+       } else {
+               struct sym_hist *h = annotation__histogram(notes, evidx);
+               unsigned int hits = 0;
+
+               while (offset < end)
+                       hits += h->addr[offset++];
+
+               if (h->sum)
+                       percent = 100.0 * hits / h->sum;
+       }
+
+       return percent;
+}
+
+static int disasm_line__print(struct disasm_line *dl, struct symbol *sym, u64 start,
+                     struct perf_evsel *evsel, u64 len, int min_pcnt, int printed,
+                     int max_lines, struct disasm_line *queue)
 {
        static const char *prev_line;
        static const char *prev_color;
 
-       if (oline->offset != -1) {
+       if (dl->offset != -1) {
                const char *path = NULL;
-               unsigned int hits = 0;
-               double percent = 0.0;
+               double percent, max_percent = 0.0;
+               double *ppercents = &percent;
+               int i, nr_percent = 1;
                const char *color;
                struct annotation *notes = symbol__annotation(sym);
-               struct source_line *src_line = notes->src->lines;
-               struct sym_hist *h = annotation__histogram(notes, evidx);
-               s64 offset = oline->offset;
-               struct objdump_line *next;
-
-               next = objdump__get_next_ip_line(&notes->src->source, oline);
+               s64 offset = dl->offset;
+               const u64 addr = start + offset;
+               struct disasm_line *next;
 
-               while (offset < (s64)len &&
-                      (next == NULL || offset < next->offset)) {
-                       if (src_line) {
-                               if (path == NULL)
-                                       path = src_line[offset].path;
-                               percent += src_line[offset].percent;
-                       } else
-                               hits += h->addr[offset];
+               next = disasm__get_next_ip_line(&notes->src->source, dl);
 
-                       ++offset;
+               if (perf_evsel__is_group_event(evsel)) {
+                       nr_percent = evsel->nr_members;
+                       ppercents = calloc(nr_percent, sizeof(double));
+                       if (ppercents == NULL)
+                               return -1;
                }
 
-               if (src_line == NULL && h->sum)
-                       percent = 100.0 * hits / h->sum;
+               for (i = 0; i < nr_percent; i++) {
+                       percent = disasm__calc_percent(notes,
+                                       notes->src->lines ? i : evsel->idx + i,
+                                       offset,
+                                       next ? next->offset : (s64) len,
+                                       &path);
+
+                       ppercents[i] = percent;
+                       if (percent > max_percent)
+                               max_percent = percent;
+               }
 
-               if (percent < min_pcnt)
+               if (max_percent < min_pcnt)
                        return -1;
 
                if (max_lines && printed >= max_lines)
@@ -155,14 +684,14 @@ static int objdump_line__print(struct objdump_line *oline, struct symbol *sym,
 
                if (queue != NULL) {
                        list_for_each_entry_from(queue, &notes->src->source, node) {
-                               if (queue == oline)
+                               if (queue == dl)
                                        break;
-                               objdump_line__print(queue, sym, evidx, len,
+                               disasm_line__print(queue, sym, start, evsel, len,
                                                    0, 0, 1, NULL);
                        }
                }
 
-               color = get_percent_color(percent);
+               color = get_percent_color(max_percent);
 
                /*
                 * Also color the filename and line if needed, with
@@ -178,30 +707,65 @@ static int objdump_line__print(struct objdump_line *oline, struct symbol *sym,
                        }
                }
 
-               color_fprintf(stdout, color, " %7.2f", percent);
+               for (i = 0; i < nr_percent; i++) {
+                       percent = ppercents[i];
+                       color = get_percent_color(percent);
+                       color_fprintf(stdout, color, " %7.2f", percent);
+               }
+
                printf(" :      ");
-               color_fprintf(stdout, PERF_COLOR_BLUE, "%s\n", oline->line);
+               color_fprintf(stdout, PERF_COLOR_MAGENTA, "  %" PRIx64 ":", addr);
+               color_fprintf(stdout, PERF_COLOR_BLUE, "%s\n", dl->line);
+
+               if (ppercents != &percent)
+                       free(ppercents);
+
        } else if (max_lines && printed >= max_lines)
                return 1;
        else {
+               int width = 8;
+
                if (queue)
                        return -1;
 
-               if (!*oline->line)
-                       printf("         :\n");
+               if (perf_evsel__is_group_event(evsel))
+                       width *= evsel->nr_members;
+
+               if (!*dl->line)
+                       printf(" %*s:\n", width, " ");
                else
-                       printf("         :      %s\n", oline->line);
+                       printf(" %*s:   %s\n", width, " ", dl->line);
        }
 
        return 0;
 }
 
+/*
+ * symbol__parse_objdump_line() parses objdump output (with -d --no-show-raw)
+ * which looks like following
+ *
+ *  0000000000415500 <_init>:
+ *    415500:       sub    $0x8,%rsp
+ *    415504:       mov    0x2f5ad5(%rip),%rax        # 70afe0 <_DYNAMIC+0x2f8>
+ *    41550b:       test   %rax,%rax
+ *    41550e:       je     415515 <_init+0x15>
+ *    415510:       callq  416e70 <__gmon_start__@plt>
+ *    415515:       add    $0x8,%rsp
+ *    415519:       retq
+ *
+ * it will be parsed and saved into struct disasm_line as
+ *  <offset>       <name>  <ops.raw>
+ *
+ * The offset will be a relative offset from the start of the symbol and -1
+ * means that it's not a disassembly line so should be treated differently.
+ * The ops.raw part will be parsed further according to type of the instruction.
+ */
 static int symbol__parse_objdump_line(struct symbol *sym, struct map *map,
                                      FILE *file, size_t privsize)
 {
        struct annotation *notes = symbol__annotation(sym);
-       struct objdump_line *objdump_line;
-       char *line = NULL, *tmp, *tmp2, *c;
+       struct disasm_line *dl;
+       char *line = NULL, *parsed_line, *tmp, *tmp2, *c;
        size_t line_len;
        s64 line_ip, offset = -1;
 
@@ -219,6 +783,7 @@ static int symbol__parse_objdump_line(struct symbol *sym, struct map *map,
                *c = 0;
 
        line_ip = -1;
+       parsed_line = line;
 
        /*
         * Strip leading spaces:
@@ -246,14 +811,17 @@ static int symbol__parse_objdump_line(struct symbol *sym, struct map *map,
                offset = line_ip - start;
                if (offset < 0 || (u64)line_ip > end)
                        offset = -1;
+               else
+                       parsed_line = tmp2 + 1;
        }
 
-       objdump_line = objdump_line__new(offset, line, privsize);
-       if (objdump_line == NULL) {
-               free(line);
+       dl = disasm_line__new(offset, parsed_line, privsize);
+       free(line);
+
+       if (dl == NULL)
                return -1;
-       }
-       objdump__add_line(&notes->src->source, objdump_line);
+
+       disasm__add(&notes->src->source, dl);
 
        return 0;
 }
@@ -296,7 +864,7 @@ fallback:
                free_filename = false;
        }
 
-       if (dso->symtab_type == SYMTAB__KALLSYMS) {
+       if (dso->symtab_type == DSO_BINARY_TYPE__KALLSYMS) {
                char bf[BUILD_ID_SIZE * 2 + 16] = " with build id ";
                char *build_id_msg = NULL;
 
@@ -313,7 +881,7 @@ fallback:
                pr_err("Can't annotate %s:\n\n"
                       "No vmlinux file%s\nwas found in the path.\n\n"
                       "Please use:\n\n"
-                      "  perf buildid-cache -av vmlinux\n\n"
+                      "  perf buildid-cache -vu vmlinux\n\n"
                       "or:\n\n"
                       "  --vmlinux vmlinux\n",
                       sym->name, build_id_msg ?: "");
@@ -328,9 +896,10 @@ fallback:
                 dso, dso->long_name, sym, sym->name);
 
        snprintf(command, sizeof(command),
-                "objdump %s%s --start-address=0x%016" PRIx64
+                "%s %s%s --start-address=0x%016" PRIx64
                 " --stop-address=0x%016" PRIx64
                 " -d %s %s -C %s|grep -v %s|expand",
+                objdump_path ? objdump_path : "objdump",
                 disassembler_style ? "-M " : "",
                 disassembler_style ? disassembler_style : "",
                 map__rip_2objdump(map, sym->start),
@@ -361,85 +930,181 @@ static void insert_source_line(struct rb_root *root, struct source_line *src_lin
        struct source_line *iter;
        struct rb_node **p = &root->rb_node;
        struct rb_node *parent = NULL;
+       int i, ret;
 
        while (*p != NULL) {
                parent = *p;
                iter = rb_entry(parent, struct source_line, node);
 
-               if (src_line->percent > iter->percent)
+               ret = strcmp(iter->path, src_line->path);
+               if (ret == 0) {
+                       for (i = 0; i < src_line->nr_pcnt; i++)
+                               iter->p[i].percent_sum += src_line->p[i].percent;
+                       return;
+               }
+
+               if (ret < 0)
                        p = &(*p)->rb_left;
                else
                        p = &(*p)->rb_right;
        }
 
+       for (i = 0; i < src_line->nr_pcnt; i++)
+               src_line->p[i].percent_sum = src_line->p[i].percent;
+
        rb_link_node(&src_line->node, parent, p);
        rb_insert_color(&src_line->node, root);
 }
 
+static int cmp_source_line(struct source_line *a, struct source_line *b)
+{
+       int i;
+
+       for (i = 0; i < a->nr_pcnt; i++) {
+               if (a->p[i].percent_sum == b->p[i].percent_sum)
+                       continue;
+               return a->p[i].percent_sum > b->p[i].percent_sum;
+       }
+
+       return 0;
+}
+
+static void __resort_source_line(struct rb_root *root, struct source_line *src_line)
+{
+       struct source_line *iter;
+       struct rb_node **p = &root->rb_node;
+       struct rb_node *parent = NULL;
+
+       while (*p != NULL) {
+               parent = *p;
+               iter = rb_entry(parent, struct source_line, node);
+
+               if (cmp_source_line(src_line, iter))
+                       p = &(*p)->rb_left;
+               else
+                       p = &(*p)->rb_right;
+       }
+
+       rb_link_node(&src_line->node, parent, p);
+       rb_insert_color(&src_line->node, root);
+}
+
+static void resort_source_line(struct rb_root *dest_root, struct rb_root *src_root)
+{
+       struct source_line *src_line;
+       struct rb_node *node;
+
+       node = rb_first(src_root);
+       while (node) {
+               struct rb_node *next;
+
+               src_line = rb_entry(node, struct source_line, node);
+               next = rb_next(node);
+               rb_erase(node, src_root);
+
+               __resort_source_line(dest_root, src_line);
+               node = next;
+       }
+}
+
 static void symbol__free_source_line(struct symbol *sym, int len)
 {
        struct annotation *notes = symbol__annotation(sym);
        struct source_line *src_line = notes->src->lines;
+       size_t sizeof_src_line;
        int i;
 
-       for (i = 0; i < len; i++)
-               free(src_line[i].path);
+       sizeof_src_line = sizeof(*src_line) +
+                         (sizeof(src_line->p) * (src_line->nr_pcnt - 1));
 
-       free(src_line);
+       for (i = 0; i < len; i++) {
+               free(src_line->path);
+               src_line = (void *)src_line + sizeof_src_line;
+       }
+
+       free(notes->src->lines);
        notes->src->lines = NULL;
 }
 
 /* Get the filename:line for the colored entries */
 static int symbol__get_source_line(struct symbol *sym, struct map *map,
-                                  int evidx, struct rb_root *root, int len,
+                                  struct perf_evsel *evsel,
+                                  struct rb_root *root, int len,
                                   const char *filename)
 {
        u64 start;
-       int i;
+       int i, k;
+       int evidx = evsel->idx;
        char cmd[PATH_MAX * 2];
        struct source_line *src_line;
        struct annotation *notes = symbol__annotation(sym);
        struct sym_hist *h = annotation__histogram(notes, evidx);
+       struct rb_root tmp_root = RB_ROOT;
+       int nr_pcnt = 1;
+       u64 h_sum = h->sum;
+       size_t sizeof_src_line = sizeof(struct source_line);
+
+       if (perf_evsel__is_group_event(evsel)) {
+               for (i = 1; i < evsel->nr_members; i++) {
+                       h = annotation__histogram(notes, evidx + i);
+                       h_sum += h->sum;
+               }
+               nr_pcnt = evsel->nr_members;
+               sizeof_src_line += (nr_pcnt - 1) * sizeof(src_line->p);
+       }
 
-       if (!h->sum)
+       if (!h_sum)
                return 0;
 
-       src_line = notes->src->lines = calloc(len, sizeof(struct source_line));
+       src_line = notes->src->lines = calloc(len, sizeof_src_line);
        if (!notes->src->lines)
                return -1;
 
-       start = map->unmap_ip(map, sym->start);
+       start = map__rip_2objdump(map, sym->start);
 
        for (i = 0; i < len; i++) {
                char *path = NULL;
                size_t line_len;
                u64 offset;
                FILE *fp;
+               double percent_max = 0.0;
 
-               src_line[i].percent = 100.0 * h->addr[i] / h->sum;
-               if (src_line[i].percent <= 0.5)
-                       continue;
+               src_line->nr_pcnt = nr_pcnt;
+
+               for (k = 0; k < nr_pcnt; k++) {
+                       h = annotation__histogram(notes, evidx + k);
+                       src_line->p[k].percent = 100.0 * h->addr[i] / h->sum;
+
+                       if (src_line->p[k].percent > percent_max)
+                               percent_max = src_line->p[k].percent;
+               }
+
+               if (percent_max <= 0.5)
+                       goto next;
 
                offset = start + i;
                sprintf(cmd, "addr2line -e %s %016" PRIx64, filename, offset);
                fp = popen(cmd, "r");
                if (!fp)
-                       continue;
+                       goto next;
 
                if (getline(&path, &line_len, fp) < 0 || !line_len)
-                       goto next;
+                       goto next_close;
 
-               src_line[i].path = malloc(sizeof(char) * line_len + 1);
-               if (!src_line[i].path)
-                       goto next;
+               src_line->path = malloc(sizeof(char) * line_len + 1);
+               if (!src_line->path)
+                       goto next_close;
 
-               strcpy(src_line[i].path, path);
-               insert_source_line(root, &src_line[i]);
+               strcpy(src_line->path, path);
+               insert_source_line(&tmp_root, src_line);
 
-       next:
+       next_close:
                pclose(fp);
+       next:
+               src_line = (void *)src_line + sizeof_src_line;
        }
 
+       resort_source_line(root, &tmp_root);
        return 0;
 }
 
@@ -458,25 +1123,34 @@ static void print_summary(struct rb_root *root, const char *filename)
 
        node = rb_first(root);
        while (node) {
-               double percent;
+               double percent, percent_max = 0.0;
                const char *color;
                char *path;
+               int i;
 
                src_line = rb_entry(node, struct source_line, node);
-               percent = src_line->percent;
-               color = get_percent_color(percent);
+               for (i = 0; i < src_line->nr_pcnt; i++) {
+                       percent = src_line->p[i].percent_sum;
+                       color = get_percent_color(percent);
+                       color_fprintf(stdout, color, " %7.2f", percent);
+
+                       if (percent > percent_max)
+                               percent_max = percent;
+               }
+
                path = src_line->path;
+               color = get_percent_color(percent_max);
+               color_fprintf(stdout, color, " %s", path);
 
-               color_fprintf(stdout, color, " %7.2f %s", percent, path);
                node = rb_next(node);
        }
 }
 
-static void symbol__annotate_hits(struct symbol *sym, int evidx)
+static void symbol__annotate_hits(struct symbol *sym, struct perf_evsel *evsel)
 {
        struct annotation *notes = symbol__annotation(sym);
-       struct sym_hist *h = annotation__histogram(notes, evidx);
-       u64 len = sym->end - sym->start, offset;
+       struct sym_hist *h = annotation__histogram(notes, evsel->idx);
+       u64 len = symbol__size(sym), offset;
 
        for (offset = 0; offset < len; ++offset)
                if (h->addr[offset] != 0)
@@ -485,30 +1159,44 @@ static void symbol__annotate_hits(struct symbol *sym, int evidx)
        printf("%*s: %" PRIu64 "\n", BITS_PER_LONG / 2, "h->sum", h->sum);
 }
 
-int symbol__annotate_printf(struct symbol *sym, struct map *map, int evidx,
-                           bool full_paths, int min_pcnt, int max_lines,
-                           int context)
+int symbol__annotate_printf(struct symbol *sym, struct map *map,
+                           struct perf_evsel *evsel, bool full_paths,
+                           int min_pcnt, int max_lines, int context)
 {
        struct dso *dso = map->dso;
-       const char *filename = dso->long_name, *d_filename;
+       char *filename;
+       const char *d_filename;
        struct annotation *notes = symbol__annotation(sym);
-       struct objdump_line *pos, *queue = NULL;
+       struct disasm_line *pos, *queue = NULL;
+       u64 start = map__rip_2objdump(map, sym->start);
        int printed = 2, queue_len = 0;
        int more = 0;
        u64 len;
+       int width = 8;
+       int namelen;
+
+       filename = strdup(dso->long_name);
+       if (!filename)
+               return -ENOMEM;
 
        if (full_paths)
                d_filename = filename;
        else
                d_filename = basename(filename);
 
-       len = sym->end - sym->start;
+       len = symbol__size(sym);
+       namelen = strlen(d_filename);
 
-       printf(" Percent |      Source code & Disassembly of %s\n", d_filename);
-       printf("------------------------------------------------\n");
+       if (perf_evsel__is_group_event(evsel))
+               width *= evsel->nr_members;
+
+       printf(" %-*.*s|        Source code & Disassembly of %s\n",
+              width, width, "Percent", d_filename);
+       printf("-%-*.*s-------------------------------------\n",
+              width+namelen, width+namelen, graph_dotted_line);
 
        if (verbose)
-               symbol__annotate_hits(sym, evidx);
+               symbol__annotate_hits(sym, evsel);
 
        list_for_each_entry(pos, &notes->src->source, node) {
                if (context && queue == NULL) {
@@ -516,8 +1204,9 @@ int symbol__annotate_printf(struct symbol *sym, struct map *map, int evidx,
                        queue_len = 0;
                }
 
-               switch (objdump_line__print(pos, sym, evidx, len, min_pcnt,
-                                           printed, max_lines, queue)) {
+               switch (disasm_line__print(pos, sym, start, evsel, len,
+                                           min_pcnt, printed, max_lines,
+                                           queue)) {
                case 0:
                        ++printed;
                        if (context) {
@@ -546,6 +1235,8 @@ int symbol__annotate_printf(struct symbol *sym, struct map *map, int evidx,
                }
        }
 
+       free(filename);
+
        return more;
 }
 
@@ -561,32 +1252,56 @@ void symbol__annotate_decay_histogram(struct symbol *sym, int evidx)
 {
        struct annotation *notes = symbol__annotation(sym);
        struct sym_hist *h = annotation__histogram(notes, evidx);
-       struct objdump_line *pos;
-       int len = sym->end - sym->start;
+       int len = symbol__size(sym), offset;
 
        h->sum = 0;
-
-       list_for_each_entry(pos, &notes->src->source, node) {
-               if (pos->offset != -1 && pos->offset < len) {
-                       h->addr[pos->offset] = h->addr[pos->offset] * 7 / 8;
-                       h->sum += h->addr[pos->offset];
-               }
+       for (offset = 0; offset < len; ++offset) {
+               h->addr[offset] = h->addr[offset] * 7 / 8;
+               h->sum += h->addr[offset];
        }
 }
 
-void objdump_line_list__purge(struct list_head *head)
+void disasm__purge(struct list_head *head)
 {
-       struct objdump_line *pos, *n;
+       struct disasm_line *pos, *n;
 
        list_for_each_entry_safe(pos, n, head, node) {
                list_del(&pos->node);
-               objdump_line__free(pos);
+               disasm_line__free(pos);
+       }
+}
+
+static size_t disasm_line__fprintf(struct disasm_line *dl, FILE *fp)
+{
+       size_t printed;
+
+       if (dl->offset == -1)
+               return fprintf(fp, "%s\n", dl->line);
+
+       printed = fprintf(fp, "%#" PRIx64 " %s", dl->offset, dl->name);
+
+       if (dl->ops.raw[0] != '\0') {
+               printed += fprintf(fp, "%.*s %s\n", 6 - (int)printed, " ",
+                                  dl->ops.raw);
        }
+
+       return printed + fprintf(fp, "\n");
+}
+
+size_t disasm__fprintf(struct list_head *head, FILE *fp)
+{
+       struct disasm_line *pos;
+       size_t printed = 0;
+
+       list_for_each_entry(pos, head, node)
+               printed += disasm_line__fprintf(pos, fp);
+
+       return printed;
 }
 
-int symbol__tty_annotate(struct symbol *sym, struct map *map, int evidx,
-                        bool print_lines, bool full_paths, int min_pcnt,
-                        int max_lines)
+int symbol__tty_annotate(struct symbol *sym, struct map *map,
+                        struct perf_evsel *evsel, bool print_lines,
+                        bool full_paths, int min_pcnt, int max_lines)
 {
        struct dso *dso = map->dso;
        const char *filename = dso->long_name;
@@ -596,20 +1311,20 @@ int symbol__tty_annotate(struct symbol *sym, struct map *map, int evidx,
        if (symbol__annotate(sym, map, 0) < 0)
                return -1;
 
-       len = sym->end - sym->start;
+       len = symbol__size(sym);
 
        if (print_lines) {
-               symbol__get_source_line(sym, map, evidx, &source_line,
+               symbol__get_source_line(sym, map, evsel, &source_line,
                                        len, filename);
                print_summary(&source_line, filename);
        }
 
-       symbol__annotate_printf(sym, map, evidx, full_paths,
+       symbol__annotate_printf(sym, map, evsel, full_paths,
                                min_pcnt, max_lines, 0);
        if (print_lines)
                symbol__free_source_line(sym, len);
 
-       objdump_line_list__purge(&symbol__annotation(sym)->src->source);
+       disasm__purge(&symbol__annotation(sym)->src->source);
 
        return 0;
 }