search

vmatare / thinkfan

The minimalist fan control program
GNU General Public License v3.0
552 stars 62 forks source link

ERROR: Failed to read temperature(s) from /proc/acpi/ibm/thermal: Success #4

Closed ghost closed 8 years ago

ghost commented 8 years ago

can't launch it

WARNING: Using default fan control in /proc/acpi/ibm/fan.
Using default temperature inputs in /proc/acpi/ibm/thermal.

ERROR: Failed to read temperature(s) from /proc/acpi/ibm/thermal: Success

Backtrace:
/usr/sbin/thinkfan() [0x445235]
/usr/sbin/thinkfan(_ZN8thinkfan5ErrorC2ERKSs+0x5f) [0x4453d5]
/usr/sbin/thinkfan(_ZN8thinkfan13ExpectedErrorC1ERKSs+0x23) [0x42e615]
/usr/sbin/thinkfan(_ZN8thinkfan11SystemErrorC1ERKSs+0x23) [0x42e6a9]
/usr/sbin/thinkfan(_ZNK8thinkfan14TpSensorDriver10read_tempsEv+0x1c0) [0x43a822]
/usr/sbin/thinkfan(_ZN8thinkfan3runERKNS_6ConfigE+0x12b) [0x42bf27]
/usr/sbin/thinkfan(main+0x226) [0x42dece]
/usr/lib/libc.so.6(__libc_start_main+0xf0) [0x7f057d2c9610]
/usr/sbin/thinkfan(_start+0x29) [0x42ba09]

same happens if I just run /usr/bin/thinkfan as root

also my /proc/acpi/ibm/thermal

[localhost hj]# cat /proc/acpi/ibm/thermal
temperatures:   52 0 52 0 0 0 35 0
vmatare commented 8 years ago

897dde3 should fix this, but I can't test it since my system doesn't have /proc/acpi/ibm/thermal any more. Please let me know wether it works with the updated master.

ghost commented 8 years ago

yes that works but now it hangs(I guess it doesn't give a proper status back to the shell)

eg after a start

[hj@localhost ~]$ rc-status
Runlevel: default
 dbus                                                                                     [  started  ]
 bluetooth                                                                                [  started  ]
 thinkfan                                                                                 [ starting  ]
 syslog-ng                                                                                [  started  ]
 NetworkManager                                                                           [  started  ]
 cupsd                                                                                    [  started  ]
 consolekit                                                                               [  started  ]
 netmount                                                                                 [  started  ]
 xdm                                                                                      [  started  ]
 acpid                                                                                    [  started  ]
 cronie                                                                                   [  started  ]
 alsasound                                                                                [  started  ]

if I add an '&' to the init file it will also be started eg.

-start-stop-daemon --start --exec /usr/sbin/thinkfan -- -q -s5 -c /etc/thinkfan.conf
+start-stop-daemon --start --exec /usr/sbin/thinkfan -- -q -s5 -c /etc/thinkfan.conf &
vmatare commented 8 years ago

Oh yeah that's right. The old versions used to daemonize by default, but the newer version doesn't daemonize at all. That's why you have to put the & at the end, to make the shell daemonize it. But thanks for reporting this issue. I'll see about implementing daemon mode again.