All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 15:25 Arjan van de Ven
  0 siblings, 0 replies; 12+ messages in thread
From: Arjan van de Ven @ 2014-07-07 15:25 UTC (permalink / raw)
  To: powertop

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


> not sure that powertop performs any specific MSRs at all, since most of MSRs are
> called correspondingly to c-state flags (has_cXXX), e.g.
>
> __init:
> 	if (model == 0x45 || model ==0x3D)
> 		has_c8c9c10_res = 1;
>
>
>
> and later:
>
> 	if (has_c8c9c10_res) {
> 		c8_before    = get_msr(number, MSR_PKG_C8_RESIDENCY);
> 		c9_before    = get_msr(number, MSR_PKG_C9_RESIDENCY);
> 		c10_before    = get_msr(number, MSR_PKG_C10_RESIDENCY);
> 	}
>
> 	and so on.


but if the set is empty, you don't get any data, and the "emulation" side of powertop
is likely a better choice... since there are C states, just not counters.



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-09 15:30 Valentin Hoebel
  0 siblings, 0 replies; 12+ messages in thread
From: Valentin Hoebel @ 2014-07-09 15:30 UTC (permalink / raw)
  To: powertop

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

Hi,

not sure if this is an important information or not, but I am getting 
the "138% idle" behaviour for all of the systems where reading out the 
current CPU frequencies via PowerTop is not possible.

Best regards
Valentin


Am 08.07.2014 13:05, schrieb Sergey Senozhatsky:
> On (07/07/14 08:25), Arjan van de Ven wrote:
>>> not sure that powertop performs any specific MSRs at all, since most of MSRs are
>>> called correspondingly to c-state flags (has_cXXX), e.g.
>>>
>>> __init:
>>> 	if (model == 0x45 || model ==0x3D)
>>> 		has_c8c9c10_res = 1;
>>>
>>>
>>>
>>> and later:
>>>
>>> 	if (has_c8c9c10_res) {
>>> 		c8_before    = get_msr(number, MSR_PKG_C8_RESIDENCY);
>>> 		c9_before    = get_msr(number, MSR_PKG_C9_RESIDENCY);
>>> 		c10_before    = get_msr(number, MSR_PKG_C10_RESIDENCY);
>>> 	}
>>>
>>> 	and so on.
>>
>>
>> but if the set is empty, you don't get any data, and the "emulation" side of powertop
>> is likely a better choice... since there are C states, just not counters.
>>
>
> looks like powertop checks C6 state `get_msr(number, MSR_PKG_C6_RESIDENCY)' in start
> and end measurement for any intel cpu, along with `get_msr(first_cpu, MSR_TSC)'. anyway,
> need to investigate. the output that Valentin sees is rather strange, saying that cpu0
> is 138% Idle.
>
> 	          CPU 0
> 	Idle      138%
>
> 	          CPU 2
> 	Idle      111,7%
>
> 	          CPU 1
> 	Idle      111,7%
>
> 	          CPU 3
> 	Idle      111,7%
>
> 	-ss
>

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-08 11:05 Sergey Senozhatsky
  0 siblings, 0 replies; 12+ messages in thread
From: Sergey Senozhatsky @ 2014-07-08 11:05 UTC (permalink / raw)
  To: powertop

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

On (07/07/14 08:25), Arjan van de Ven wrote:
> >not sure that powertop performs any specific MSRs at all, since most of MSRs are
> >called correspondingly to c-state flags (has_cXXX), e.g.
> >
> >__init:
> >	if (model == 0x45 || model ==0x3D)
> >		has_c8c9c10_res = 1;
> >
> >
> >
> >and later:
> >
> >	if (has_c8c9c10_res) {
> >		c8_before    = get_msr(number, MSR_PKG_C8_RESIDENCY);
> >		c9_before    = get_msr(number, MSR_PKG_C9_RESIDENCY);
> >		c10_before    = get_msr(number, MSR_PKG_C10_RESIDENCY);
> >	}
> >
> >	and so on.
> 
> 
> but if the set is empty, you don't get any data, and the "emulation" side of powertop
> is likely a better choice... since there are C states, just not counters.
> 

looks like powertop checks C6 state `get_msr(number, MSR_PKG_C6_RESIDENCY)' in start
and end measurement for any intel cpu, along with `get_msr(first_cpu, MSR_TSC)'. anyway,
need to investigate. the output that Valentin sees is rather strange, saying that cpu0
is 138% Idle.

	          CPU 0
	Idle      138%

	          CPU 2
	Idle      111,7%

	          CPU 1
	Idle      111,7%

	          CPU 3
	Idle      111,7%

	-ss

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 14:18 Sergey Senozhatsky
  0 siblings, 0 replies; 12+ messages in thread
From: Sergey Senozhatsky @ 2014-07-07 14:18 UTC (permalink / raw)
  To: powertop

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

On (07/07/14 07:00), Arjan van de Ven wrote:
> >>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
> >>                                             ⏎
> >>
> >
> >thanks! ok, so cpu model 0x17 is not listed as a supported one in
> >src/cpu/intel_cpus.cpp:intel_cpu_models[] to begin with, and no
> >C-states are initialized as a result. Let's wait for Intel devs
> >to add cpu/core/package models and corresponding C-states.
> >
> >Alexandra, will you take care of this?
> >
> >	-ss
> >
> >>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
> 
> so turns out the L5410 is one generation prior to NHM, and does not support some of the internal MSRs we're using.
> I'll talk to Alexandra (pending vacations, it's that season) to see what we can do
> 

not sure that powertop performs any specific MSRs at all, since most of MSRs are
called correspondingly to c-state flags (has_cXXX), e.g.

__init:
	if (model == 0x45 || model ==0x3D)
		has_c8c9c10_res = 1;



and later:

	if (has_c8c9c10_res) {
		c8_before    = get_msr(number, MSR_PKG_C8_RESIDENCY);
		c9_before    = get_msr(number, MSR_PKG_C9_RESIDENCY);
		c10_before    = get_msr(number, MSR_PKG_C10_RESIDENCY);
	}

	and so on.

	-ss

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 14:00 Arjan van de Ven
  0 siblings, 0 replies; 12+ messages in thread
From: Arjan van de Ven @ 2014-07-07 14:00 UTC (permalink / raw)
  To: powertop

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

On 7/7/2014 4:24 AM, Sergey Senozhatsky wrote:
> On (07/07/14 13:13), Valentin Hoebel wrote:
>> 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
>>                                              ⏎
>>
>
> thanks! ok, so cpu model 0x17 is not listed as a supported one in
> src/cpu/intel_cpus.cpp:intel_cpu_models[] to begin with, and no
> C-states are initialized as a result. Let's wait for Intel devs
> to add cpu/core/package models and corresponding C-states.
>
> Alexandra, will you take care of this?
>
> 	-ss
>
>> 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

so turns out the L5410 is one generation prior to NHM, and does not support some of the internal MSRs we're using.
I'll talk to Alexandra (pending vacations, it's that season) to see what we can do



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 11:24 Sergey Senozhatsky
  0 siblings, 0 replies; 12+ messages in thread
From: Sergey Senozhatsky @ 2014-07-07 11:24 UTC (permalink / raw)
  To: powertop

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

On (07/07/14 13:13), Valentin Hoebel wrote:
> 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
>                                             ⏎
>

thanks! ok, so cpu model 0x17 is not listed as a supported one in
src/cpu/intel_cpus.cpp:intel_cpu_models[] to begin with, and no
C-states are initialized as a result. Let's wait for Intel devs
to add cpu/core/package models and corresponding C-states.

Alexandra, will you take care of this?

	-ss

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 11:13 Valentin Hoebel
  0 siblings, 0 replies; 12+ messages in thread
From: Valentin Hoebel @ 2014-07-07 11:13 UTC (permalink / raw)
  To: powertop

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 10:28 Sergey Senozhatsky
  0 siblings, 0 replies; 12+ messages in thread
From: Sergey Senozhatsky @ 2014-07-07 10:28 UTC (permalink / raw)
  To: powertop

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07 10:26 Sergey Senozhatsky
  0 siblings, 0 replies; 12+ messages in thread
From: Sergey Senozhatsky @ 2014-07-07 10:26 UTC (permalink / raw)
  To: powertop

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


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-07  8:21 Valentin Hoebel
  0 siblings, 0 replies; 12+ messages in thread
From: Valentin Hoebel @ 2014-07-07  8:21 UTC (permalink / raw)
  To: powertop

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

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?

Best regards
Valentin


Am 03.07.2014 15:59, schrieb Sergey Senozhatsky:
> On (07/03/14 14:26), Valentin Hoebel wrote:
>> Hello,
>>
>> I have been using PowerTop for some time now and would like to thank all
>> contributors for their work, especially Intel!
>>
>> However, when using PowerTop on Debian 7 (Kernel 3.2, PowerTop package
>> 2.0-0.3) and navigating to "Frequency stats", it does not show the MHz
>> numbers. Instead, it shows me the following:
>
> Hello,
> am not really familiar with debian versioning, but if "2.0" means
> upstream release v2.0, then you use a really-really outdated powertop
> (several years). in that case I'd recommend to switch to recent version
> (2.6 IIRC).
>
> 	-ss
>
>>            CPU 0
>> Idle      138%
>>
>>            CPU 2
>> Idle      111,7%
>>
>>            CPU 1
>> Idle      111,7%
>>
>>            CPU 3
>> Idle      111,7%
>>
>> Is the CPU model Intel Xeon L5410 simply not supported or is there no way to
>> read out the frequencies?
>>
>> Browsing the source code, I did not stumple accross this CPU model.
>>
>> Thanks in advance.
>>
>> Best regards
>> Valentin
>>
>>
>> --
>> 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
>> _______________________________________________
>> PowerTop mailing list
>> PowerTop(a)lists.01.org
>> https://lists.01.org/mailman/listinfo/powertop
>>

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-03 13:59 Sergey Senozhatsky
  0 siblings, 0 replies; 12+ messages in thread
From: Sergey Senozhatsky @ 2014-07-03 13:59 UTC (permalink / raw)
  To: powertop

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

On (07/03/14 14:26), Valentin Hoebel wrote:
> Hello,
> 
> I have been using PowerTop for some time now and would like to thank all
> contributors for their work, especially Intel!
> 
> However, when using PowerTop on Debian 7 (Kernel 3.2, PowerTop package
> 2.0-0.3) and navigating to "Frequency stats", it does not show the MHz
> numbers. Instead, it shows me the following:

Hello,
am not really familiar with debian versioning, but if "2.0" means
upstream release v2.0, then you use a really-really outdated powertop
(several years). in that case I'd recommend to switch to recent version
(2.6 IIRC).

	-ss

>           CPU 0
> Idle      138%
> 
>           CPU 2
> Idle      111,7%
> 
>           CPU 1
> Idle      111,7%
> 
>           CPU 3
> Idle      111,7%
> 
> Is the CPU model Intel Xeon L5410 simply not supported or is there no way to
> read out the frequencies?
> 
> Browsing the source code, I did not stumple accross this CPU model.
> 
> Thanks in advance.
> 
> Best regards
> Valentin
> 
> 
> -- 
> 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
> _______________________________________________
> PowerTop mailing list
> PowerTop(a)lists.01.org
> https://lists.01.org/mailman/listinfo/powertop
> 

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ?
@ 2014-07-03 12:26 Valentin Hoebel
  0 siblings, 0 replies; 12+ messages in thread
From: Valentin Hoebel @ 2014-07-03 12:26 UTC (permalink / raw)
  To: powertop

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

Hello,

I have been using PowerTop for some time now and would like to thank all 
contributors for their work, especially Intel!

However, when using PowerTop on Debian 7 (Kernel 3.2, PowerTop package 
2.0-0.3) and navigating to "Frequency stats", it does not show the MHz 
numbers. Instead, it shows me the following:

           CPU 0
Idle      138%

           CPU 2
Idle      111,7%

           CPU 1
Idle      111,7%

           CPU 3
Idle      111,7%

Is the CPU model Intel Xeon L5410 simply not supported or is there no 
way to read out the frequencies?

Browsing the source code, I did not stumple accross this CPU model.

Thanks in advance.

Best regards
Valentin


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

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2014-07-09 15:30 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-07 15:25 [Powertop] Does PowerTop support MHz numbers for Intel Xeon L5410 CPUs ? Arjan van de Ven
  -- strict thread matches above, loose matches on Subject: below --
2014-07-09 15:30 Valentin Hoebel
2014-07-08 11:05 Sergey Senozhatsky
2014-07-07 14:18 Sergey Senozhatsky
2014-07-07 14:00 Arjan van de Ven
2014-07-07 11:24 Sergey Senozhatsky
2014-07-07 11:13 Valentin Hoebel
2014-07-07 10:28 Sergey Senozhatsky
2014-07-07 10:26 Sergey Senozhatsky
2014-07-07  8:21 Valentin Hoebel
2014-07-03 13:59 Sergey Senozhatsky
2014-07-03 12:26 Valentin Hoebel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.