Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1626753 > unrolled thread
| Started by | Andi Kleen <andi@firstfloor.org> |
|---|---|
| First post | 2017-04-19 22:00 +0200 |
| Last post | 2017-04-20 00:00 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] perf, tools: Fix display of data source snoop indication Andi Kleen <andi@firstfloor.org> - 2017-04-19 22:00 +0200
Re: [PATCH] perf, tools: Fix display of data source snoop indication Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-04-20 00:00 +0200
| From | Andi Kleen <andi@firstfloor.org> |
|---|---|
| Date | 2017-04-19 22:00 +0200 |
| Subject | [PATCH] perf, tools: Fix display of data source snoop indication |
| Message-ID | <ty3VE-7vf-41@gated-at.bofh.it> |
From: Andi Kleen <ak@linux.intel.com>
perf mem report doesn't display the data source snoop indication correctly.
In the kernel API the definition is
but the table used by the perf tools exchanged Hit and Miss
"None",
"Miss",
"Hit",
Fix the table in perf.
Cc: eranian@google.com
Signed-off-by: Andi Kleen <ak@linux.intel.com>
---
tools/perf/util/mem-events.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/util/mem-events.c b/tools/perf/util/mem-events.c
index 1d4ab53c60ca..865ba3c39284 100644
--- a/tools/perf/util/mem-events.c
+++ b/tools/perf/util/mem-events.c
@@ -205,8 +205,8 @@ int perf_mem__lvl_scnprintf(char *out, size_t sz, struct mem_info *mem_info)
static const char * const snoop_access[] = {
"N/A",
"None",
- "Miss",
"Hit",
+ "Miss",
"HitM",
};
--
2.9.3
[toc] | [next] | [standalone]
| From | Arnaldo Carvalho de Melo <acme@kernel.org> |
|---|---|
| Date | 2017-04-20 00:00 +0200 |
| Message-ID | <ty5NL-b4-3@gated-at.bofh.it> |
| In reply to | #1626753 |
Em Wed, Apr 19, 2017 at 12:53:37PM -0700, Andi Kleen escreveu:
> From: Andi Kleen <ak@linux.intel.com>
>
> perf mem report doesn't display the data source snoop indication correctly.
>
> In the kernel API the definition is
I fixed this already, prepending the lines starting with # with two
spaces, so that it doesn't vanishes one 3rd time 8-)
- Arnaldo
> but the table used by the perf tools exchanged Hit and Miss
>
> "None",
> "Miss",
> "Hit",
>
> Fix the table in perf.
>
> Cc: eranian@google.com
> Signed-off-by: Andi Kleen <ak@linux.intel.com>
> ---
> tools/perf/util/mem-events.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/util/mem-events.c b/tools/perf/util/mem-events.c
> index 1d4ab53c60ca..865ba3c39284 100644
> --- a/tools/perf/util/mem-events.c
> +++ b/tools/perf/util/mem-events.c
> @@ -205,8 +205,8 @@ int perf_mem__lvl_scnprintf(char *out, size_t sz, struct mem_info *mem_info)
> static const char * const snoop_access[] = {
> "N/A",
> "None",
> - "Miss",
> "Hit",
> + "Miss",
> "HitM",
> };
>
> --
> 2.9.3
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web