diff options
author | Andi Kleen <ak@linux.intel.com> | 2017-04-19 13:49:40 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-04-24 11:33:32 -0400 |
commit | 166ebdd2442660e7f942d657fc5e629000e58ec3 (patch) | |
tree | 566e6548905768ed5065640064097b91266af53d /tools/perf | |
parent | 8c2b7cac78e17886e8089389a570a290c9b5ca67 (diff) |
perf mem: Fix display of data source snoop indication
'perf mem report' doesn't display the data source snoop indication correctly.
In the kernel API the definition is:
#define PERF_MEM_SNOOP_NONE 0x02 /* no snoop */
#define PERF_MEM_SNOOP_HIT 0x04 /* snoop hit */
#define PERF_MEM_SNOOP_MISS 0x08 /* snoop miss */
but the table used by the perf tools exchanged "Hit" and "Miss":
"None",
"Miss",
"Hit",
Fix the table in perf.
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/20170419174940.13641-1-andi@firstfloor.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/util/mem-events.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/mem-events.c b/tools/perf/util/mem-events.c index c56d52f90b54..06f5a3a4295c 100644 --- a/tools/perf/util/mem-events.c +++ b/tools/perf/util/mem-events.c | |||
@@ -206,8 +206,8 @@ int perf_mem__lvl_scnprintf(char *out, size_t sz, struct mem_info *mem_info) | |||
206 | static const char * const snoop_access[] = { | 206 | static const char * const snoop_access[] = { |
207 | "N/A", | 207 | "N/A", |
208 | "None", | 208 | "None", |
209 | "Miss", | ||
210 | "Hit", | 209 | "Hit", |
210 | "Miss", | ||
211 | "HitM", | 211 | "HitM", |
212 | }; | 212 | }; |
213 | 213 | ||