Changeset 4761

Timestamp:

09/05/07 10:20:35 (6 years ago)

Author:

khali

Message:

Optimize the memory consumption in sensors_read_dynamic_chip(). The
idea is to have separate indexing for subfeatures with a compute
mapping and for subfeatures without it. They still follow each
other in the big table but this avoids wasting memory due to the
numbering gap between them.

This cuts the amount of memory (temporarily) allocated by
sensors_read_dynamic_chip() almost by half, and also speeds it up
a little as it now takes less iterations to walk the sparse array.

Files:

• lm-sensors/branches/lm-sensors-3.0.0/lib/sysfs.c (modified) (3 diffs)

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

@@ -38 +38 @@
 
 #define MAX_SENSORS_PER_TYPE    16
-#define MAX_SUB_FEATURES        22
+#define MAX_SUB_FEATURES        6
 /* Room for all 3 types (in, fan, temp) with all their subfeatures + VID */
-#define ALL_POSSIBLE_FEATURES   (MAX_SENSORS_PER_TYPE * MAX_SUB_FEATURES * 3 \
+#define ALL_POSSIBLE_FEATURES   (MAX_SENSORS_PER_TYPE * MAX_SUB_FEATURES * 6 \
                                  + MAX_SENSORS_PER_TYPE)
 
@@ -121 +121 @@
                    sorted table */
                 if (type == SENSORS_FEATURE_VID) {
-                        i = nr + MAX_SENSORS_PER_TYPE * MAX_SUB_FEATURES * 3;
+                        i = nr + MAX_SENSORS_PER_TYPE * MAX_SUB_FEATURES * 6;
                 } else {
                         i = (type >> 8) * MAX_SENSORS_PER_TYPE *
-                                MAX_SUB_FEATURES + nr * MAX_SUB_FEATURES +
-                                (type & 0xFF);
+                            MAX_SUB_FEATURES * 2 + nr * MAX_SUB_FEATURES * 2 +
+                            ((type & 0x10) >> 4) * MAX_SUB_FEATURES +
+                            (type & 0x0F);
                 }
 
@@ -145 +146 @@
                 } else {
                         /* sub feature */
-                        feature.data.mapping = i - i % MAX_SUB_FEATURES;
+                        feature.data.mapping = i - i % (MAX_SUB_FEATURES * 2);
                         if (!(type & 0x10))
                                 feature.data.flags |= SENSORS_COMPUTE_MAPPING;