linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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; 6+ 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	[flat|nested] 6+ 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; 6+ 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] 6+ 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; 6+ 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] 6+ 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; 6+ 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	[flat|nested] 6+ messages in thread

* Re: [PATCH] perf, tools: Fix display of data source snoop indication
  2017-04-19 19:53 [PATCH] perf, tools: " Andi Kleen
@ 2017-04-19 21:53 ` Arnaldo Carvalho de Melo
  0 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2017-04-19 21:53 UTC (permalink / raw)
  To: Andi Kleen; +Cc: jolsa, linux-kernel, Andi Kleen, eranian

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] perf, tools: Fix display of data source snoop indication
@ 2017-04-19 19:53 Andi Kleen
  2017-04-19 21:53 ` Arnaldo Carvalho de Melo
  0 siblings, 1 reply; 6+ messages in thread
From: Andi Kleen @ 2017-04-19 19:53 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	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-04-24 21:12 UTC | newest]

Thread overview: 6+ 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
2017-04-19 19:53 [PATCH] perf, tools: " Andi Kleen
2017-04-19 21:53 ` Arnaldo Carvalho de Melo

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).