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

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