lkml.org 
[lkml]   [2010]   [Mar]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip:perf/urgent] perf record: Add ID and to recorded event data when recording multiple events
    Commit-ID:  8907fd607b66e36636469a2de9833db643869db8
    Gitweb: http://git.kernel.org/tip/8907fd607b66e36636469a2de9833db643869db8
    Author: Eric B Munson <ebmunson@us.ibm.com>
    AuthorDate: Fri, 5 Mar 2010 12:51:05 -0300
    Committer: Ingo Molnar <mingo@elte.hu>
    CommitDate: Wed, 10 Mar 2010 13:53:46 +0100

    perf record: Add ID and to recorded event data when recording multiple events

    Currently perf record does not write the ID or the to disk for
    events. This doesn't allow report to tell if an event stream
    contains one or more types of events. This patch adds this
    entry to the list of data that record will write to disk if more
    than one event was requested.

    Signed-off-by: Eric B Munson <ebmunson@us.ibm.com>
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
    Cc: Paul Mackerras <paulus@samba.org>
    LKML-Reference: <1267804269-22660-2-git-send-email-acme@infradead.org>
    Signed-off-by: Ingo Molnar <mingo@elte.hu>
    ---
    tools/perf/builtin-record.c | 3 +++
    1 files changed, 3 insertions(+), 0 deletions(-)

    diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c
    index 771533c..f573bbb 100644
    --- a/tools/perf/builtin-record.c
    +++ b/tools/perf/builtin-record.c
    @@ -244,6 +244,9 @@ static void create_counter(int counter, int cpu, pid_t pid)

    attr->sample_type |= PERF_SAMPLE_IP | PERF_SAMPLE_TID;

    + if (nr_counters > 1)
    + attr->sample_type |= PERF_SAMPLE_ID;
    +
    if (freq) {
    attr->sample_type |= PERF_SAMPLE_PERIOD;
    attr->freq = 1;

    \
     
     \ /
      Last update: 2010-03-10 14:17    [W:0.025 / U:60.104 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site