From: Pavel Machek <pavel@suse.cz>
To: Len Brown <len.brown@intel.com>
Cc: Dax Kelson <dax@gurulabs.com>,
trenn@suse.de, seife@suse.de,
Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Allow userspace do something special on overtemp
Date: Fri, 13 Aug 2004 01:21:47 +0200 [thread overview]
Message-ID: <20040812232147.GH15138@elf.ucw.cz> (raw)
In-Reply-To: <1092351080.5021.198.camel@dhcppc4>
Hi!
> > hmm, yes, but it still would be nice to properly shutdown instead of
> > fail.
>
> The reality is that most of the critical temperature events
> are false positives, and for those that are not, the hardware
> will keep itself from burning even when the OS control fails.
>
> If we confuse some self-supporting kernel types, that is too bad.
> If they're supporting themselves, they should read the change logs
> for the kernels that they download. I don't think
> this is of a magnitude that it needs to wait for 2.7 to be fixed.
There's nothing to fix. It is not broken. It just does /sbin/poweroff;
that's correct.
/sbin/poweroff is there on almost all systems; that is not case with
acpid. Currently *noone* has acpid that handles critical properly,
right?
So I believe that change is bad idea. /sbin/overtemp lets user
configure it etc.
Ouch and btw I've done some torturing on one prototype (AMD). It had
thermal at 98Celsius (specs for this cpu said 95C max), and I ended my
test at 105Celsius. I do not know about TM1/TM2 etc, but in this case
hardware clearly failed to do the right thing.
I do not know why acpid should be involved in this. execing binary
seems safer to me -- acpid might have died (OOM? segfault?), and exec
does not strike me like too ugly.
Pavel
--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!
next prev parent reply other threads:[~2004-08-12 23:22 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-08-11 8:53 Allow userspace do something special on overtemp Pavel Machek
2004-08-11 9:03 ` Arjan van de Ven
2004-08-11 9:06 ` Pavel Machek
2004-08-11 14:49 ` Len Brown
2004-08-11 14:52 ` Wichert Akkerman
2004-08-11 15:28 ` Thomas Renninger
2004-08-11 21:08 ` Kronos
2004-08-11 20:26 ` Pavel Machek
2004-08-11 9:18 ` Måns Rullgård
2004-08-11 18:22 ` Olaf Hering
[not found] ` <411A239C.8060606@blue-labs.org>
2004-08-11 20:22 ` Pavel Machek
2004-08-12 0:08 ` Dax Kelson
2004-08-12 3:29 ` Len Brown
2004-08-12 7:40 ` Pavel Machek
2004-08-12 14:28 ` Len Brown
2004-08-12 20:24 ` Pavel Machek
2004-08-12 22:51 ` Len Brown
2004-08-12 23:21 ` Pavel Machek [this message]
2004-08-12 7:35 ` Pavel Machek
2004-08-12 15:19 ` Len Brown
2004-08-12 15:16 ` Randy.Dunlap
2004-08-12 15:50 ` Stefan Seyfried
2004-08-12 17:27 ` [ACPI] " Stefan Dösinger
[not found] ` <200408121927.11277.stefandoesinger-RbZlAiThDcE@public.gmane.org>
2004-08-12 22:53 ` Len Brown
2004-08-12 22:53 ` [ACPI] " Len Brown
[not found] <fa.fd8nc62.oig6ao@ifi.uio.no>
[not found] ` <fa.g1p407b.1c569pj@ifi.uio.no>
2004-08-13 4:22 ` Robert Hancock
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=20040812232147.GH15138@elf.ucw.cz \
--to=pavel@suse.cz \
--cc=dax@gurulabs.com \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=seife@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.