acpi-support should let laptop-mode-tools run properly

good work thanks

This seems to make a lot of sense. But what if laptop_mode is not enabled (which is default), will there be no spindown time set at all? I agree it doesn't belong here anyway, it's not an ACPI thing.

If laptop-mode is not enabled (by default) then nothing changes. Currently, without laptop-mode enabled there is not spindown time as well.

It's interesting to know that current acpi-support in Debian unstable (1.109-5) has this comment in /etc/default/acpi-support:

# Note: to enable "laptop mode" (to spin down your hard drive for longer
# periods of time), install the laptop-mode-tools package and configure
# it in /etc/laptop-mode/laptop-mode.conf.
#
# (Note to upgraders: earlier versions of the acpi-support package contained
# an option to enable/disable laptop mode. This option has never actually
# worked, and for that reason it has been removed.)

> Currently, without laptop-mode enabled there is not spindown time as well.

I think I can confirm this, yes. I have ENABLE_LAPTOP_MODE=false in /etc/default/acpi-support.

However, I am now trying out the debdiff3 in bug #255446 (includes your patch from bug #250935), and with this version the hard disk spins down nevertheless. (This is more a comment to those two bugs than this one though)

I found out why: there wasn't any space around the "=" in the check in the hooks (in patch for bug #250935). [ xtrue=xfalse] yields true.

Alexey!
I'd like to say thank you.
For providing patches to the laptop-mode related things in ubuntu.
It was good to see your work apear.

This bug was fixed in the package acpi-support - 0.110

---------------
acpi-support (0.110) intrepid; urgency=low

  * Let laptop-mode-tools run properly (closes LP: #250938)
    - power.sh: remove laptop-mode invocations, should be handled by pm-utils
    - acpi-support: remove SPINDOWN_TIME, should be controlled by laptop-mode

 -- Alexey Borzenkov <email address hidden> Tue, 22 Jul 2008 23:59:07 +0400

Hmm, laptop-mode-tools still doesn't work properly.... the logic to control laptop-mode from ACPI-support has been removed, but the logic within the laptop-mode-tools package itself has not been fixed.
Note the following line in the changelog: "- usr/sbin/laptop_mode: Do not parse arguments, ignore "force" and let start/stop mean enable/disable."

laptop-mode-tools (1.45-1ubuntu1) intrepid; urgency=low

  [ Tormod Volden ]
  * Merge from debian unstable (LP: #255446), remaining changes:
    - etc/init.d/laptop-mode: Check if laptop mode is disabled in
      /etc/default/laptop-mode (that's the file laptop_mode looks into, too,
      but it incorrectly evaluates the ENABLE_LAPTOP_MODE setting) as well
      as in /etc/default/acpi-support.
    - usr/sbin/laptop_mode: Do not parse arguments, ignore "force" and
      let start/stop mean enable/disable.
    - /usr/sbin/laptop_mode: Do not read $ACTIVATE_WITH_POSSIBLE_DATA_LOSS
      from /var/run/laptop-mode-state to see if state has changed.
    - debian/rules: Do not ship acpi/apm scripts (we handle that ourselves)
    - debian/laptop-mode-tools.preinst: Remove any old acpi/apm scripts
    - debian/laptop-mode-tools.postinst: Create /etc/default/laptop-mode
      (with ENABLE_LAPTOP_MODE=false) if the file does not exist yet and we
      are on ppc.
    - Also accept "stop" as second argument since the init script
      calls laptop_mode with "init stop"

  [ Alexey Borzenkov ]
  * Change default settings to match acpi-support (LP: #250935)
    - etc/laptop-mode/laptop-mode.conf: change default spindown time and
      power management modes
  * Add pm-utils hooks to ensure laptop-mode is running as expected after
    power change or thaw/resume
    - power.d/laptop-mode: runs laptop-mode on power change
    - sleep.d/96laptop-mode: runs laptop-mode on thaw/resume

 -- Tormod Volden <email address hidden> Wed, 06 Aug 2008 20:06:08 +0200