Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 13 Jul 2012 11:16:21 +0300
From:      Andriy Gapon <avg@FreeBSD.org>
To:        Vitaly Magerya <vmagerya@gmail.com>
Cc:        "freebsd-acpi@freebsd.org" <freebsd-acpi@FreeBSD.org>, Giovanni Trematerra <gianni@FreeBSD.org>, John Baldwin <jhb@FreeBSD.org>
Subject:   Re: improve cx_lowest logic
Message-ID:  <4FFFD955.9030200@FreeBSD.org>
In-Reply-To: <CAL409Ky6qmeBUaTeCqrRcvYEH%2BgDUqqUgvpFiKSCt2d361j64g@mail.gmail.com>
References:  <4FF95F79.30309@FreeBSD.org> <CAL409Ky6qmeBUaTeCqrRcvYEH%2BgDUqqUgvpFiKSCt2d361j64g@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
on 10/07/2012 19:30 Vitaly Magerya said the following:
> Andriy Gapon <avg@freebsd.org> wrote:
>> I would like to propose the following change for review and testing:
>> http://people.freebsd.org/~avg/acpi_cpu_cx_lowest.diff
> 
> I just tested this on a laptop that jumps between reporting only
> C1, C1 and C2, and C1 through C3; in all cases your patch does
> the right thing, so this removes the need for kludges I had to
> do earlier; thank you, Andriy.
> 
> Will you also change how /etc/rc.d/power_profile selects
> lowest_value? It currently uses the maximum from dev.cpu.0.cx_supported,
> but with your patch it'll be the right thing to simply use Cmax.

I've put this on my to do list.  Hope to commit this trivial change soon-ish.

-- 
Andriy Gapon





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