Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 31 Jul 2012 11:02:44 -0700
From:      Sean Bruno <seanbru@yahoo-inc.com>
To:        Alexander Motin <mav@freebsd.org>
Cc:        "svn-src-head@FreeBSD.org" <svn-src-head@freebsd.org>, "svn-src-all@FreeBSD.org" <svn-src-all@freebsd.org>, "src-committers@FreeBSD.org" <src-committers@freebsd.org>
Subject:   Re: svn commit: r238943 - head/sys/dev/acpica
Message-ID:  <1343757764.2957.12.camel@powernoodle.corp.yahoo.com>
In-Reply-To: <1343751146.2957.3.camel@powernoodle.corp.yahoo.com>
References:  <201207311058.q6VAwp18019777@svn.freebsd.org> <1343751146.2957.3.camel@powernoodle.corp.yahoo.com>

next in thread | previous in thread | raw e-mail | index | archive | help
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 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
> 
> Sean
> 

Upon further review and discussion, I remove my objection to this
commit.  It does *not* break power_profile in my test cases.

sean




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1343757764.2957.12.camel>