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>