http://bugzilla.kernel.org/show_bug.cgi?id=9139
[EMAIL PROTECTED] changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |RESOLVED
Resolution| |CODE_FIX
------- Comment #8 from [EMAIL PROTECTED] 2007-12-03 23:17 -------
Re: the patch in comment #1 to disable poweroff from the kernel
critical thermal handler always... I think one could argue
this either way, but we wouldn't want to make such a change
suddenly, as some people could be relying on the old code.
I agree that "thermal.nocrt" should just disable the shutdown action
from the kernel, and should not disable the event to user-space.
If somebody wants to disable both, they can use "thermal.crt=-1"
So I think the patch in comment #5 is correct -- independent
of the debate over what the default should be.
Re: the patch in comment #6 for Kconfig to set thermal.nocrt=1.
Actually, I think it would be better to combine this with
the patch in comment #1. ie. If you're going to have a config
option for this, then have it surround the poweroff code in the kernel.
It would default to Y, which would be no change from today.
Then we can debate on the list if the default should some day change,
and if everybody goes for that, then the option and the code it
includes can some day be deleted entirely. In the mean time,
you'll are able to build the kernel you want and not have to
mess with boot parameters.
--
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell. From the desktop to the data center, Linux is going
mainstream. Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
