X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fsensors.c;h=fde8dcd7ec00e2a93e6d4826a92d0d67775c9cc5;hb=7e5df1a2c6611bd4ac9fb8ac4b78106f9139ae6e;hp=7cf787c7a8b155b9a74f1398a2e4c550f42372bc;hpb=845b2d2577461a4bc2bf4385cb521b4ad575651d;p=collectd.git diff --git a/src/sensors.c b/src/sensors.c index 7cf787c7..fde8dcd7 100644 --- a/src/sensors.c +++ b/src/sensors.c @@ -1,11 +1,10 @@ /** * collectd - src/sensors.c - * Copyright (C) 2005,2006 Florian octo Forster + * Copyright (C) 2005-2007 Florian octo Forster * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License as published by the - * Free Software Foundation; either version 2 of the License, or (at your - * option) any later version. + * Free Software Foundation; only version 2 of the License is applicable. * * This program is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty of @@ -34,10 +33,6 @@ #include "plugin.h" #include "configfile.h" #include "utils_ignorelist.h" -#include "utils_debug.h" - -#define MODULE_NAME "sensors" -#define MODULE_NAME_VOLTAGE MODULE_NAME"_voltage" #if defined(HAVE_SENSORS_SENSORS_H) # include @@ -51,57 +46,31 @@ # define SENSORS_HAVE_READ 0 #endif -#define BUFSIZE 512 - -/* temperature and fan sensors */ -static char *ds_def[] = -{ - "DS:value:GAUGE:"COLLECTD_HEARTBEAT":U:U", - NULL -}; -static int ds_num = 1; - -/* voltage sensors */ -static char *sensor_voltage_ds_def[] = -{ - "DS:voltage:GAUGE:"COLLECTD_HEARTBEAT":U:U", - NULL -}; -static int sensor_voltage_ds_num = 1; - -/* old naming */ -static char *old_filename_format = "sensors-%s.rrd"; -/* end old naming */ - -/* new naming default is to create chip-feature - * 1 => use new naming scheme chip-bus-address/type-feature +#ifndef SENSORS_CONF_PATH +# define SENSORS_CONF_PATH "/etc/sensors.conf" +#endif + +static const char *conffile = SENSORS_CONF_PATH; +/* SENSORS_CONF_PATH */ + +/* + * remember stat of the loaded config */ -static int sensor_extended_naming = 0; +static time_t sensors_config_mtime = 0; -#ifdef HAVE_LIBSENSORS typedef struct featurelist { const sensors_chip_name *chip; @@ -188,9 +159,8 @@ typedef struct featurelist } featurelist_t; featurelist_t *first_feature = NULL; -#endif /* defined (HAVE_LIBSENSORS) */ -static int sensors_config (char *key, char *value) +static int sensors_config (const char *key, const char *value) { if (sensor_list == NULL) sensor_list = ignorelist_create (1); @@ -199,7 +169,8 @@ static int sensors_config (char *key, char *value) { if (ignorelist_add (sensor_list, value)) { - syslog (LOG_EMERG, "Cannot add value to ignorelist."); + ERROR ("sensors plugin: " + "Cannot add value to ignorelist."); return (1); } } @@ -211,15 +182,6 @@ static int sensors_config (char *key, char *value) || (strcasecmp (value, "On") == 0)) ignorelist_set_invert (sensor_list, 0); } - else if (strcasecmp (key, "ExtendedSensorNaming") == 0) - { - if ((strcasecmp (value, "True") == 0) - || (strcasecmp (value, "Yes") == 0) - || (strcasecmp (value, "On") == 0)) - sensor_extended_naming = 1; - else - sensor_extended_naming = 0; - } else { return (-1); @@ -228,44 +190,79 @@ static int sensors_config (char *key, char *value) return (0); } -static void collectd_sensors_init (void) +void sensors_free_features (void) +{ + featurelist_t *thisft; + featurelist_t *nextft; + + if (first_feature == NULL) + return; + + sensors_cleanup (); + + for (thisft = first_feature; thisft != NULL; thisft = nextft) + { + nextft = thisft->next; + sfree (thisft); + } + first_feature = NULL; +} + +static int sensors_load_conf (void) { -#ifdef HAVE_LIBSENSORS FILE *fh; featurelist_t *last_feature = NULL; - featurelist_t *new_feature; + featurelist_t *new_feature = NULL; const sensors_chip_name *chip; int chip_num; const sensors_feature_data *data; int data_num0, data_num1; + + struct stat statbuf; + int status; - new_feature = first_feature; - while (new_feature != NULL) + status = stat (conffile, &statbuf); + if (status != 0) { - last_feature = new_feature->next; - free (new_feature); - new_feature = last_feature; + char errbuf[1024]; + ERROR ("sensors plugin: stat (%s) failed: %s", conffile, + sstrerror (errno, errbuf, sizeof (errbuf))); + sensors_config_mtime = 0; } -#ifdef assert - assert (new_feature == NULL); - assert (last_feature == NULL); -#endif + if ((sensors_config_mtime != 0) + && (sensors_config_mtime == statbuf.st_mtime)) + return (0); - if ((fh = fopen ("/etc/sensors.conf", "r")) == NULL) - return; + if (sensors_config_mtime != 0) + { + NOTICE ("sensors plugin: Reloading config from %s", + conffile); + sensors_free_features (); + sensors_config_mtime = 0; + } - if (sensors_init (fh)) + fh = fopen (conffile, "r"); + if (fh == NULL) { - fclose (fh); - syslog (LOG_ERR, "sensors: Cannot initialize sensors. " - "Data will not be collected."); - return; + char errbuf[1024]; + ERROR ("sensors plugin: fopen(%s) failed: %s", conffile, + sstrerror (errno, errbuf, sizeof (errbuf))); + return (-1); } + status = sensors_init (fh); fclose (fh); + if (status != 0) + { + ERROR ("sensors plugin: Cannot initialize sensors. " + "Data will not be collected."); + return (-1); + } + + sensors_config_mtime = statbuf.st_mtime; chip_num = 0; while ((chip = sensors_get_detected_chips (&chip_num)) != NULL) @@ -283,7 +280,7 @@ static void collectd_sensors_init (void) continue; /* Only known features */ - for (i = 0; known_features[i].type >= 0; i++) + for (i = 0; i < known_features_num; i++) { if (strcmp (data->name, known_features[i].label) != 0) continue; @@ -292,17 +289,16 @@ static void collectd_sensors_init (void) if (sensors_get_ignored (*chip, data->number) == 0) break; - DBG ("Adding feature: %s-%s-%s", + DEBUG ("Adding feature: %s-%s-%s", chip->prefix, - sensor_type_prefix[known_features[i].type], + sensor_to_type[known_features[i].type], data->name); if ((new_feature = (featurelist_t *) malloc (sizeof (featurelist_t))) == NULL) { - DBG ("sensors plugin: malloc: %s", - strerror (errno)); - syslog (LOG_ERR, "sensors plugin: malloc: %s", - strerror (errno)); + char errbuf[1024]; + ERROR ("sensors plugin: malloc: %s", + sstrerror (errno, errbuf, sizeof (errbuf))); break; } @@ -329,153 +325,122 @@ static void collectd_sensors_init (void) } /* while sensors_get_detected_chips */ if (first_feature == NULL) + { sensors_cleanup (); -#endif /* defined(HAVE_LIBSENSORS) */ + INFO ("sensors plugin: lm_sensors reports no " + "features. Data will not be collected."); + return (-1); + } - return; -} + return (0); +} /* int sensors_load_conf */ -static void sensors_voltage_write (char *host, char *inst, char *val) +static int sensors_shutdown (void) { - char file[BUFSIZE]; - int status; - - /* skip ignored in our config */ - if (ignorelist_match (sensor_list, inst)) - return; - - /* extended sensor naming */ - if(sensor_extended_naming) - status = snprintf (file, BUFSIZE, extended_filename_format, inst); - else - status = snprintf (file, BUFSIZE, old_filename_format, inst); + sensors_free_features (); + ignorelist_free (sensor_list); - if ((status < 1) || (status >= BUFSIZE)) - return; - - rrd_update_file (host, file, val, sensor_voltage_ds_def, sensor_voltage_ds_num); -} + return (0); +} /* int sensors_shutdown */ -static void sensors_write (char *host, char *inst, char *val) +static void sensors_submit (const char *plugin_instance, + const char *type, const char *type_instance, + double val) { - char file[BUFSIZE]; + char match_key[1024]; int status; - /* skip ignored in our config */ - if (ignorelist_match (sensor_list, inst)) - return; - - /* extended sensor naming */ - if (sensor_extended_naming) - status = snprintf (file, BUFSIZE, extended_filename_format, inst); - else - status = snprintf (file, BUFSIZE, old_filename_format, inst); + value_t values[1]; + value_list_t vl = VALUE_LIST_INIT; - if ((status < 1) || (status >= BUFSIZE)) + status = snprintf (match_key, sizeof (match_key), "%s/%s-%s", + plugin_instance, type, type_instance); + if ((status < 1) || (status >= sizeof (match_key))) return; + match_key[sizeof (match_key) - 1] = '\0'; - rrd_update_file (host, file, val, ds_def, ds_num); -} - -#if SENSORS_HAVE_READ -static void sensors_submit (const char *feat_name, - const char *chip_prefix, double value, int type) -{ - char buf[BUFSIZE]; - char inst[BUFSIZE]; - - if (snprintf (inst, BUFSIZE, "%s-%s", chip_prefix, feat_name) - >= BUFSIZE) - return; + if (sensor_list != NULL) + { + DEBUG ("sensors plugin: Checking ignorelist for `%s'", match_key); + if (ignorelist_match (sensor_list, match_key)) + return; + } - /* skip ignored in our config */ - if (ignorelist_match (sensor_list, inst)) - return; + values[0].gauge = val; - if (snprintf (buf, BUFSIZE, "%u:%.3f", (unsigned int) curtime, - value) >= BUFSIZE) - return; + vl.values = values; + vl.values_len = 1; + vl.time = time (NULL); + strcpy (vl.host, hostname_g); + strcpy (vl.plugin, "sensors"); + strcpy (vl.plugin_instance, plugin_instance); + strcpy (vl.type_instance, type_instance); - if (type == SENSOR_TYPE_VOLTAGE) - { - DBG ("%s: %s/%s, %s", MODULE_NAME_VOLTAGE, - sensor_type_prefix[type], inst, buf); - plugin_submit (MODULE_NAME_VOLTAGE, inst, buf); - } - else - { - DBG ("%s: %s/%s, %s", MODULE_NAME, - sensor_type_prefix[type], inst, buf); - plugin_submit (MODULE_NAME, inst, buf); - } -} + plugin_dispatch_values (type, &vl); +} /* void sensors_submit */ -static void sensors_read (void) +static int sensors_read (void) { featurelist_t *feature; double value; - char chip_fullprefix[BUFSIZE]; + + char plugin_instance[DATA_MAX_NAME_LEN]; + char type_instance[DATA_MAX_NAME_LEN]; + + if (sensors_load_conf () != 0) + return (-1); for (feature = first_feature; feature != NULL; feature = feature->next) { if (sensors_get_feature (*feature->chip, feature->data->number, &value) < 0) continue; - if (sensor_extended_naming) + /* full chip name logic borrowed from lm_sensors */ + if (feature->chip->bus == SENSORS_CHIP_NAME_BUS_ISA) { - /* full chip name logic borrowed from lm_sensors */ - if (feature->chip->bus == SENSORS_CHIP_NAME_BUS_ISA) - { - if (snprintf (chip_fullprefix, BUFSIZE, "%s-isa-%04x/%s", - feature->chip->prefix, - feature->chip->addr, - sensor_type_prefix[feature->type]) - >= BUFSIZE) - continue; - } - else if (feature->chip->bus == SENSORS_CHIP_NAME_BUS_DUMMY) - { - if (snprintf (chip_fullprefix, BUFSIZE, "%s-%s-%04x/%s", - feature->chip->prefix, - feature->chip->busname, - feature->chip->addr, - sensor_type_prefix[feature->type]) - >= BUFSIZE) - continue; - } - else - { - if (snprintf (chip_fullprefix, BUFSIZE, "%s-i2c-%d-%02x/%s", - feature->chip->prefix, - feature->chip->bus, - feature->chip->addr, - sensor_type_prefix[feature->type]) - >= BUFSIZE) - continue; - } - - sensors_submit (feature->data->name, - chip_fullprefix, - value, feature->type); + if (snprintf (plugin_instance, DATA_MAX_NAME_LEN, "%s-isa-%04x", + feature->chip->prefix, + feature->chip->addr) + >= 512) + continue; + } + else if (feature->chip->bus == SENSORS_CHIP_NAME_BUS_DUMMY) + { + if (snprintf (plugin_instance, 512, "%s-%s-%04x", + feature->chip->prefix, + feature->chip->busname, + feature->chip->addr) + >= 512) + continue; } else { - sensors_submit (feature->data->name, - feature->chip->prefix, - value, feature->type); + if (snprintf (plugin_instance, 512, "%s-i2c-%d-%02x", + feature->chip->prefix, + feature->chip->bus, + feature->chip->addr) + >= 512) + continue; } - } -} -#else -# define sensors_read NULL + + strncpy (type_instance, feature->data->name, DATA_MAX_NAME_LEN); + + sensors_submit (plugin_instance, + sensor_to_type[feature->type], + type_instance, + value); + } /* for feature = first_feature .. NULL */ + + return (0); +} /* int sensors_read */ #endif /* SENSORS_HAVE_READ */ void module_register (void) { - plugin_register (MODULE_NAME, collectd_sensors_init, sensors_read, sensors_write); - plugin_register (MODULE_NAME_VOLTAGE, NULL, NULL, sensors_voltage_write); - cf_register (MODULE_NAME, sensors_config, config_keys, config_keys_num); -} - -#undef BUFSIZE -#undef MODULE_NAME +#if SENSORS_HAVE_READ + plugin_register_config ("sensors", sensors_config, + config_keys, config_keys_num); + plugin_register_read ("sensors", sensors_read); + plugin_register_shutdown ("sensors", sensors_shutdown); +#endif +} /* void module_register */