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

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