Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 22 Sep 2007 22:49:11 -0400
From:      "Constantine A. Murenin" <cnst@FreeBSD.org>
To:        "Constantine A. Murenin" <cnst@FreeBSD.org>
Cc:        freebsd-hardware@FreeBSD.org, Alexander Leidinger <Alexander@leidinger.net>, Shteryana Shopova <syrinx@FreeBSD.org>, freebsd-current@FreeBSD.org, "Constantine A. Murenin" <cnst@FreeBSD.org>
Subject:   Re: GSoC2007: cnst-sensors.2007-09-13.patch
Message-ID:  <46F5D427.9020504@FreeBSD.org>
In-Reply-To: <46E9FC0C.70607@FreeBSD.org>
References:  <200709132302.l8DN2Tv5076033@repoman.freebsd.org> <46E9FC0C.70607@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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
>>     * <sys/sensors.h>
>>     * assorted bug-fixes and HW_SENSORS definition for <sys/sysctl.h>
>>     * 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 ...
>>
> 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?46F5D427.9020504>