All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] Kernel-provided labels
Date: Tue, 12 May 2009 20:55:08 +0000	[thread overview]
Message-ID: <20090512225508.7ab00c02@hyperion.delvare> (raw)

Hi all,

Kernel-provided sensor labels can be problematic when the user tries to
apply a compute or ignore statement. In general, one can comment out
all label statements in the configuration file or run "sensors
-c /dev/null" to see the symbolic names of all inputs (in0, temp1
etc...) However, in the case where kernel-provided labels are
available, libsensors will process them nevertheless, so the user has
no way to see the symbolic name for the input.

I can think of different approaches to solve this issue. I wonder which
one other developers and users think should be implemented.

1* A new command line parameter to disable the processing of
   kernel-provided labels (say: --no-kernel-labels.)

2* A new command line parameter to disable the processing of
   kernel-provided labels and configuration files (say: --raw.)
   This is a shorter alternative to "sensors -c /dev/null
   --no-kernel-label".

3* A new sensors output format which would display all available
   symbolic names of a device, possibly with kernel-provided and
   user-provided labels in a table.

4* Allow kernel-provided labels to be used in compute statements
   instead of symbolic names.

5* Something else, speak up :)

Option 4 seems technically complex to me, and error-prone
(kernel-provided labels are not guaranteed to be unique.) Option 1 is
very specific, I tend to prefer option 2 which is more generic and can
evolve over time. Option 3 may be interesting for other reasons,
including debugging and a valuable alternative to walking the sysfs
tree.

As far as I can see, options 1, 2 and 3 all require that we extend the
libsensors API.

Opinion anyone?

-- 
Jean Delvare

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

             reply	other threads:[~2009-05-12 20:55 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-12 20:55 Jean Delvare [this message]
2009-05-12 21:19 ` [lm-sensors] Kernel-provided labels Hubert Kario
2009-07-22 12:20 ` Jean Delvare

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=20090512225508.7ab00c02@hyperion.delvare \
    --to=khali@linux-fr.org \
    --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.