From: Colin Ian King <colin.king@canonical.com>
Date: Thu, 15 Nov 2012 19:02:00 +0000 (-0500)
Subject: tools/power x86_energy_perf_policy: close /proc/stat in for_every_cpu()
X-Git-Tag: v5.15~21018^2~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=84764a415c707b43e751deb579a421776f190a95;p=platform%2Fkernel%2Flinux-starfive.git

tools/power x86_energy_perf_policy: close /proc/stat in for_every_cpu()

Instead of returning out of for_every_cpu() we should break out of the loop=
 which will then tidy up correctly by closing the file /proc/stat.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Len Brown <len.brown@intel.com>
---

diff --git a/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c b/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c
index 33c5c7e..40b3e54 100644
--- a/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c
+++ b/tools/power/x86/x86_energy_perf_policy/x86_energy_perf_policy.c
@@ -289,7 +289,7 @@ void for_every_cpu(void (func)(int))
 			"cpu%u %*d %*d %*d %*d %*d %*d %*d %*d %*d %*d\n",
 			&cpu);
 		if (retval != 1)
-			return;
+			break;
 
 		func(cpu);
 	}