All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hubert Kario <hka@qbs.com.pl>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Kernel-provided labels
Date: Tue, 12 May 2009 21:19:52 +0000	[thread overview]
Message-ID: <200905122319.52876.hka@qbs.com.pl> (raw)
In-Reply-To: <20090512225508.7ab00c02@hyperion.delvare>

On Tuesday 12 May 2009 22:55:08 you wrote:
> 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.

I'm voting or this one
this could be a "verbose" output. IMO as the complexity of lm-sensors 
increases the need for such output mode will only rise

also it could be used to insert detailed descriptions of specific sensors, so, 
for example, the user could get info that the "CPU temp" is, in fact the 
"Tcase CPU temp" ─ a heat spreader temperature ;)

this would increase user friendliness greatly

>
> 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?

-- 
Hubert Kario
QBS - Quality Business Software
ul. Ksawerów 30/85
02-656 Warszawa
POLAND
tel. +48 (22) 646-61-51, 646-74-24
fax +48 (22) 646-61-50


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

  reply	other threads:[~2009-05-12 21:19 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-12 20:55 [lm-sensors] Kernel-provided labels Jean Delvare
2009-05-12 21:19 ` Hubert Kario [this message]
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=200905122319.52876.hka@qbs.com.pl \
    --to=hka@qbs.com.pl \
    --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.