Changeset 51

Timestamp:

12/08/98 23:52:05 (14 years ago)

Author:

frodo

Message:

Introduced /proc/sys/dev/sensors/chips file

See doc/sysctl for an explanation why it is needed.

Also fixed two very small cleanup bugs in lm75/lm78, which would probably be
impossible to trigger anyway, but still.

Location:

lm-sensors/trunk

Files:

• TODO (modified) (1 diff)
• doc/sysctl (added)
• kernel/chips/lm75.c (modified) (1 diff)
• kernel/chips/lm78.c (modified) (1 diff)
• kernel/include/sensors.h (modified) (2 diffs)
• kernel/sensors.c (modified) (6 diffs)
• src/lm75.c (modified) (1 diff)
• src/lm78.c (modified) (1 diff)
• src/sensors.c (modified) (6 diffs)
• src/sensors.h (modified) (2 diffs)

lm-sensors/trunk/TODO

@@ -11 +11 @@
   places, probably (everywhere where global vars are accessed). This must be
   done for the i2c modules, too.
-* Extend sensors.c with a file /proc/sys/dev/sensors/chips, which lists
-  all directories under /proc/sys/dev/sensors with their SYSCTL id
-  (needed for good sysctl access).
 * Extend the mod_inc_use/mod_dec_use through the fill_inode trick, for all
   new /proc files.

lm-sensors/trunk/kernel/chips/lm75.c

@@ -332 +332 @@
       return res;
     }
-  } else
     lm75_initialized --;
+  }
 
   return 0;

lm-sensors/trunk/kernel/chips/lm78.c

@@ -879 +879 @@
       return res;
     }
-  } else
     lm78_initialized --;
-
+  }
   return 0;
 }

lm-sensors/trunk/kernel/include/sensors.h

@@ -74 +74 @@
 extern void sensors_deregister_entry(int id);
 
-
 #endif /* def __KERNEL__ */
 
+
+/* The maximum length of the prefix */
+#define SENSORS_PREFIX_MAX 20
 
 /* Driver IDs */
@@ -90 +92 @@
                           dev table */
 #endif /* def DEV_HWMON */
+
+#define SENSORS_CHIPS 1
+struct sensors_chips_data {
+  int sysctl_id;
+  char name[SENSORS_PREFIX_MAX + 13];
+};
 
 #define LM78_SYSCTL_IN0 1000  /* Volts * 100 */

lm-sensors/trunk/kernel/sensors.c

@@ -42 +42 @@
 static void sensors_write_reals(int nrels,void *buffer,int *bufsize,
                                 long *results, int magnitude);
+static int sensors_proc_chips(ctl_table *ctl, int write, struct file * filp,
+                              void *buffer, size_t *lenp);
+static int sensors_sysctl_chips (ctl_table *table, int *name, int nlen, 
+                                 void *oldval, size_t *oldlenp, void *newval,
+                                 size_t newlen, void **context);
 
 static int sensors_init(void);
@@ -63 +68 @@
   { 0 }
 };
+
+static ctl_table sensors_proc_dev_sensors[] = {
+  { SENSORS_CHIPS, "chips", NULL, 0, 0644, NULL, &sensors_proc_chips, 
+    &sensors_sysctl_chips },
+  { 0 }
+};
+
+static ctl_table sensors_proc_dev[] = {
+  { DEV_SENSORS, "sensors", NULL, 0, 0555, sensors_proc_dev_sensors },
+  { 0 },
+};
+
+
+static ctl_table sensors_proc[] = {
+  { CTL_DEV, "dev", NULL, 0, 0555, sensors_proc_dev },
+  { 0 }
+};
+
+
+static struct ctl_table_header *sensors_proc_header;
+static int sensors_initialized;
 
 /* This returns a nice name for a new directory; for example lm78-isa-0310
@@ -207 +233 @@
 }
 #endif /* (LINUX_VERSION_CODE >= KERNEL_VERSION(2,1,58)) */
+
+int sensors_proc_chips(ctl_table *ctl, int write, struct file * filp,
+                       void *buffer, size_t *lenp)
+{
+  char BUF[SENSORS_PREFIX_MAX + 30];
+  int buflen,curbufsize,i;
+  struct ctl_table *client_tbl;
+
+  if (write)
+    return 0;
+
+  /* If buffer is size 0, or we try to read when not at the start, we
+     return nothing. Note that I think writing when not at the start
+     does not work either, but anyway, this is straight from the kernel
+     sources. */
+  if (!*lenp || (filp->f_pos && !write)) {
+    *lenp = 0;
+    return 0;
+  }
+  curbufsize = 0;
+  for (i = 0; i < SENSORS_ENTRY_MAX; i ++)
+    if (sensors_entries[i]) {
+      client_tbl = sensors_entries[i]->ctl_table->child->child;
+      buflen = sprintf(BUF,"%d\t%s\n",client_tbl->ctl_name,
+                       client_tbl->procname);
+      if (buflen + curbufsize > *lenp)
+        buflen=*lenp-curbufsize;
+      copy_to_user(buffer,BUF,buflen);
+      curbufsize += buflen;
+      (char *) buffer += buflen;
+    }
+  *lenp = curbufsize;
+  filp->f_pos += curbufsize;
+  return 0;
+}
+
+int sensors_sysctl_chips (ctl_table *table, int *name, int nlen, void *oldval,
+                          size_t *oldlenp, void *newval, size_t newlen,
+                          void **context)
+{
+  struct sensors_chips_data data;
+  int i,oldlen,nrels,maxels;
+  struct ctl_table *client_tbl;
+
+  if (oldval && oldlenp && ! get_user_data(oldlen,oldlenp) && oldlen) {
+    maxels = oldlen / sizeof(struct sensors_chips_data);
+    nrels = 0;
+    for (i = 0; (i < SENSORS_ENTRY_MAX) && (nrels < maxels); i++)
+      if (sensors_entries[i]) {
+        client_tbl = sensors_entries[i]->ctl_table->child->child;
+        data.sysctl_id = client_tbl->ctl_name;
+        strcpy(data.name,client_tbl->procname);
+        copy_to_user(oldval,&data,sizeof(struct sensors_chips_data));
+        (char *) oldval += sizeof(struct sensors_chips_data);
+        nrels++;
+      }
+    oldlen = nrels * sizeof(struct sensors_chips_data);
+    put_user(oldlen,oldlenp);
+  }
+  return 0;
+}
+
 
 /* This funcion reads or writes a 'real' value (encoded by the combination
@@ -263 +351 @@
 
     /* And write them to buffer, converting to reals */
-    sensors_write_reals(nrels,buffer,lenp,results,mag);
     sensors_write_reals(nrels,buffer,lenp,results,mag);
     filp->f_pos += *lenp;
@@ -478 +565 @@
 {
   printk("sensors.o version %s (%s)\n",LM_VERSION,LM_DATE);
+  sensors_initialized = 0;
+  if (! (sensors_proc_header = register_sysctl_table(sensors_proc,0)))
+    return -ENOMEM;
+  sensors_initialized ++;
   return 0;
 }
@@ -483 +574 @@
 int sensors_cleanup(void)
 {
+  if (sensors_initialized >= 1) {
+    unregister_sysctl_table(sensors_proc_header);
+    sensors_initialized --;
+  }
   return 0;
 }

lm-sensors/trunk/src/lm75.c

@@ -332 +332 @@
       return res;
     }
-  } else
     lm75_initialized --;
+  }
 
   return 0;

lm-sensors/trunk/src/lm78.c

@@ -879 +879 @@
       return res;
     }
-  } else
     lm78_initialized --;
-
+  }
   return 0;
 }

lm-sensors/trunk/src/sensors.c

@@ -42 +42 @@
 static void sensors_write_reals(int nrels,void *buffer,int *bufsize,
                                 long *results, int magnitude);
+static int sensors_proc_chips(ctl_table *ctl, int write, struct file * filp,
+                              void *buffer, size_t *lenp);
+static int sensors_sysctl_chips (ctl_table *table, int *name, int nlen, 
+                                 void *oldval, size_t *oldlenp, void *newval,
+                                 size_t newlen, void **context);
 
 static int sensors_init(void);
@@ -63 +68 @@
   { 0 }
 };
+
+static ctl_table sensors_proc_dev_sensors[] = {
+  { SENSORS_CHIPS, "chips", NULL, 0, 0644, NULL, &sensors_proc_chips, 
+    &sensors_sysctl_chips },
+  { 0 }
+};
+
+static ctl_table sensors_proc_dev[] = {
+  { DEV_SENSORS, "sensors", NULL, 0, 0555, sensors_proc_dev_sensors },
+  { 0 },
+};
+
+
+static ctl_table sensors_proc[] = {
+  { CTL_DEV, "dev", NULL, 0, 0555, sensors_proc_dev },
+  { 0 }
+};
+
+
+static struct ctl_table_header *sensors_proc_header;
+static int sensors_initialized;
 
 /* This returns a nice name for a new directory; for example lm78-isa-0310
@@ -207 +233 @@
 }
 #endif /* (LINUX_VERSION_CODE >= KERNEL_VERSION(2,1,58)) */
+
+int sensors_proc_chips(ctl_table *ctl, int write, struct file * filp,
+                       void *buffer, size_t *lenp)
+{
+  char BUF[SENSORS_PREFIX_MAX + 30];
+  int buflen,curbufsize,i;
+  struct ctl_table *client_tbl;
+
+  if (write)
+    return 0;
+
+  /* If buffer is size 0, or we try to read when not at the start, we
+     return nothing. Note that I think writing when not at the start
+     does not work either, but anyway, this is straight from the kernel
+     sources. */
+  if (!*lenp || (filp->f_pos && !write)) {
+    *lenp = 0;
+    return 0;
+  }
+  curbufsize = 0;
+  for (i = 0; i < SENSORS_ENTRY_MAX; i ++)
+    if (sensors_entries[i]) {
+      client_tbl = sensors_entries[i]->ctl_table->child->child;
+      buflen = sprintf(BUF,"%d\t%s\n",client_tbl->ctl_name,
+                       client_tbl->procname);
+      if (buflen + curbufsize > *lenp)
+        buflen=*lenp-curbufsize;
+      copy_to_user(buffer,BUF,buflen);
+      curbufsize += buflen;
+      (char *) buffer += buflen;
+    }
+  *lenp = curbufsize;
+  filp->f_pos += curbufsize;
+  return 0;
+}
+
+int sensors_sysctl_chips (ctl_table *table, int *name, int nlen, void *oldval,
+                          size_t *oldlenp, void *newval, size_t newlen,
+                          void **context)
+{
+  struct sensors_chips_data data;
+  int i,oldlen,nrels,maxels;
+  struct ctl_table *client_tbl;
+
+  if (oldval && oldlenp && ! get_user_data(oldlen,oldlenp) && oldlen) {
+    maxels = oldlen / sizeof(struct sensors_chips_data);
+    nrels = 0;
+    for (i = 0; (i < SENSORS_ENTRY_MAX) && (nrels < maxels); i++)
+      if (sensors_entries[i]) {
+        client_tbl = sensors_entries[i]->ctl_table->child->child;
+        data.sysctl_id = client_tbl->ctl_name;
+        strcpy(data.name,client_tbl->procname);
+        copy_to_user(oldval,&data,sizeof(struct sensors_chips_data));
+        (char *) oldval += sizeof(struct sensors_chips_data);
+        nrels++;
+      }
+    oldlen = nrels * sizeof(struct sensors_chips_data);
+    put_user(oldlen,oldlenp);
+  }
+  return 0;
+}
+
 
 /* This funcion reads or writes a 'real' value (encoded by the combination
@@ -263 +351 @@
 
     /* And write them to buffer, converting to reals */
-    sensors_write_reals(nrels,buffer,lenp,results,mag);
     sensors_write_reals(nrels,buffer,lenp,results,mag);
     filp->f_pos += *lenp;
@@ -478 +565 @@
 {
   printk("sensors.o version %s (%s)\n",LM_VERSION,LM_DATE);
+  sensors_initialized = 0;
+  if (! (sensors_proc_header = register_sysctl_table(sensors_proc,0)))
+    return -ENOMEM;
+  sensors_initialized ++;
   return 0;
 }
@@ -483 +574 @@
 int sensors_cleanup(void)
 {
+  if (sensors_initialized >= 1) {
+    unregister_sysctl_table(sensors_proc_header);
+    sensors_initialized --;
+  }
   return 0;
 }

lm-sensors/trunk/src/sensors.h

@@ -74 +74 @@
 extern void sensors_deregister_entry(int id);
 
-
 #endif /* def __KERNEL__ */
 
+
+/* The maximum length of the prefix */
+#define SENSORS_PREFIX_MAX 20
 
 /* Driver IDs */
@@ -90 +92 @@
                           dev table */
 #endif /* def DEV_HWMON */
+
+#define SENSORS_CHIPS 1
+struct sensors_chips_data {
+  int sysctl_id;
+  char name[SENSORS_PREFIX_MAX + 13];
+};
 
 #define LM78_SYSCTL_IN0 1000  /* Volts * 100 */