[Bug 222458] Re: /lib/udev/hdparm Compares Against $DEVNAME.

[Launchpad Bug Tracker]

This bug was fixed in the package hdparm - 9.42-1ubuntu1

---------------
hdparm (9.42-1ubuntu1) raring; urgency=low

  * Merge from Debian unstable:
    + Adds support to the udev script for hdparm.conf stanzas that point at
      a symlink of the device rather than the device itself.  LP: #222458.
  * Remaining changes:
    + debian/control:
      - Add Breaks on old udev
      - Do not depend on lsb-base (we no longer use init script)
    + debian/hpdarm.dirs:
      - Drop unused /etc/udev/rules.d
    + debian/rules, debian/hdparm.udev-script:
      - Install udev-script, drop init file stuff
    + debian/rules:
      - Install a suspend.d symlink to also call the pm-utils hook on
        resuming.
  * Dropped changes, included in Debian:
    + debian/control:
      - Add Homepage
      - cdbs build-dependency bump
      - Recommend powermgmt-base
    + debian/rules:
      - Pass --priority=85 to dh_installudev
    + debian/hdparm.postinst, debian/rules: move udev rules to /lib/udev
    + Addition of debian/95hdparm-apm and debian/hdparm-functions
    + Add a new option to hdparm.conf, apm_battery, which is used in place of
      apm when we detect that we're on battery power.
    + debian/hdparm.udev-script, debian/hdparm.init:
      - only use the 'apm' setting when we're not on battery; when we are on
        battery, apply the separate 'apm_battery' setting.
      - Don't call hdparm -f for every drive
      - don't call sync
    + debian/hdparm.install:
      - Use hpdarm.udev instead of .rules
    + debian/hdparm.conf, debian/hdparm.conf.5: drop 'command_line' example
  * Dropped changes:
    - pm-utils Breaks/Replaces: Ubuntu-specific and no longer needed for
      upgrades.
    - adding lib/udev for our udev-script: dh_installudev takes care of this
      for us.
    - Remove broken symlink on upgrade: pre-lucid upgrades, no longer needed.
  * Don't trigger the udev rule on sd[a-z]*; yes, this means we aren't
    handling /dev/sdaa, but it also means we avoid spurious triggers on
    /dev/sda{1,2,...}

hdparm (9.42-1) unstable; urgency=low

  * New upstream release
  * Allow udev rule to process large numbers of devices (closes: #675624)
  * Changed hdparm-functions to allow links instead of devices in hdparm.conf.
  * Taken from Ubuntu:
    Set spindown policy back to -B128 instead of -B127. Too many drives
    misbehave too badly with this setting, possibly leading to drive failure.
    We should revisit this once we understand why people's drives are spinning
    back *up* so frequently (closes: #684241)

hdparm (9.39-1) unstable; urgency=low

  * New upstream version (closes: #602522, #631614, #639660)
  * Added myself to uploaders
  * Added Homepage field (closes: #533190, #545005)
  * Changed linking stage to not strip binary, otherwise the debug package
    would be empty
  * Renamed XC-Package-Type to Package-Type
  * No longer install idectl, that relies on options that were removed, and
    ultrabayd, that relies on idectl (closes: #511283)
  * Install UDEV rule to /lib/udev/rules.d (closes: #563762, #622568)
  * Bumped build dependency on debhelper for correct udev handling.
  * Install wiper.sh into its own subdirectory together with its README.txt
  * Recommend powermgmt-base
  * Thanks to Dmitry Smirnov <onlyjob at member.fsf.org>:
    + Added description to init file
    + Updated watch file to fetch from sourceforge
    + Clean up old rules from /etc/udev in postinst
  * Taken from Ubuntu:
    + pm-utils script 95-hdparm-apm:
      - run hdparm on wakeup (closes: #468307)
      - full integration with pm-utils (closes: #510676)
      - adjust hard drive APM settings (closes: #664041)
    + /lib/hdparm/hdparm-functions:
      -  prepare to replace our init script eventually
      -  special handling for Firewire/USB (closes: #654873)
    + hdparm.init:
      - only use the 'apm' setting when we're not on battery; when we are on
        battery, apply the separate 'apm_battery' setting.
      - Don't call hdparm -f for every drive; no clear rationale was ever given
        for doing this, and doing so slows down hard drive access at boot time.
      - also don't call sync... (closes: #546813)
    * hdparm.conf.5
  * hdparm.conf: added apm_battery, removed command_line example
  * Bumped Standards-Version to 3.9.3, no changes needed
 -- Steve Langasek <steve.langasek at ubuntu.com>   Wed, 24 Oct 2012 21:41:35 -0700

** Changed in: hdparm (Ubuntu)
       Status: Confirmed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to hdparm in Ubuntu.
https://bugs.launchpad.net/bugs/222458

Title:
  /lib/udev/hdparm Compares Against $DEVNAME.

Status in “hdparm” package in Ubuntu:
  Fix Released

Bug description:
  Binary package hint: hdparm

  Ubuntu 7.10, hdparm 7.5-1ubuntu1.  My /etc/hdparm.conf has entries
  like

      /dev/disk/by-id/ata-ST3160023A_4JS03MMK {
          spindown_time = 60
      }

  which work just fine with /etc/init.d/hdparm.  However, it appears
  init.d/hdparm is on its way out and has been replaced with
  /lib/udev/hdparm.  That only executes hdparm(8) for bits of
  /etc/hdparm.conf that match $DEVNAME, presumably passed in by udev.  I
  guess it'll have values like /dev/sda or /dev/hda.  However, I don't
  necessarily know what the /dev name will be, so I've specified it by
  /dev/disk/by-id instead which is a symlink.

      $ stat -c %N /dev/disk/by-id/ata-ST3160023A_4JS03MMK
      `/dev/disk/by-id/ata-ST3160023A_4JS03MMK' -> `../../hda'

  It seems that /lib/udev/hdparm needs to do some readlink(1) or similar
  for each of the discs specified in /etc/hdparm.conf, e.g. `readlink -e
  /dev/disk/by-id/ata-ST3160023A_4JS03MMK' gives /dev/hda, and only then
  compare against $DEVNAME.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/hdparm/+bug/222458/+subscriptions