Date: Thu, 16 Aug 2007 02:15:08 +0100 From: Rui Paulo <rpaulo@fnop.net> To: Ben Kaduk <minimarmot@gmail.com> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: 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 ... Message-ID: <46C3A51C.4050201@fnop.net> In-Reply-To: <47d0403c0708151424j746d003l7a224f3e45819a6@mail.gmail.com> References: <200708151926.l7FJQ3Md029693@repoman.freebsd.org> <20070815204708.GA11724@tirith.brixandersen.dk> <47d0403c0708151424j746d003l7a224f3e45819a6@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Ben Kaduk wrote: > On 8/15/07, Henrik Brix Andersen <henrik@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@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@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@. -- Rui Paulo
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?46C3A51C.4050201>