(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.
(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) 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.
I agree that this should be fixed, and the existing /usr/lib/
* 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.