always runs 95hdparm-apm twice for no reason

Bug #437796 reported by Tormod Volden
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pm-utils (Ubuntu)
Undecided
Unassigned

Bug Description

Binary package hint: pm-utils

At sleep and resume, two identical 95hdparm-apm scripts are run, first the one in sleep.d, then through sleep.d/00powersave also the one in power.d since 1.2.5-2ubuntu4 added it there also (bug 385949).

At sleep, this does not matter, since the script does nothing for "sleep" or "hibernate" actions (the sleep.d one), but only for the "false" (or "true) action (the power.d one).

At resume, the same action is run for "resume" as well as for "false" (or "true"). This adds unnecessary to the execution time, by adding among others two hdparm calls.

Since the one in power.d will be run anyway, we can safely delete the one in sleep.d.

ProblemType: Bug
Architecture: i386
Date: Sun Sep 27 21:12:15 2009
DistroRelease: Ubuntu 9.10
Package: pm-utils 1.2.5-2ubuntu5
PackageArchitecture: all
ProcEnviron:
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-11.36-generic
SourcePackage: pm-utils
Uname: Linux 2.6.31-11-generic i686

Related branches

Revision history for this message
Tormod Volden (tormodvolden) wrote :
Revision history for this message
Tormod Volden (tormodvolden) wrote :
Revision history for this message
Tormod Volden (tormodvolden) wrote :

In this debdiff I have also added a detection of the "nohdparm" boot option.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pm-utils - 1.2.5-2ubuntu6

---------------
pm-utils (1.2.5-2ubuntu6) karmic; urgency=low

  * debian/rules: Do not install sleep.d/95hdparm-apm since
    sleep.d/00powersave will call power.d/95hdparm-apm anyway
    (LP: #437796)
  * debian/95hdparm-apm: skip if booted with "nohdparm", using the existing
    convention of the hdparm package.

 -- Tormod Volden <email address hidden> Sun, 27 Sep 2009 21:04:23 +0200

Changed in pm-utils (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers