Comment 2 for bug 738757

Revision history for this message
Martin Pitt (pitti) wrote :

(This should be fixed in hdparm, closing pm-utils task).

Thanks heckneck for this report!

So essentially your script changes two things:

  * It runs at suspend/resume time instead of when changing AC <-> Battery (by being placed into suspend.d instead of power.d)
     I agree that this should be fixed, and the existing /usr/lib/pm-utils/power.d/95hdparm-apm even supports being called as suspend.d already. I'm happy to add a symlink to suspend.d/ to this.

  * It adds handling of the -S option from /etc/hdparm.conf. We don't specify one by default, so adding support for this sounds safe, as it won't change default behaviour.

However, don't think we should handle -S only for the power.d script and -B only for suspend, but both in both modes.