lkml.org 
[lkml]   [2011]   [May]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip:perf/urgent] perf tools: Honour the cpu list parameter when also monitoring a thread list
    Commit-ID:  b90194181988063266f3da0b7bf3e57268c627c8
    Gitweb: http://git.kernel.org/tip/b90194181988063266f3da0b7bf3e57268c627c8
    Author: Arnaldo Carvalho de Melo <acme@redhat.com>
    AuthorDate: Mon, 25 Apr 2011 16:25:20 -0300
    Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
    CommitDate: Sun, 15 May 2011 09:32:52 -0300

    perf tools: Honour the cpu list parameter when also monitoring a thread list

    The perf_evlist__create_maps was discarding the --cpu parameter when a
    --pid or --tid was specified, fix that.

    Cc: Frederic Weisbecker <fweisbec@gmail.com>
    Cc: Ingo Molnar <mingo@elte.hu>
    Cc: Mike Galbraith <efault@gmx.de>
    Cc: Paul Mackerras <paulus@samba.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Stephane Eranian <eranian@google.com>
    Cc: Tom Zanussi <tzanussi@gmail.com>
    Link: http://lkml.kernel.org/r/20110426204401.GB1746@ghostprotocols.net
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/util/evlist.c | 2 +-
    1 files changed, 1 insertions(+), 1 deletions(-)

    diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c
    index 45da8d1..1884a7c 100644
    --- a/tools/perf/util/evlist.c
    +++ b/tools/perf/util/evlist.c
    @@ -348,7 +348,7 @@ int perf_evlist__create_maps(struct perf_evlist *evlist, pid_t target_pid,
    if (evlist->threads == NULL)
    return -1;

    - if (target_tid != -1)
    + if (cpu_list == NULL && target_tid != -1)
    evlist->cpus = cpu_map__dummy_new();
    else
    evlist->cpus = cpu_map__new(cpu_list);

    \
     
     \ /
      Last update: 2011-05-15 19:53    [W:3.301 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site