Changeset 4629

Timestamp:

07/19/07 22:46:09 (1 year ago)

Author:

khali

Message:

The way we use sensors_feature_get_type() is rather suboptimal. It is
first called during the library initialization to generate the feature
tables. Then it is called again by sensors itself. This is because we
do not store the result the first time. I propose that we add a type
field to struct sensors_feature_data, where libsensors would store
the result of sensors_feature_get_type(). That way, the application
can get the information without calling sensors_feature_get_type()
again.

This change has the following benefits:
* Obviously, a small speed-up.
* sensors_feature_get_type() can be removed from the public interface.

This means that we can turn it into something that fits the
libsensors needs better, allowing for more optimizations (see next
patches.)

Note: the patch looks bigger that it really is, because I had to move
definitions around in sensors.h.

Files:

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

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

@@ -30 +30 @@
-                                                             int feature);
-
+sensors_feature_type sensors_feature_get_type(const sensors_feature_data *feature);
+
-#endif /* def LIB_SENSORS_ACCESS_H */

lm-sensors/branches/lm-sensors-3.0.0/lib/libsensors.3

@@ -119 +119 @@
-  const char *name;
-.br
+  sensors_feature_type type;
+.br
-  int mapping;
-.br
-unused
+compute_mapping
-.br
-  int mode;

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

@@ -126 +126 @@
-#define SENSORS_NO_MAPPING -1
-
-/* This structure is used when you want to get all features of a specific
-   chip. */
-typedef struct sensors_feature_data {
-  int number;
-  char *name;
-  int mapping;
-  int compute_mapping;
-  int mode;
-} sensors_feature_data;
-
-/* This returns all features of a specific chip. They are returned in 
-   bunches: everything with the same mapping is returned just after each
-   other, with the master feature in front (that feature does not map to
-   itself, but has SENSORS_NO_MAPPING as mapping field). nr1 and nr2 are
-   two internally used variables. Set both to zero to start again at the
-   begin of the list. If no more features are found NULL is returned.
-   Do not try to change the returned structure; you will corrupt internal
-   data structures. */
-extern const sensors_feature_data *sensors_get_all_features 
-             (sensors_chip_name name, int *nr1,int *nr2);
-
-/* This enum contains some "magic" used by sensors_read_dynamic_chip() from
-   lib/sysfs.c. All the sensor types (in, fan, temp, vid) are a multiple of
@@ -187 +166 @@
-} sensors_feature_type;
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-
-#ifdef __cplusplus

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

@@ -149 +149 @@
-                /* fill in the other feature members */
-                feature.data.number = i + 1;
+                feature.data.type = type;
-                        
-                if ((type & 0x00FF) == 0) {

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

@@ -47 +47 @@
-      iter->mapping != SENSORS_NO_MAPPING &&
-      iter->mapping == feature->number) {
-    sensors_feature_type type = sensors_feature_get_type(iter);
-    int indx;
-    
-
-
-
-
+
-    if (indx < 0 || indx >= size) {
-      printf("ERROR: Bug in sensors: index out of bound");
@@ -346 +342 @@
-      continue;
-    
-(sensors_feature_get_type(feature)
+ (feature->type
-      case SENSORS_FEATURE_TEMP:
-        print_generic_chip_temp(name, feature, i, j, label_size); break;