2016-11-30 23:39:59

by Piotr Luc

Subject: [PATCH] perf/x86/intel: Enable C-state residency events for Knights Mill

The Knights Mill is enough close to Knights Landing so the path reuses
C-state residency support of the latter.

Signed-off-by: Piotr Luc <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: "H. Peter Anvin" <[email protected]>
Cc: [email protected]
Cc: [email protected]
---
arch/x86/events/intel/cstate.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/x86/events/intel/cstate.c b/arch/x86/events/intel/cstate.c
index 4f5ac72..da51e5a 100644
--- a/arch/x86/events/intel/cstate.c
+++ b/arch/x86/events/intel/cstate.c
@@ -540,6 +540,7 @@ static const struct x86_cpu_id intel_cstates_match[] __initconst = {
X86_CSTATES_MODEL(INTEL_FAM6_SKYLAKE_DESKTOP, snb_cstates),

X86_CSTATES_MODEL(INTEL_FAM6_XEON_PHI_KNL, knl_cstates),
+ X86_CSTATES_MODEL(INTEL_FAM6_XEON_PHI_KNM, knl_cstates),
{ },
};
MODULE_DEVICE_TABLE(x86cpu, intel_cstates_match);
--
2.10.1

2016-11-30 23:42:22

by Thomas Gleixner

Subject: Re: [PATCH] perf/x86/intel: Enable C-state residency events for Knights Mill

On Thu, 1 Dec 2016, Piotr Luc wrote:

> The Knights Mill is enough close to Knights Landing so the path reuses
> C-state residency support of the latter.
>
> Signed-off-by: Piotr Luc <[email protected]>
> Cc: Thomas Gleixner <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: "H. Peter Anvin" <[email protected]>
> Cc: [email protected]
> Cc: [email protected]

Please consult MAINTAINERS to figure out who should be on the CC list.

Thanks,

tglx

2016-11-30 23:57:28

by Piotr Luc

Subject: Re: [PATCH] perf/x86/intel: Enable C-state residency events for Knights Mill

On Thu, 2016-12-01 at 00:39 +0100, Thomas Gleixner wrote:
> On Thu, 1 Dec 2016, Piotr Luc wrote:
>
> >
> > The Knights Mill is enough close to Knights Landing so the path
> > reuses
> > C-state residency support of the latter.
> >
> > Signed-off-by: Piotr Luc <[email protected]>
> > Cc: Thomas Gleixner <[email protected]>
> > Cc: Ingo Molnar <[email protected]>
> > Cc: "H. Peter Anvin" <[email protected]>
> > Cc: [email protected]
> > Cc: [email protected]
>
> Please consult MAINTAINERS to figure out who should be on the CC
> list.
>
> Thanks,
>
> tglx

Thanks for picking this up.
I counted on get_maintainers.pl script, but it seems it used the
generic rule instead the specific one. I will correct CC list right
now.

Thanks
Piotr 

2016-12-01 00:09:22

by Piotr Luc

Subject: [RESEND PATCH] perf/x86/intel: Enable C-state residency events for Knights Mill

The Knights Mill is enough close to Knights Landing so the path reuses
C-state residency support of the latter.

Signed-off-by: Piotr Luc <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Arnaldo Carvalho de Melo <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: [email protected]
Cc: [email protected]
---
arch/x86/events/intel/cstate.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/x86/events/intel/cstate.c b/arch/x86/events/intel/cstate.c
index 4f5ac72..da51e5a 100644
--- a/arch/x86/events/intel/cstate.c
+++ b/arch/x86/events/intel/cstate.c
@@ -540,6 +540,7 @@ static const struct x86_cpu_id intel_cstates_match[] __initconst = {
X86_CSTATES_MODEL(INTEL_FAM6_SKYLAKE_DESKTOP, snb_cstates),

X86_CSTATES_MODEL(INTEL_FAM6_XEON_PHI_KNL, knl_cstates),
+ X86_CSTATES_MODEL(INTEL_FAM6_XEON_PHI_KNM, knl_cstates),
{ },
};
MODULE_DEVICE_TABLE(x86cpu, intel_cstates_match);
--
2.10.1

Fixed CC list.

+ Cc: Peter Zijlstra <[email protected]>
+ Cc: Arnaldo Carvalho de Melo <[email protected]>
+ Cc: Alexander Shishkin <[email protected]>
- Cc: Thomas Gleixner <[email protected]>
- Cc: "H. Peter Anvin" <[email protected]>

2016-12-06 10:13:38

by tip-bot for Vasyl Gomonovych

Subject: [tip:perf/urgent] perf/x86/intel: Enable C-state residency events for Knights Mill

Commit-ID: 1dba23b12f49d7cf3d4504171c62541122b55141
Gitweb: http://git.kernel.org/tip/1dba23b12f49d7cf3d4504171c62541122b55141
Author: Piotr Luc <[email protected]>
AuthorDate: Thu, 1 Dec 2016 01:08:53 +0100
Committer: Ingo Molnar <[email protected]>
CommitDate: Tue, 6 Dec 2016 09:44:27 +0100

perf/x86/intel: Enable C-state residency events for Knights Mill

The Knights Mill is enough close to Knights Landing so the path reuses
C-state residency support of the latter.

Signed-off-by: Piotr Luc <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: Arnaldo Carvalho de Melo <[email protected]>
Cc: Arnaldo Carvalho de Melo <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Stephane Eranian <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Cc: Vince Weaver <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Ingo Molnar <[email protected]>
---
arch/x86/events/intel/cstate.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/x86/events/intel/cstate.c b/arch/x86/events/intel/cstate.c
index 4f5ac72..da51e5a 100644
--- a/arch/x86/events/intel/cstate.c
+++ b/arch/x86/events/intel/cstate.c
@@ -540,6 +540,7 @@ static const struct x86_cpu_id intel_cstates_match[] __initconst = {
X86_CSTATES_MODEL(INTEL_FAM6_SKYLAKE_DESKTOP, snb_cstates),

X86_CSTATES_MODEL(INTEL_FAM6_XEON_PHI_KNL, knl_cstates),
+ X86_CSTATES_MODEL(INTEL_FAM6_XEON_PHI_KNM, knl_cstates),
{ },
};
MODULE_DEVICE_TABLE(x86cpu, intel_cstates_match);