00powersave sleep hook misses claimed patch

Bug #1548853 reported by Dirk F
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pm-utils (Ubuntu)
Fix Released
Low
Unassigned
Nominated for Trusty by Christian Ehrhardt 

Bug Description

The pm-utils sleep hook installed at /usr/lib/pm-utils/sleep.d/00powersave was supposed to have been patched to prevent it running on ARM architectures. However a new case was added without deleting the preceding existing one (lines 12 on):

case $1 in
    suspend|hibernate) pm-powersave false ;;
    suspend|hibernate) [ "$ARCH" != "${ARCH#arm}" ] || pm-powersave false ;;
    resume|thaw) pm-powersave ;;
    *) exit $NA ;;
esac

Observed in and affects 14.04.3,4.; seems to be OK in 14.10-15.10.

ProblemType: Bug
DistroRelease: Ubuntu 14.04
Package: pm-utils 1.4.1-13ubuntu0.2 [modified: usr/lib/pm-utils/pm-functions]
ProcVersionSignature: Ubuntu 4.2.0-29.34~14.04.1-generic 4.2.8-ckt3
Uname: Linux 4.2.0-29-generic i686
ApportVersion: 2.14.1-0ubuntu3.19
Architecture: i386
Date: Tue Feb 23 13:51:14 2016
InstallationDate: Installed on 2016-02-21 (1 days ago)
InstallationMedia: Lubuntu 14.04.4 LTS "Trusty Tahr" - Release i386 (20160217.1)
PackageArchitecture: all
SourcePackage: pm-utils
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Dirk F (fieldhouse) wrote :
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

This is the second bug I've seen from you and I feel embarrassed that nobody looked at them yet, Iso I wanted to beg a pardon and thank you for your work.

To fix this it would mean backporting debian/patches/16-no-powersave-before-suspend.patch for an SRU.
Not complex to but somewhat discussion worthy if the effect is worth an SRU.

The fix documents this as:
"do not run pm-powersave on ARM during suspend; the 1.5 seconds that it takes to run it don't nearly compensate the potentially slightly slower suspend operation in low power mode"

Fixed in 1.4.1-14 so everything >=Vivid is good.

Changed in pm-utils (Ubuntu):
status: New → Triaged
importance: Undecided → Low
tags: added: bitesize patch
removed: package-from-proposed
Changed in pm-utils (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.