On Fri, Feb 05, 2016 at 10:01:46PM +0900, Namhyung Kim wrote: SNIP
> + printed += fprintf(fp, "%s\n", buf); > + > + if (symbol_conf.use_callchain && he->leaf) { > + u64 total = hists__total_period(hists); > + > + printed += hist_entry_callchain__fprintf(he, total, 0, fp); > + goto out; > + } > + > +out: > + return printed; > +} > + > static int hist_entry__fprintf(struct hist_entry *he, size_t size, > struct hists *hists, > char *bf, size_t bfsz, FILE *fp) > @@ -423,6 +488,13 @@ static int hist_entry__fprintf(struct hist_entry *he, > size_t size, > if (size == 0 || size > bfsz) > size = hpp.size = bfsz; > > + if (symbol_conf.report_hierarchy) { it'd be great for review to have symbol_conf.report_hierarchy already present, so I could try it early in the patchset like with this change jirka