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