From owner-freebsd-current@FreeBSD.ORG Sun Sep 23 02:49:34 2007 Return-Path: Delivered-To: freebsd-current@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C10D416A418; Sun, 23 Sep 2007 02:49:34 +0000 (UTC) (envelope-from cnst@FreeBSD.org) Received: from mojo.ru (mojo.ru [84.252.152.63]) by mx1.freebsd.org (Postfix) with ESMTP id 3292413C447; Sun, 23 Sep 2007 02:49:33 +0000 (UTC) (envelope-from cnst@FreeBSD.org) Received: from [192.168.0.16] (nc-76-4-28-21.dhcp.embarqhsd.net [76.4.28.21]) (authenticated bits=0) by mojo.ru (8.12.11.20060308/8.12.10) with ESMTP id l8N2nY0v016657 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sun, 23 Sep 2007 06:49:41 +0400 Message-ID: <46F5D427.9020504@FreeBSD.org> Date: Sat, 22 Sep 2007 22:49:11 -0400 From: "Constantine A. Murenin" Organization: Google Summer of Code 2007 Student @ The FreeBSD Project User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.7.5) Gecko/20041217 X-Accept-Language: en-gb, en-gb-oed, en, en-us, ru, ru-ru, ru-su MIME-Version: 1.0 To: "Constantine A. Murenin" References: <200709132302.l8DN2Tv5076033@repoman.freebsd.org> <46E9FC0C.70607@FreeBSD.org> In-Reply-To: <46E9FC0C.70607@FreeBSD.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Cc: freebsd-hardware@FreeBSD.org, Alexander Leidinger , Shteryana Shopova , freebsd-current@FreeBSD.org, "Constantine A. Murenin" Subject: Re: GSoC2007: cnst-sensors.2007-09-13.patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 23 Sep 2007 02:49:34 -0000 Dear freebsd-{current,hardware}@, Let me a give a few guiding comments for potential testers and integrators of this patch. First of all, let's start with a common pitfall: * You have to update your /boot/device.hints, and then reboot. In no other circumstance will the isa-based lm(4) or it(4) be probed. Manually updating the hints with kenv(1) after the system has already been booted has no effect on the isa modules. * Second, since this patch is not only about the framework, but about some Super I/O Hardware Monitoring drivers too, let me once again reiterate on the popularity of the chips that are supported by the lm(4) and it(4) drivers. I have four boxes here, all run FreeBSD and OpenBSD. Let's see what Super I/O chips they have: * AOpen AX4G-N (845G + Intel Pentium 4 Northwood): Winbond W83627HF-AM * ASUS Terminator C3 (VIA CLE266 + VIA C3 Samuel 2): Winbond W83627THF-A * ASUS V3-P5G965 (G965 + Intel Core 2 Duo Allendale): Winbond W83627DHG-A * PCCHIPS V21G (VIA CN700 + VIA C7 Esther): ITE IT8716F-S Out of the above, Winbond W83627* are supported by lm(4), and ITE IT8716F-S by it(4). (Obviously, the drivers support other chips, too. ;) In general, most boards from Taiwanese manufacturers have either a Winbond or ITE Tech Super I/O chips, supported by lm(4) and it(4) respectively; e.g. if you have a Gigabyte or ASUS mainboard, then the probability of you having one of these chips is quite substantial -- consider testing the patch if you are interested. Supermicro boards also feature Winbond chips quite often -- feel free to test. Intel-branded boards, on the other hand, often use an SMBus-interfaced Hardware Monitoring solution. I have none of those boards, so none of the SMBus drivers were ported to FreeBSD yet. Anyhow, I hope this information helps some potential testers and integrators. ;) Cheers, Constantine. On 13/09/2007 23:12, Constantine A. Murenin wrote: > Dear freebsd-{arch,current,hackers}@, > > On this 256th day of 2007, it is my great pleasure to announce the > completion of my GSoC2007 project on porting the sysctl hardware sensors > framework from OpenBSD to FreeBSD. > > All of the things that were planned to be ported from OpenBSD base > system to FreeBSD have now been ported. > > The userland part of the framework is entirely source-code compatible > with OpenBSD. For example, you can take OpenBSD's stock sensorsd(8), > and it'll compile and work on FreeBSD with no modifications. > > The framework is quite self-contained, so I think it is a safe bet to at > least try to get it into the tree even at this point, when the code > freeze is taking place in preparation for RELENG_7 branching. > > Therefore, I hereby request that this patch be considered for immediate > inclusion into FreeBSD's main CVS repository. > > The complete CVS patch is available from: > http://mojo.ru/us/GSoC2007.cnst-sensors.2007-09-13.patch.gz > > For backup purposes, a copy of this CVS patch is also available in my > perforce branch, although it has tainted $P4$ tags in individual files, > so use perforce as a last resort: > http://p4web.freebsd.org//depot/projects/soc2007/cnst-sensors/cnst-sensors.2007-09-13.patch > > > Exact details on how to apply and test the patch are available in my > LiveJournal, along with certain other comments: > > http://cnst.livejournal.com/38421.html#directions > > If you have an Intel Core 2 processor, or a Winbond or ITE Tech Super > I/O chip on your board, then please test and report back on how your > tests went. > > Best regards, > Constantine Aleksandrovich Murenin, > Google Summer of Code 2007 Student @ The FreeBSD Project. ;) > > > On 13/09/2007 19:02, Constantine A. Murenin wrote: > >> http://perforce.freebsd.org/chv.cgi?CH=126384 >> >> Change 126384 by cnst@dale on 2007/09/13 23:01:55 >> >> On this 256th day of 2007, it is my great pleasure to >> present a feature-complete port of the hardware sensors >> framework from OpenBSD to FreeBSD. >> >> Below is a complete `cvs diff` of cnst-sensors GSoC2007 >> project as of 2007-256. >> >> It includes the following components, listed below in >> the very same order as they are appearing in this diff: >> >> * sample configuration file for sensorsd >> * rc(8) script and glue code for sensorsd(8) >> * sysctl(3) doc fixes for CTL_HW tree >> * sysctl(3) documentation for hardware sensors >> * sysctl(8) documentation for hardware sensors >> * assorted KNF and bug-fixes for sysctl(8) >> * support for the sensor structure for sysctl(8) >> * coretemp(4) documentation >> * it(4) documentation >> * lm(4) documentation >> * rc.conf(5) documentation for starting sensorsd(8) >> * sensor_attach(9) et al documentation >> * coretemp(4) conversion to the hw.sensors framework >> * it(4) isa driver ported from OpenBSD >> * lm(4) isa driver ported from OpenBSD >> * /sys/kern/kern_sensors.c >> o sensor_attach(9) API for drivers to register ksensors >> o sensor_task_register(9) API for the update task >> o sysctl(3) glue code >> o hw.sensors shadow tree for sysctl(8) internal magic >> * assorted KNF and bug-fixes for /sys/kern/kern_sysctl.c >> * it(4) module for testing sensor_attach/detach et al >> * lm(4) module for testing sensor_attach/detach et al >> * >> * assorted bug-fixes and HW_SENSORS definition for >> * sensors display for systat(1), including all documentation >> * sensorsd(8) and all applicable documentation >> >> The userland part of the framework is entirely source-code >> compatible with OpenBSD 4.1, 4.2 and -current as of today. >> >> All sensor readings can be viewed with `sysctl hw.sensors`, >> monitored in semi-realtime with `systat -sensors` and also >> logged with `sensorsd`. Third-party tools, for example a >> plug-in for nagios, are also available. A separate patch >> for ports/sysutils/symon will be provided upon request. >> >> Submitted by: cnst@FreeBSD.org (Constantine A. Murenin) >> Obtained from: generated by sensors.cvsdiff.sh from >> //depot/projects/soc2007/cnst-sensors/ >> Sponsored by: Google Summer of Code 2007 >> >> >> Obtained from: >> http://mojo.ru/us/GSoC2007.cnst-sensors.2007-09-13.patch.gz >> >> Details at: http://cnst.livejournal.com/38421.html >> >> Affected files ... >> >> .. >> //depot/projects/soc2007/cnst-sensors/cnst-sensors.2007-09-13.patch#1 add >> >> Differences ... >> >