Description of problem: Window Maker is responsible for 4 wakeups when totally idle (as shown by powertop). Version-Release number of selected component (if applicable): wmaker-0.92.0 (from www.windowmaker.info website) How reproducible: I am not a Fedora user (sorry), but I bet that wmaker also wakes up approx. 4 times in the latest fedora. Steps to Reproduce: 1. Run wmaker without any dockapps running 2. Start powertop on a xterm and stand still (no mouse movement, no key presses, etc) 3. Actual results: wmaker appears with 3.5 to 4 wakeups Expected results: It should have 0 wakeups when totally idle, as it should be event driven. Additional info: Two months ago I've made a patch for my own usage (see this post in powertop list: http://www.bughost.org/pipermail/power/2007-June/000614.html) and I've been using my own version of wmaker with 0 wakeups since then, with no known regression or any problems. On my website (url provided above) I have this patch, plus an additional one from mandriva to make the vanilla wmaker-0.92.0 compile with gcc 4. Well, I divided the wakeup patch into 3 separated patches, according to what it does. The -remove_nopolling.patch is more evolved, since it replaces the default behavior of polling whether the configuration files inside ~/GNUstep/Defaults have changed by the kernel dnotify mechanism. Note that I adapted the dnotify example from the kernel source Documentation/dnotify.txt to my needs. With this patch wmaker is notified when the conf. files changed. E.g., if I modify the WMRootMenu by hand, or with the genmenu script (also on my webpage, but someone else wrote the script under GPL), wmaker automatically loads the new menu. The other two patches just remove what I _think_ are useless timers. However, I am not so sure (theoretically) what the delayedAction function really does, except that by removing the timer which calls it (with -remove_delayedAction.patch) I experimentally don't see any bad side effects. Well, I hope these patches turn out to be useful for someone. They are for me. I really like not seeing wmaker in powertop's shame list.
Created attachment 181001 [details] Substitute polling with kernel's dnotify mechanism
Created attachment 181041 [details] Remove timer which calls delayedAction
Created attachment 181061 [details] Remove useless timer
Created attachment 181081 [details] Compile with gcc 4 (from mandriva)
Thanks for submitting your patches. I don't see a general problem with wakeups (come on ... it does not hurt your battery that much ;) ) I will try to integrate it into the wmaker package for fedora. As we don't really have a devel tree at the moment I will wait till the f8 release, do some testing here locally in the meantime and add it to rawhide/f9 asap. If it works well and there are no complains I will consider issuing upgrades for f8 and f7.
Build for devel.
WindowMaker-0.92.0-15.fc8 has been pushed to the Fedora 8 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update WindowMaker'
I can confirm that WindowMaker-0.92.0-15.fc8 eliminates wakeups from wmaker. Will this also be pushed to Fedora 7? I have a machine running F7 and WindowMaker that could use this.
WindowMaker-0.92.0-15.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report.