Commit 0c788d47 authored Jun 15, 2017 by Kim Phillips Committed by Arnaldo Carvalho de Melo Jun 19, 2017

perf coresight: Remove superfluous check before use



The cs_etm_evsel variable is guaranteed to be set at this point in
cs_etm_recording_options().

Signed-off-by: Kim Phillips <kim.phillips@arm.com>
Acked-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: linux-arm-kernel@lists.infradead.org
Link: http://lkml.kernel.org/r/20170615125521.80cc128dc856bc1f2e61b730@arm.com


Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

parent 5c97cac6

tools/perf/arch/arm/util/cs-etm.c

+12 −13

Original line number	Diff line number	Diff line
		@@ -203,19 +203,18 @@ static int cs_etm_recording_options(struct auxtrace_record *itr,
		pr_debug2("%s snapshot size: %zu\n", CORESIGHT_ETM_PMU_NAME,
		opts->auxtrace_snapshot_size);

		if (cs_etm_evsel) {
		/*
		* To obtain the auxtrace buffer file descriptor, the auxtrace
		* event must come first.
		*/
		perf_evlist__to_front(evlist, cs_etm_evsel);

		/*
		* In the case of per-cpu mmaps, we need the CPU on the
		* AUX event.
		*/
		if (!cpu_map__empty(cpus))
		perf_evsel__set_sample_bit(cs_etm_evsel, CPU);
		}

		/* Add dummy event to keep tracking */
		if (opts->full_auxtrace) {