Changeset 4644

Timestamp:

07/24/07 10:20:53 (1 year ago)

Author:

khali

Message:

Let libsensors handle ignore statements by itself, rather than
delegating the task to the user applications.

For now, I am calling sensors_get_ignored() in
sensors_get_all_features(), because this is the least intrusive way.
This is in no way optimal though, it would be better to not add
ignored features to the feature list in the first place. However,
doing so would require that the configuration file is read before
sysfs is scanned for features, which isn't currently the case. So
this improvement is left for later.

Note that this patch adds a small cost in terms of performance. This
is because we now honor ignore statements on all features (main ones
and subfeatures) while in practice "sensors" was only checking for
main features. It would be trivial to stop checking for subfeatures,
but it seems to me that supporting ignore on subfeatures is a good
move, as the user may actually want to ignore a specific subfeature.
Individual alarms come to mind. This will also be handy to debug the
generic print code in "sensors".

Files:

• lm-sensors/branches/lm-sensors-3.0.0/lib/access.c (modified) (2 diffs)
• lm-sensors/branches/lm-sensors-3.0.0/lib/sensors.h (modified) (1 diff)
• lm-sensors/branches/lm-sensors-3.0.0/prog/sensors/chips.c (modified) (1 diff)

lm-sensors/branches/lm-sensors-3.0.0/lib/access.c

@@ -194 +194 @@
-}
-
-
+
+
+
+
-{
-        const sensors_chip *chip;
@@ -344 +347 @@
-                if (sensors_match_chip(sensors_proc_chips[i].chip, name)) {
-                        feature_list = sensors_proc_chips[i].feature;
+                        while (feature_list[*nr].data.name
+                            && sensors_get_ignored(name, feature_list[*nr].data.number) != 1)
+                                (*nr)++;
-                        if (!feature_list[*nr].data.name)
-                                return NULL;

lm-sensors/branches/lm-sensors-3.0.0/lib/sensors.h

@@ -83 +83 @@
-extern int sensors_get_label(sensors_chip_name name, int feature, 
-                             char **result);
-
-/* Looks up whether a feature should be ignored. Returns <0 on failure,
-   0 if it should be ignored, 1 if it is valid. This function takes
-   logical mappings into account. */
-extern int sensors_get_ignored(sensors_chip_name name, int fature);
-
-/* Read the value of a feature of a certain chip. Note that chip should not

lm-sensors/branches/lm-sensors-3.0.0/prog/sensors/chips.c

@@ -92 +92 @@
-  int err;
-  err = sensors_get_label(name,feature,label);
-
-
-
-
-
-
+
-  return err;
-}