Subject: [PATCH] powercap / RAPL: Support Knights Landing

This patch enables intel_rapl driver to run on the
next-generation Intel(R) Xeon Phi Microarchitecture
code named "Knights Landing"

Signed-off-by: Dasaratharaman Chandramouli <[email protected]>
---
drivers/powercap/intel_rapl.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c
index fd24323..5796332 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -1069,6 +1069,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
+ RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing */
{}
};
MODULE_DEVICE_TABLE(x86cpu, rapl_ids);
--
1.8.1.5


2015-05-19 23:06:38

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] powercap / RAPL: Support Knights Landing

On Tuesday, May 19, 2015 03:37:49 PM Dasaratharaman Chandramouli wrote:
> This patch enables intel_rapl driver to run on the
> next-generation Intel(R) Xeon Phi Microarchitecture
> code named "Knights Landing"
>
> Signed-off-by: Dasaratharaman Chandramouli <[email protected]>

Jacob?

> ---
> drivers/powercap/intel_rapl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c
> index fd24323..5796332 100644
> --- a/drivers/powercap/intel_rapl.c
> +++ b/drivers/powercap/intel_rapl.c
> @@ -1069,6 +1069,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
> RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
> RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
> RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
> + RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing */
> {}
> };
> MODULE_DEVICE_TABLE(x86cpu, rapl_ids);
>

--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.

2015-05-20 00:10:10

by Jacob Pan

[permalink] [raw]
Subject: Re: [PATCH] powercap / RAPL: Support Knights Landing

On Wed, 20 May 2015 01:31:54 +0200
"Rafael J. Wysocki" <[email protected]> wrote:

> On Tuesday, May 19, 2015 03:37:49 PM Dasaratharaman Chandramouli
> wrote:
> > This patch enables intel_rapl driver to run on the
> > next-generation Intel(R) Xeon Phi Microarchitecture
> > code named "Knights Landing"
> >
> > Signed-off-by: Dasaratharaman Chandramouli
> > <[email protected]>
>
> Jacob?
looks good.
Acked-by:Jacob Pan <[email protected]>
>
> > ---
> > drivers/powercap/intel_rapl.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/powercap/intel_rapl.c
> > b/drivers/powercap/intel_rapl.c index fd24323..5796332 100644
> > --- a/drivers/powercap/intel_rapl.c
> > +++ b/drivers/powercap/intel_rapl.c
> > @@ -1069,6 +1069,7 @@ static const struct x86_cpu_id rapl_ids[]
> > __initconst = { RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
> > RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
> > RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
> > + RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights
> > Landing */ {}
> > };
> > MODULE_DEVICE_TABLE(x86cpu, rapl_ids);
> >
>

[Jacob Pan]

2015-05-21 00:36:13

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] powercap / RAPL: Support Knights Landing

On Tuesday, May 19, 2015 05:10:05 PM Jacob Pan wrote:
> On Wed, 20 May 2015 01:31:54 +0200
> "Rafael J. Wysocki" <[email protected]> wrote:
>
> > On Tuesday, May 19, 2015 03:37:49 PM Dasaratharaman Chandramouli
> > wrote:
> > > This patch enables intel_rapl driver to run on the
> > > next-generation Intel(R) Xeon Phi Microarchitecture
> > > code named "Knights Landing"
> > >
> > > Signed-off-by: Dasaratharaman Chandramouli
> > > <[email protected]>
> >
> > Jacob?
> looks good.
> Acked-by:Jacob Pan <[email protected]>

Missing space in the tag.

Anyway, queued up for 4.2, thanks!


--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.