perf report: Use perf_evlist__force_leader to support '--group'
authorJin Yao <yao.jin@linux.intel.com>
Mon, 21 May 2018 14:57:45 +0000 (22:57 +0800)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 21 May 2018 17:41:01 +0000 (14:41 -0300)
Since we created a new function perf_evlist__force_leader(), remove the
old code and use that new evlist method.

Signed-off-by: Jin Yao <yao.jin@linux.intel.com>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1526914666-31839-3-git-send-email-yao.jin@linux.intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-report.c

index 4c931afb2e80dd9c6e53a33480fe1ce8de5a21fb..ad978e3ee2b8708dddffaf513fc39b393962e8eb 100644 (file)
@@ -194,20 +194,11 @@ out:
        return err;
 }
 
-/*
- * Events in data file are not collect in groups, but we still want
- * the group display. Set the artificial group and set the leader's
- * forced_leader flag to notify the display code.
- */
 static void setup_forced_leader(struct report *report,
                                struct perf_evlist *evlist)
 {
-       if (report->group_set && !evlist->nr_groups) {
-               struct perf_evsel *leader = perf_evlist__first(evlist);
-
-               perf_evlist__set_leader(evlist);
-               leader->forced_leader = true;
-       }
+       if (report->group_set)
+               perf_evlist__force_leader(evlist);
 }
 
 static int process_feature_event(struct perf_tool *tool,