SavingTheWorld
From http://gould.cx/ted/blog/Saving_the_world_one__w_at_a_time
Please, for gnome-related packages use this upstream page: http://live.gnome.org/GnomeGoals/UseTimeoutAddSeconds
Saving the world one uW at a time
I'm here at UDS and joined a session between the kernel and desktop teams talking about power. Like all of such sessions the kernel team thinks that the desktop should fix all the problems and vice versa. At their suggestion, I took an action item.
One of the features of GTK+ is that it provides two timeout functions: gtk_timeout_add and gtk_timeout_add_seconds [1]. The difference is much more important than not having to add a "* 1000" in one and not the other. The important thing that the second function does is that it tried to group wake-ups to ensure that the program wakes up as little as possible. This is never perfect, but it's a small thing that can help to save some power and reduce the number of wake-ups.
So I looked through the Ubuntu main archive to figure out how many applications use the more precise function, but just pass in several seconds as the parameter. I figured this would be a few apps, and I'd submit a couple patches, then I could go back to blaming the kernel team for all our problems. There are more than a couple packages. There are a lot of packages. The results of the grep and a cleaned up list of packages that are likely to have easy fixes.
I realize now that I can't do this by myself, the only hope is to document how to fix it.
So if you'd like to help save a little power on your machine, follow these instructions.
Please, for gnome-related packages use this upstream page: http://live.gnome.org/GnomeGoals/UseTimeoutAddSeconds
Take one of the packages in the list above and find it in the package repository (instructions).
- Branch it:
$ bzr branch http://package-import.ubuntu.com/x/xproject/jaunty localfix
- Find the instances that might be causing the problem.
$ rgrep timeout_add *
- Commit and build a package:
$ bzr commit -m "Saving the world"
$ dch -i
# aptitude build-dep xproject
$ debuild
- Test it on your system, make sure things still work reasonably.
- Generate a patch:
$ bzr diff -r ancestor:
[1] Note that PyGObject has timeout_add_seconds() but it was not documented.
Add your name to the Assignee column if you wish to tackle that package. After you have submitted the patch upstream, change row's background to yellow by adding <rowbgcolor="#ff9"> in the first cell of the column. After the patch has been commited upstream, change row's background to green by adding <rowbgcolor="#9f9"> in the first cell of the column. This will let people know that the package in that row has been hugged.
Package |
Assignee |
Upstream Bug |
foo (example) |
Andrewsomething |
|
avahi |
|
|
bacula |
|
|
consolekit |
|
|
gcompris |
|
|
gimp |
|
|
gksu |
|
|
hal |
|
|
irssi |
|
|
jockey |
|
|
ldm |
|
|
libbtctl |
|
|
libhildonfm |
|
|
liferea |
Already fixed upstream, all timeouts over 1s using g_timeout_add_seconds in SVN trunk Fixes for noted instances |
|
midbrowser |
|
|
network-manager |
|
|
nvidia-setting |
|
|
oem-config |
|
|
openoffice.org-l10n |
|
|
pbbuttonsd |
|
|
pidgin |
||
pigment-python |
|
|
policykit |
Patch sent to f.d.o development mailing list, mail currently pending approval |
|
screem |
|
|
screen-resolution-extra |
|
|
system-config-cluster |
|
|
system-config-printer |
|
|
thin-client-manager |
|
|
tracker |
||
transmission |
Andrewsomething |
|
tsclient |
|
|
twisted |
|
|
ubiquity |
|
|
update-notifier |
|
|
usb-creator |
|
|
vim |
|
|
webkit |
|
|
xsane |
|
|
xscreensaver |
|
|
Gnome-related packages, only here for historical reasons. Please, use http://live.gnome.org/GnomeGoals/UseTimeoutAddSeconds instead
Package |
Assignee |
Upstream Bug |
brasero |
Debarshi Ray |
|
dbus-python |
All test/example code |
|
deskbar-applet |
||
dia |
||
ekiga |
|
False alarm. |
evolution |
||
gconf |
||
gedit |
||
gftp |
||
gnome-applets |
||
gnome-games |
|
|
gnome-mount |
||
gnome-power-manager |
||
gnome-system-monitor |
Debarshi Ray |
|
gtksourceview |
Reviewed, only use is in debug code (Gnome SVN trunk 24/2/09) |
|
gvfs |
Awalton |
|
metacity |
GNOME Bugzilla #564073 - Closed, Unneeded - Look at bug for more |
|
nautilus |
Fixed, GNOME Bugzilla #574033 |
|
nautilus-sendto |
||
network-manager-applet |
||
pidgin |
||
sabayon |
Debarshi Ray |
|
seahorse |
Debarshi Ray |
|
totem |
Philip Withnall |
|
vinagre |
all instances < 1000 ms |
|
vino |
||
xchat-gnome |
SavingTheWorld (last edited 2013-05-31 17:42:34 by brian-murray)