From: Kay Sievers <kay.sievers@vrfy.org>
To: Joey Lee <jlee@novell.com>
Cc: trenn@suse.de, rui.zhang@intel.com, hmacht@suse.de,
rwysocki@suse.de, linux-acpi@vger.kernel.org
Subject: Re: [RFC] Shutdown on thermal HOT event if no userspace tool registered being able to do S4
Date: Thu, 01 Apr 2010 15:40:26 +0200 [thread overview]
Message-ID: <1270129226.1767.12.camel@yio.site> (raw)
In-Reply-To: <4BB537F10200002300015C07@novprvlin0050.provo.novell.com>
On Thu, 2010-04-01 at 07:18 -0600, Joey Lee wrote:
> 於 二,2010-03-30 於 17:31 +0100,Thomas Renninger 提到:
> > On Tuesday 30 March 2010 18:12:40 Joey Lee wrote:
> > > 於 二,2010-03-30 於 10:23 +0100,Thomas Renninger 提到:
> > > > What userspace tools are candidates to implement this if this
> > > > makes sense?
> > >
> > > HAL + pm-util or DeviceKit-power might the candidates. But, if there
> > > have no those component in userland, what will acpi thermal module do?
> > > Direct shutdown?
> > > And, how can kernel space know userland can do this job well? Might wait
> > > 10 seconds if system doesn't S4 or shutdown?
> > Yep, my idea is to e.g. provide:
> > /sys/class/thermal/S4_capable
> > By default you get:
> > cat /sys/class/thermal/S4_capable
> > 0
> > and the thermal driver will call the same shutdown method
> > if the hot thermal tp is reached as if the critical is reached.
> >
> > If pm-utils does:
> > echo 1 >/sys/class/thermal/S4_capable
> > it has to make sure it picks up acpi thermal hot event and initiates
> > S4. If S4 fails for whatever reasons (no swap,
> > stroking some kernel drivers fails, unmounting file systems fails, ...),
> > it has to make sure to force a system shutdown (just call /sbin/poweroff
> > the same way the kernel does currently with a critical thermal event?).
> > Emergency shutdown is then out of kernels hand (as it is currently for HOT
> > anyway which is dangerous).
What would be the event source? pm-utils is not a process, it can't pick
stuff up, I EXPECT, unless the kernel would fork a certain binary.
> > Hmm, this is acpi specific, possibly it should be a new dir here:
> > /sys/firmware/acpi/thermal/S4_capable
> > or something else?
> >
>
> Your idea looks good to me, but, the DeviceKit-power will rename to
> upower,
Yes, it did already a while ago.
> We are discussing about how to handle the situtation if BIOS ACPI didn't
> return the _CRT value, and even have _HOT value, if S4 fail, what can we
> do on kernel space and userland.
>
> _CRT value is the shutdown critical temperature
> _HOT value is the critical temperature for sleep (entry to S4)
>
> Per ACPI spec:
> The platform vendor should define _HOT to be far enough below _CRT so as
> to allow OSPM enough time to transition the system into the S4 sleeping
> state.
>
> Currently, there have a bit platform provider (OEM/ODM) only return the
> _HOT, but didn't have _CRT value. But, if _HOT S4 fail, the system
> components might dangerous if there have no any hardware protected
> Mechanism.
>
> Thomas's idea is add a sysfs interface /sys/class/thermal/S4_capable for
> userland to tell kernel userland component (pm-util or upower?) already
> take the _HOT event and try to do S4.
>
> How do you think about this idea?
I'm not so sure, this sounds a bit like an ad-hoc interface for a pretty
specific problem. I would expect, it needs more integration than a
global sysfs flag that keeps its state even when no userland process is
taking care anymore.
The details should probably be discussed on a mailing list like
devkit-devel@lists.freedesktop.org and the people working on the
userspace integration side should comment on it.
Cheers,
Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2010-04-01 13:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-01 13:18 [RFC] Shutdown on thermal HOT event if no userspace tool registered being able to do S4 Joey Lee
2010-04-01 13:40 ` Kay Sievers [this message]
-- strict thread matches above, loose matches on Subject: below --
2010-04-12 2:06 Joey Lee
2010-03-30 16:12 Joey Lee
2010-03-30 16:31 ` Thomas Renninger
2010-03-30 9:23 Thomas Renninger
2010-03-31 0:14 ` Henrique de Moraes Holschuh
2010-04-12 2:38 ` Zhang Rui
2010-04-13 8:54 ` Thomas Renninger
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=1270129226.1767.12.camel@yio.site \
--to=kay.sievers@vrfy.org \
--cc=hmacht@suse.de \
--cc=jlee@novell.com \
--cc=linux-acpi@vger.kernel.org \
--cc=rui.zhang@intel.com \
--cc=rwysocki@suse.de \
--cc=trenn@suse.de \
/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;
as well as URLs for NNTP newsgroup(s).