Bug 221648 - powerdevil only autosuspends once/twice
Summary: powerdevil only autosuspends once/twice
Status: RESOLVED FIXED
Alias: None
Product: solid
Classification: Frameworks and Libraries
Component: powermanagement-daemon (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Dario Freddi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-07 14:08 UTC by S. Burmeister
Modified: 2010-10-02 12:50 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch to reset d->status in PowerDevilDaemon::resumeFromIdle() (307 bytes, patch)
2010-04-12 21:39 UTC, Felix Geyer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description S. Burmeister 2010-01-07 14:08:59 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

This might be related to bug 221637. After powerdevil autosuspends I immediately resume, it autosupspends again (see the mentioned bug) and I resume again. After that it never autosuspends again although the relevant power profiles are set to do so.

there might have been a powerlevel change in-between but that should not matter.
Comment 1 Felix Geyer 2010-04-12 21:39:31 UTC
Created attachment 42730 [details]
Patch to reset d->status in PowerDevilDaemon::resumeFromIdle()

When a suspension due to the idle time is trigger, d->status is set to Action in PowerDevilDaemon::poll().
After that d->status is never reset to NoAction (except if "dim display when idle" is activated) as poll is only called with idle == settings->readEntry("idleTime").toInt() * 60 * 1000.

The attached patch fixes this by resetting d->status resumeFromIdle().
Comment 2 Felix Geyer 2010-04-15 20:40:06 UTC
SVN commit 1115250 by fgeyer:

Fix bug that causes idle actions to be triggered only once per session.
d->status was never reset to NoAction after such an action.

BUG: 221648


 M  +2 -0      PowerDevilDaemon.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1115250
Comment 3 Felix Geyer 2010-04-15 20:42:54 UTC
SVN commit 1115252 by fgeyer:

Backport r1115250.

Fix bug that causes idle actions to be triggered only once per session.
d->status was never reset to NoAction after such an action.

CCBUG: 221648


 M  +2 -0      PowerDevilDaemon.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1115252