commit 20ae04f824d968d439c0bf6403e50b9b0d79e505
author Borislav Petkov <bp@amd64.org> Thu Apr 29 17:13:13 2010 +0200
committer Dominik Brodowski <linux@dominikbrodowski.net> Thu Apr 29 18:05:15 2010 +0200
tree 38e39fe2790049619ea8dea26d27dbc866e5c491
parent af4932119e8b234109e8a4f62648a137c20ef332

aperf: Print cpunum even when offlined

Remove completed items from ToDo list while at it.

Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>

utils/aperf.c

diff --git a/utils/aperf.c b/utils/aperf.c
index 063f4a4..c5ceb4b 100644
--- a/utils/aperf.c
+++ b/utils/aperf.c
@@ -31,8 +31,6 @@
  *
  * Possible ToDos/Enhancments:
  *
- *  - Use cpuid assmbler command to retrieve cpuid
- *    -No dependency to the cpuid driver
  *  - Refresh the screen when mulitple cpus are poked and display results
  *    on one screen
  *    -This would introduce a lot more complexity, not sure whether it's
@@ -219,11 +217,9 @@
 	return 0;
 }
 
-static void print_cpu_stats(unsigned int cpu, unsigned long average,
-			    struct timeval c0_time, struct timeval cx_time,
-			    unsigned int c0_percent)
+static void print_cpu_stats(unsigned long average, struct timeval c0_time,
+			    struct timeval cx_time, unsigned int c0_percent)
 {
-	printf("%.3u\t", cpu);
 	printf("%.7lu\t\t\t", average);
 	printf("%.2lu sec %.3lu ms\t", c0_time.tv_sec, c0_time.tv_usec);
 	printf("%.2lu sec %.3lu ms\t", cx_time.tv_sec, cx_time.tv_usec);
@@ -255,9 +251,11 @@
 		if (!cpu_info.is_valid)
 			continue;
 
+		printf("%.3u\t", cpu);
+
 		ret = get_aperf_mperf(cpu, &current_aperf, &current_mperf);
 		if (ret < 0) {
-			printf("\t[offline]\n");
+			printf("[offline]\n");
 			continue;
 		}
 
@@ -272,8 +270,7 @@
 					   aperf_diff, mperf_diff);
 		cpu_info.saved_mperf = current_mperf;
 		cpu_info.saved_aperf = current_aperf;
-		print_cpu_stats(cpu, average, C0_time, CX_time,
-				c0_percent);
+		print_cpu_stats(average, C0_time, CX_time, c0_percent);
 
 		if (once) {
 			printf("\n");
@@ -320,11 +317,13 @@
 
 		for (cpu = 0; cpu < cpus; cpu++) {
 
+			printf("%.3u\t", cpu);
+
 			ret = get_aperf_mperf(cpu, &current_aperf,
 					      &current_mperf);
 
 			if ((ret < 0) || !cpu_list[cpu].is_valid) {
-				printf("\t[offline]\n");
+				printf("[offline]\n");
 				continue;
 			}
 
@@ -339,8 +338,7 @@
 						   aperf_diff, mperf_diff);
 			cpu_list[cpu].saved_mperf = current_mperf;
 			cpu_list[cpu].saved_aperf = current_aperf;
-			print_cpu_stats(cpu, average, C0_time, CX_time,
-					c0_percent);
+			print_cpu_stats(average, C0_time, CX_time, c0_percent);
 			printf("\n");
 		}
 		if (once)