powerpc/180593: PowerMac G5 shuts down when building pcre

[Julio Merino]

The following reply was made to PR powerpc/180593; it has been noted by GNATS.

From: Julio Merino <julio at meroh.net>
To: bug-followup at freebsd.org
Cc: nwhitehorn at freebsd.org
Subject: Re: powerpc/180593: PowerMac G5 shuts down when building pcre
Date: Sun, 15 Sep 2013 11:45:21 -0400

 --089e014942b4e38f9604e66dfa61
 Content-Type: text/plain; charset=ISO-8859-1
 
 It seems to me that the powermac_thermal driver should cope with
 possibly-faulty sensors (or just by bad readings from them) by not
 trusting a single reading to perform such a drastic action as shutting
 the machine down.
 
 The attached patch makes the driver consider several readings in a row
 before shutting off.
 
 With this patch, building pcre in the machine I have results in the
 following log:
 
 WARNING: Current temperature (U3 HEATSINK: 84.3 C) exceeds critical
 temperature (80.0 C); count=1
 WARNING: Current temperature (U3 HEATSINK: 84.3 C) exceeds critical
 temperature (80.0 C); count=2
 WARNING: Current temperature (U3 HEATSINK: 121.5 C) exceeds critical
 temperature (80.0 C); count=1
 WARNING: Current temperature (U3 HEATSINK: 121.5 C) exceeds critical
 temperature (80.0 C); count=2
 WARNING: Current temperature (U3 HEATSINK: 82.0 C) exceeds critical
 temperature (80.0 C); count=1
 WARNING: Current temperature (U3 HEATSINK: 82.0 C) exceeds critical
 temperature (80.0 C); count=2
 WARNING: Current temperature (U3 HEATSINK: 91.8 C) exceeds critical
 temperature (80.0 C); count=1
 WARNING: Current temperature (U3 HEATSINK: 91.8 C) exceeds critical
 temperature (80.0 C); count=2
 WARNING: Current temperature (U3 HEATSINK: 91.8 C) exceeds critical
 temperature (80.0 C); count=3
 
 Note the big jumps from previously-good temperatures to supposedly-bad
 temperatures (80C to 121.5C) and how quickly (2-3 readings with a
 period of hz) they go down. I don't know if this is caused by a bad
 sensor or just by bad individual readings.
 
 -- 
 Julio Merino / @jmmv
 
 --089e014942b4e38f9604e66dfa61
 Content-Type: application/octet-stream; name="powermac_thermal.diff"
 Content-Disposition: attachment; filename="powermac_thermal.diff"
 Content-Transfer-Encoding: base64
 X-Attachment-Id: f_hlmfe7ap0
 
 SW5kZXg6IHBvd2VybWFjX3RoZXJtYWwuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBwb3dlcm1hY190aGVybWFs
 LmMJKHJldmlzaW9uIDI1NTQ4OCkKKysrIHBvd2VybWFjX3RoZXJtYWwuYwkod29ya2luZyBjb3B5
 KQpAQCAtNjgsNiArNjgsOCBAQAogc3RydWN0IHBtYWNfc2Vuc19sZSB7CiAJc3RydWN0IHBtYWNf
 dGhlcm0JCSpzZW5zb3I7CiAJaW50CQkJCWxhc3RfdmFsOworI2RlZmluZSBNQVhfQ1JJVElDQUxf
 Q09VTlQgNgorCWludAkJCQljcml0aWNhbF9jb3VudDsKIAlTTElTVF9FTlRSWShwbWFjX3NlbnNf
 bGUpCWVudHJpZXM7CiB9Owogc3RhdGljIFNMSVNUX0hFQUQocG1hY19mYW5zLCBwbWFjX2Zhbl9s
 ZSkgZmFucyA9IFNMSVNUX0hFQURfSU5JVElBTElaRVIoZmFucyk7CkBAIC0xMDYsMTQgKzEwOCwy
 NyBAQAogCQkJc2Vuc29yLT5sYXN0X3ZhbCA9IHRlbXA7CiAKIAkJaWYgKHNlbnNvci0+bGFzdF92
 YWwgPiBzZW5zb3ItPnNlbnNvci0+bWF4X3RlbXApIHsKKwkJCXNlbnNvci0+Y3JpdGljYWxfY291
 bnQrKzsKIAkJCXByaW50ZigiV0FSTklORzogQ3VycmVudCB0ZW1wZXJhdHVyZSAoJXM6ICVkLiVk
 IEMpICIKLQkJCSAgICAiZXhjZWVkcyBjcml0aWNhbCB0ZW1wZXJhdHVyZSAoJWQuJWQgQykhICIK
 LQkJCSAgICAiU2h1dHRpbmcgZG93biFcbiIsIHNlbnNvci0+c2Vuc29yLT5uYW1lLAotCQkJICAg
 ICAgIChzZW5zb3ItPmxhc3RfdmFsIC0gWkVST19DX1RPX0spIC8gMTAsCi0JCQkgICAgICAgKHNl
 bnNvci0+bGFzdF92YWwgLSBaRVJPX0NfVE9fSykgJSAxMCwKLQkJCSAgICAgICAoc2Vuc29yLT5z
 ZW5zb3ItPm1heF90ZW1wIC0gWkVST19DX1RPX0spIC8gMTAsCi0JCQkgICAgICAgKHNlbnNvci0+
 c2Vuc29yLT5tYXhfdGVtcCAtIFpFUk9fQ19UT19LKSAlIDEwKTsKLQkJCXNodXRkb3duX25pY2Uo
 UkJfUE9XRVJPRkYpOworCQkJICAgICJleGNlZWRzIGNyaXRpY2FsIHRlbXBlcmF0dXJlICglZC4l
 ZCBDKTsgIgorCQkJICAgICJjb3VudD0lZFxuIiwKKwkJCSAgICBzZW5zb3ItPnNlbnNvci0+bmFt
 ZSwKKwkJCSAgICAoc2Vuc29yLT5sYXN0X3ZhbCAtIFpFUk9fQ19UT19LKSAvIDEwLAorCQkJICAg
 IChzZW5zb3ItPmxhc3RfdmFsIC0gWkVST19DX1RPX0spICUgMTAsCisJCQkgICAgKHNlbnNvci0+
 c2Vuc29yLT5tYXhfdGVtcCAtIFpFUk9fQ19UT19LKSAvIDEwLAorCQkJICAgIChzZW5zb3ItPnNl
 bnNvci0+bWF4X3RlbXAgLSBaRVJPX0NfVE9fSykgJSAxMCwKKwkJCSAgICBzZW5zb3ItPmNyaXRp
 Y2FsX2NvdW50KTsKKwkJCWlmIChzZW5zb3ItPmNyaXRpY2FsX2NvdW50ID49IE1BWF9DUklUSUNB
 TF9DT1VOVCkgeworCQkJCXByaW50ZigiV0FSTklORzogJXMgdGVtcGVyYXR1cmUgZXhjZWVkZWQg
 IgorCQkJCSAgICAiY3JpdGljYWwgdGVtcGVyYXR1cmUgJWQgdGltZXMgaW4gYSByb3c7ICIKKwkJ
 CQkgICAgInNodXR0aW5nIGRvd24hXG4iLAorCQkJCSAgICBzZW5zb3ItPnNlbnNvci0+bmFtZSwK
 KwkJCQkgICAgc2Vuc29yLT5jcml0aWNhbF9jb3VudCk7CisJCQkJc2h1dGRvd25fbmljZShSQl9Q
 T1dFUk9GRik7CisJCQl9CisJCX0gZWxzZSB7CisJCQlpZiAoc2Vuc29yLT5jcml0aWNhbF9jb3Vu
 dCA+IDApCisJCQkJc2Vuc29yLT5jcml0aWNhbF9jb3VudC0tOwogCQl9CiAJfQogCkBAIC0xNzcs
 NiArMTkyLDggQEAKIAlsaXN0X2VudHJ5ID0gbWFsbG9jKHNpemVvZihzdHJ1Y3QgcG1hY19zZW5z
 X2xlKSwgTV9QTUFDVEhFUk0sCiAJICAgIE1fWkVSTyB8IE1fV0FJVE9LKTsKIAlsaXN0X2VudHJ5
 LT5zZW5zb3IgPSBzZW5zb3I7CisJbGlzdF9lbnRyeS0+bGFzdF92YWwgPSAwOworCWxpc3RfZW50
 cnktPmNyaXRpY2FsX2NvdW50ID0gMDsKIAogCVNMSVNUX0lOU0VSVF9IRUFEKCZzZW5zb3JzLCBs
 aXN0X2VudHJ5LCBlbnRyaWVzKTsKIH0K
 --089e014942b4e38f9604e66dfa61--