From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Sun, 04 Jul 2010 21:26:45 +0000 Subject: Re: [lm-sensors] fan control based on hard-drive temperatures Message-Id: <20100704212645.GA23336@ericsson.com> List-Id: References: <4C3099F1.5030703@powercraft.nl> In-Reply-To: <4C3099F1.5030703@powercraft.nl> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: lm-sensors@vger.kernel.org Hi Jelle, On Sun, Jul 04, 2010 at 04:47:05PM -0400, Jelle de Jong wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 >=20 > Hi Guenter, >=20 > On 04-07-10 17:51, Guenter Roeck wrote: > > It might be possible to trick fancontrol into using temperature output > > from a regular file, though I am not sure. Others may know better. > >=20 > > If that works, you could write a little script to run in the background= =20 > > which keeps reading temperature values from the hard drives and writes > > the average (or maybe better max ?) into that file, where fancontrol > > would pick it up from there. >=20 > It seems to work! I created and tested a little bash program that > integrates with fancontrol. Would be great if somebody reviews the > attachment. > Excellent! This looks like it might be quite useful for others as well. Wonder if it would make sense to integrate it info fancontrol and/or pwmcon= fig. Couple of minor comments inline. =20 > Thanks in advance, >=20 > With kind regards, >=20 > Jelle de Jong > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.10 (GNU/Linux) > Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ >=20 > iJwEAQECAAYFAkww80YACgkQ1WclBW9j5HmDtwQAnK0CZ61PzjkErdCqyBlUTrS3 > QhGNRlZ72WXDpmZxlDmHUYH1bZbv4c+OXETSpXxfJI5xNMqX8u8r+QXfFP/Tr4vY > VHJO1uDH/agG/uFrbAyr0VzSyF95mXSlJwbU70+x43gqqfyH/9GfAR6EwfKJBoTH > pH3gDbjHncxtqRO3vj8> =3DpxSU > -----END PGP SIGNATURE----- > #!/bin/bash >=20 > # Company: PowerCraft Technology > # Author: Copyright Jelle de Jong > # Note: Please send me an email if you enhanced the document > # Date: 2010-07-04 > # License: CC-BY-SA >=20 > # This document is free documentation; you can redistribute it and/or That isn't really a documentation, but an installation shell script. Not that I care much, it is just odd to have such a script with an ending of .txt. Is that common nowadays or just your personal touch ? > # modify it under the terms of the Creative Commons Attribution Share > # Alike as published by the Creative Commons Foundation; either version > # 3.0 of the License, or (at your option) any later version. > # > # This document is distributed in the hope that it will be useful, > # but WITHOUT ANY WARRANTY; without even the implied warranty of > # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > # Creative Commons BY-SA License for more details. > # > # http://creativecommons.org/licenses/by-sa/ >=20 > #----------------------------------------------------------------------- >=20 > apt-get install bc hddtemp file fancontrol >=20 > #----------------------------------------------------------------------- >=20 > echo '#!/bin/bash -e >=20 > # Company: PowerCraft Technology > # Author: Copyright Jelle de Jong > # Note: Please send me an email if you enhanced the script > # Version: 0.0.2 > # Date: 2010-07-04 >=20 > # This program is free software; you can redistribute it and/or modify > # it under the terms of the GNU General Public License as published by > # the Free Software Foundation; either version 3 of the License, or > # (at your option) any later version. > # > # This program is distributed in the hope that it will be useful, > # but WITHOUT ANY WARRANTY; without even the implied warranty of > # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > # GNU General Public License for more details. > # > # You should have received a copy of the GNU General Public License > # along with this program; if not, write to the Free Software > # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, > # MA 02110-1301, USA. >=20 > PATH=3D/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/bin/ >=20 > LC_ALL=3DC >=20 > # set -x > # exec 2>>/var/log/pct-fancontrol-hddtemp.log > # date=3D$(date) >=20 > DISKS=3D"[abcdefghijklmnop]" >=20 > # located in /var/lib/ so it will be there for fancontrol during boot > LOCATION=3D/var/lib/pct-fancontrol-hddtemp > FILE_MAX=3D"$LOCATION"/pct-fancontrol-hddtemp-maximum > FILE_AVG=3D"$LOCATION"/pct-fancontrol-hddtemp-average >=20 > DEGUG=3D0 >=20 > [[ -e $LOCATION ]] || mkdir "$LOCATION" >=20 > while true > do > temperature=3D0 > summation=3D0 > maximum=3D0 > average=3D0 > count=3D0 > for value in $(hddtemp -n /dev/sd$DISKS 2>/dev/null) > do > temperature=3D"${value//[!0-9]}" > ((summation+=3D$temperature)) > ((count+=3D1)) > if [ $temperature -gt $maximum ]; then > maximum=3D$temperature > fi > average=3D$(echo "scale=3D1; $summation / $count" | bc) "average=3D$(($summation / $count))" would probably be sufficient here. For a "final" code version, it would be sufficient to calculate the average after the loop. But then you would have to check for $count =3D= 0, so I am not sure if it would be worth it. > if [ $DEGUG =3D 1 ]; then > echo $summation > echo $maximum > echo $average > echo $count > echo "-----" > fi > done > # fancontrol output format > echo "($maximum * 1000)/1" | bc > "$FILE_MAX" > echo "($average * 1000)/1" | bc > "$FILE_AVG" "echo $(($average * 1000))" seems to work as well and would be a bit simple= r. What is the "/1" for, anyway ? bc doesn't seem to need it. Also, you are loosing precision because average is calculated (and rounded = down) first. You could avoid that with "echo $(($average * 1000 / $count))" if you check for the $count=3D0 case. But then that doesn't really make much of a differ= ence, so I am not sure if it would be worth it either. > sleep 45 > done >=20 > exit' | tee /usr/local/bin/pct-fancontrol-hddtemp >=20 > chown root:root /usr/local/bin/pct-fancontrol-hddtemp > chmod 755 /usr/local/bin/pct-fancontrol-hddtemp > ls -hal /usr/local/bin/pct-fancontrol-hddtemp > cat /usr/local/bin/pct-fancontrol-hddtemp >=20 > #----------------------------------------------------------------------- >=20 > /usr/local/bin/pct-fancontrol-hddtemp & > cat /var/lib/pct-fancontrol-hddtemp/pct-fancontrol-hddtemp-maximum > cat /var/lib/pct-fancontrol-hddtemp/pct-fancontrol-hddtemp-average > cat /var/lib/pct-fancontrol-hddtemp/* >=20 > watch cat /var/lib/pct-fancontrol-hddtemp/* >=20 > #----------------------------------------------------------------------- >=20 > vim /etc/fancontrol > INTERVAL=10 > DEVPATH=3Dhwmon0=DEvices/platform/it87.552 > DEVNAME=3Dhwmon0=3Dit8720 > FCTEMPS=3Dhwmon0/device/pwm2=3D/var/lib/pct-fancontrol-hddtemp/pct-fa= ncontrol-hddtemp-maximum hwmon0/device/pwm1=3Dhwmon0/device/temp2_input > FCFANS=3Dhwmon0/device/pwm2=3Dhwmon0/device/fan2_input hwmon0/device/= pwm1=3Dhwmon0/device/fan1_input > MINTEMP=3Dhwmon0/device/pwm2A hwmon0/device/pwm15 > MAXTEMP=3Dhwmon0/device/pwm2Q hwmon0/device/pwm1U > MINSTART=3Dhwmon0/device/pwm2X hwmon0/device/pwm1p > MINSTOP=3Dhwmon0/device/pwm2X hwmon0/device/pwm1f >=20 > /etc/init.d/fancontrol restart >=20 > #----------------------------------------------------------------------- >=20 > /etc/init.d/fancontrol stop >=20 > whereis fancontrol > file /usr/sbin/fancontrol >=20 > bash -x /usr/sbin/fancontrol >=20 > #----------------------------------------------------------------------- >=20 > vim /etc/rc.local > /usr/local/bin/pct-fancontrol-hddtemp >=20 Might be better to add a startup script in /etc/init.d and/or /etc/rc*.d and make sure it runs before fancontrol is started. > cat /etc/rc.local >=20 > #----------------------------------------------------------------------- >=20 > ps aux | grep fancontrol >=20 > #----------------------------------------------------------------------- _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors