Date: Wed, 17 Nov 2010 17:41:55 -0500 From: Jung-uk Kim <jkim@FreeBSD.org> To: freebsd-stable@FreeBSD.org Cc: freebsd-acpi@freebsd.org, "Alexandre \"Sunny\" Kovalenko" <gaijin.k@ovi.com>, Andriy Gapon <avg@freebsd.org> Subject: Re: acpi(ca) mfc for 8.2 Message-ID: <201011171741.57389.jkim@FreeBSD.org> In-Reply-To: <1290031441.1899.8.camel@RabbitsDen.RabbitsLawn.verizon.net> References: <4CE42244.8060309@freebsd.org> <1290031441.1899.8.camel@RabbitsDen.RabbitsLawn.verizon.net>
next in thread | previous in thread | raw e-mail | index | archive | help
--Boundary-00=_1oF5MOfv2lvbHEN Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline 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/TPX60 > 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? Thanks, Jung-uk Kim --Boundary-00=_1oF5MOfv2lvbHEN Content-Type: text/plain; charset="iso-8859-1"; name="acpi_thermal.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="acpi_thermal.diff" Index: sys/dev/acpica/acpi_thermal.c =================================================================== --- sys/dev/acpica/acpi_thermal.c (revision 215429) +++ sys/dev/acpica/acpi_thermal.c (working copy) @@ -181,14 +181,16 @@ static int acpi_tz_cooling_unit = -1; static int acpi_tz_probe(device_t dev) { - int result; + char *name; - if (acpi_get_type(dev) == ACPI_TYPE_THERMAL && !acpi_disabled("thermal")) { - device_set_desc(dev, "Thermal Zone"); - result = -10; - } else - result = ENXIO; - return (result); + if (!acpi_disabled("thermal")) { + name = acpi_name(acpi_get_handle(dev)); + if (name != NULL && strcmp(name, "\\_TZ_") == 0) { + device_set_desc(dev, "Thermal Zone"); + return (-10); + } + } + return (ENXIO); } static int --Boundary-00=_1oF5MOfv2lvbHEN--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201011171741.57389.jkim>