Commit ffb273dd authored Aug 07, 2013 by David Ahern Committed by Arnaldo Carvalho de Melo Aug 12, 2013

perf sched: Remove thread lookup in sample handler



Not used in the function, so no sense in doing the lookup here. Thread
look up will be done in the timehist command, and no sense in doing it
twice.

Signed-off-by: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1375930261-77273-4-git-send-email-dsahern@gmail.com


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

parent ad9def7c

tools/perf/builtin-sched.c

+0 −7

Original line number	Diff line number	Diff line
		@@ -1425,15 +1425,8 @@ static int perf_sched__process_tracepoint_sample(struct perf_tool *tool __maybe_
		struct perf_evsel *evsel,
		struct machine *machine)
		{
		struct thread *thread = machine__findnew_thread(machine, sample->tid);
		int err = 0;

		if (thread == NULL) {
		pr_debug("problem processing %s event, skipping it.\n",
		perf_evsel__name(evsel));
		return -1;
		}

		evsel->hists.stats.total_period += sample->period;
		hists__inc_nr_events(&evsel->hists, PERF_RECORD_SAMPLE);