svn commit: r238943 - head/sys/dev/acpica

2012-07-31 Thread Alexander Motin
Author: mav Date: Tue Jul 31 10:58:50 2012 New Revision: 238943 URL: http://svn.freebsd.org/changeset/base/238943 Log: Add several performance optimizations to acpi_cpu_idle(). For C1 and C2 states use cpu_ticks() to measure sleep time instead of much slower ACPI timer. We can't do it

Re: svn commit: r238943 - head/sys/dev/acpica

2012-07-31 Thread Andriy Gapon
on 31/07/2012 13:58 Alexander Motin said the following: To help with diagnostics, add C-state type into dev.cpu.X.cx_supported I think that this really should have been a separate commit. Besides it is a POLA violation for those who may parse output of this sysctl for whatever reason.

Re: svn commit: r238943 - head/sys/dev/acpica

2012-07-31 Thread Sean Bruno
On Tue, 2012-07-31 at 03:58 -0700, Alexander Motin wrote: To help with diagnostics, add C-state type into dev.cpu.X.cx_supported. Pretty sure that this breaks /etc/rc.d/power_profile Andiry and I went back and forth on this specific thing quite a bit. If you note my commit/revert at svn

Re: svn commit: r238943 - head/sys/dev/acpica

2012-07-31 Thread Alexander Motin
On 31.07.2012 19:12, Sean Bruno wrote: On Tue, 2012-07-31 at 03:58 -0700, Alexander Motin wrote: To help with diagnostics, add C-state type into dev.cpu.X.cx_supported. Pretty sure that this breaks /etc/rc.d/power_profile No, it doesn't. I've checked it beforehand. At least in my case

Re: svn commit: r238943 - head/sys/dev/acpica

2012-07-31 Thread Sean Bruno
On Tue, 2012-07-31 at 09:12 -0700, Sean Bruno wrote: On Tue, 2012-07-31 at 03:58 -0700, Alexander Motin wrote: To help with diagnostics, add C-state type into dev.cpu.X.cx_supported. Pretty sure that this breaks /etc/rc.d/power_profile Andiry and I went back and forth on this specific