All of lore.kernel.org
 help / color / mirror / Atom feed
From: sergio <mailbox@sergio.spb.ru>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] fancontrol
Date: Wed, 25 Mar 2009 15:33:04 +0000	[thread overview]
Message-ID: <49CA4EB0.8000006@sergio.spb.ru> (raw)
In-Reply-To: <1122217880.3578.8.camel@localhost.localdomain>

Jean Delvare wrote:

> That being said, the script approach has the advantage that
> it is extremely easy for users to customize for their own needs. They
> don't have to download the sources and rebuild a binary. For a fan
> control script this seems to have value. Users might miss this
> flexibility if we switch to C. One advantage though is that fancontrol
> would finally honor temperature adjustments made in sensors.conf.
Another advantage, that it's more simple to include it in embedded
system. Current fancontrol doesn't work in busybox or dash.
I don't believe, that shell is good for daemons.

> A third possibility which was discussed in the thread I mentioned above
> is a helper binary, linked with libsensors, which would take care of
> translating hwmonN names to libsensors chip names and back.
IMHO this is worst way.

> A fifth possibility is to try harder to always load the hwmon drivers
> in the same order. In this respect, lm-sensors 3.1.0 should behave much
> better than previous versions did. Did you give it a try?
I have two similar machines with identical sensors. One with debian
lenny (3.0.2) other with sid (3.1.0). The sensors are: acpitz, k8temp
and it8718. On both machines acpitz and k8temp loads automagically and
it8718 from /etc/modules. If k8temp also present in /etc/modules it is
don't matter in which order this modules are written in this file.
acpitz always will hwmon0, k8temp -> hwmon1, and it8718 -> hwmon1 (if it
present in modules)
What difference should i see?

(Really, on machine with lenny it87 loads via force_load in
initramfs-tools hook, so it87 is (always?) first.)

So hwmon names jumping it's not very big problem for me.
It is never happens unexpectedly. Only when i change or upgrade something.

> Honestly, I am not sure if it makes much sense to rewrite fancontrol as
> it exists today in C. fancontrol is very limited in what it can do and
> the conditions it checks. There are other control daemons out there
> which are much more flexible, as they can change behaviors not only
> based on temperature, and they can change a lot more settings than just
> fan speeds. So, instead of rewriting fancontrol, I'd rather work on
> integrating sensors better in existing daemons.
OK. Let's leave fancontrol as simple script. But i don't know any other
fan control daemon. Can you give examples of such programs?

May be should add possibility to define real path to device 
(/sys/devices/platform/it87.552)?
Or this path isn't always identical?
(And not all hwmons has a device)

And what about udev for name stabilizing?
 > The usual solution (udev) doesn't work for us because hwmon devices
 > have no device nodes in /dev, only a sysfs interface.
Network interfaces is not present in /dev, but udev takes care about 
they names.

-- 
sergio

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

  parent reply	other threads:[~2009-03-25 15:33 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-24 17:15 [lm-sensors] Fancontrol Henry
2005-07-25 13:44 ` Jim Cromie
2005-07-26  9:45 ` Rudolf Marek
2005-07-26 18:13 ` Henry
2005-07-27  9:23 ` Rudolf Marek
2005-07-27 16:05 ` Henry
2005-07-28 10:21 ` Rudolf Marek
2005-07-28 15:18 ` Henry
2005-07-28 15:19 ` Henry
2005-07-28 15:41 ` Henry
2005-07-30  7:02 ` Henry
2009-03-25  9:04 ` [lm-sensors] fancontrol Jean Delvare
2009-03-25 15:33 ` sergio [this message]
2009-03-25 21:35 ` Jean Delvare
2009-03-27  3:03 ` sergio
2009-03-27  8:39 ` Jean Delvare
2009-03-27 16:30 ` sergio
2009-05-09 11:43 ` Jean Delvare
2010-03-29 11:46 ` Ioan Rusan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49CA4EB0.8000006@sergio.spb.ru \
    --to=mailbox@sergio.spb.ru \
    --cc=lm-sensors@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.