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 */

Hi, thanks a lot for your efforts, I appreciate it. My co-worker applied your patch and sent me the following output: -------------------------------------------------- / ./powertop -t 1 --csv ⏎

Loaded 0 prior measurements Cannot load from file /var/cache/powertop/saved_parameters.powertop check cpu model 0x17 ERROR: not supported RAPL device for cpu 0 RAPL device for cpu 0 check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported check cpu model 0x17 ERROR: not supported Cannot load from file /var/cache/powertop/saved_parameters.powertop Preparing to take measurements unknown op '{' Taking 1 measurement(s) for a duration of 1 second(s) each. PowerTOP outputing using base filename powertop.csv \ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ I hope this helps. Best regards Valentin Am 07.07.2014 12:28, schrieb Sergey Senozhatsky: >On (07/07/14 19:26), Sergey Senozhatsky wrote: >>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 >> > >oh. the patch is against the current upstream (iow, git) powertop source code. > > -ss > >>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 */ >> -- Mit freundlichen Grüßen Valentin Höbel Cloud Architect NFON AG Leonrodstraße 68 D-80636 München Telefon +49 89 45300-407 valentin.hoebel(a)nfon.com www.nfon.com Support-Hotline der NFON AG Telefon 0800 - 6366 555 Die gesetzlichen Pflichtangaben finden Sie unter https://www.nfon.net/pflichtangaben