From owner-freebsd-stable@FreeBSD.ORG Tue Nov 22 16:06:55 2005 Return-Path: X-Original-To: freebsd-stable@freebsd.org Delivered-To: freebsd-stable@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 68A9316A424; Tue, 22 Nov 2005 16:06:55 +0000 (GMT) (envelope-from pldrouin@pldrouin.net) Received: from smtp.cyberfingers.net (smtp.cyberfingers.net [198.177.254.227]) by mx1.FreeBSD.org (Postfix) with ESMTP id 205E543D5A; Tue, 22 Nov 2005 16:06:53 +0000 (GMT) (envelope-from pldrouin@pldrouin.net) Received: from [134.117.23.170] (pldrouinlap-pc.physics.carleton.ca [134.117.23.170]) by smtp.cyberfingers.net (Postfix) with ESMTP id 10618170177; Tue, 22 Nov 2005 11:06:54 -0500 (EST) Message-ID: <4383421B.6060403@pldrouin.net> Date: Tue, 22 Nov 2005 11:06:51 -0500 From: Pierre-Luc Drouin User-Agent: Mozilla Thunderbird 1.0.7 (X11/20051107) X-Accept-Language: en-us, en MIME-Version: 1.0 To: acpi@freebsd.org, freebsd-stable@freebsd.org References: <4377775B.3080606@pldrouin.net> <20051114105854.GA1041@galgenberg.net> <4378CC14.2020109@pldrouin.net> <437A3901.8010001@pldrouin.net> <437A3B96.4040300@root.org> <437B42C0.9040605@pldrouin.net> <437CAEB0.9060202@pldrouin.net> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Subject: Re: Performance problem since updating from 6.0-RELEASE to 6.0-STABLE last friday X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Nov 2005 16:06:55 -0000 Hajimu UMEMOTO wrote: >Hi, > > > >>>>>>On Thu, 17 Nov 2005 11:24:16 -0500 >>>>>>Pierre-Luc Drouin said: >>>>>> >>>>>> > > > >>Ok, there is new development. I realized by playing with >>debug.acpi.disabled="smbat", debug.acpi.disabled="smbat cmbat" and >>debug.acpi.disabled="cmbat", that my laptop battery is not a smbat, >>but a cmbat. When I played with hw.acpi.battery.info_expire after to >>have applied the patch for acpi_smbat.c, it was freezing less often >>because that sysctl variable was shared by both cmbat and smbat. So I >>can only get battery status from cmbat (disabling cmbat disables the >>use of acpiconf -i loop). To get the status of my battery via cmbat >>was working fine up to 6.0-RELEASE (included), but makes my laptop to >>freeze since I upgraded to 6.0-stable with Nov 10th sources. What >>change related to cmbat between 6.0-release and 6.0-stable could be >>causing this? >> >> > >pldrouin> Has someone found how to fix this problem in -stable? > >Perhaps, I found the cause. acpi_cmbat_get_bif() is heavy process, >and it was called only when ACPIIO_CMBAT_GET_BIF ioctl was issued >explicitly, until smbat stuff was committed. However, >acpiio_cmbat_get_bif() is called from every >acpi_battery_get_battinfo() call, now. >The attached patch will bring back to former behavior. Please try it >and let me know the result. >It is against 7-CURRENT as of today. If you want to try it on >6-STABLE, you need to apply following diff before applying it: > > > Wonderful! It fixes it!! I applied both patches and everything seams to be working fine now!