From owner-cvs-src@FreeBSD.ORG Wed Aug 15 21:24:20 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 D4DBB16A468 for ; Wed, 15 Aug 2007 21:24:20 +0000 (UTC) (envelope-from minimarmot@gmail.com) Received: from ug-out-1314.google.com (ug-out-1314.google.com [66.249.92.168]) by mx1.freebsd.org (Postfix) with ESMTP id 63F6713C46E for ; Wed, 15 Aug 2007 21:24:20 +0000 (UTC) (envelope-from minimarmot@gmail.com) Received: by ug-out-1314.google.com with SMTP id o4so236430uge for ; Wed, 15 Aug 2007 14:24:19 -0700 (PDT) DKIM-Signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=RBcUvLZZbagY/ixsBj7qv+nmG3nUGWOkY35bBuLExmgL1lzOhIfhO2ZFqJifIRNWc6wT5cPUVAnNo1Nbbq5/uQmwahujLQbCCMpM7red/MI/r0pknJ22GVDfvLX44HxYcDH6J1mxsZbIQzdGADptp8I4QLddGuhLZhiB4gwk1dg= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=mlXVwD9CFTDv3GOtsTg1JSmcZ0F//RB7UBFGNd4YAfkjQaCmZiEpbpM4bIs8LQY5kNdq+YPgXmZHy0580vCpAZPnYSNo/DVIzJn13AeoO8kQWaiX5h0vSZ7qt6EEjMHTdW1I44dt1oIGMpPqHAmqvbVRCu68WodtcciVRQK/tcE= Received: by 10.143.13.6 with SMTP id q6mr54426wfi.1187213057339; Wed, 15 Aug 2007 14:24:17 -0700 (PDT) Received: by 10.143.1.8 with HTTP; Wed, 15 Aug 2007 14:24:17 -0700 (PDT) Message-ID: <47d0403c0708151424j746d003l7a224f3e45819a6@mail.gmail.com> Date: Wed, 15 Aug 2007 17:24:17 -0400 From: "Ben Kaduk" To: src-committers@freebsd.org, cvs-src@freebsd.org, cvs-all@freebsd.org In-Reply-To: <20070815204708.GA11724@tirith.brixandersen.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <200708151926.l7FJQ3Md029693@repoman.freebsd.org> <20070815204708.GA11724@tirith.brixandersen.dk> Cc: 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: Wed, 15 Aug 2007 21:24:20 -0000 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 the sysctl node is not named as well as it could be, perhaps? -Ben Kaduk