On 6/18/07, Arjan van de Ven <[EMAIL PROTECTED]> wrote:
Hi,

I just released PowerTOP 1.7

In addition to various bugfixes and translation updates, there are 3
new features in this release:
* A suggestion to disable TV out (saves +/- 1 Watt if left enabled)
   [only works on new enough video drivers]
* A suggestion to disable Wake-on-Lan of the ethernet card; WoL keeps
the phy powered up even when the interface is down, this consumes
extra power
* PowerTOP now also shows the P-state (CPU frequency) data on the main
screen, so that you can see how much time is spent at which frequency.

I think you might have chosen wrong value for rounding up. Or maybe,
most likely, I misunderstood your code.

$ cat /sys/devices/system/cpu/cpu0/cpufreq/stats/time_in_state
1667000 106337
1333000 655
1000000 236125

before patch:
Cn          Avg residency (5s)          P-states (frequencies)
C0 (cpu running)        (47.5%)
C1                0.0ms ( 0.0%)         1.72 Ghz   100.0%
C2                0.1ms (52.5%)         1333 Mhz     0.0%
C3                0.0ms ( 0.0%)         1000 Mhz     0.0%

after patch:
Cn          Avg residency (10s)         P-states (frequencies)
C0 (cpu running)        (46.5%)
C1                0.0ms ( 0.0%)         1.67 Ghz   100.0%
C2                0.1ms (53.5%)         1333 Mhz     0.0%
C3                0.0ms ( 0.0%)         1000 Mhz     0.0%


$ diff -urN cpufreqstats.c.asli cpufreqstats.c
--- cpufreqstats.c.asli 2007-06-17 14:07:47.000000000 +0700
+++ cpufreqstats.c      2007-06-18 00:09:18.000000000 +0700
@@ -77,7 +77,7 @@
               sprintf(buffer, _("%6lli Mhz"), (Hz+500)/1000);

       if (Hz>1500000)
-               sprintf(buffer, _("%6.2f Ghz"), (Hz+50000.0)/1000000);
+               sprintf(buffer, _("%6.2f Ghz"), (Hz+5000.0)/1000000);


       return buffer;


Regards,
Andika Triwidada
_______________________________________________
Power mailing list
[email protected]
http://www.bughost.org/mailman/listinfo/power

Reply via email to