From owner-freebsd-ppc@FreeBSD.ORG Sun Sep 15 15:50:01 2013 Return-Path: Delivered-To: freebsd-ppc@smarthost.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id C05B3D2F for ; Sun, 15 Sep 2013 15:50:01 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 9CE8C280A for ; Sun, 15 Sep 2013 15:50:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.7/8.14.7) with ESMTP id r8FFo1d1033402 for ; Sun, 15 Sep 2013 15:50:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.7/8.14.7/Submit) id r8FFo1I5033401; Sun, 15 Sep 2013 15:50:01 GMT (envelope-from gnats) Date: Sun, 15 Sep 2013 15:50:01 GMT Message-Id: <201309151550.r8FFo1I5033401@freefall.freebsd.org> To: freebsd-ppc@FreeBSD.org Cc: From: Julio Merino Subject: Re: powerpc/180593: PowerMac G5 shuts down when building pcre X-BeenThere: freebsd-ppc@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: Julio Merino List-Id: Porting FreeBSD to the PowerPC List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 15 Sep 2013 15:50:01 -0000 The following reply was made to PR powerpc/180593; it has been noted by GNATS. From: Julio Merino To: bug-followup@freebsd.org Cc: nwhitehorn@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--