From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============1608335696700712037==" MIME-Version: 1.0 From: Valentin Hoebel Subject: Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ? Date: Mon, 07 Jul 2014 13:13:59 +0200 Message-ID: <53BA80F7.5050101@nfon.com> In-Reply-To: 20140707102850.GB1135@swordfish To: powertop@lists.01.org List-ID: --===============1608335696700712037== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable 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 =E2=8F=8E 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 t= he >>> 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) th= at we >> don't set any of C-state flags. if none of C-states set, then no MSR wor= k 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 =3D 0; intel_cpu_models[i] !=3D 0; i++) >> if (model =3D=3D 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 =3D 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 =3D 0; >> has_c2c7_res =3D 0; >> >> + fprintf(stderr, "nhm_package model: 0x%X\n", model); >> + >> switch(model) { >> case 0x2A: /* SNB */ >> case 0x2D: /* SNB Xeon */ >> -- = Mit freundlichen Gr=C3=BC=C3=9Fen Valentin H=C3=B6bel Cloud Architect NFON AG Leonrodstra=C3=9Fe 68 D-80636 M=C3=BCnchen 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 --===============1608335696700712037==--