Shutdown dialog appears on resume

Bug #1508563 reported by Michael Terry on 2015-10-21
28
This bug affects 14 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Medium
kevin gunn
unity8 (Ubuntu)
Medium
Michael Terry

Bug Description

This is very similar to old friends bug 1383277 and bug 1410830.

But we never fully vanquished the beast. The dialog still appears on resume in some situations where the system is very loaded.
(also when not at all loaded)

I suspect what is happening is that we process the power-button-release event so much later than the -press event because of the load. And think that the user has been holding it down this whole time.

But instead, we should compare event timestamps to determine that. (Which I'm assuming we aren't doing because this bug exists.)

Related branches

Michael Terry (mterry) wrote :

Specifically, the load seems to only matter for dbus (which makes sense, it would delay IPC). So this mostly comes up when we see bug 1480877.

The following line reproduces this bug fairly easily if you press the power button a few times after running it:
sudo cpulimit -l 1 -c 1 -p `pidof dbus-daemon`

I have a proposed fix, but want broader comment on whether it's crack or not.

Michael Terry (mterry) on 2015-10-21
Changed in unity8 (Ubuntu):
assignee: nobody → Michael Terry (mterry)
status: New → In Progress
Michael Terry (mterry) wrote :

Uh... Use the following command instead of pidof to get the pid:
ps ax | grep dbus-daemon | head -n 1 | awk '{print $1;}'

I forgot there are two dbus-daemons.

Michał Sawicz (saviq) on 2015-10-26
Changed in unity8 (Ubuntu):
importance: Undecided → Medium
Pat McGowan (pat-mcgowan) wrote :

FWIW I reproduced getting the dialog on resume quite easily , I simply pressed the power button for more than 1 sec and less than 5, released it just before the dialog appeared. It was there when I resumed.
arale and krillin

description: updated
summary: - Shutdown dialog appears on resume under high load
+ Shutdown dialog appears on resume
Changed in canonical-devices-system-image:
status: New → In Progress
importance: Undecided → Medium
milestone: none → ww46-2015
Jean-Baptiste Lallement (jibel) wrote :

Fixed in unity8 (8.11+15.04.20151104-0ubuntu1)

Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.11+16.04.20151104-0ubuntu1

---------------
unity8 (8.11+16.04.20151104-0ubuntu1) xenial; urgency=medium

  [ Albert Astals Cid ]
  * Add an Item that proxies for old and new audio roles (LP: #1493851)
  * CroppedImageMinimumSourceSize: Fix 'Binding loop detected for
    property "imageAspectRatio"'
  * Make the scope settings list scroll to text visible area (LP:
    #1499084)
  * Move to importing QtQuick 2.4 & friends
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Andrea Cimitan ]
  * Add sharing widget to zoomable image and video playback
  * Make zoomable and video playback widgets edge to edge
  * Updated all 1.3 UbuntuShape to use new APIs where possible
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ CI Train Bot ]
  * Resync trunk.

  [ Christopher Lee ]
  * Quick spelling fix in process control output.

  [ Daniel d'Andrada ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Josh Arenson ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)

  [ Michael Terry ]
  * Avoid showing the shutdown dialog when turning on the screen if your
    device is under heavy load. (LP: #1508563)
  * Handle lifecycle policy exceptions ourselves, instead of letting
    qtmir do it for us and allow non-Touch apps to opt-out of the Touch
    lifecycle.

  [ Michael Zanetti ]
  * Fixes for the panel buttons (LP: #1510360, #1504269, #1431566,
    #1443319)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Michał Sawicz ]
  * Add missing copyright to Cursor.qml
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

  [ Nick Dedekind ]
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

 -- Michał Sawicz <email address hidden> Wed, 04 Nov 2015 14:58:41 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
kevin gunn (kgunn72) on 2015-11-06
Changed in canonical-devices-system-image:
assignee: nobody → kevin gunn (kgunn72)
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers