* [PATCH] perf, tools: Fix display of data source snoop indication
@ 2017-04-19 17:49 Andi Kleen
2017-04-19 18:46 ` Arnaldo Carvalho de Melo
2017-04-24 21:10 ` [tip:perf/core] perf mem: " tip-bot for Andi Kleen
0 siblings, 2 replies; 4+ messages in thread
From: Andi Kleen @ 2017-04-19 17:49 UTC (permalink / raw)
To: acme; +Cc: jolsa, linux-kernel, Andi Kleen, eranian
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
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] perf, tools: Fix display of data source snoop indication
2017-04-19 17:49 [PATCH] perf, tools: Fix display of data source snoop indication Andi Kleen
@ 2017-04-19 18:46 ` Arnaldo Carvalho de Melo
2017-04-19 18:50 ` Andi Kleen
2017-04-24 21:10 ` [tip:perf/core] perf mem: " tip-bot for Andi Kleen
1 sibling, 1 reply; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2017-04-19 18:46 UTC (permalink / raw)
To: Andi Kleen; +Cc: jolsa, linux-kernel, Andi Kleen, eranian
Em Wed, Apr 19, 2017 at 10:49:40AM -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
missing something?
> 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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] perf, tools: Fix display of data source snoop indication
2017-04-19 18:46 ` Arnaldo Carvalho de Melo
@ 2017-04-19 18:50 ` Andi Kleen
0 siblings, 0 replies; 4+ messages in thread
From: Andi Kleen @ 2017-04-19 18:50 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo; +Cc: Andi Kleen, jolsa, linux-kernel, eranian
On Wed, Apr 19, 2017 at 03:46:13PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Wed, Apr 19, 2017 at 10:49:40AM -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
>
> missing something?
#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.
> >
> > 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
^ permalink raw reply [flat|nested] 4+ messages in thread
* [tip:perf/core] perf mem: Fix display of data source snoop indication
2017-04-19 17:49 [PATCH] perf, tools: Fix display of data source snoop indication Andi Kleen
2017-04-19 18:46 ` Arnaldo Carvalho de Melo
@ 2017-04-24 21:10 ` tip-bot for Andi Kleen
1 sibling, 0 replies; 4+ messages in thread
From: tip-bot for Andi Kleen @ 2017-04-24 21:10 UTC (permalink / raw)
To: linux-tip-commits
Cc: linux-kernel, hpa, ak, tglx, mingo, acme, eranian, jolsa
Commit-ID: 166ebdd2442660e7f942d657fc5e629000e58ec3
Gitweb: http://git.kernel.org/tip/166ebdd2442660e7f942d657fc5e629000e58ec3
Author: Andi Kleen <ak@linux.intel.com>
AuthorDate: Wed, 19 Apr 2017 10:49:40 -0700
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Mon, 24 Apr 2017 12:33:32 -0300
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>
---
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 c56d52f..06f5a3a 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)
static const char * const snoop_access[] = {
"N/A",
"None",
- "Miss",
"Hit",
+ "Miss",
"HitM",
};
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-04-24 21:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-19 17:49 [PATCH] perf, tools: Fix display of data source snoop indication Andi Kleen
2017-04-19 18:46 ` Arnaldo Carvalho de Melo
2017-04-19 18:50 ` Andi Kleen
2017-04-24 21:10 ` [tip:perf/core] perf mem: " tip-bot for Andi Kleen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).