aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Gallatin <gallatin@FreeBSD.org>2023-05-22 00:47:28 +0000
committerAndrew Gallatin <gallatin@FreeBSD.org>2023-05-25 13:40:26 +0000
commitfd96685a4a579fc84031e8e66d8f8b1ce8cdf1e5 (patch)
treea2eb88e5e886b4c137b3b03471b9150c75ade147
parent430d064ba5b0cb2e91a26af34c15df48d290e417 (diff)
downloadsrc-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-xlibexec/rc/rc.d/powerd7
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"