Re: [lm-sensors] AMD Phenom temperature driver

[Achim Gottinger <achim@ag-web.biz>]

Rudolf Marek schrieb:
> Hello all,
>
> I think I put all parties up in CC. Thanks to Andreas, I have received 
> missing information about the TControlMax temperature. It means we can 
> get further to a new driver.
>
> I did some experimental driver which works fine (attached as k8temp.c) 
> Kryzstof Went even further and made changes to original k8temp driver.
>
> I will go for vacation soonish (middle of next week) and I'm very 
> busy, but at least we can start. I will write a short list of things 
> which needs to be discussed in the meanwhile:
>
> 1) driver name
>
> AMD proposes the change to amdtemp (from k8temp) I think we can merge 
> the drivers together, it should be still simple.
>
> Jean, please do you remember how autoloading/driver name change is 
> handled in the kernel? I have some feeling that it should be simple to 
> handle. Please confirm.
>
> The name change can be done if we know that old name will work, so we 
> dont break too much things around.
>
> 2) Maximum temperature.
>
> The new Phenom temperature is non-physical it means we must know the 
> limit so we know when we cross it.
>
> Therefore our new driver must export either temp1_max or temp1_crit. 
> The TcontrolMax is for all fam 10h CPUs 70 degrees.
>
> The Intel driver has temp1_max as maximum temperature from which all 
> fans must start cooling. The temp1_crit is when the CPU will fail.
>
> I think the thremtrip_l is asserted for much higher temperatures, I 
> think we can go for temp1_max.
>
> We can put the TcontrolMax into a variable, because for future 
> revisions we may need cpuid to put there right value.
>
> 3) Errata #319
>
> I think all CPU revisions still may have problem with the temp sensor 
> inside the CPU (not the diode interface which is analog and works fine).
>
> Please can someone test mine k8temp.c attached, and report back the 
> values you got? If we detect a CPU with the errata we should print a 
> warning to syslog.
Hi Rudolf,

I tried your attached code  with an 9950BE and the temps reported look 
equal to those everest reports under winxp 32bit with similar load 
conditions. I have a few other phenoms here and will report if i find 
one whos affected by the errata.

achim~


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors