After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 707649 - screen blanking only works once
screen blanking only works once
Status: RESOLVED FIXED
Product: mutter
Classification: Core
Component: general
3.9.x
Other Linux
: Normal normal
: ---
Assigned To: mutter-maint
mutter-maint
Depends on:
Blocks:
 
 
Reported: 2013-09-06 18:53 UTC by Sebastian Keller
Modified: 2013-09-09 13:30 UTC
See Also:
GNOME target: 3.10
GNOME version: ---

Attachments
Monitor: don't shortcut set_power_save if the value is the same as the old one (1022 bytes, patch)
2013-09-09 11:13 UTC, Giovanni Campagna 		committed Details | Review
Rename META_POWER_SAVE_UNKNOWN to UNSUPPORTED (2.15 KB, patch)
2013-09-09 11:14 UTC, Giovanni Campagna 		committed Details | Review

Description Sebastian Keller 2013-09-06 18:53:58 UTC 
Lock the screen -> the screen gets turned off as expected
unlock the screen and lock it again -> the screen stays on
restart gnome shell and it works again (one time)

I've tracked this down to org.gnome.Mutter.DisplayConfig.PowerSaveMode not being reset when the screen gets turned on again. If I manually set it to 0 again after unlocking, blanking works again one time.
Comment 1 drago01 2013-09-07 11:41:16 UTC 
https://bugzilla.gnome.org/show_bug.cgi?id=696169#c40

*** This bug has been marked as a duplicate of bug 696169 ***
Comment 2 drago01 2013-09-07 11:41:44 UTC 
Oh sorry does not sound to be the same one.
Comment 3 Giovanni Campagna 2013-09-09 11:13:44 UTC 
Created attachment 254468 [details] [review]
Monitor: don't shortcut set_power_save if the value is the same as the old one

We don't get notifications from X11 when the mode is reset, so
our cached value can get stale. To work around that, always forward
requests to the backend (and let it deal with ignoring the change
if wanted)
Comment 4 Giovanni Campagna 2013-09-09 11:14:07 UTC 
Created attachment 254469 [details] [review]
Rename META_POWER_SAVE_UNKNOWN to UNSUPPORTED

It conveys more clearly what this value means.
Comment 5 drago01 2013-09-09 11:19:07 UTC 
Review of attachment 254468 [details] [review]:

OK.
Comment 6 drago01 2013-09-09 11:19:54 UTC 
Review of attachment 254469 [details] [review]:

Way better.
Comment 7 Giovanni Campagna 2013-09-09 13:30:14 UTC 
Attachment 254468 [details] pushed as 016e2ae - Monitor: don't shortcut set_power_save if the value is the same as the old one
Attachment 254469 [details] pushed as cacdaa0 - Rename META_POWER_SAVE_UNKNOWN to UNSUPPORTED