lkml.org 
[lkml]   [2013]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 04/21] perf sched: Remove thread lookup in sample handler
    Date
    From: David Ahern <dsahern@gmail.com>

    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>
    ---
    tools/perf/builtin-sched.c | 7 -------
    1 file changed, 7 deletions(-)

    diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c
    index 34ce57d..5285024 100644
    --- a/tools/perf/builtin-sched.c
    +++ b/tools/perf/builtin-sched.c
    @@ -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);

    --
    1.8.1.4


    \
     
     \ /
      Last update: 2013-08-15 09:01    [W:3.849 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site