mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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!

  reply	other threads:[~2004-08-12 23:22 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-11  8:53 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
2004-08-12 22:53     ` 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome