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

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@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@lists.01.org https://lists.01.org/mailman/listinfo/powertop