The __perf_evlist__set_leader() was setting the leader for all events in the list except the first. Which means it assumed the first event already had event->leader = event. Seems like this should be the role of the function to also do this. This is a requirement for an upcoming patch set.
Signed-off-by: Stephane Eranian <[email protected]> --- diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c index dc8aee9..050d5bc 100644 --- a/tools/perf/util/evlist.c +++ b/tools/perf/util/evlist.c @@ -119,8 +119,7 @@ void __perf_evlist__set_leader(struct list_head *list) leader = list_entry(list->next, struct perf_evsel, node); list_for_each_entry(evsel, list, node) { - if (evsel != leader) - evsel->leader = leader; + evsel->leader = leader; } } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
