From owner-freebsd-stable@FreeBSD.ORG Wed Nov 17 23:08:06 2010 Return-Path: Delivered-To: freebsd-stable@FreeBSD.org Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by hub.freebsd.org (Postfix) with ESMTP id DF0521065670; Wed, 17 Nov 2010 23:08:05 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: freebsd-stable@FreeBSD.org Date: Wed, 17 Nov 2010 18:07:54 -0500 User-Agent: KMail/1.6.2 References: <4CE42244.8060309@freebsd.org> <201011171741.57389.jkim@FreeBSD.org> <201011171748.11701.jkim@FreeBSD.org> In-Reply-To: <201011171748.11701.jkim@FreeBSD.org> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201011171807.57841.jkim@FreeBSD.org> Cc: freebsd-acpi@freebsd.org, "Alexandre \"Sunny\" Kovalenko" , Andriy Gapon Subject: Re: acpi(ca) mfc for 8.2 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: Wed, 17 Nov 2010 23:08:06 -0000 On Wednesday 17 November 2010 05:48 pm, Jung-uk Kim wrote: > On Wednesday 17 November 2010 05:41 pm, Jung-uk Kim wrote: > > On Wednesday 17 November 2010 05:04 pm, Alexandre "Sunny" > > Kovalenko > > > > wrote: > > > On Wed, 2010-11-17 at 20:43 +0200, Andriy Gapon wrote: > > > > I want to do MFC of ACPICA imports to stable/8 before 8.2 > > > > release. This would obviously include commits that fix > > > > mismerges or remove obsolete code. Plus some other small > > > > enhancements/fixes in our ACPI code. > > > > > > > > This is what I currently have: > > > > svn status: http://people.freebsd.org/~avg/acpi-stable-8.txt > > > > svn diff: http://people.freebsd.org/~avg/acpi-stable-8.diff > > > > svn log ...: http://people.freebsd.org/~avg/acpi-stable-8.log > > > > > > > > Please note that the svn diff above can not be used with > > > > patch(1), because of some svn peculiarities related to files > > > > in vendor area. > > > > Here's a plain diff that should be more useful: > > > > http://people.freebsd.org/~avg/acpi-stable-8.plain.diff > > > > > > > > I will appreciate any testing and reviews. > > > > Mobile users who track stable/8 would be the primary > > > > candidates, I guess :-) Thanks a lot! > > > > > > On my system: > > > > > > FreeBSD RabbitsDen.RabbitsLawn.verizon.net 8.1-STABLE FreeBSD > > > 8.1-STABLE #0 r215086: Wed Nov 10 11:07:30 EST 2010 > > > root@RabbitsDen.RabbitsLawn.verizon.net:/usr/obj/usr/src/sys/TP > > >X6 0 i386 > > > > > > running on ThinkPad X60, I get proliferation of > > > > > > "acpi_tz0: error fetching current temperature -- AE_NOT_FOUND" > > > > > > messages after applying the patch. > > > > > > Verbose dmesg from "before" and "after" cold boots are > > > available at: > > > > > > http://members.verizon.net/~akovalenko/ACPI/dmesg.before.bz2 > > > http://members.verizon.net/~akovalenko/ACPI/dmesg.after.bz2 > > > > > > respectively. > > > > > > Now, on this machine tz0 is not a real thermal zone but some > > > kind of implement to initiate system shutdown for some case I > > > never had enough time or inclination to track through ASL. Real > > > thermal zone is tz1. > > > > > > hw.acpi.thermal.min_runtime: 0 > > > hw.acpi.thermal.polling_rate: 10 > > > hw.acpi.thermal.user_override: 1 > > > hw.acpi.thermal.tz0.temperature: 45.0C > > > hw.acpi.thermal.tz0.active: -1 > > > hw.acpi.thermal.tz0.passive_cooling: 0 > > > hw.acpi.thermal.tz0.thermal_flags: 0 > > > hw.acpi.thermal.tz0._PSV: -1 > > > hw.acpi.thermal.tz0._HOT: -1 > > > hw.acpi.thermal.tz0._CRT: 127.0C > > > hw.acpi.thermal.tz0._ACx: -1 -1 -1 -1 -1 -1 -1 -1 -1 -1 > > > hw.acpi.thermal.tz0._TC1: -1 > > > hw.acpi.thermal.tz0._TC2: -1 > > > hw.acpi.thermal.tz0._TSP: -1 > > > hw.acpi.thermal.tz1.temperature: 43.0C > > > hw.acpi.thermal.tz1.active: -1 > > > hw.acpi.thermal.tz1.passive_cooling: 1 > > > hw.acpi.thermal.tz1.thermal_flags: 0 > > > hw.acpi.thermal.tz1._PSV: 75.0C > > > hw.acpi.thermal.tz1._HOT: -1 > > > hw.acpi.thermal.tz1._CRT: 97.0C > > > hw.acpi.thermal.tz1._ACx: -1 -1 -1 -1 -1 -1 -1 -1 -1 -1 > > > hw.acpi.thermal.tz1._TC1: 5 > > > hw.acpi.thermal.tz1._TC2: 4 > > > hw.acpi.thermal.tz1._TSP: 600 > > > > > > If I can provide any additional information or test any > > > patches, please, let me know. > > > > Ouch... Can you please try the attached patch? > > Please ignore this patch. I need little bit more thinking. I think I know what's going on. Andriy, it seems this change is missing from your patchset (maybe more): http://svn.freebsd.org/viewvc/base/head/sys/contrib/dev/acpica/utilities/utglobal.c?r1=210976&r2=213806 Jung-uk Kim