From owner-cvs-src@FreeBSD.ORG Thu Nov 20 18:08:27 2003 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 4641116A4D0 for ; Thu, 20 Nov 2003 18:08:27 -0800 (PST) Received: from rootlabs.com (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 087EF43FE0 for ; Thu, 20 Nov 2003 18:08:25 -0800 (PST) (envelope-from nate@rootlabs.com) Received: (qmail 73185 invoked by uid 1000); 21 Nov 2003 02:08:26 -0000 Date: Thu, 20 Nov 2003 18:08:26 -0800 (PST) From: Nate Lawson To: Marcel Moolenaar In-Reply-To: <20031121012502.GA914@dhcp01.pn.xcllnt.net> Message-ID: <20031120180509.G73176@root.org> References: <20031120153849.H72721@root.org> <20031121012502.GA914@dhcp01.pn.xcllnt.net> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org cc: John Baldwin Subject: Re: cvs commit: src/sys/dev/acpica acpi.c acpi_cpu.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 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: Fri, 21 Nov 2003 02:08:27 -0000 On Thu, 20 Nov 2003, Marcel Moolenaar wrote: > On Thu, Nov 20, 2003 at 03:40:58PM -0800, Nate Lawson wrote: > > No good ideas over here. Took me a while to even come up with > > hw.acpi.cpu.cx_lowest. You want to indicate higher states will be also > > used as determined by the scheduling algorithm and that states with higher > > indices provide more power savings but also more latency. cx_lowest_idx > > seems kinda lame. Suggestions? > > Maybe make the Cx state type an "unimportant" attribute? > > MIB-like: > > hw.acpi.cpu.cx.count=4 > hw.acpi.cpu.cx.lowest=0 > hw.acpi.cpu.cx.0.type=C1 > hw.acpi.cpu.cx.0.latency=1 > hw.acpi.cpu.cx.0.longsleep=x > hw.acpi.cpu.cx.0.shortsleep=y > hw.acpi.cpu.cx.1.type=C2 > : > hw.acpu.cpu.cx.2.type=C3 > : > hw.acpu.cpu.cx.3.type=C3 > : > > Something like this makes the sysctl variables less magical, better > to parse mechanically and puts the emphasis on the index. It's more > verbose though... Nice, but too verbose for my tastes. Since this is something that will only be looked at by someone while tuning the system, I think more compact is better. Also it's really good to be able to do sysctl hw.acpi.cpu and get only 5-10 lines. I plan to reduce the verboseness of the throttling entries as well once more of the power control/devd stuff is committed. -Nate