[SOLVED] Re: [patch] enhance powerd(8) to handle max temperature

[Pietro Cerutti]

Pietro Cerutti wrote:
> Hajimu UMEMOTO wrote:
>> Hi,
>>
>>>>>>> On Tue, 31 Jul 2007 07:54:20 +0200
>>>>>>> Pietro Cerutti <gahr at gahr.ch> said:
>> gahr> Thanks for your help!
>>
>> I cannot see _TC1, _TC2 nor _TSP in your `acpidump -dt' output.
>> Further, there is no _PSV definition in anywhere, in the first place.
>> It seems to me that your ACPI BIOS doesn't support passive cooling at
>> all.
> 
> Yep that's what I suspected.
> 
>> gahr> hw.acpi.thermal.user_override: 1
>> gahr> hw.acpi.thermal.tz0._PSV: 80.0C
>>
>> I suspect that you set hw.acpi.thermal.tz0._PSV manually.
> 
> Yes I did..
> 
>> Of cource, it is insufficient to make passive cooling work with your ACPI BIOS,
>> and you need the definition of _TC1, _TC2 and _TSP as well.
>> If you define apropreate definition of them, passive cooling will
>> work.  But, we don't provide overriding the values of them.  Perhaps,
>> modifying your ASL (`acpidump -dt' output) to add the definitions of
>> _TC1, _TC2, _TSP and _PSV, and loading it from loader will make
>> passive cooling work.
> 
> Do you have an example? I'm really far from being familiar with the ASL
> code...

Nevermind, I finally got my passive cooling working.

Here's the diff against my acpidump -dt:

--- MSI1034.asl.orig	2007-07-31 12:37:55.000000000 +0200
+++ MSI1034.asl	2007-07-31 12:37:57.000000000 +0200
@@ -2612,6 +2612,10 @@
                         {
                             Return (KLV (0x64))
                         }
+                        Name (_TC1, 0x00)
+                        Name (_TC2, 0x0C)
+                        Name (_TSP, 0x28)
+
                     }
                 }

@@ -4425,8 +4429,11 @@
         Zero,
         Zero
     })
+
+    /*
     If (SS1)
     {
+    */
         Name (_S1, Package (0x04)
         {
             One,
@@ -4434,10 +4441,14 @@
             Zero,
             Zero
         })
+    /*
     }
+    */

+    /*
     If (SS3)
     {
+    */
         Name (_S3, Package (0x04)
         {
             0x05,
@@ -4445,10 +4456,14 @@
             Zero,
             Zero
         })
+    /*
     }
+    */

+    /*
     If (SS4)
     {
+    */
         Name (_S4, Package (0x04)
         {
             0x06,
@@ -4456,7 +4471,9 @@
             Zero,
             Zero
         })
+    /*
     }
+    */

     Name (_S5, Package (0x04)
     {

I don't know if the values of _TC1, _TC2 and TSP are meaningful, but it
seems to work for me, having a _PSV value of 65C.
Any suggestions on those values are welcome.


acpi_tz0: temperature 65.8C: decreasing clock speed from 1743 MHz to
1452 MHz
acpi_tz0: temperature 65.8C: decreasing clock speed from 1452 MHz to
1162 MHz
acpi_tz0: temperature 63.8C: increasing clock speed from 1162 MHz to
1452 MHz
acpi_tz0: temperature 64.8C: increasing clock speed from 1452 MHz to
1660 MHz
acpi_tz0: temperature 64.8C: increasing clock speed from 1660 MHz to
1743 MHz
acpi_tz0: temperature 65.8C: decreasing clock speed from 1743 MHz to
1452 MHz
acpi_tz0: temperature 64.8C: increasing clock speed from 1452 MHz to
1660 MHz
acpi_tz0: temperature 64.8C: increasing clock speed from 1660 MHz to
1743 MHz
acpi_tz0: temperature 66.8C: decreasing clock speed from 1743 MHz to
1162 MHz
acpi_tz0: temperature 64.8C: increasing clock speed from 1162 MHz to
1328 MHz
...............

This seems to keep my laptop's temperature around 65C.

Thanks again,

> 
>> Sincerely,
> 
> Thank you
> 
>> --
>> Hajimu UMEMOTO @ Internet Mutual Aid Society Yokohama, Japan
>> ume at mahoroba.org  ume@{,jp.}FreeBSD.org
>> http://www.imasy.org/~ume/
> 


-- 
Pietro Cerutti

PGP Public Key:
http://gahr.ch/pgp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: OpenPGP digital signature
Url : http://lists.freebsd.org/pipermail/freebsd-acpi/attachments/20070731/e12a6f23/signature.pgp