#695363 pm-utils: /usr/lib/pm-utils/power.d/sata_alpm imposes max_power

Package:
pm-utils
Source:
pm-utils
Submitter:
Stefan Monnier
Date:
2023-12-21 18:02:08 UTC
Severity:
normal
#695363#5
Date:
2012-12-07 15:24:38 UTC
From:
To:
Dear Maintainer,

Whenever I plug my power adapter, /usr/lib/pm-utils/power.d/sata_alpm
sets the link_power_management_policy to max_performance.  I think that
SATA_ALPM_ENABLE should be extended to 4 different values:
- don't touch.
- choose min_power or max_performance depending on ac power.
- always set max_performance.
- always set min_power.

Currently, I can get the second behavior by setting SATA_ALPM_ENABLE to
"true", I can sort-of get the third behavior by leaving SATA_ALPM_ENABLE
at some other value, and to get the first behavior I have to
"touch /etc/pm/power.d/sata_alpm".  And there's no way to get the fourth
other than to do it outside of pm-utils (after doing the "touch").


	Stefan
#695363#10
Date:
2012-12-07 15:41:01 UTC
From:
To:
Hi,

The sata_alpm hook is disabled by default, you need to explicitly enable
it e.g. via
echo SATA_ALPM_ENABLE=true > /etc/pm/config.d/sata_alpm

This is what is what you get if SATA_ALPM_ENABLE=true

What would be the point of that? If you don't want pm-utils to change
the alpm state, set it outside of pm-utils and use SATA_ALPM_ENABLE=false.

Michael
#695363#15
Date:
2012-12-07 17:11:19 UTC
From:
To:
When SATA_ALPM_ENABLE=false you get "always set max_performance" because
sata_alpm does:

   case $1 in
       true) [ "$SATA_ALPM_ENABLE" = true ] && set_sata_alpm min_power;;
       false) set_sata_alpm max_performance;;
       help) help;;
       *) exit $NA;;
   esac

i.e. it calls "set_sata_alpm max_performance" regardless of
SATA_ALPM_ENABLE when you plug the power cord.

That's what the subject of this bug report says and is my main complaint.


        Stefan
#695363#20
Date:
2012-12-07 17:32:09 UTC
From:
To:
Ok, got it.

Maybe we should use something like this:
case $1 in
    true) [ "$SATA_ALPM_ENABLE" = true ] && set_sata_alpm min_power;;
    false) [ "$SATA_ALPM_ENABLE" = true ] && set_sata_alpm max_performance;;


This somehow seems more logical to me and would be more consistent with
/usr/lib/pm-utils/power.d/intel-audio-powersave,
/usr/lib/pm-utils/power.d/pci_devices and
/usr/lib/pm-utils/power.d/usb_bluetooth


Michael
#695363#25
Date:
2012-12-07 18:52:23 UTC
From:
To:
Yes.


        Stefan
#695363#30
Date:
2013-01-01 20:08:12 UTC
From:
To:
A Happy New Year to everyone!


Am Freitag, den 07.12.2012, 13:52 -0500 schrieb Stefan Monnier:

Stefan, thank you for your report and help. It would be awesome, if you
could prepare a patch with a nice commit message. Even better if you
could submit it upstream (list and/or bug tracker) too, although it is
currently not maintained as far as I can see. (OT: Is there a
replacement for it?) But Michael seems to have upstream commit right.


Thanks,

Paul
#695363#35
Date:
2023-12-21 17:56:09 UTC
From:
To:
Dear submitter,

as the package pm-utils has just been removed from the Debian archive
unstable we hereby close the associated bug reports.  We are sorry
that we couldn't deal with your issue properly.

For details on the removal, please see https://bugs.debian.org/1058701

The version of this package that was in Debian prior to this removal
can still be found using https://snapshot.debian.org/.

Please note that the changes have been done on the master archive and
will not propagate to any mirrors until the next dinstall run at the
earliest.

This message was generated automatically; if you believe that there is
a problem with it please contact the archive administrators by mailing
ftpmaster@ftp-master.debian.org.

Debian distribution maintenance software
pp.
Thorsten Alteholz (the ftpmaster behind the curtain)