From owner-freebsd-current@FreeBSD.ORG Mon Jul 25 18:41:09 2005 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 37B5316A41F; Mon, 25 Jul 2005 18:41:09 +0000 (GMT) (envelope-from wb@freebie.xs4all.nl) Received: from smtp-vbr11.xs4all.nl (smtp-vbr11.xs4all.nl [194.109.24.31]) by mx1.FreeBSD.org (Postfix) with ESMTP id 22A7F43D53; Mon, 25 Jul 2005 18:41:07 +0000 (GMT) (envelope-from wb@freebie.xs4all.nl) Received: from freebie.xs4all.nl (freebie.xs4all.nl [213.84.32.253]) by smtp-vbr11.xs4all.nl (8.13.3/8.13.3) with ESMTP id j6PIf2J0041889; Mon, 25 Jul 2005 20:41:02 +0200 (CEST) (envelope-from wb@freebie.xs4all.nl) Received: from freebie.xs4all.nl (localhost [127.0.0.1]) by freebie.xs4all.nl (8.13.3/8.13.3) with ESMTP id j6PIf1be041045; Mon, 25 Jul 2005 20:41:01 +0200 (CEST) (envelope-from wb@freebie.xs4all.nl) Received: (from wb@localhost) by freebie.xs4all.nl (8.13.3/8.13.1/Submit) id j6PIf1Zx041044; Mon, 25 Jul 2005 20:41:01 +0200 (CEST) (envelope-from wb) Date: Mon, 25 Jul 2005 20:41:01 +0200 From: Wilko Bulte To: Nate Lawson Message-ID: <20050725184101.GA41023@freebie.xs4all.nl> References: <42E1481F.5040306@root.org> <20050723133819.36efb537.lists@yazzy.org> <42E29CAA.1020007@root.org> <20050724215818.55f951ae.lists@yazzy.org> <42E42F10.3030309@root.org> <20050725091221.106a0368.lists@yazzy.org> <42E5134F.706@root.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <42E5134F.706@root.org> X-OS: FreeBSD 5.4-STABLE User-Agent: Mutt/1.5.9i X-Virus-Scanned: by XS4ALL Virus Scanner Cc: acpi@freebsd.org, kjelderg@gmail.com, Marcin Jessa , freebsd-current@freebsd.org Subject: Re: acpi battery rework patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 25 Jul 2005 18:41:09 -0000 On Mon, Jul 25, 2005 at 09:29:03AM -0700, Nate Lawson wrote.. > Marcin Jessa wrote: > >On Sun, 24 Jul 2005 17:15:12 -0700 > >Nate Lawson wrote: > >>As far as Eric's result, I'll check that on my systems as well. I > >>_think_ the previous behavior was more correct (-1 = unknown) so I'll > >>commit this minor change. The "state" variable should not have changed > >>so please let me know for sure if it has. "1" means "discharging" which > >>doesn't sound right if you're on AC power. If you just recently plugged > >>in the laptop, try polling the status a few more times (apm(8) gives > >>more detail) and see if it catches up. Some systems lag reporting > >>various values until they are sure they've restabilized after an AC line > >>change. > >> > >>Marcin, if your system reports a valid rate of use, we can calculate > >>time. If you're on AC power, there is no time available (since your > >>batteries aren't discharging at any rate). Try a previous > >>kernel/acpi.ko to be certain what the behavior used to be. > > > > > >[yazzy@lapdance:~]> uname -a > >FreeBSD lapdance.yazzy.net 6.0-BETA1 FreeBSD 6.0-BETA1 #4: Sat Jul 23 > >17:35:13 CEST 2005 > >root@lapdance.yazzy.net:/usr/obj/usr/src/sys/LAPDANCE-6.0 i386 > > > >Running on AC: > >[yazzy@lapdance:~]> sysctl -a | grep battery > >hw.acpi.battery.life: 23 > >hw.acpi.battery.time: -1 > >hw.acpi.battery.state: 2 > >hw.acpi.battery.units: 1 > >hw.acpi.battery.info_expire: 5 > > > >Running on battery: > >[yazzy@lapdance:~]> sysctl -a | grep battery > >hw.acpi.battery.life: 23 > >hw.acpi.battery.time: 41 > >hw.acpi.battery.state: 1 > >hw.acpi.battery.units: 1 > >hw.acpi.battery.info_expire: 5 > >[yazzy@lapdance:~]> > > And with a 7-current kernel, everything is the same except "time" is > always -1 both on and off AC power? > > Please try this patch with 7-current and report its output both on and > off power. It prints out more debugging info each time the battery is read. On an Compaq EVO N160, on AC power Trying to mount root from ufs:/dev/ad0s2a batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 ichss0: transition to 1 failed ichss0: transition to 1 failed ichss0: transition to 1 failed batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 ichss0: transition to 1 failed ichss0: transition to 1 failed batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 2 cap 0 valid_rate 0 batt pass2: min 0 total_min 0 total_cap 0 ichss0: transition to 1 failed APM version: 1.2 APM Management: Enabled AC Line status: on-line Battery Status: charging Remaining battery life: 0% Remaining battery time: 0:00:00 Number of batteries: 1 Battery 0: Battery Status: charging Remaining battery life: 0% Remaining battery time: 0:00:00 wb@chuck ~: Unplugging AC power: batt pass1: dev 0xc15e3100 ok valid state 1 cap 0 valid_rate 1082 batt pass2: min 166 total_min 166 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 1 cap 0 valid_rate 1082 batt pass2: min 166 total_min 166 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 1 cap 0 valid_rate 1082 batt pass2: min 166 total_min 166 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 1 cap 0 valid_rate 1082 batt pass2: min 166 total_min 166 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 1 cap 0 valid_rate 1082 batt pass2: min 166 total_min 166 total_cap 0 batt pass1: dev 0xc15e3100 ok valid state 1 cap 0 valid_rate 1082 batt pass2: min 166 total_min 166 total_cap 0 wb@chuck ~: wb@chuck ~: apm APM version: 1.2 APM Management: Enabled AC Line status: off-line Battery Status: low Remaining battery life: 0% Remaining battery time: 2:31:00 Number of batteries: 1 Battery 0: Battery Status: low Remaining battery life: 0% Remaining battery time: 2:31:00 wb@chuck ~: -- Wilko Bulte wilko@FreeBSD.org