diff options
| author |   Leo Yan <leo.yan@linaro.org> | 2020-10-14 06:09:16 +0100 |
|---|---|---|
| committer |   Arnaldo Carvalho de Melo <acme@redhat.com> | 2020-10-15 09:34:41 -0300 |
| commit | 6d662d730d6cdc16a19c4bc076c03e2cd21c242d (patch) | |
| tree | 6559b12edf54321928d38f0f2d5b5d9c83158e7d /tools/perf/builtin-c2c.c | |
| parent | perf c2c: Display "Total Stores" as a standalone metrics (diff) | |
| download | wireguard-linux-6d662d730d6cdc16a19c4bc076c03e2cd21c242d.tar.xz wireguard-linux-6d662d730d6cdc16a19c4bc076c03e2cd21c242d.zip | |
perf c2c: Organize metrics based on memory hierarchy
The metrics are not organized based on memory hierarchy, e.g. the tool
doesn't organize the metrics order based on memory nodes from the close
node (e.g. L1/L2 cache) to far node (e.g. L3 cache and DRAM).
To output metrics with more friendly form, this patch refines the
metrics order based on memory hierarchy:
"Core Load Hit" => "LLC Load Hit" => "LLC Ld Miss" => "Load Dram"
Signed-off-by: Leo Yan <leo.yan@linaro.org>
Tested-by: Joe Mario <jmario@redhat.com>
Acked-by: Jiri Olsa <jolsa@redhat.com>
Link: https://lore.kernel.org/r/20201014050921.5591-4-leo.yan@linaro.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | tools/perf/builtin-c2c.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/perf/builtin-c2c.c b/tools/perf/builtin-c2c.c index 51250f37ac9c..8947a8c48d5a 100644 --- a/tools/perf/builtin-c2c.c +++ b/tools/perf/builtin-c2c.c @@ -2852,10 +2852,10 @@ static int perf_c2c__report(int argc, const char **argv) "tot_loads," "tot_stores," "stores_l1hit,stores_l1miss," - "dram_lcl,dram_rmt," - "ld_llcmiss," "ld_fbhit,ld_l1hit,ld_l2hit," - "ld_lclhit,ld_rmthit", + "ld_lclhit,ld_rmthit," + "ld_llcmiss," + "dram_lcl,dram_rmt", c2c.display == DISPLAY_TOT ? "tot_hitm" : c2c.display == DISPLAY_LCL ? "lcl_hitm" : "rmt_hitm" ); |