Skip to contentSkip to conversation

powercap/rapl: add ids for future xeon cpus

PATCH In review Jacob Pan 2 participants 3 messages Dec 15, 2014 – Dec 16, 2014 Power management core

lore
Share & tools
lore

Jacob Pan

author
#0 lore raw

PATCH 1/2 powercap/rapl: add ids for future xeon cpus

Enable RAPL driver on Xeon cpu id 0x56.

Signed-off-by Jacob Pan <[email protected]>
---
 drivers/powercap/intel_rapl.c | 1 +
 1 file changed, 1 insertion(+)
drivers/powercap/intel_rapl.c +1 -0
index c71443c..ee00ffb 100644
@@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
 	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
 	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
 	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
+	RAPL_CPU(0x56, rapl_defaults_atom),/* Future Xeon */
 	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
 	{}
 };
Signature · 1 line
1.9.1
#1 lore raw
Quoted 12 lines · Jacob Pan

Enable RAPL driver on Xeon cpu id 0x56.

Signed-off-by Jacob Pan <[email protected]>
---
 drivers/powercap/intel_rapl.c | 1 +
 1 file changed, 1 insertion(+)
drivers/powercap/intel_rapl.c +0 -0
index c71443c..ee00ffb 100644

drivers/powercap/intel_rapl.c static const struct x86_cpu_id rapl_ids[] = { · line 1044

Quoted 4 lines
drivers/powercap/intel_rapl.c +1 -0
@@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
 	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
 	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
 	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
+	RAPL_CPU(0x56, rapl_defaults_atom),/* Future Xeon */

Is rapl_defaults_atom correct, or should this be rapl_defaults_core ?

Guenter

Quoted 13 lines
drivers/powercap/intel_rapl.c +0 -0
 	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
 	{}
 };
Signature · 7 lines
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
author
#2 lore raw
Quoted 8 lines · On Mon, 15 Dec 2014 16:36:48 -0800 Guenter Roeck <[email protected]> wrote:
Quoted 7 lines · Jacob Pan

Enable RAPL driver on Xeon cpu id 0x56.

Signed-off-by Jacob Pan <[email protected]>
---
 drivers/powercap/intel_rapl.c | 1 +
 1 file changed, 1 insertion(+)
Quoted 12 lines

drivers/powercap/intel_rapl.c static const struct x86_cpu_id rapl_ids[] = { · line 1044

Quoted 9 lines
(quoted diff) +0 -0

b/drivers/powercap/intel_rapl.c index c71443c..ee00ffb 100644

drivers/powercap/intel_rapl.c +1 -0
@@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
 	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
 	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
 	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
+	RAPL_CPU(0x56, rapl_defaults_atom),/* Future Xeon */

Is rapl_defaults_atom correct, or should this be rapl_defaults_core ?

you are right should be core, will fix.

thanks

Quoted 15 lines

Guenter

Quoted 13 lines
drivers/powercap/intel_rapl.c +0 -0
 	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
 	{}
 };
Signature · 7 lines
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe
linux-kernel" in the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Jacob Pan]