- revert adapter greediness

[packages/lm_sensors.git] / fancontrol.init

diff --git a/fancontrol.init b/fancontrol.init
index 55be28d4a51295ef9742a1fb927121ca7c501475..a1822613e8843d98b137235b98884b308cf7cb2e 100644 (file)
--- a/fancontrol.init
+++ b/fancontrol.init
@@ -1,41 +1,53 @@
 #!/bin/sh
 #
-# fancontrol   fancontrol short service description
+# fancontrol   Fans speed control according to temperature service
 #
-# chkconfig:   345 <start_level> <stop_level>
+# Starts after all hotplug-like things, which could load proper drivers
+# for fans and dies last possible
+# chkconfig:   345 09 99
 #
-# description: fancontrol long service description
+# description: It is crucial to proper configure this service, or severe
+#              damadge could occur. For this, be sure, to run init first
+#              and be sure the configuration is correct.
 #
 # $Id$
 #
 # TODO:
-# - find out, if it should be dependant of sensors deamon
 # - check this, case this is my first 'advanced' init script
-# - register proper start and stop levels
-# - describe that
+# - maybe add some more stop control, like re-running fancontrol and killing
+#   it once more, if killproc fails
 #
 
 # Source function library
 . /etc/rc.d/init.d/functions
 
-# Get service config - may override defaults
-# [ -f /etc/sysconfig/fancontrol ] && . /etc/sysconfig/fancontrol
+# Get service config
+ [ -f /etc/sysconfig/fancontrol ] && . /etc/sysconfig/fancontrol
 
 start() {
-       # Fancontrol needs to check system for pwm control
-       if [ ! -f /etc/sysconfig/fancontrol ]; then
-               nls "Fancontrol not initialized."
-               nls "Try \`%s init' before start." "$0"
-               exit 6
-       fi
-       # Check if the service is already running?
-       if [ ! -f /var/lock/subsys/fancontrol ]; then
-               msg_starting fancontrol
-               daemon fancontrol
-               RETVAL=$?
-               [ $RETVAL -eq 0 ] && touch /var/lock/subsys/fancontrol
+       # sensors deamon needs to be started
+       # or at least proper modules loaded (provided by sensorsd deamon)
+       if [ -f /var/lock/subsys/sensors ]; then
+               # Check if the service is already running?
+               if [ ! -f /var/lock/subsys/fancontrol ]; then
+                       # Fancontrol needs to check system for pwm control
+                       if [ ! -f /etc/sysconfig/fancontrol ]; then
+                               nls "Fancontrol not initialized."
+                               nls "Try \`%s init' before start." "$0"
+                               exit 6
+                       fi
+
+                       msg_starting fancontrol
+                       # won't start without an output
+                       daemon --fork 'fancontrol >/dev/null'
+                       RETVAL=$?
+                       [ $RETVAL -eq 0 ] && touch /var/lock/subsys/fancontrol
+               else
+                       msg_already_running fancontrol
+               fi
        else
-               msg_already_running fancontrol
+               echo "Start sensors daemon first"
+               exit 1
        fi
 }
 
@@ -43,7 +55,10 @@ stop() {
        if [ -f /var/lock/subsys/fancontrol ]; then
                # Stop daemons.
                msg_stopping fancontrol
-               killproc fancontrol
+               # it's crucial to be sure it ended ok, otherwise we get
+               # the fans set and the computer working without any control
+               # and that could cause MASSIVE damage
+               killproc --waitforname fancontrol --waitfortime 30 fancontrol -TERM
                rm -f /var/lock/subsys/fancontrol
        else
                msg_not_running fancontrol
@@ -66,19 +81,6 @@ case "$1" in
   restart)
        restart
        ;;
-  reload)
-       if [ -f /var/lock/subsys/fancontrol ]; then
-               msg_reloading fancontrol
-               killproc fancontrol -HUP
-               RETVAL=$?
-       else
-               msg_not_running fancontrol >&2
-               RETVAL=7
-       fi
-       ;;
-  force-reload)
-       restart
-       ;;
   init)
        pwmconfig
        exit $?
@@ -88,7 +90,7 @@ case "$1" in
        RETVAL=$?
        ;;
   *)
-       msg_usage "$0 {start|stop|restart|reload|force-reload|status|init}"
+       msg_usage "$0 {start|stop|restart|status|init}"
        exit 3
 esac