Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?

[Sergey Senozhatsky <sergey.senozhatsky at gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1862 bytes --]

On (07/07/14 10:21), Valentin Hoebel wrote:
> Hi,
> 
> as you already guessed Debian still uses a 2.0 version.
> However, I git cloned 2.6.1 and the "head" versions from GitHub, built them
> both and tried them - still the same result.
> 
> Is it known if this particular CPU model does not support reading out the
> CPU frequency values or could this be another problem?

hm... a *wild* guess. can you check what cpu model is getting passed to
nhm_core::nhm_core(int model)? the problem might be (or might be not) that we
don't set any of C-state flags. if none of C-states set, then no MSR work is
getting done, and hence nothing is reported back. (yeah... we could check
documentation, search the web, etc. but let's do it the lazy way).


something like this


NOTE:
NOT TESTED, NOT EVEN COMPILE-TESTED. NOT A PATCH, NOT A FIX.
FOR TESTING PURPOSE ONLY.

---

 src/cpu/intel_cpus.cpp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/cpu/intel_cpus.cpp b/src/cpu/intel_cpus.cpp
index e3cdaf2..ae083cf 100644
--- a/src/cpu/intel_cpus.cpp
+++ b/src/cpu/intel_cpus.cpp
@@ -64,10 +64,13 @@ int is_supported_intel_cpu(int model)
 {
 	int i;
 
+	fprintf(stderr, "check cpu model 0x%X\n", model);
+
 	for (i = 0; intel_cpu_models[i] != 0; i++)
 		if (model == intel_cpu_models[i])
 			return 1;
 
+	printf(stderr, "ERROR: not supported\n");
 	return 0;
 }
 
@@ -91,6 +94,8 @@ nhm_core::nhm_core(int model)
 {
 	has_c2c7_res = 0;
 
+	fprintf(stderr, "nhm_core model: 0x%X\n", model);
+
 	switch(model) {
 		case 0x2A:	/* SNB */
 		case 0x2D:	/* SNB Xeon */
@@ -260,6 +265,8 @@ nhm_package::nhm_package(int model)
 	has_c8c9c10_res = 0;
 	has_c2c7_res = 0;
 
+	fprintf(stderr, "nhm_package model: 0x%X\n", model);
+
 	switch(model) {
 		case 0x2A:	/* SNB */
 		case 0x2D:	/* SNB Xeon */