Messages in this thread | | | Date | Wed, 26 Aug 2009 17:23:42 +0100 | From | Matthew Garrett <> | Subject | Re: [PATCH 4/6] thermal: add sanity check for the passive attribute |
| |
On Wed, Aug 26, 2009 at 06:17:23PM +0200, Frans Pop wrote: > Values below 40000 milli-celsius (limit is somewhat arbitrary) > don't make sense and can cause the system to go into a thermal > heart attack: the actual temperature will always be lower and > thus the system will be throttled down to its lowest setting.
Not keen on this - it's a pretty arbitrary cutoff, and there are some cases where someone might want this value. Policy belongs in userspace, and all that.
-- Matthew Garrett | mjg59@srcf.ucam.org
| |