lkml.org 
[lkml]   [2016]   [Oct]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 29/68] perf c2c report: Add hitm percent sort key
    Date
    From: Jiri Olsa <jolsa@kernel.org>

    It is to be displayed in the main cachelines overall output:

    percent_hitm

    It displays HITMs percentage for cacheline.

    It counts remote HITMs at the moment, but it is changed later to support
    local as well, based on the sort configuration.

    Signed-off-by: Jiri Olsa <jolsa@kernel.org>
    Cc: Andi Kleen <andi@firstfloor.org>
    Cc: David Ahern <dsahern@gmail.com>
    Cc: Don Zickus <dzickus@redhat.com>
    Cc: Joe Mario <jmario@redhat.com>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
    Link: http://lkml.kernel.org/n/tip-czd17qsh5u5z0yc1estz9l2y@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/builtin-c2c.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++
    1 file changed, 91 insertions(+)

    diff --git a/tools/perf/builtin-c2c.c b/tools/perf/builtin-c2c.c
    index 2411fe025bc7..dd356d88285c 100644
    --- a/tools/perf/builtin-c2c.c
    +++ b/tools/perf/builtin-c2c.c
    @@ -565,6 +565,87 @@ tot_loads_cmp(struct perf_hpp_fmt *fmt __maybe_unused,
    return tot_recs_left - tot_recs_right;
    }

    +typedef double (get_percent_cb)(struct c2c_hist_entry *);
    +
    +static int
    +percent_color(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp,
    + struct hist_entry *he, get_percent_cb get_percent)
    +{
    + struct c2c_hist_entry *c2c_he;
    + int width = c2c_width(fmt, hpp, he->hists);
    + double per;
    +
    + c2c_he = container_of(he, struct c2c_hist_entry, he);
    + per = get_percent(c2c_he);
    +
    + return hpp_color_scnprintf(hpp, "%*.2f%%", width - 1, per);
    +}
    +
    +static double percent_hitm(struct c2c_hist_entry *c2c_he)
    +{
    + struct c2c_hists *hists;
    + struct c2c_stats *stats;
    + struct c2c_stats *total;
    + int tot, st;
    + double p;
    +
    + hists = container_of(c2c_he->he.hists, struct c2c_hists, hists);
    + stats = &c2c_he->stats;
    + total = &hists->stats;
    +
    + st = stats->rmt_hitm;
    + tot = total->rmt_hitm;
    +
    + p = tot ? (double) st / tot : 0;
    +
    + return 100 * p;
    +}
    +
    +#define PERC_STR(__s, __v) \
    +({ \
    + scnprintf(__s, sizeof(__s), "%.2F%%", __v); \
    + __s; \
    +})
    +
    +static int
    +percent_hitm_entry(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp,
    + struct hist_entry *he)
    +{
    + struct c2c_hist_entry *c2c_he;
    + int width = c2c_width(fmt, hpp, he->hists);
    + char buf[10];
    + double per;
    +
    + c2c_he = container_of(he, struct c2c_hist_entry, he);
    + per = percent_hitm(c2c_he);
    + return scnprintf(hpp->buf, hpp->size, "%*s", width, PERC_STR(buf, per));
    +}
    +
    +static int
    +percent_hitm_color(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp,
    + struct hist_entry *he)
    +{
    + return percent_color(fmt, hpp, he, percent_hitm);
    +}
    +
    +static int64_t
    +percent_hitm_cmp(struct perf_hpp_fmt *fmt __maybe_unused,
    + struct hist_entry *left, struct hist_entry *right)
    +{
    + struct c2c_hist_entry *c2c_left;
    + struct c2c_hist_entry *c2c_right;
    + double per_left;
    + double per_right;
    +
    + c2c_left = container_of(left, struct c2c_hist_entry, he);
    + c2c_right = container_of(right, struct c2c_hist_entry, he);
    +
    + per_left = percent_hitm(c2c_left);
    + per_right = percent_hitm(c2c_right);
    +
    + return per_left - per_right;
    +}
    +
    #define HEADER_LOW(__h) \
    { \
    .line[1] = { \
    @@ -768,6 +849,15 @@ static struct c2c_dimension dim_tot_loads = {
    .width = 7,
    };

    +static struct c2c_dimension dim_percent_hitm = {
    + .header = HEADER_LOW("%hitm"),
    + .name = "percent_hitm",
    + .cmp = percent_hitm_cmp,
    + .entry = percent_hitm_entry,
    + .color = percent_hitm_color,
    + .width = 7,
    +};
    +
    static struct c2c_dimension *dimensions[] = {
    &dim_dcacheline,
    &dim_offset,
    @@ -790,6 +880,7 @@ static struct c2c_dimension *dimensions[] = {
    &dim_ld_llcmiss,
    &dim_tot_recs,
    &dim_tot_loads,
    + &dim_percent_hitm,
    NULL,
    };

    --
    2.7.4
    \
     
     \ /
      Last update: 2016-10-11 19:40    [W:4.424 / U:0.044 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site