Screen never sleeps; missing power management

[Daniel van Vugt]

Effect Touch as well

Using XMir under unity-system-compositor I notice the screen never goes to sleep. I guess it's to be expected right now, but that's something we really need to fix.

Test case:
xset dpms force off

[Ubuntu QA Website (ubuntuqa)]

This bug has been reported on the Ubuntu Package testing tracker.

A list of all reports related to this bug can be found here:
http://packages.qa.ubuntu.com/qatracker/reports/bugs/1193222

[Ritesh Khadgaray (khadgaray)]

XMir crashes on resume ( using pm-suspend) .

[kevin gunn (kgunn72)]

critical for 13.10

[Robert Carr (robertcarr)]

I see 3 steps:

1. In mir, expose API from graphics platform for powering on and off CRTCs. Expose to client API, i.e. mir_connection_request_display_power_mode (WIP :p). Likely this should be gated by a component:

msh::PowerManagementPolicy

Which receives messages like

bool session_is_allowed_to_turn_off_displays()

2. The system compositor gains a PowerManagementPolicy to allow XMir to turn off displays.

3. xmir gains a DPMS impl in terms of the new mir client API.

[Daniel van Vugt (vanvugt)]

See also bug 1189770 where we are turning off the monitor by accident :)

[Seth Forshee (sforshee)]

I don't see that this has anything at all to do with powerd, so marking that task invalid.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:mir at revision None, scheduled for release in mir, milestone 0.0.11

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Launchpad Janitor (janitor)]

This bug was fixed in the package mir - 0.0.11+13.10.20130924.1-0ubuntu1

---------------
mir (0.0.11+13.10.20130924.1-0ubuntu1) saucy; urgency=low

  [ kg ]
  * bump version for ABI break (LP: #1229212)

  [ Robert Ancell ]
  * Allow an application to override the options being populated.
  * Pass the program options to parse_options().
  * Add missing include for std::cerr.
  * Report when paused and resumed via configuration. (LP: #1192843)
  * Add missing directory separator when searching for a config file to
    parse.

  [ Kevin Gunn ]
  * change test timeouts and fix fence.

  [ Alexandros Frantzis ]
  * examples: Only check key information for key events This fixes a
    memory error reported by valgrind for examples that use eglapp.
  * shell: Apply the base configuration on a hardware change only if no
    per-session configuration is active.
  * graphics: consolidated fixed for nested code and examples on android
    stack.

  [ Eleni Maria Stea ]
  * graphics: Pull in Eleni's changes to get the DRM fd to init GBM from
    the host Mir instance.

  [ Didier Roche ]
  * remove (unused in the ppa as we have libmirserver3) hack to force
    depending and building against the exact same version.

  [ Robert Carr ]
  * Add DPMS configuration API. (LP: #1193222)
  * Deduplicate mtd::NullDisplayConfig and
    mtd::NullDisplayConfiguration.
  * Fix multiple internal client surfaces on android. (LP: #1228144)
  * change test timeouts and fix fence.
  * Add DPMS API and GBM/android impls.

  [ Kevin DuBois ]
  * platform, graphics: support nested (mir-on-mir) rendering on the
    Android platform.

  [ Michael Terry ]
  * Change how Mir chooses socket locations to make it simpler for a
    nested-Mir world, by using MIR_SOCKET as the host socket if no other
    host socket is provided and passing MIR_SOCKET on to any children.
    Also, change --nested-mode to --host-socket for clarity and add --
    standalone to force standalone mode.

  [ Daniel d'Andrada ]
  * android-input housekeeping - Updated README - Removed some dead
    code.

  [ Daniel van Vugt ]
  * Add DPMS configuration API. (LP: #1193222)
  * Add a "flags" field to MirBufferPackage so that clients can find out
    if the buffer they've been given is scanout-capable. This is
    normally something a client should never need to know. However there
    are two specialized cases where it's required to fix bugs in the
    intel and radeon X drivers:   LP: #1218735, LP: #1218815 The intel
    fix (already landed) contains a hack which will be updated after
    this branch lands. (LP: #1218815, #1218735)
  * GBM: Ensure that we don't create scanout buffers if bypass is
    explicitly disabled from the environment. (LP: #1227133) . (LP:
    #1227133)

  [ Alan Griffiths ]
  * graphics: Pull in Eleni's changes to get the DRM fd to init GBM from
    the host Mir instance.
  * input: Separate the code for dispatching input from that reading it.
  * graphics: Hook up nested surfaces events to input.
  * input: Connect nested input relay to input dispatch.
  * graphics: Simplify NativeAndroidPlatform out of existence.
  * tests: Fixes to get the tests "passing" on android/arm stack. (LP...

[Till Kamppeter (till-kamppeter)]

I got mir 0.0.11+13.10.20130924.1-0ubuntu1 with the latest update and after that I have still the problem that the screen does not get into sleep mode.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:powerd at revision 94, scheduled for release in powerd, milestone ubuntu-13.05

[Launchpad Janitor (janitor)]

This bug was fixed in the package powerd - 0.13+13.10.20130926-0ubuntu1

---------------
powerd (0.13+13.10.20130926-0ubuntu1) saucy; urgency=low

  [ Ricardo Mendoza ]
  * Allow powerd to work with unity-mir's DBus API for screen power
    control, selective using the backend available according to the
    running display server. (LP: #1193222)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 94
 -- Ubuntu daily release <email address hidden> Thu, 26 Sep 2013 18:34:30 +0000

[Eduardo (ludentico)]

Installing powerd solved the bug... So, I guess this has to be a «recommended» package when installing mir

[Joseph Wakeling (webdrake)]

Screen still not turning off -- this is on a ThinkPad T420 with Intel graphics [Intel Corporation 2nd Generation Core Processor Family Integrated Graphics Controller (rev 09)].

Is powerd now considered necessary also for Ubuntu Desktop? If so as Eduardo says it should probably be added as a recommend or dependency of mir.

[Daniel van Vugt (vanvugt)]

Joseph,

Please join bug 1231857 if that's what you see.