On Sat, 29 Sep 2007 15:39:19 +0200, Jean Delvare wrote: > So, from the above, it seems that what we both agree on is that a > separate error value for I/O errors would be needed. I'll work up a > patch implementing this. I've committed a patch: http://www.lm-sensors.org/changeset/4899 While testing it, I found that libsysfs will silently discard attributes which return errors when being read. This means that an attribute which returns an error at libsensors initialization time will be forever missing, even if it can be successfully read again later. This is no good. One more reason to get rid of libsysfs, I presume. This also revealed that libsysfs reads the value from all attributes when we do the device feature detection, even though we did not ask for it and don't need the attribute values at all at this point. This triggers a cache refresh at the device driver level (for most drivers at least). No big deal for "sensors", but it's needlessly slowing down "sensors -s". It might even have hidden driver bugs, as it means the cache is always up-to-date when "sensors -s" is run, while drivers should be able to cope with "sensors -s" when the cache is not yet populated. Oh well, we really have to move away from libsysfs. I've created a ticket in trac for that, it's scheduled for lm-sensors 3.0.1: http://www.lm-sensors.org/ticket/2262 -- Jean Delvare
