Re: [Powertop] [PATCH] enumerate cpu based on different string on ARM

Alexandra Yates <alexandra.yates at linux.intel.com>
Newsgroups dev.linux.lists.powertop
Message-ID <[email protected]>
Patch added.

Sanjay thank you for sending the patch.

Sergey thank you for looking at this.

Alex
> On (06/04/14 11:27), Sanjay Singh Rawat wrote:
>> bogomips is removed by timer-backed delay loop (commit 9fc2105a)
>> in ARM kernel. Using other suitable string to decide to enumerate each
>> cpu.
>
>
> looks good.
>
> 	-ss
>
>> Signed-off-by: Sanjay Singh Rawat <sanjay.rawat(a)linaro.org>
>> ---
>>  src/cpu/cpu.cpp |   12 ++++++++++++
>>  1 file changed, 12 insertions(+)
>>
>> diff --git a/src/cpu/cpu.cpp b/src/cpu/cpu.cpp
>> index b3b6166..105c066 100644
>> --- a/src/cpu/cpu.cpp
>> +++ b/src/cpu/cpu.cpp
>> @@ -296,6 +296,18 @@ void enumerate_cpus(void)
>>  				number = -2;
>>  			}
>>  		}
>> +		/* bogomips is removed in ARM, using CPU revision to enumerate */
>> +		if (strncasecmp(line, "CPU revision\t", 13) == 0) {
>> +			if (number == -1) {
>> +				/* Not all /proc/cpuinfo include "processor\t". */
>> +				number = 0;
>> +			}
>> +			if (number >= 0) {
>> +				handle_one_cpu(number, vendor, family, model);
>> +				set_max_cpu(number);
>> +				number = -2;
>> +			}
>> +		}
>>  	}
>>
>>
>> --
>> 1.7.10.4
>>
>> _______________________________________________
>> PowerTop mailing list
>> PowerTop(a)lists.01.org
>> https://lists.01.org/mailman/listinfo/powertop
>>
> _______________________________________________
> PowerTop mailing list
> PowerTop(a)lists.01.org
> https://lists.01.org/mailman/listinfo/powertop
>


Thank you,
Alexandra.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.