lkml.org 
[lkml]   [2017]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3 6/6] perf script: support time percent and multiple time ranges
    Date
    perf script has a --time option to limit the time range of output.
    It only supports absolute time.

    Now this option is extended to support multiple time ranges and
    support the percent of time.

    For example:

    1. Select the first and second 10% time slices
    perf script --time 10%/1,10%/2

    2. Select from 0% to 10% and 30% to 40% slices
    perf script --time 0%-10%,30%-40%

    Change log:
    -----------
    v3: Since the definitions of first_sample_time/last_sample_time
    are moved from perf_session to perf_evlist so change the
    related code.

    Signed-off-by: Jin Yao <yao.jin@linux.intel.com>
    ---
    tools/perf/Documentation/perf-script.txt | 16 ++++++++++++++++
    tools/perf/builtin-script.c | 21 ++++++++++++++++++---
    2 files changed, 34 insertions(+), 3 deletions(-)

    diff --git a/tools/perf/Documentation/perf-script.txt b/tools/perf/Documentation/perf-script.txt
    index bcc1ba3..2c1f2b9 100644
    --- a/tools/perf/Documentation/perf-script.txt
    +++ b/tools/perf/Documentation/perf-script.txt
    @@ -321,6 +321,22 @@ include::itrace.txt[]
    stop time is not given (i.e, time string is 'x.y,') then analysis goes
    to end of file.

    + Also support time percent with multipe time range. Time string is
    + 'a%/n,b%/m,...' or 'a%-b%,c%-%d,...'. The maximum number of slices is 10.
    +
    + For example:
    + Select the second 10% time slice
    + perf script --time 10%/2
    +
    + Select from 0% to 10% time slice
    + perf script --time 0%-10%
    +
    + Select the first and second 10% time slices
    + perf script --time 10%/1,10%/2
    +
    + Select from 0% to 10% and 30% to 40% slices
    + perf script --time 0%-10%,30%-40%
    +
    --max-blocks::
    Set the maximum number of program blocks to print with brstackasm for
    each sample.
    diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
    index 9092de0..7fd3063 100644
    --- a/tools/perf/builtin-script.c
    +++ b/tools/perf/builtin-script.c
    @@ -1357,6 +1357,8 @@ static void print_sample_synth(struct perf_sample *sample,
    }
    }

    +#define PTIME_RANGE_MAX 10
    +
    struct perf_script {
    struct perf_tool tool;
    struct perf_session *session;
    @@ -1370,6 +1372,8 @@ struct perf_script {
    int name_width;
    const char *time_str;
    struct perf_time_interval ptime;
    + struct perf_time_interval ptime_range[PTIME_RANGE_MAX];
    + int range_num;
    };

    static int perf_evlist__max_name_len(struct perf_evlist *evlist)
    @@ -1565,8 +1569,11 @@ static int process_sample_event(struct perf_tool *tool,
    struct perf_script *scr = container_of(tool, struct perf_script, tool);
    struct addr_location al;

    - if (perf_time__skip_sample(&scr->ptime, sample->time))
    + if (perf_time__skip_sample(&scr->ptime, sample->time) ||
    + perf_time__ranges_skip_sample(scr->ptime_range, scr->range_num,
    + sample->time)) {
    return 0;
    + }

    if (debug_mode) {
    if (sample->time < last_timestamp) {
    @@ -3104,8 +3111,16 @@ int cmd_script(int argc, const char **argv)

    /* needs to be parsed after looking up reference time */
    if (perf_time__parse_str(&script.ptime, script.time_str) != 0) {
    - pr_err("Invalid time string\n");
    - return -EINVAL;
    + script.range_num = perf_time__percent_parse_str(
    + script.ptime_range, PTIME_RANGE_MAX,
    + script.time_str,
    + session->evlist->first_sample_time,
    + session->evlist->last_sample_time);
    +
    + if (script.range_num < 0) {
    + pr_err("Invalid time string\n");
    + return -EINVAL;
    + }
    }

    err = __cmd_script(&script);
    --
    2.7.4
    \
     
     \ /
      Last update: 2017-09-28 06:50    [W:2.202 / U:0.156 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site