Date: Wed, 15 Aug 2007 17:24:17 -0400 From: "Ben Kaduk" <minimarmot@gmail.com> To: src-committers@freebsd.org, cvs-src@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: <47d0403c0708151424j746d003l7a224f3e45819a6@mail.gmail.com> In-Reply-To: <20070815204708.GA11724@tirith.brixandersen.dk> References: <200708151926.l7FJQ3Md029693@repoman.freebsd.org> <20070815204708.GA11724@tirith.brixandersen.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
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 the sysctl node is not named as well as it could be, perhaps? -Ben Kaduk
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?47d0403c0708151424j746d003l7a224f3e45819a6>