Re: Re: Re: [announce] Intel announces the PowerTOP utility for Linux

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Matt Mackall <mpm@...>
Cc: Arjan van de Ven <arjan@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-acpi@...>
Date: Saturday, May 12, 2007 - 12:09 am

Words by Matt Mackall [Fri, May 11, 2007 at 09:39:05PM -0500]:

Then again, perhaps you have /proc/tstats instead.

If so apply this (well, you get the idea):

--- powertop/powertop.c 2007-05-12 05:01:15.000000000 +0100
+++ powertop_new/powertop.c     2007-05-12 05:08:46.000000000 +0100
@@ -212,8 +212,8 @@
 void stop_timerstats(void)
 {
        FILE *file;
-       file = fopen("/proc/timer_stats","w");
-       if (!file) {
+       if (!(file = fopen("/proc/timer_stats","w")) &&
+               !(file = fopen("/proc/stats","w")) ) {
                nostats = 1;
                return;
        }
@@ -223,8 +223,8 @@
 void start_timerstats(void)
 {
        FILE *file;
-       file = fopen("/proc/timer_stats","w");
-       if (!file) {
+       if (!(file = fopen("/proc/timer_stats","w")) &&
+               !(file = fopen("/proc/stats","w")) ) {
                nostats = 1;
                return;
        }
@@ -388,7 +388,7 @@
                i = 0;
                totalticks = 0;
                if (!nostats)
-                       file = popen("cat /proc/timer_stats | sort -n | tail -190", "r");
+                       file = popen("cat /proc/timer_stats 2>>/dev/null|| cat /proc/tstats | sort -n | tail -190", "r");
                while (file && !feof(file) && i<190) {
                        char *count, *pid, *process, *func;
                        int cnt;


-- 
Jose Celestino
----------------------------------------------------------------
http://www.msversus.org/     ; http://techp.org/petition/show/1
http://www.vinc17.org/noswpat.en.html
----------------------------------------------------------------
"And on the trillionth day, Man created Gods." -- Thomas D. Pate
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[announce] Intel announces the PowerTOP utility for Linux, Arjan van de Ven, (Fri May 11, 7:07 pm)
Re: [announce] Intel announces the PowerTOP utility for Linux, Alistair John Strachan, (Sat May 12, 5:12 pm)
Re: [announce] Intel announces the PowerTOP utility for Linux, Alistair John Strachan, (Mon May 14, 4:42 pm)
Re: [announce] Intel announces the PowerTOP utility for Linux, Arjan van de Ven, (Sat May 12, 1:44 pm)
Re: Re: Re: [announce] Intel announces the PowerTOP utility ..., Jose Celestino, (Sat May 12, 12:09 am)