Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c8280cec authored by Jiri Olsa's avatar Jiri Olsa Committed by Arnaldo Carvalho de Melo
Browse files

perf stat: Move enable_on_exec setup under earlier code 



It's more readable this way and we can save one
perf_evsel__is_group_leader condition in current code.

Signed-off-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1449133606-14429-7-git-send-email-jolsa@kernel.org


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 67ccdecd

tools/perf/builtin-stat.c

+9 −6
Original line number Diff line number Diff line
@@ -173,17 +173,20 @@ static int create_perf_stat_counter(struct perf_evsel *evsel) 
	 * either manually by us or by kernel via enable_on_exec

	 * set later.

	 */

	if (perf_evsel__is_group_leader(evsel))

	if (perf_evsel__is_group_leader(evsel)) {

		attr->disabled = 1;



	if (target__has_cpu(&target))

		return perf_evsel__open_per_cpu(evsel, perf_evsel__cpus(evsel));



	if (!target__has_task(&target) && perf_evsel__is_group_leader(evsel)) {

		if (!initial_delay)

		/*

		 * In case of initial_delay we enable tracee

		 * events manually.

		 */

		if (target__none(&target) && !initial_delay)

			attr->enable_on_exec = 1;

	}



	if (target__has_cpu(&target))

		return perf_evsel__open_per_cpu(evsel, perf_evsel__cpus(evsel));



	return perf_evsel__open_per_thread(evsel, evsel_list->threads);

}