FS#31355 - [cpupower] Remove /usr/lib/pm-utils/power.d/cpupower
Attached to Project:
Community Packages
Opened by Thomas Bächler (brain0) - Friday, 31 August 2012, 10:14 GMT
Last edited by Sébastien Luttringer (seblu) - Saturday, 15 September 2012, 18:23 GMT
Opened by Thomas Bächler (brain0) - Friday, 31 August 2012, 10:14 GMT
Last edited by Sébastien Luttringer (seblu) - Saturday, 15 September 2012, 18:23 GMT
|
Details
The addition of the file /usr/lib/pm-utils/power.d/cpupower
enables it by default. The user is not informed that the
installation of cpupower messes with his power settings.
On top of that, the default behaviour is complete nonsense: It switches to 'conservative' when on battery power. On modern CPUs, you should always use the 'ondemand' governor, regardless of battery or AC. If a user wishes such behaviour, he should add the file himself. |
This task depends upon
Closed by Sébastien Luttringer (seblu)
Saturday, 15 September 2012, 18:23 GMT
Reason for closing: Fixed
Additional comments about closing: cpupower 3.5-5
Saturday, 15 September 2012, 18:23 GMT
Reason for closing: Fixed
Additional comments about closing: cpupower 3.5-5
Before adding this, I thought like you about conservative/ondemand. But reading kernel doc on conservative vs ondemand, I changed my mind. Conversative increase speed step by step where ondemand give full power when directly. So better governor for battery would be conservative.
Why do you see it differently?
About disabling, like modules in /usr/lib/modules-load.d, files are loaded by default in /usr/lib/pm-utils/power.d/. The upstream way to disable it is to put an empty file in /etc/pm/power.d, like for module-load.d, unlike systemd services.
"conservative" is only meant for CPUs that can't handle the short transition times of ondemand.
BTW, the upstream way in cpupower is not to provide this file. Arch should follow this.
Having read the file, I agree with Thomas: We should never default to anything else than on-demand (which is why I pushed to change our default to that in the kernel some time ago). Users might want to do other things in special cases (weird/broken hardware,++), but that should be done on a case-by-case basis only.