Re: [PATCH v6 09/25] perf hists: Basic support of hierarchical report view

From: Namhyung Kim
Date: Sun Feb 21 2016 - 14:59:02 EST


Hi Jiri,

On Sun, Feb 21, 2016 at 12:18:45AM +0100, Jiri Olsa wrote:
> On Tue, Feb 16, 2016 at 11:08:27PM +0900, Namhyung Kim wrote:
>
> SNIP
>
> > + int64_t cmp;
> > +
> > + while (*p != NULL) {
> > + parent = *p;
> > + iter = rb_entry(parent, struct hist_entry, rb_node_in);
> > +
> > + cmp = fmt->collapse(fmt, iter, he);
> > + if (!cmp) {
> > + he_stat__add_stat(&iter->stat, &he->stat);
> > + return iter;
> > + }
> > +
> > + if (cmp < 0)
> > + p = &parent->rb_left;
> > + else
> > + p = &parent->rb_right;
> > + }
> > +
> > + new = hist_entry__new(he, true);
> > + if (new == NULL)
> > + return NULL;
> > +
> > + hists__apply_filters(hists, new);
> > + hists->nr_entries++;
> > +
> > + /* save related format for output */
> > + new->fmt = fmt;
> > +
> > + /* it's now passed to 'new' */
> > + he->trace_output = NULL;
>
> should you do the same to he->srcline ?

Right! And he->srcflie too.. Actually they needs to be passed to a
corresponding fmt rather than the first one. Will change to that
direction in the v7.

Thanks,
Namhyung