From owner-cvs-src@FreeBSD.ORG Mon Mar 17 18:44:31 2008 Return-Path: Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 816FC106566B; Mon, 17 Mar 2008 18:44:31 +0000 (UTC) (envelope-from sobomax@FreeBSD.org) Received: from sippysoft.com (gk1.360sip.com [72.236.70.240]) by mx1.freebsd.org (Postfix) with ESMTP id 4703F8FC1C; Mon, 17 Mar 2008 18:44:31 +0000 (UTC) (envelope-from sobomax@FreeBSD.org) Received: from [192.168.0.36] ([204.244.149.125]) (authenticated bits=0) by sippysoft.com (8.13.8/8.13.8) with ESMTP id m2HIiTnx042601 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Mon, 17 Mar 2008 11:44:30 -0700 (PDT) (envelope-from sobomax@FreeBSD.org) Message-ID: <47DEBC02.8060700@FreeBSD.org> Date: Mon, 17 Mar 2008 11:44:18 -0700 From: Maxim Sobolev Organization: Sippy Software, Inc. User-Agent: Thunderbird 2.0.0.12 (Windows/20080213) MIME-Version: 1.0 To: Poul-Henning Kamp References: <3709.1205762533@critter.freebsd.dk> In-Reply-To: <3709.1205762533@critter.freebsd.dk> Content-Type: text/plain; charset=KOI8-U; format=flowed Content-Transfer-Encoding: 7bit Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, John Baldwin Subject: Re: cvs commit: src/sys/i386/cpufreq est.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 17 Mar 2008 18:44:31 -0000 Poul-Henning Kamp wrote: > In message <200803170933.48212.jhb@freebsd.org>, John Baldwin writes: >> On Monday 17 March 2008 05:01:43 am Poul-Henning Kamp wrote: >>> phk 2008-03-17 09:01:43 UTC >>> >>> FreeBSD src repository >>> >>> Modified files: >>> sys/i386/cpufreq est.c >>> Log: >>> Increase time we wait for things to settle to 1 millisecond, >>> 10 microseconds is too short. >>> >>> Always set the cpu to the highest frequency so that we get through >>> boot and don't handicap cpus where powerd(8) is not used. >> Hmm, I actually consider this a feature when I'm not running powerd to use >> less battery. I think we should only bump up the CPU on battery power when >> using powerd so that it can be lowered again to save battery power when the >> CPU is idle. > > We have cpufreq enabled by default now, badly configured machines run > at 50% of rated CPU power because people don't know that they need to > enable powerd(8) on servers. > > This is only going to get worse when more EnergyStar compliant servers > hit the channel. > > I think setting full speed is the correct choice, if people care about > powersaving, they need to configured it, if they don't they should get > their moneys worth out of their hardware. Perhaps make it a kernel option? But I agree that CPU should run at full speed by default, otherwise it would break POLA for users upgrading from previous releases. -Maxim