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 variable was correctly set to C3.

Andiry and I went back and forth on this specific thing quite a bit.  If
you note my commit/revert at svn r238004 and r238009 where I did a
similar thing and then had to yank it out.

there's a lot of discussion around this point on the acpi mailing lists.
http://lists.freebsd.org/pipermail/freebsd-acpi/2012-July/007645.html

I knew about that change and its revert. But it was much more invasive, substituting C-state index with C-state type, that I also consider incorrect. My change is much more simple. Yes, it may affect some POLA, but that is all. I wanted to do this for several years, and I believe that in this case benefits overweight negative side. But if someone tell I should back it out, I will.

--
Alexander Motin
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to