diff options
author | Andrew Gallatin <gallatin@FreeBSD.org> | 2023-05-22 00:47:28 +0000 |
---|---|---|
committer | Andrew Gallatin <gallatin@FreeBSD.org> | 2023-05-25 13:40:26 +0000 |
commit | fd96685a4a579fc84031e8e66d8f8b1ce8cdf1e5 (patch) | |
tree | a2eb88e5e886b4c137b3b03471b9150c75ade147 | |
parent | 430d064ba5b0cb2e91a26af34c15df48d290e417 (diff) | |
download | src-fd96685a4a579fc84031e8e66d8f8b1ce8cdf1e5.tar.gz src-fd96685a4a579fc84031e8e66d8f8b1ce8cdf1e5.zip |
Revert "When stopping powerd, set the CPU frequency back to its maximum value"
This reverts commit 1dcb6ad173e57b489a859ea59ed6eaa733bdb5bc.
As of "8cb16fdbea6b Restore original frequency on exit.", powerd
restores the original frequency itself.
Further, if the original frequency is not the same as the
first frequency found in the frequency list, then the restoration
done by the powerd_poststop will restore the wrong frequency.
This can happen on Intel machines where Turbo is not enabled,
but the turbo frequency is first in the list of frequencies.
In this case, turbo will be enabled when the user did not want
it to be.
Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D40197
Reviewed by: imp, mav
-rwxr-xr-x | libexec/rc/rc.d/powerd | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/libexec/rc/rc.d/powerd b/libexec/rc/rc.d/powerd index 2fc783a627e9..6f63bb96ff42 100755 --- a/libexec/rc/rc.d/powerd +++ b/libexec/rc/rc.d/powerd @@ -14,13 +14,6 @@ name="powerd" desc="Modify the power profile based on AC line state" rcvar="powerd_enable" command="/usr/sbin/${name}" -stop_postcmd=powerd_poststop - -powerd_poststop() -{ - sysctl dev.cpu.0.freq=`sysctl -n dev.cpu.0.freq_levels | - sed -e 's:/.*::'` > /dev/null -} load_rc_config $name run_rc_command "$1" |