Re: thermal zone causing insta-reboot on bootup of 2.6.3

[Pavel Machek <pavel-+ZI9xUNit7I@public.gmane.org>]

Hi!

> The following patch is from Pavel, and maybe it is suitable for this
> case. --Luming

Probably not. In my case, sensor randomly lies. In his case, it seems
to report 255 all the time...
								Pavel

> @@ -457,7 +460,17 @@
>  		return_VALUE(-EINVAL);
>  
>  	if (tz->temperature >= tz->trips.critical.temperature) {
> +		long old_temperature = tz->temperature;
>  		ACPI_DEBUG_PRINT((ACPI_DB_WARN, "Critical trip point\n"));
> +
> +		result = acpi_thermal_get_temperature(tz);
> +		if (!result) {
> +			if (tz->temperature < (tz->trips.critical.temperature - 100)) {
> +				printk(KERN_ALERT "ACPI changed its mind about temperature, was %ld C, now %ld C", 
> +				       KELVIN_TO_CELSIUS(old_temperature), KELVIN_TO_CELSIUS(tz->temperature));
> +				return_VALUE(0);
> +			}
> +		}
>  		tz->trips.critical.flags.enabled = 1;
>  	}
>  	else if (tz->trips.critical.flags.enabled)
> 
>   
> 

-- 
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click