From owner-cvs-src@FreeBSD.ORG Thu Aug 16 01:40:29 2007 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3C56716A419; Thu, 16 Aug 2007 01:40:29 +0000 (UTC) (envelope-from rpaulo@fnop.net) Received: from core.fnop.net (mx.fnop.net [82.102.11.82]) by mx1.freebsd.org (Postfix) with ESMTP id AE88F13C468; Thu, 16 Aug 2007 01:40:28 +0000 (UTC) (envelope-from rpaulo@fnop.net) Received: from core.fnop.net (mx.fnop.net [82.102.11.82]) by core.fnop.net (Postfix) with ESMTP id 7EC37690F4B; Thu, 16 Aug 2007 02:19:15 +0100 (WEST) Received: by core.fnop.net (Postfix, from userid 1015) id 4877B69100B; Thu, 16 Aug 2007 02:19:15 +0100 (WEST) X-Spam-Checker-Version: SpamAssassin 3.1.7 (2006-10-05) on core.fnop.net X-Spam-Level: * X-Spam-Status: No, score=1.2 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DSBL, RCVD_IN_SORBS_DUL autolearn=no version=3.1.7 Received: from epsilon.local (unknown [83.144.141.37]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by core.fnop.net (Postfix) with ESMTP id 6E871690F4B; Thu, 16 Aug 2007 02:19:14 +0100 (WEST) Message-ID: <46C3A51C.4050201@fnop.net> Date: Thu, 16 Aug 2007 02:15:08 +0100 From: Rui Paulo User-Agent: Thunderbird 2.0.0.6 (Macintosh/20070728) MIME-Version: 1.0 To: Ben Kaduk References: <200708151926.l7FJQ3Md029693@repoman.freebsd.org> <20070815204708.GA11724@tirith.brixandersen.dk> <47d0403c0708151424j746d003l7a224f3e45819a6@mail.gmail.com> In-Reply-To: <47d0403c0708151424j746d003l7a224f3e45819a6@mail.gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV using ClamSMTP 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 ... X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Aug 2007 01:40:29 -0000 Ben Kaduk wrote: > On 8/15/07, Henrik Brix Andersen 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 >>> 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: on cpu0 >> coretemp1: 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