From: Zhang Rui <rui.zhang@intel.com>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: linux-acpi@vger.kernel.org, lm-sensors@lm-sensors.org,
"Mark M. Hoffman" <mhoffman@lightlink.com>,
Jean Delvare <khali@linux-fr.org>, Len Brown <lenb@kernel.org>,
"Thomas, Sujith" <sujith.thomas@intel.com>
Subject: Re: The new thermal management sysfs class, and hwmon
Date: Sun, 03 Feb 2008 17:31:12 +0800 [thread overview]
Message-ID: <1202031072.3042.54.camel@acpi-sony.sh.intel.com> (raw)
In-Reply-To: <20080203022622.GA995@khazad-dum.debian.net>
Hi, henrique,
On Sun, 2008-02-03 at 10:26 +0800, Henrique de Moraes Holschuh wrote:
> The new thermal managemeny sysfs class that was just merged into
> acpi-test,
> especially when dealing with temperature measurement and fan control,
> has a
> lot of common ground with the hwmon interface.
Yes, that's true.
The idea is from Len's ols paper in 2007, "Cool Hand Linux - Handheld
Thermal Extensions". The new thermal management sysfs class can be used
by handheld devices which use device throttling for thermal control and
may have no fan at all.
Please refer to
http://www.kernel.org/pub/linux/kernel/people/lenb/acpi/doc/OLS2007-cool-web/
> However, the ACPI model for thermal cooling devices (fans, etc) and
> thermal
> zones (temp sensors) as currently implemented in the sysfs class
> appears at
> first glance to be a lot more simplified than what is available
> through the
> hwmon sysfs ABI (for a lack of a better term to describe the sysfs
> attribute
> specifications).
>
> And the two sysfs ABIs are incompatible. The ACPI one uses
> low-precision
> units, (temperature in 10^0 degrees Celcius), while the hwmon ABI uses
> medium precision units (10^-3 degrees Celcius), for example. There is
> also
> no tachometer feedback for fans, etc.
Yes, currently ACPI is the only user of the thermal sysfs I/F. We can
add new sys I/F if someone really need it.
>
> IMHO, we can probably do better than two incompatible sysfs ABIs for
> what
> ammounts to the same functionality for many userspace applications
> (i.e.
> thermal monitor apps, and fan control and monitoring apps). And it
> would be
> really neat if the new thermal management stuff could just plug into
> the
> already available temperature sensors and fan controllers that follow
> the
> hwmon sysfs ABI.
Yes, we do want to use the hwmon sysfs ABI at the beginning.
But there are several gaps that making us turn to a new thermal sysfs
class. And the biggest one is that hwmon does NOT support ACPI passive
cooling devices like processor, video, etc. intel menlow platform even
has a ACPI memory controller device which can be throttled by changing
the bandwidth. These are quite different from the hwmon's fan-based
thermal management, right?
thanks,
rui
next prev parent reply other threads:[~2008-02-03 9:32 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-03 2:26 The new thermal management sysfs class, and hwmon Henrique de Moraes Holschuh
2008-02-03 9:31 ` Zhang Rui [this message]
2008-02-03 16:44 ` Mark M. Hoffman
2008-02-03 17:50 ` Henrique de Moraes Holschuh
2008-02-05 10:14 ` Thomas, Sujith
2008-02-05 13:57 ` Mark M. Hoffman
2008-02-05 14:55 ` [lm-sensors] " Henrique de Moraes Holschuh
2008-02-07 7:01 ` Len Brown
2008-02-07 12:30 ` Henrique de Moraes Holschuh
2008-02-06 5:23 ` Thomas, Sujith
2008-02-13 15:10 ` Thomas Renninger
2008-02-14 14:08 ` Matthew Garrett
2008-02-15 11:04 ` Thomas, Sujith
2008-02-15 11:56 ` Matthew Garrett
2008-02-15 12:19 ` [lm-sensors] " Henrique de Moraes Holschuh
2008-02-22 5:47 ` zhangrui
2008-02-22 8:00 ` [lm-sensors] " Hans de Goede
2008-02-22 8:33 ` Zhang, Rui
2008-02-22 10:54 ` Hans de Goede
2008-02-23 7:43 ` Len Brown
2008-02-23 8:29 ` Jean Delvare
2008-02-24 22:52 ` Zhang, Rui
2008-02-25 8:53 ` Hans de Goede
2008-02-23 20:39 ` [lm-sensors] " Henrique de Moraes Holschuh
2008-02-07 6:41 ` Len Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1202031072.3042.54.camel@acpi-sony.sh.intel.com \
--to=rui.zhang@intel.com \
--cc=hmh@hmh.eng.br \
--cc=khali@linux-fr.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=mhoffman@lightlink.com \
--cc=sujith.thomas@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox