Date: Sat, 16 Feb 2008 16:17:32 +0900 From: Hajimu UMEMOTO <ume@freebsd.org> To: "Alexandre \"Sunny\" Kovalenko" <alex.kovalenko@verizon.net> Cc: acpi@freebsd.org, Johannes Dieterich <dieterich.joh@googlemail.com>, Ian Smith <smithi@nimnet.asn.au> Subject: Re: [RFC] Patch to enable temperature ceiling in powerd Message-ID: <ygeodahjroz.wl%ume@mahoroba.org> In-Reply-To: <1203136636.821.4.camel@RabbitsDen> References: <20080208045605.15C874500E@ptavv.es.net> <47ABF402.7030904@root.org> <1202475519.7014.7.camel@RabbitsDen> <yge1w7komar.wl%ume@mahoroba.org> <1203126071.833.19.camel@RabbitsDen> <yger6fdk328.wl%ume@mahoroba.org> <1203136636.821.4.camel@RabbitsDen>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi,
>>>>> On Fri, 15 Feb 2008 23:37:16 -0500
>>>>> "Alexandre \"Sunny\" Kovalenko" <alex.kovalenko@verizon.net> said:
alex> Thank you, it certainly did:
alex> sunny:RabbitsDen>sysctl hw.acpi.thermal
alex> hw.acpi.thermal.min_runtime: 0
alex> hw.acpi.thermal.polling_rate: 10
alex> hw.acpi.thermal.user_override: 0
alex> hw.acpi.thermal.tz0.temperature: 65.0C
alex> hw.acpi.thermal.tz0.active: -1
alex> hw.acpi.thermal.tz0.passive_cooling: 0
alex> hw.acpi.thermal.tz0.thermal_flags: 0
alex> hw.acpi.thermal.tz0._PSV: -1
alex> hw.acpi.thermal.tz0._HOT: -1
alex> hw.acpi.thermal.tz0._CRT: 127.0C
alex> hw.acpi.thermal.tz0._ACx: -1 -1 -1 -1 -1 -1 -1 -1 -1 -1
alex> hw.acpi.thermal.tz1.temperature: 64.0C
alex> hw.acpi.thermal.tz1.active: -1
alex> ==> hw.acpi.thermal.tz1.passive_cooling: 1
alex> hw.acpi.thermal.tz1.thermal_flags: 0
alex> hw.acpi.thermal.tz1._PSV: 92.5C
alex> hw.acpi.thermal.tz1._HOT: -1
alex> hw.acpi.thermal.tz1._CRT: 97.0C
alex> hw.acpi.thermal.tz1._ACx: -1 -1 -1 -1 -1 -1 -1 -1 -1 -1
Thank you for confirmation. I've just committed it.
alex> I have one question regarding this patch: it enables passive cooling for
alex> the thermal zone, which has _PSV, _TSP, _TC1 and _TC2. Isn't it
alex> necessary to also have _PSL in the thermal zone to have passive cooling?
Since cpufreq enumerates all CPUs for us, we ignore _PSL for now.
However, it's possible in the future _PSL will reference non-CPU
devices so we may want to support it then.
Sincerely,
--
Hajimu UMEMOTO @ Internet Mutual Aid Society Yokohama, Japan
ume@mahoroba.org ume@{,jp.}FreeBSD.org
http://www.imasy.org/~ume/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ygeodahjroz.wl%ume>