Re: Avoid critical temp shutdowns on specific ThinkPad T4x(p) and R50p

[Thomas Renninger <trenn-l3A5Bk7waGM@public.gmane.org>]

On Wednesday 30 July 2008 19:52:27 Henrique de Moraes Holschuh wrote:
> On Wed, 30 Jul 2008, Thomas Renninger wrote:
> > frequency (what we probably will do again soon). It has been tested that
> > specific ThinkPads do not throw a thermal event when exceeding the
> > passive trip point. Even thermal polling was not enough...
>
> Urgh.
>
> > +			DMI_MATCH(DMI_BIOS_VENDOR,"IBM"),
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad T41"),
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad T42"),
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad T41p"),
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad T42p"),
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad R50p"),
>
> Just match BIOS 1R* with VENDOR IBM.  The T40 is affected as well for sure.
> It uses the same BIOS and EC firmware, it *has* to be.  And it is missing
> on this list.
>
> For IBM ThinkPads, it is *always* better to match on BIOS version than on
> product version.
>
> See here for model-BIOS table:
> http://www.thinkwiki.org/wiki/BIOS_Upgrade_Downloads
>
> and here for DMI info table:
> http://www.thinkwiki.org/wiki/List_of_DMI_IDs
>
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad T43"),
> > +			DMI_MATCH(DMI_PRODUCT_VERSION,"ThinkPad T43p"),
I thought they tested the T43 or T43p and it threw thermal events and did not 
shut down.
I will have a look at the bug again, I have to touch this one anyway.
Or just lower passive to around 80 C for all of them as you suggested below.

Thanks for the comments..., give me some time and I repost.

       Thomas
>
> > +	if (dmi_check_system(thermal_psv_dmi_table)) {
> > +		if (tz->trips.passive.flags.valid &&
> > +		    tz->trips.passive.temperature > CELSIUS_TO_KELVIN(85)) {
> > +			printk (KERN_INFO "Adjust passive trip point from %lu"
> > +				" to %lu\n",
> > +				KELVIN_TO_CELSIUS(tz->trips.passive.temperature),
> > +				KELVIN_TO_CELSIUS(tz->trips.passive.temperature - 150));
> > +			tz->trips.passive.temperature -= 150;
> > +			acpi_thermal_set_polling(tz, 5);
> > +		}
> > +	}
>
> If this is a safe test that won't misstrigger or cause issues, maybe just
> enable it on anything with BIOS_VENDOR IBM, and PRODUCT_VERSION "ThinkPad
> [TR]*" ?  None of them have CPUs that should go over 85C, AFAIK.



-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/