diff mbox

[RESEND,10/11] powercap / RAPL: Add Knights Mill CPUID

Message ID 20161013153105.2517-11-piotr.luc@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Rafael Wysocki
Headers show

Commit Message

Piotr Luc Oct. 13, 2016, 3:31 p.m. UTC
Add Knights Mill (KNM) to the list of CPUIDs supported by intel_rapl

Signed-off-by: Piotr Luc <piotr.luc@intel.com>
Reviewed-by: Dave Hansen <dave.hansen@intel.com>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: linux-pm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mill 
to Intel family

 drivers/powercap/intel_rapl.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Piotr Luc Oct. 25, 2016, 10:41 a.m. UTC | #1
SGkgUmFmYWVsLA0KDQpDb3VsZCB5b3UgdGVsbCBtZSB3aGF0IG1vcmUgZG8gSSBuZWVkIHRvIGRv
IHRvIGdldCB0aGUgcGF0Y2gNCmFjY2VwdGVkPw0KDQpUaGUgbWVudGlvbmVkIHBhdGggMDMvMTEg
aXMgYWxyZWFkeSBtZXJnZWQgaW4gNC45LnJjMi4NCg0KVGhhbmtzLA0KUGlvdHINCg0KDQpPbiBU
aHUsIDIwMTYtMTAtMTMgYXQgMTc6MzEgKzAyMDAsIFBpb3RyIEx1YyB3cm90ZToNCj4gQWRkIEtu
aWdodHMgTWlsbCAoS05NKSB0byB0aGUgbGlzdCBvZiBDUFVJRHMgc3VwcG9ydGVkIGJ5IGludGVs
X3JhcGwNCj4gDQo+IFNpZ25lZC1vZmYtYnk6IFBpb3RyIEx1YyA8cGlvdHIubHVjQGludGVsLmNv
bT4NCj4gUmV2aWV3ZWQtYnk6IERhdmUgSGFuc2VuIDxkYXZlLmhhbnNlbkBpbnRlbC5jb20+DQo+
IENjOiAiUmFmYWVsIEouIFd5c29ja2kiIDxyandAcmp3eXNvY2tpLm5ldD4NCj4gQ2M6IGxpbnV4
LXBtQHZnZXIua2VybmVsLm9yZw0KPiBDYzogbGludXgta2VybmVsQHZnZXIua2VybmVsLm9yZw0K
PiAtLS0NCj4gVGhpcyBwYXRjaCBkZXBlbmRzIG9uIFtQQVRDSCAwMy8xMV0geDg2L2NwdS9pbnRl
bDogQWRkIEtuaWdodHMgTWlsbMKgDQo+IHRvIEludGVsIGZhbWlseQ0KPiANCj4gwqBkcml2ZXJz
L3Bvd2VyY2FwL2ludGVsX3JhcGwuYyB8IDEgKw0KPiDCoDEgZmlsZSBjaGFuZ2VkLCAxIGluc2Vy
dGlvbigrKQ0KPiANCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvcG93ZXJjYXAvaW50ZWxfcmFwbC5j
DQo+IGIvZHJpdmVycy9wb3dlcmNhcC9pbnRlbF9yYXBsLmMNCj4gaW5kZXggZmJhYjI5ZC4uNmE1
OGZmNyAxMDA2NDQNCj4gLS0tIGEvZHJpdmVycy9wb3dlcmNhcC9pbnRlbF9yYXBsLmMNCj4gKysr
IGIvZHJpdmVycy9wb3dlcmNhcC9pbnRlbF9yYXBsLmMNCj4gQEAgLTExNjAsNiArMTE2MCw3IEBA
IHN0YXRpYyBjb25zdCBzdHJ1Y3QgeDg2X2NwdV9pZCByYXBsX2lkc1tdDQo+IF9faW5pdGNvbnN0
ID0gew0KPiDCoAlSQVBMX0NQVShJTlRFTF9GQU02X0FUT01fREVOVkVSVE9OLAlyYXBsX2RlZmF1
bHRzX2Nvcg0KPiBlKSwNCj4gwqANCj4gwqAJUkFQTF9DUFUoSU5URUxfRkFNNl9YRU9OX1BISV9L
TkwsCXJhcGxfZGVmYXVsdHNfaHN3X3MNCj4gZXJ2ZXIpLA0KPiArCVJBUExfQ1BVKElOVEVMX0ZB
TTZfWEVPTl9QSElfS05NLAlyYXBsX2RlZmF1bHRzX2hzd19zDQo+IGVydmVyKSwNCj4gwqAJe30N
Cj4gwqB9Ow0KPiDCoE1PRFVMRV9ERVZJQ0VfVEFCTEUoeDg2Y3B1LCByYXBsX2lkcyk7
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Piotr Luc Nov. 30, 2016, 10:33 p.m. UTC | #2
Hi Rafael,

Kindly reminder.
Could you pick this patch up for kernel 4.10?
The RedHat acceptance window is closing soon, I need to pass them the
commit id.

Thanks,
Piotr

On Tue, 2016-10-25 at 12:35 +0200, Luc, Piotr wrote:
> Hi Rafael,

> 

> Could you tell me what more do I need to do to get the patch

> accepted?

> 

> The mentioned path 03/11 is already merged in 4.9.rc2.

> 

> Thanks,

> Piotr

> 

> 

> On Thu, 2016-10-13 at 17:31 +0200, Piotr Luc wrote:

> > 

> > Add Knights Mill (KNM) to the list of CPUIDs supported by

> > intel_rapl

> > 

> > Signed-off-by: Piotr Luc <piotr.luc@intel.com>

> > Reviewed-by: Dave Hansen <dave.hansen@intel.com>

> > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>

> > Cc: linux-pm@vger.kernel.org

> > Cc: linux-kernel@vger.kernel.org

> > ---

> > This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights

> > Mill 

> > to Intel family

> > 

> >  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 fbab29d..6a58ff7 100644

> > --- a/drivers/powercap/intel_rapl.c

> > +++ b/drivers/powercap/intel_rapl.c

> > @@ -1160,6 +1160,7 @@ static const struct x86_cpu_id rapl_ids[]

> > __initconst = {

> >  	RAPL_CPU(INTEL_FAM6_ATOM_DENVERTON,	rapl_defaults_c

> > or

> > e),

> >  

> >  	RAPL_CPU(INTEL_FAM6_XEON_PHI_KNL,	rapl_defaults_hsw

> > _s

> > erver),

> > +	RAPL_CPU(INTEL_FAM6_XEON_PHI_KNM,	rapl_defaults_hsw

> > _s

> > erver),

> >  	{}

> >  };

> >  MODULE_DEVICE_TABLE(x86cpu, rapl_ids);
Rafael J. Wysocki Nov. 30, 2016, 10:42 p.m. UTC | #3
On Wed, Nov 30, 2016 at 11:33 PM, Luc, Piotr <Piotr.Luc@intel.com> wrote:
> Hi Rafael,
>
> Kindly reminder.
> Could you pick this patch up for kernel 4.10?
> The RedHat acceptance window is closing soon, I need to pass them the
> commit id.

Done now, thanks!

Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c
index fbab29d..6a58ff7 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -1160,6 +1160,7 @@  static const struct x86_cpu_id rapl_ids[] __initconst = {
 	RAPL_CPU(INTEL_FAM6_ATOM_DENVERTON,	rapl_defaults_core),
 
 	RAPL_CPU(INTEL_FAM6_XEON_PHI_KNL,	rapl_defaults_hsw_server),
+	RAPL_CPU(INTEL_FAM6_XEON_PHI_KNM,	rapl_defaults_hsw_server),
 	{}
 };
 MODULE_DEVICE_TABLE(x86cpu, rapl_ids);