X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fapcups.c;h=a0629d5f6ae0a87e13b08f1e53f6d69b0c1a85f3;hb=c089c920528313b72a535dabfcdbd91a66c8d977;hp=cd64426afa01e42125f3b210b7e39dc40f991c12;hpb=df6f5e387b589b665f3be9099f6a27a84a805e09;p=collectd.git diff --git a/src/apcups.c b/src/apcups.c index cd64426a..a0629d5f 100644 --- a/src/apcups.c +++ b/src/apcups.c @@ -1,8 +1,9 @@ /* * collectd - src/apcups.c - * Copyright (C) 2006 Anthony Gialluca - * Copyright (C) 2000-2004 Kern Sibbald - * Copyright (C) 1996-99 Andre M. Hedrick + * Copyright (C) 2006-2007 Florian octo Forster + * Copyright (C) 2006 Anthony Gialluca + * Copyright (C) 2000-2004 Kern Sibbald + * Copyright (C) 1996-1999 Andre M. Hedrick * * This program is free software; you can redistribute it and/or * modify it under the terms of version 2 of the GNU General @@ -20,20 +21,13 @@ * * Authors: * Anthony Gialluca + * Florian octo Forster **/ -/* - * FIXME: Don't know why but without this here atof() was not returning - * correct values for me. This is behavior that I don't understand and - * should be examined in closer detail. - */ -#include - #include "collectd.h" #include "common.h" /* rrd_update_file */ #include "plugin.h" /* plugin_register, plugin_submit */ #include "configfile.h" /* cf_register */ -#include "utils_debug.h" #if HAVE_SYS_TYPES_H # include @@ -49,77 +43,15 @@ # include #endif -#ifndef APCMAIN -# define APCMAIN 0 -#endif - #define NISPORT 3551 #define MAXSTRING 256 #define MODULE_NAME "apcups" #define APCUPS_DEFAULT_HOST "localhost" -/* Default values for contacting daemon */ -static char *global_host = NULL; -static int global_port = NISPORT; - -/* - * The following are only if not compiled to test the module with its own main. -*/ -#if !APCMAIN -static char *bvolt_file_template = "apcups/voltage-%s.rrd"; -static char *bvolt_ds_def[] = -{ - "DS:voltage:GAUGE:"COLLECTD_HEARTBEAT":0:U", -}; -static int bvolt_ds_num = 1; - -static char *load_file_template = "apcups/load_percent.rrd"; -static char *load_ds_def[] = -{ - "DS:percent:GAUGE:"COLLECTD_HEARTBEAT":0:110", -}; -static int load_ds_num = 1; - -static char *charge_file_template = "apcups/charge_percent.rrd"; -static char *charge_ds_def[] = -{ - "DS:percent:GAUGE:"COLLECTD_HEARTBEAT":0:110", -}; -static int charge_ds_num = 1; - -static char *time_file_template = "apcups/timeleft.rrd"; -static char *time_ds_def[] = -{ - "DS:timeleft:GAUGE:"COLLECTD_HEARTBEAT":0:100", -}; -static int time_ds_num = 1; - -static char *temp_file_template = "apcups/temperature.rrd"; -static char *temp_ds_def[] = -{ - /* -273.15 is absolute zero */ - "DS:value:GAUGE:"COLLECTD_HEARTBEAT":-274:U", -}; -static int temp_ds_num = 1; - -static char *freq_file_template = "apcups/frequency-%s.rrd"; -static char *freq_ds_def[] = -{ - "DS:frequency:GAUGE:"COLLECTD_HEARTBEAT":0:U", -}; -static int freq_ds_num = 1; - -static char *config_keys[] = -{ - "Host", - "Port", - NULL -}; -static int config_keys_num = 2; - -#endif /* if APCMAIN */ - +/* + * Private data types + */ struct apc_detail_s { double linev; @@ -132,32 +64,48 @@ struct apc_detail_s double linefreq; }; -#define BIG_BUF 4096 +/* + * Private variables + */ +/* Default values for contacting daemon */ +static char *conf_host = NULL; +static int conf_port = NISPORT; + +static int global_sockfd = -1; + +static const char *config_keys[] = +{ + "Host", + "Port", + NULL +}; +static int config_keys_num = 2; -#if APCMAIN /* Close the network connection */ -static void net_close (int *fd) +static int apcups_shutdown (void) { uint16_t packet_size = 0; - assert (*fd >= 0); + if (global_sockfd < 0) + return (0); - DBG ("Gracefully shutting down socket %i.", *fd); + DEBUG ("Gracefully shutting down socket %i.", global_sockfd); /* send EOF sentinel */ - swrite (*fd, (void *) &packet_size, sizeof (packet_size)); + swrite (global_sockfd, (void *) &packet_size, sizeof (packet_size)); - close (*fd); - *fd = -1; -} -#endif /* APCMAIN */ + close (global_sockfd); + global_sockfd = -1; + + return (0); +} /* int apcups_shutdown */ /* * Open a TCP connection to the UPS network server * Returns -1 on error * Returns socket file descriptor otherwise */ -static int net_open (char *host, char *service, int port) +static int net_open (char *host, int port) { int sd; int status; @@ -169,8 +117,7 @@ static int net_open (char *host, char *service, int port) assert ((port > 0x00000000) && (port <= 0x0000FFFF)); /* Convert the port to a string */ - snprintf (port_str, 8, "%i", port); - port_str[7] = '\0'; + ssnprintf (port_str, sizeof (port_str), "%i", port); /* Resolve name */ memset ((void *) &ai_hints, '\0', sizeof (ai_hints)); @@ -180,7 +127,11 @@ static int net_open (char *host, char *service, int port) status = getaddrinfo (host, port_str, &ai_hints, &ai_return); if (status != 0) { - DBG ("getaddrinfo failed: %s", status == EAI_SYSTEM ? strerror (errno) : gai_strerror (status)); + char errbuf[1024]; + INFO ("getaddrinfo failed: %s", + (status == EAI_SYSTEM) + ? sstrerror (errno, errbuf, sizeof (errbuf)) + : gai_strerror (status)); return (-1); } @@ -196,7 +147,7 @@ static int net_open (char *host, char *service, int port) if (sd < 0) { - DBG ("Unable to open a socket"); + DEBUG ("Unable to open a socket"); freeaddrinfo (ai_return); return (-1); } @@ -207,12 +158,14 @@ static int net_open (char *host, char *service, int port) if (status != 0) /* `connect(2)' failed */ { - DBG ("connect failed: %s", strerror (errno)); + char errbuf[1024]; + INFO ("connect failed: %s", + sstrerror (errno, errbuf, sizeof (errbuf))); close (sd); return (-1); } - DBG ("Done opening a socket %i", sd); + DEBUG ("Done opening a socket %i", sd); return (sd); } /* int net_open (char *host, char *service, int port) */ @@ -240,7 +193,7 @@ static int net_recv (int *sockfd, char *buf, int buflen) packet_size = ntohs (packet_size); if (packet_size > buflen) { - DBG ("record length too large"); + DEBUG ("record length too large"); return (-2); } @@ -297,52 +250,47 @@ static int apc_query_server (char *host, int port, int n; char recvline[1024]; char *tokptr; + char *toksaveptr; char *key; double value; - static int sockfd = -1; - static complain_t compl; - #if APCMAIN # define PRINT_VALUE(name, val) printf(" Found property: name = %s; value = %f;\n", name, val) #else # define PRINT_VALUE(name, val) /**/ #endif - if (sockfd < 0) + if (global_sockfd < 0) { - if ((sockfd = net_open (host, NULL, port)) < 0) + global_sockfd = net_open (host, port); + if (global_sockfd < 0) { - plugin_complain (LOG_ERR, &compl, "apcups plugin: " - "Connecting to the apcupsd failed."); + ERROR ("apcups plugin: Connecting to the " + "apcupsd failed."); return (-1); } - else - { - plugin_relief (LOG_NOTICE, &compl, "apcups plugin: " - "Connection re-established to the apcupsd."); - } } - if (net_send (&sockfd, "status", 6) < 0) + if (net_send (&global_sockfd, "status", 6) < 0) { - syslog (LOG_ERR, "apcups plugin: Writing to the socket failed."); + ERROR ("apcups plugin: Writing to the socket failed."); return (-1); } - while ((n = net_recv (&sockfd, recvline, sizeof (recvline) - 1)) > 0) + while ((n = net_recv (&global_sockfd, recvline, sizeof (recvline) - 1)) > 0) { - assert (n < sizeof (recvline)); + assert ((unsigned int)n < sizeof (recvline)); recvline[n] = '\0'; #if APCMAIN printf ("net_recv = `%s';\n", recvline); #endif /* if APCMAIN */ - tokptr = strtok (recvline, " :\t"); + toksaveptr = NULL; + tokptr = strtok_r (recvline, " :\t", &toksaveptr); while (tokptr != NULL) { key = tokptr; - if ((tokptr = strtok (NULL, " :\t")) == NULL) + if ((tokptr = strtok_r (NULL, " :\t", &toksaveptr)) == NULL) continue; value = atof (tokptr); @@ -365,60 +313,29 @@ static int apc_query_server (char *host, int port, else if (strcmp ("TIMELEFT", key) == 0) apcups_detail->timeleft = value; - tokptr = strtok (NULL, ":"); + tokptr = strtok_r (NULL, ":", &toksaveptr); } /* while (tokptr != NULL) */ } if (n < 0) { - syslog (LOG_WARNING, "apcups plugin: Error reading from socket"); + WARNING ("apcups plugin: Error reading from socket"); return (-1); } -#if APCMAIN - else - { - /* close the opened socket */ - net_close (&sockfd); - } -#endif /* APCMAIN */ return (0); } -#if APCMAIN -/* - * This is used for testing apcups in a standalone mode. - * Usefull for debugging. - */ -int main (int argc, char **argv) -{ - /* we are not really going to use this */ - struct apc_detail_s apcups_detail; - - openlog ("apcups", LOG_PID | LOG_NDELAY | LOG_LOCAL1, LOG_USER); - - if (global_host == NULL || strcmp (global_host, "0.0.0.0") == 0) - global_host = "localhost"; - - if(apc_query_server (global_host, global_port, &apcups_detail) < 0) - { - printf("apcups: Failed...\n"); - return(-1); - } - - return 0; -} -#else -static int apcups_config (char *key, char *value) +static int apcups_config (const char *key, const char *value) { if (strcasecmp (key, "host") == 0) { - if (global_host != NULL) + if (conf_host != NULL) { - free (global_host); - global_host = NULL; + free (conf_host); + conf_host = NULL; } - if ((global_host = strdup (value)) == NULL) + if ((conf_host = strdup (value)) == NULL) return (1); } else if (strcasecmp (key, "Port") == 0) @@ -426,10 +343,10 @@ static int apcups_config (char *key, char *value) int port_tmp = atoi (value); if (port_tmp < 1 || port_tmp > 65535) { - syslog (LOG_WARNING, "apcups plugin: Invalid port: %i", port_tmp); + WARNING ("apcups plugin: Invalid port: %i", port_tmp); return (1); } - global_port = port_tmp; + conf_port = port_tmp; } else { @@ -438,83 +355,37 @@ static int apcups_config (char *key, char *value) return (0); } -static void apcups_init (void) -{ - return; -} - -static void apc_write_voltage (char *host, char *inst, char *val) +static void apc_submit_generic (char *type, char *type_inst, double value) { - char file[512]; - int status; + value_t values[1]; + value_list_t vl = VALUE_LIST_INIT; - status = snprintf (file, 512, bvolt_file_template, inst); - if ((status < 1) || (status >= 512)) - return; + values[0].gauge = value; - rrd_update_file (host, file, val, bvolt_ds_def, bvolt_ds_num); -} + vl.values = values; + vl.values_len = 1; + sstrncpy (vl.host, hostname_g, sizeof (vl.host)); + sstrncpy (vl.plugin, "apcups", sizeof (vl.plugin)); + sstrncpy (vl.plugin_instance, "", sizeof (vl.plugin_instance)); + sstrncpy (vl.type, type, sizeof (vl.type)); + sstrncpy (vl.type_instance, type_inst, sizeof (vl.type_instance)); -static void apc_write_charge (char *host, char *inst, char *val) -{ - rrd_update_file (host, charge_file_template, val, charge_ds_def, charge_ds_num); -} - -static void apc_write_percent (char *host, char *inst, char *val) -{ - rrd_update_file (host, load_file_template, val, load_ds_def, load_ds_num); -} - -static void apc_write_timeleft (char *host, char *inst, char *val) -{ - rrd_update_file (host, time_file_template, val, time_ds_def, time_ds_num); -} - -static void apc_write_temperature (char *host, char *inst, char *val) -{ - rrd_update_file (host, temp_file_template, val, temp_ds_def, temp_ds_num); -} - -static void apc_write_frequency (char *host, char *inst, char *val) -{ - char file[512]; - int status; - - status = snprintf (file, 512, freq_file_template, inst); - if ((status < 1) || (status >= 512)) - return; - - rrd_update_file (host, file, val, freq_ds_def, freq_ds_num); -} - -static void apc_submit_generic (char *type, char *inst, - double value) -{ - char buf[512]; - int status; - - status = snprintf (buf, 512, "%u:%f", - (unsigned int) curtime, value); - if ((status < 1) || (status >= 512)) - return; - - DBG ("plugin_submit (%s, %s, %s);", type, inst, buf); - plugin_submit (type, inst, buf); + plugin_dispatch_values (&vl); } static void apc_submit (struct apc_detail_s *apcups_detail) { - apc_submit_generic ("apcups_voltage", "input", apcups_detail->linev); - apc_submit_generic ("apcups_voltage", "output", apcups_detail->outputv); - apc_submit_generic ("apcups_voltage", "battery", apcups_detail->battv); - apc_submit_generic ("apcups_charge", "-", apcups_detail->bcharge); - apc_submit_generic ("apcups_charge_pct", "-", apcups_detail->loadpct); - apc_submit_generic ("apcups_timeleft", "-", apcups_detail->timeleft); - apc_submit_generic ("apcups_temp", "-", apcups_detail->itemp); - apc_submit_generic ("apcups_frequency", "input", apcups_detail->linefreq); + apc_submit_generic ("voltage", "input", apcups_detail->linev); + apc_submit_generic ("voltage", "output", apcups_detail->outputv); + apc_submit_generic ("voltage", "battery", apcups_detail->battv); + apc_submit_generic ("charge", "", apcups_detail->bcharge); + apc_submit_generic ("percent", "load", apcups_detail->loadpct); + apc_submit_generic ("timeleft", "", apcups_detail->timeleft); + apc_submit_generic ("temperature", "", apcups_detail->itemp); + apc_submit_generic ("frequency", "input", apcups_detail->linefreq); } -static void apcups_read (void) +static int apcups_read (void) { struct apc_detail_s apcups_detail; int status; @@ -528,10 +399,10 @@ static void apcups_read (void) apcups_detail.itemp = -300.0; apcups_detail.linefreq = -1.0; - status = apc_query_server (global_host == NULL + status = apc_query_server (conf_host == NULL ? APCUPS_DEFAULT_HOST - : global_host, - global_port, &apcups_detail); + : conf_host, + conf_port, &apcups_detail); /* * if we did not connect then do not bother submitting @@ -539,28 +410,23 @@ static void apcups_read (void) */ if (status != 0) { - DBG ("apc_query_server (%s, %i) = %i", - global_host == NULL + DEBUG ("apc_query_server (%s, %i) = %i", + conf_host == NULL ? APCUPS_DEFAULT_HOST - : global_host, - global_port, status); - return; + : conf_host, + conf_port, status); + return (-1); } apc_submit (&apcups_detail); + + return (0); } /* apcups_read */ void module_register (void) { - plugin_register (MODULE_NAME, apcups_init, apcups_read, NULL); - plugin_register ("apcups_voltage", NULL, NULL, apc_write_voltage); - plugin_register ("apcups_charge", NULL, NULL, apc_write_charge); - plugin_register ("apcups_charge_pct", NULL, NULL, apc_write_percent); - plugin_register ("apcups_timeleft", NULL, NULL, apc_write_timeleft); - plugin_register ("apcups_temp", NULL, NULL, apc_write_temperature); - plugin_register ("apcups_frequency", NULL, NULL, apc_write_frequency); - cf_register (MODULE_NAME, apcups_config, config_keys, config_keys_num); -} - -#endif /* if APCMAIN */ -#undef MODULE_NAME + plugin_register_config ("apcups", apcups_config, config_keys, + config_keys_num); + plugin_register_read ("apcups", apcups_read); + plugin_register_shutdown ("apcups", apcups_shutdown); +} /* void module_register */