cvs commit: src/sys/amd64/include specialreg.h src/sys/conf files.amd64 files.i386 src/sys/dev/coretemp coretemp.c src/sys/i386/include specialreg.h src/sys/modules Makefile src/sys/modules/coretemp Makefile src/sys/amd64/conf NOTES ...

[Rui Paulo]

Ben Kaduk wrote:
> On 8/15/07, Henrik Brix Andersen <henrik at brixandersen.dk> wrote:
>> On Wed, Aug 15, 2007 at 07:26:03PM +0000, Dag-Erling Smorgrav wrote:
>>> des         2007-08-15 19:26:03 UTC
>>>
>>>   FreeBSD src repository
>>>
>>>   Modified files:
>>>     sys/amd64/include    specialreg.h
>>>     sys/conf             files.amd64 files.i386
>>>     sys/i386/include     specialreg.h
>>>     sys/modules          Makefile
>>>     sys/amd64/conf       NOTES
>>>     sys/i386/conf        NOTES
>>>   Added files:
>>>     sys/dev/coretemp     coretemp.c
>>>     sys/modules/coretemp Makefile
>>>   Log:
>>>   Add a driver for the on-die digital thermal sensor found on Intel Core
>>>   and newer CPUs (including Core 2 and Core / Core 2 based Xeons).  The
>>>   driver attaches to each cpu device and creates a sysctl node in that
>>>   device's sysctl context (dev.cpu.N.temperature).  When invoked, the
>>>   handler binds to the appropriate CPU to ensure a correct reading.
>>>
>>>   Submitted by:   Rui Paulo <rpaulo at fnop.net>
>>>   Sponsored by:   Google Summer of Code 2007
>>>   Tested by:      des, marcus, Constantine A. Murenin, Ian FREISLICH
>>>   Approved by:    re (kensmith)
>>>   MFC after:      3 weeks
>> Thanks for working on this. I just tested it on my Core Duo laptop (an
>> IBM/Lenovo X60s with an L2400 @ 1.66GHz) - it doesn't seem to provide
>> useful readings:
>>
>> $ uname -a
>> FreeBSD lothlorien.brixandersen.dk 7.0-CURRENT FreeBSD 7.0-CURRENT #63: Wed Aug 15 22:13:01 CEST 2007     root at lothlorien.brixandersen.dk:/usr/obj/usr/src/sys/LOTHLORIEN  i386
>>
>> $ dmesg | grep coretemp
>> coretemp0: <CPU On-Die Thermal Sensors> on cpu0
>> coretemp1: <CPU On-Die Thermal Sensors> on cpu1
>>
>> $ sysctl dev.cpu.0.temperature
>> dev.cpu.0.temperature: -50
>>
>> $ sysctl dev.cpu.1.temperature
>> dev.cpu.1.temperature: -49
>>
> 
> The original patch reported the delta between the maximum temperature
> (allowed before auto-shutdown?) and the actual temperature.  It seems

This is a different driver than the one written by des at .

-- 
Rui Paulo