Tree wide: Reformat with clang-format.
[collectd.git] / src / gmond.c
index 0743d8e..fc9a232 100644 (file)
 
 #include "collectd.h"
 
-#include "plugin.h"
 #include "common.h"
+#include "plugin.h"
 #include "utils_avltree.h"
 
 #if HAVE_NETDB_H
-# include <netdb.h>
+#include <netdb.h>
 #endif
 #if HAVE_NETINET_IN_H
-# include <netinet/in.h>
+#include <netinet/in.h>
 #endif
 #if HAVE_ARPA_INET_H
-# include <arpa/inet.h>
+#include <arpa/inet.h>
 #endif
 #if HAVE_POLL_H
-# include <poll.h>
+#include <poll.h>
 #endif
 
 #include <gm_protocol.h>
 
 #ifndef IPV6_ADD_MEMBERSHIP
-# ifdef IPV6_JOIN_GROUP
-#  define IPV6_ADD_MEMBERSHIP IPV6_JOIN_GROUP
-# else
-#  error "Neither IP_ADD_MEMBERSHIP nor IPV6_JOIN_GROUP is defined"
-# endif
+#ifdef IPV6_JOIN_GROUP
+#define IPV6_ADD_MEMBERSHIP IPV6_JOIN_GROUP
+#else
+#error "Neither IP_ADD_MEMBERSHIP nor IPV6_JOIN_GROUP is defined"
+#endif
 #endif /* !IP_ADD_MEMBERSHIP */
 
 #ifdef GANGLIA_MAX_MESSAGE_LEN
-# define BUFF_SIZE GANGLIA_MAX_MESSAGE_LEN
+#define BUFF_SIZE GANGLIA_MAX_MESSAGE_LEN
 #else
-# define BUFF_SIZE 1400
+#define BUFF_SIZE 1400
 #endif
 
-struct socket_entry_s
-{
-  int                     fd;
+struct socket_entry_s {
+  int fd;
   struct sockaddr_storage addr;
-  socklen_t               addrlen;
+  socklen_t addrlen;
 };
 typedef struct socket_entry_s socket_entry_t;
 
-struct staging_entry_s
-{
+struct staging_entry_s {
   char key[2 * DATA_MAX_NAME_LEN];
   value_list_t vl;
   int flags;
 };
 typedef struct staging_entry_s staging_entry_t;
 
-struct metric_map_s
-{
-  char  *ganglia_name;
-  char  *type;
-  char  *type_instance;
-  char  *ds_name;
-  int    ds_type;
+struct metric_map_s {
+  char *ganglia_name;
+  char *type;
+  char *type_instance;
+  char *ds_name;
+  int ds_type;
   size_t ds_index;
 };
 typedef struct metric_map_s metric_map_t;
 
 #define MC_RECEIVE_GROUP_DEFAULT "239.2.11.71"
-static char          *mc_receive_group = NULL;
+static char *mc_receive_group = NULL;
 #define MC_RECEIVE_PORT_DEFAULT "8649"
-static char          *mc_receive_port = NULL;
+static char *mc_receive_port = NULL;
 
 static struct pollfd *mc_receive_sockets = NULL;
-static size_t         mc_receive_sockets_num = 0;
+static size_t mc_receive_sockets_num = 0;
 
-static socket_entry_t  *mc_send_sockets = NULL;
-static size_t           mc_send_sockets_num = 0;
-static pthread_mutex_t  mc_send_sockets_lock = PTHREAD_MUTEX_INITIALIZER;
+static socket_entry_t *mc_send_sockets = NULL;
+static size_t mc_send_sockets_num = 0;
+static pthread_mutex_t mc_send_sockets_lock = PTHREAD_MUTEX_INITIALIZER;
 
-static int            mc_receive_thread_loop    = 0;
-static int            mc_receive_thread_running = 0;
-static pthread_t      mc_receive_thread_id;
+static int mc_receive_thread_loop = 0;
+static int mc_receive_thread_running = 0;
+static pthread_t mc_receive_thread_id;
 
 static metric_map_t metric_map_default[] =
-{ /*---------------+-------------+-----------+-------------+------+-----*
-   * ganglia_name  ! type        ! type_inst ! data_source ! type ! idx *
-   *---------------+-------------+-----------+-------------+------+-----*/
-  { "load_one",     "load",       "",         "shortterm",     -1,   -1 },
-  { "load_five",    "load",       "",         "midterm",       -1,   -1 },
-  { "load_fifteen", "load",       "",         "longterm",      -1,   -1 },
-  { "cpu_user",     "cpu",        "user",     "value",         -1,   -1 },
-  { "cpu_system",   "cpu",        "system",   "value",         -1,   -1 },
-  { "cpu_idle",     "cpu",        "idle",     "value",         -1,   -1 },
-  { "cpu_nice",     "cpu",        "nice",     "value",         -1,   -1 },
-  { "cpu_wio",      "cpu",        "wait",     "value",         -1,   -1 },
-  { "mem_free",     "memory",     "free",     "value",         -1,   -1 },
-  { "mem_shared",   "memory",     "shared",   "value",         -1,   -1 },
-  { "mem_buffers",  "memory",     "buffered", "value",         -1,   -1 },
-  { "mem_cached",   "memory",     "cached",   "value",         -1,   -1 },
-  { "mem_total",    "memory",     "total",    "value",         -1,   -1 },
-  { "bytes_in",     "if_octets",  "",         "rx",            -1,   -1 },
-  { "bytes_out",    "if_octets",  "",         "tx",            -1,   -1 },
-  { "pkts_in",      "if_packets", "",         "rx",            -1,   -1 },
-  { "pkts_out",     "if_packets", "",         "tx",            -1,   -1 }
-};
-static size_t metric_map_len_default = STATIC_ARRAY_SIZE (metric_map_default);
+    {/*---------------+-------------+-----------+-------------+------+-----*
+      * ganglia_name  ! type        ! type_inst ! data_source ! type ! idx *
+      *---------------+-------------+-----------+-------------+------+-----*/
+     {"load_one", "load", "", "shortterm", -1, -1},
+     {"load_five", "load", "", "midterm", -1, -1},
+     {"load_fifteen", "load", "", "longterm", -1, -1},
+     {"cpu_user", "cpu", "user", "value", -1, -1},
+     {"cpu_system", "cpu", "system", "value", -1, -1},
+     {"cpu_idle", "cpu", "idle", "value", -1, -1},
+     {"cpu_nice", "cpu", "nice", "value", -1, -1},
+     {"cpu_wio", "cpu", "wait", "value", -1, -1},
+     {"mem_free", "memory", "free", "value", -1, -1},
+     {"mem_shared", "memory", "shared", "value", -1, -1},
+     {"mem_buffers", "memory", "buffered", "value", -1, -1},
+     {"mem_cached", "memory", "cached", "value", -1, -1},
+     {"mem_total", "memory", "total", "value", -1, -1},
+     {"bytes_in", "if_octets", "", "rx", -1, -1},
+     {"bytes_out", "if_octets", "", "tx", -1, -1},
+     {"pkts_in", "if_packets", "", "rx", -1, -1},
+     {"pkts_out", "if_packets", "", "tx", -1, -1}};
+static size_t metric_map_len_default = STATIC_ARRAY_SIZE(metric_map_default);
 
 static metric_map_t *metric_map = NULL;
-static size_t        metric_map_len = 0;
+static size_t metric_map_len = 0;
 
-static c_avl_tree_t   *staging_tree;
+static c_avl_tree_t *staging_tree;
 static pthread_mutex_t staging_lock = PTHREAD_MUTEX_INITIALIZER;
 
-static metric_map_t *metric_lookup (const char *key) /* {{{ */
+static metric_map_t *metric_lookup(const char *key) /* {{{ */
 {
   metric_map_t *map;
   size_t map_len;
@@ -142,17 +138,16 @@ static metric_map_t *metric_lookup (const char *key) /* {{{ */
   map = metric_map;
   map_len = metric_map_len;
   for (i = 0; i < map_len; i++)
-    if (strcmp (map[i].ganglia_name, key) == 0)
+    if (strcmp(map[i].ganglia_name, key) == 0)
       break;
 
   /* .. and fall back to the built-in table if nothing is found. */
-  if (i >= map_len)
-  {
+  if (i >= map_len) {
     map = metric_map_default;
     map_len = metric_map_len_default;
 
     for (i = 0; i < map_len; i++)
-      if (strcmp (map[i].ganglia_name, key) == 0)
+      if (strcmp(map[i].ganglia_name, key) == 0)
         break;
   }
 
@@ -164,38 +159,32 @@ static metric_map_t *metric_lookup (const char *key) /* {{{ */
   {
     const data_set_t *ds;
 
-    ds = plugin_get_ds (map[i].type);
-    if (ds == NULL)
-    {
-      WARNING ("gmond plugin: Type not defined: %s", map[i].type);
+    ds = plugin_get_ds(map[i].type);
+    if (ds == NULL) {
+      WARNING("gmond plugin: Type not defined: %s", map[i].type);
       return (NULL);
     }
 
-    if ((map[i].ds_name == NULL) && (ds->ds_num != 1))
-    {
-      WARNING ("gmond plugin: No data source name defined for metric %s, "
-          "but type %s has more than one data source.",
-          map[i].ganglia_name, map[i].type);
+    if ((map[i].ds_name == NULL) && (ds->ds_num != 1)) {
+      WARNING("gmond plugin: No data source name defined for metric %s, "
+              "but type %s has more than one data source.",
+              map[i].ganglia_name, map[i].type);
       return (NULL);
     }
 
-    if (map[i].ds_name == NULL)
-    {
+    if (map[i].ds_name == NULL) {
       map[i].ds_index = 0;
-    }
-    else
-    {
+    } else {
       size_t j;
 
       for (j = 0; j < ds->ds_num; j++)
-        if (strcasecmp (ds->ds[j].name, map[i].ds_name) == 0)
+        if (strcasecmp(ds->ds[j].name, map[i].ds_name) == 0)
           break;
 
-      if (j >= ds->ds_num)
-      {
-        WARNING ("gmond plugin: There is no data source "
-            "named `%s' in type `%s'.",
-            map[i].ds_name, ds->type);
+      if (j >= ds->ds_num) {
+        WARNING("gmond plugin: There is no data source "
+                "named `%s' in type `%s'.",
+                map[i].ds_name, ds->type);
         return (NULL);
       }
       map[i].ds_index = j;
@@ -207,181 +196,158 @@ static metric_map_t *metric_lookup (const char *key) /* {{{ */
   return (map + i);
 } /* }}} metric_map_t *metric_lookup */
 
-static int create_sockets (socket_entry_t **ret_sockets, /* {{{ */
-    size_t *ret_sockets_num,
-    const char *node, const char *service, int listen)
-{
+static int create_sockets(socket_entry_t **ret_sockets, /* {{{ */
+                          size_t *ret_sockets_num, const char *node,
+                          const char *service, int listen) {
   struct addrinfo *ai_list;
-  int              ai_return;
+  int ai_return;
 
   socket_entry_t *sockets = NULL;
-  size_t          sockets_num = 0;
+  size_t sockets_num = 0;
 
   int status;
 
   if (*ret_sockets != NULL)
     return (EINVAL);
 
-  struct addrinfo ai_hints = {
-    .ai_family = AF_UNSPEC,
-    .ai_flags = AI_ADDRCONFIG | AI_PASSIVE,
-    .ai_protocol = IPPROTO_UDP,
-    .ai_socktype = SOCK_DGRAM
-  };
+  struct addrinfo ai_hints = {.ai_family = AF_UNSPEC,
+                              .ai_flags = AI_ADDRCONFIG | AI_PASSIVE,
+                              .ai_protocol = IPPROTO_UDP,
+                              .ai_socktype = SOCK_DGRAM};
 
-  ai_return = getaddrinfo (node, service, &ai_hints, &ai_list);
-  if (ai_return != 0)
-  {
+  ai_return = getaddrinfo(node, service, &ai_hints, &ai_list);
+  if (ai_return != 0) {
     char errbuf[1024];
-    ERROR ("gmond plugin: getaddrinfo (%s, %s) failed: %s",
-        (node == NULL) ? "(null)" : node,
-        (service == NULL) ? "(null)" : service,
-        (ai_return == EAI_SYSTEM)
-        ? sstrerror (errno, errbuf, sizeof (errbuf))
-        : gai_strerror (ai_return));
+    ERROR("gmond plugin: getaddrinfo (%s, %s) failed: %s",
+          (node == NULL) ? "(null)" : node,
+          (service == NULL) ? "(null)" : service,
+          (ai_return == EAI_SYSTEM) ? sstrerror(errno, errbuf, sizeof(errbuf))
+                                    : gai_strerror(ai_return));
     return (-1);
   }
 
-  for (struct addrinfo *ai_ptr = ai_list; ai_ptr != NULL; ai_ptr = ai_ptr->ai_next) /* {{{ */
+  for (struct addrinfo *ai_ptr = ai_list; ai_ptr != NULL;
+       ai_ptr = ai_ptr->ai_next) /* {{{ */
   {
     socket_entry_t *tmp;
 
-    tmp = realloc (sockets, (sockets_num + 1) * sizeof (*sockets));
-    if (tmp == NULL)
-    {
-      ERROR ("gmond plugin: realloc failed.");
+    tmp = realloc(sockets, (sockets_num + 1) * sizeof(*sockets));
+    if (tmp == NULL) {
+      ERROR("gmond plugin: realloc failed.");
       continue;
     }
     sockets = tmp;
 
-    sockets[sockets_num].fd = socket (ai_ptr->ai_family, ai_ptr->ai_socktype,
-        ai_ptr->ai_protocol);
-    if (sockets[sockets_num].fd < 0)
-    {
+    sockets[sockets_num].fd =
+        socket(ai_ptr->ai_family, ai_ptr->ai_socktype, ai_ptr->ai_protocol);
+    if (sockets[sockets_num].fd < 0) {
       char errbuf[1024];
-      ERROR ("gmond plugin: socket failed: %s",
-          sstrerror (errno, errbuf, sizeof (errbuf)));
+      ERROR("gmond plugin: socket failed: %s",
+            sstrerror(errno, errbuf, sizeof(errbuf)));
       continue;
     }
 
-    assert (sizeof (sockets[sockets_num].addr) >= ai_ptr->ai_addrlen);
-    memcpy (&sockets[sockets_num].addr, ai_ptr->ai_addr, ai_ptr->ai_addrlen);
+    assert(sizeof(sockets[sockets_num].addr) >= ai_ptr->ai_addrlen);
+    memcpy(&sockets[sockets_num].addr, ai_ptr->ai_addr, ai_ptr->ai_addrlen);
     sockets[sockets_num].addrlen = ai_ptr->ai_addrlen;
 
     /* Sending socket: Open only one socket and don't bind it. */
-    if (listen == 0)
-    {
+    if (listen == 0) {
       sockets_num++;
       break;
-    }
-    else
-    {
+    } else {
       int yes = 1;
 
-      status = setsockopt (sockets[sockets_num].fd, SOL_SOCKET, SO_REUSEADDR,
-          (void *) &yes, sizeof (yes));
-      if (status != 0)
-      {
+      status = setsockopt(sockets[sockets_num].fd, SOL_SOCKET, SO_REUSEADDR,
+                          (void *)&yes, sizeof(yes));
+      if (status != 0) {
         char errbuf[1024];
-        WARNING ("gmond plugin: setsockopt(2) failed: %s",
-                 sstrerror (errno, errbuf, sizeof (errbuf)));
+        WARNING("gmond plugin: setsockopt(2) failed: %s",
+                sstrerror(errno, errbuf, sizeof(errbuf)));
       }
     }
 
-    status = bind (sockets[sockets_num].fd, ai_ptr->ai_addr, ai_ptr->ai_addrlen);
-    if (status != 0)
-    {
+    status = bind(sockets[sockets_num].fd, ai_ptr->ai_addr, ai_ptr->ai_addrlen);
+    if (status != 0) {
       char errbuf[1024];
-      ERROR ("gmond plugin: bind failed: %s",
-          sstrerror (errno, errbuf, sizeof (errbuf)));
-      close (sockets[sockets_num].fd);
+      ERROR("gmond plugin: bind failed: %s",
+            sstrerror(errno, errbuf, sizeof(errbuf)));
+      close(sockets[sockets_num].fd);
       continue;
     }
 
-    if (ai_ptr->ai_family == AF_INET)
-    {
+    if (ai_ptr->ai_family == AF_INET) {
       struct sockaddr_in *addr;
       int loop;
 
-      addr = (struct sockaddr_in *) ai_ptr->ai_addr;
+      addr = (struct sockaddr_in *)ai_ptr->ai_addr;
 
-      if (!IN_MULTICAST (ntohl (addr->sin_addr.s_addr)))
-      {
+      if (!IN_MULTICAST(ntohl(addr->sin_addr.s_addr))) {
         sockets_num++;
         continue;
       }
 
       loop = 1;
-      status = setsockopt (sockets[sockets_num].fd, IPPROTO_IP, IP_MULTICAST_LOOP,
-          (void *) &loop, sizeof (loop));
-      if (status != 0)
-      {
+      status = setsockopt(sockets[sockets_num].fd, IPPROTO_IP,
+                          IP_MULTICAST_LOOP, (void *)&loop, sizeof(loop));
+      if (status != 0) {
         char errbuf[1024];
-        WARNING ("gmond plugin: setsockopt(2) failed: %s",
-                 sstrerror (errno, errbuf, sizeof (errbuf)));
+        WARNING("gmond plugin: setsockopt(2) failed: %s",
+                sstrerror(errno, errbuf, sizeof(errbuf)));
       }
 
-      struct ip_mreq mreq = {
-        .imr_multiaddr.s_addr = addr->sin_addr.s_addr,
-        .imr_interface.s_addr = htonl (INADDR_ANY)
-      };
+      struct ip_mreq mreq = {.imr_multiaddr.s_addr = addr->sin_addr.s_addr,
+                             .imr_interface.s_addr = htonl(INADDR_ANY)};
 
-      status = setsockopt (sockets[sockets_num].fd, IPPROTO_IP, IP_ADD_MEMBERSHIP,
-          (void *) &mreq, sizeof (mreq));
-      if (status != 0)
-      {
+      status = setsockopt(sockets[sockets_num].fd, IPPROTO_IP,
+                          IP_ADD_MEMBERSHIP, (void *)&mreq, sizeof(mreq));
+      if (status != 0) {
         char errbuf[1024];
-        WARNING ("gmond plugin: setsockopt(2) failed: %s",
-                 sstrerror (errno, errbuf, sizeof (errbuf)));
+        WARNING("gmond plugin: setsockopt(2) failed: %s",
+                sstrerror(errno, errbuf, sizeof(errbuf)));
       }
     } /* if (ai_ptr->ai_family == AF_INET) */
-    else if (ai_ptr->ai_family == AF_INET6)
-    {
+    else if (ai_ptr->ai_family == AF_INET6) {
       struct sockaddr_in6 *addr;
       int loop;
 
-      addr = (struct sockaddr_in6 *) ai_ptr->ai_addr;
+      addr = (struct sockaddr_in6 *)ai_ptr->ai_addr;
 
-      if (!IN6_IS_ADDR_MULTICAST (&addr->sin6_addr))
-      {
+      if (!IN6_IS_ADDR_MULTICAST(&addr->sin6_addr)) {
         sockets_num++;
         continue;
       }
 
       loop = 1;
-      status = setsockopt (sockets[sockets_num].fd, IPPROTO_IPV6, IPV6_MULTICAST_LOOP,
-          (void *) &loop, sizeof (loop));
-      if (status != 0)
-      {
+      status = setsockopt(sockets[sockets_num].fd, IPPROTO_IPV6,
+                          IPV6_MULTICAST_LOOP, (void *)&loop, sizeof(loop));
+      if (status != 0) {
         char errbuf[1024];
-        WARNING ("gmond plugin: setsockopt(2) failed: %s",
-                 sstrerror (errno, errbuf, sizeof (errbuf)));
+        WARNING("gmond plugin: setsockopt(2) failed: %s",
+                sstrerror(errno, errbuf, sizeof(errbuf)));
       }
 
       struct ipv6_mreq mreq = {
-        .ipv6mr_interface = 0 /* any */
+          .ipv6mr_interface = 0 /* any */
       };
 
-      memcpy (&mreq.ipv6mr_multiaddr,
-          &addr->sin6_addr, sizeof (addr->sin6_addr));
-      status = setsockopt (sockets[sockets_num].fd, IPPROTO_IPV6, IPV6_ADD_MEMBERSHIP,
-          (void *) &mreq, sizeof (mreq));
-      if (status != 0)
-      {
+      memcpy(&mreq.ipv6mr_multiaddr, &addr->sin6_addr, sizeof(addr->sin6_addr));
+      status = setsockopt(sockets[sockets_num].fd, IPPROTO_IPV6,
+                          IPV6_ADD_MEMBERSHIP, (void *)&mreq, sizeof(mreq));
+      if (status != 0) {
         char errbuf[1024];
-        WARNING ("gmond plugin: setsockopt(2) failed: %s",
-                 sstrerror (errno, errbuf, sizeof (errbuf)));
+        WARNING("gmond plugin: setsockopt(2) failed: %s",
+                sstrerror(errno, errbuf, sizeof(errbuf)));
       }
     } /* if (ai_ptr->ai_family == AF_INET6) */
 
     sockets_num++;
   } /* }}} for (ai_ptr = ai_list; ai_ptr != NULL; ai_ptr = ai_ptr->ai_next) */
 
-  freeaddrinfo (ai_list);
+  freeaddrinfo(ai_list);
 
-  if (sockets_num == 0)
-  {
-    sfree (sockets);
+  if (sockets_num == 0) {
+    sfree(sockets);
     return (-1);
   }
 
@@ -390,66 +356,60 @@ static int create_sockets (socket_entry_t **ret_sockets, /* {{{ */
   return (0);
 } /* }}} int create_sockets */
 
-static int request_meta_data (const char *host, const char *name) /* {{{ */
+static int request_meta_data(const char *host, const char *name) /* {{{ */
 {
-  Ganglia_metadata_msg msg = { 0 };
-  char buffer[BUFF_SIZE] = { 0 };
+  Ganglia_metadata_msg msg = {0};
+  char buffer[BUFF_SIZE] = {0};
   unsigned int buffer_size;
   XDR xdr;
 
   msg.id = gmetadata_request;
-  msg.Ganglia_metadata_msg_u.grequest.metric_id.host = strdup (host);
-  msg.Ganglia_metadata_msg_u.grequest.metric_id.name = strdup (name);
+  msg.Ganglia_metadata_msg_u.grequest.metric_id.host = strdup(host);
+  msg.Ganglia_metadata_msg_u.grequest.metric_id.name = strdup(name);
 
-  if ((msg.Ganglia_metadata_msg_u.grequest.metric_id.host == NULL)
-      || (msg.Ganglia_metadata_msg_u.grequest.metric_id.name == NULL))
-  {
-    sfree (msg.Ganglia_metadata_msg_u.grequest.metric_id.host);
-    sfree (msg.Ganglia_metadata_msg_u.grequest.metric_id.name);
+  if ((msg.Ganglia_metadata_msg_u.grequest.metric_id.host == NULL) ||
+      (msg.Ganglia_metadata_msg_u.grequest.metric_id.name == NULL)) {
+    sfree(msg.Ganglia_metadata_msg_u.grequest.metric_id.host);
+    sfree(msg.Ganglia_metadata_msg_u.grequest.metric_id.name);
     return (-1);
   }
 
-  xdrmem_create (&xdr, buffer, sizeof (buffer), XDR_ENCODE);
+  xdrmem_create(&xdr, buffer, sizeof(buffer), XDR_ENCODE);
 
-  if (!xdr_Ganglia_metadata_msg (&xdr, &msg))
-  {
-    sfree (msg.Ganglia_metadata_msg_u.grequest.metric_id.host);
-    sfree (msg.Ganglia_metadata_msg_u.grequest.metric_id.name);
+  if (!xdr_Ganglia_metadata_msg(&xdr, &msg)) {
+    sfree(msg.Ganglia_metadata_msg_u.grequest.metric_id.host);
+    sfree(msg.Ganglia_metadata_msg_u.grequest.metric_id.name);
     return (-1);
   }
 
-  buffer_size = xdr_getpos (&xdr);
+  buffer_size = xdr_getpos(&xdr);
 
-  DEBUG ("gmond plugin: Requesting meta data for %s/%s.",
-      host, name);
+  DEBUG("gmond plugin: Requesting meta data for %s/%s.", host, name);
 
-  pthread_mutex_lock (&mc_send_sockets_lock);
-  for (size_t i = 0; i < mc_send_sockets_num; i++)
-  {
-    ssize_t status = sendto (mc_send_sockets[i].fd, buffer, (size_t) buffer_size,
-        /* flags = */ 0,
-        (struct sockaddr *) &mc_send_sockets[i].addr,
-        mc_send_sockets[i].addrlen);
-    if (status == -1)
-    {
+  pthread_mutex_lock(&mc_send_sockets_lock);
+  for (size_t i = 0; i < mc_send_sockets_num; i++) {
+    ssize_t status =
+        sendto(mc_send_sockets[i].fd, buffer, (size_t)buffer_size,
+               /* flags = */ 0, (struct sockaddr *)&mc_send_sockets[i].addr,
+               mc_send_sockets[i].addrlen);
+    if (status == -1) {
       char errbuf[1024];
-      ERROR ("gmond plugin: sendto(2) failed: %s",
-             sstrerror (errno, errbuf, sizeof (errbuf)));
+      ERROR("gmond plugin: sendto(2) failed: %s",
+            sstrerror(errno, errbuf, sizeof(errbuf)));
       continue;
     }
   }
-  pthread_mutex_unlock (&mc_send_sockets_lock);
+  pthread_mutex_unlock(&mc_send_sockets_lock);
 
-  sfree (msg.Ganglia_metadata_msg_u.grequest.metric_id.host);
-  sfree (msg.Ganglia_metadata_msg_u.grequest.metric_id.name);
+  sfree(msg.Ganglia_metadata_msg_u.grequest.metric_id.host);
+  sfree(msg.Ganglia_metadata_msg_u.grequest.metric_id.name);
   return (0);
 } /* }}} int request_meta_data */
 
-static staging_entry_t *staging_entry_get (const char *host, /* {{{ */
-    const char *name,
-    const char *type, const char *type_instance,
-    int values_len)
-{
+static staging_entry_t *staging_entry_get(const char *host, /* {{{ */
+                                          const char *name, const char *type,
+                                          const char *type_instance,
+                                          int values_len) {
   char key[2 * DATA_MAX_NAME_LEN];
   staging_entry_t *se;
   int status;
@@ -457,84 +417,76 @@ static staging_entry_t *staging_entry_get (const char *host, /* {{{ */
   if (staging_tree == NULL)
     return (NULL);
 
-  ssnprintf (key, sizeof (key), "%s/%s/%s", host, type,
-      (type_instance != NULL) ? type_instance : "");
+  ssnprintf(key, sizeof(key), "%s/%s/%s", host, type,
+            (type_instance != NULL) ? type_instance : "");
 
   se = NULL;
-  status = c_avl_get (staging_tree, key, (void *) &se);
+  status = c_avl_get(staging_tree, key, (void *)&se);
   if (status == 0)
     return (se);
 
   /* insert new entry */
-  se = calloc (1, sizeof (*se));
+  se = calloc(1, sizeof(*se));
   if (se == NULL)
     return (NULL);
 
-  sstrncpy (se->key, key, sizeof (se->key));
+  sstrncpy(se->key, key, sizeof(se->key));
   se->flags = 0;
 
-  se->vl.values = (value_t *) calloc (values_len, sizeof (*se->vl.values));
-  if (se->vl.values == NULL)
-  {
-    sfree (se);
+  se->vl.values = (value_t *)calloc(values_len, sizeof(*se->vl.values));
+  if (se->vl.values == NULL) {
+    sfree(se);
     return (NULL);
   }
   se->vl.values_len = values_len;
 
   se->vl.time = 0;
   se->vl.interval = 0;
-  sstrncpy (se->vl.host, host, sizeof (se->vl.host));
-  sstrncpy (se->vl.plugin, "gmond", sizeof (se->vl.plugin));
-  sstrncpy (se->vl.type, type, sizeof (se->vl.type));
+  sstrncpy(se->vl.host, host, sizeof(se->vl.host));
+  sstrncpy(se->vl.plugin, "gmond", sizeof(se->vl.plugin));
+  sstrncpy(se->vl.type, type, sizeof(se->vl.type));
   if (type_instance != NULL)
-    sstrncpy (se->vl.type_instance, type_instance,
-        sizeof (se->vl.type_instance));
+    sstrncpy(se->vl.type_instance, type_instance, sizeof(se->vl.type_instance));
 
-  status = c_avl_insert (staging_tree, se->key, se);
-  if (status != 0)
-  {
-    ERROR ("gmond plugin: c_avl_insert failed.");
-    sfree (se->vl.values);
-    sfree (se);
+  status = c_avl_insert(staging_tree, se->key, se);
+  if (status != 0) {
+    ERROR("gmond plugin: c_avl_insert failed.");
+    sfree(se->vl.values);
+    sfree(se);
     return (NULL);
   }
 
   return (se);
 } /* }}} staging_entry_t *staging_entry_get */
 
-static int staging_entry_update (const char *host, const char *name, /* {{{ */
-    const char *type, const char *type_instance,
-    size_t ds_index, int ds_type, value_t value)
-{
+static int staging_entry_update(const char *host, const char *name, /* {{{ */
+                                const char *type, const char *type_instance,
+                                size_t ds_index, int ds_type, value_t value) {
   const data_set_t *ds;
   staging_entry_t *se;
 
-  ds = plugin_get_ds (type);
-  if (ds == NULL)
-  {
-    ERROR ("gmond plugin: Looking up type %s failed.", type);
+  ds = plugin_get_ds(type);
+  if (ds == NULL) {
+    ERROR("gmond plugin: Looking up type %s failed.", type);
     return (-1);
   }
 
-  if (ds->ds_num <= ds_index)
-  {
-    ERROR ("gmond plugin: Invalid index %zu: %s has only %zu data source(s).",
-        ds_index, ds->type, ds->ds_num);
+  if (ds->ds_num <= ds_index) {
+    ERROR("gmond plugin: Invalid index %zu: %s has only %zu data source(s).",
+          ds_index, ds->type, ds->ds_num);
     return (-1);
   }
 
-  pthread_mutex_lock (&staging_lock);
+  pthread_mutex_lock(&staging_lock);
 
-  se = staging_entry_get (host, name, type, type_instance, ds->ds_num);
-  if (se == NULL)
-  {
-    pthread_mutex_unlock (&staging_lock);
-    ERROR ("gmond plugin: staging_entry_get failed.");
+  se = staging_entry_get(host, name, type, type_instance, ds->ds_num);
+  if (se == NULL) {
+    pthread_mutex_unlock(&staging_lock);
+    ERROR("gmond plugin: staging_entry_get failed.");
     return (-1);
   }
-  if (se->vl.values_len != ds->ds_num)
-  {
-    pthread_mutex_unlock (&staging_lock);
+  if (se->vl.values_len != ds->ds_num) {
+    pthread_mutex_unlock(&staging_lock);
     return (-1);
   }
 
@@ -547,38 +499,36 @@ static int staging_entry_update (const char *host, const char *name, /* {{{ */
   else if (ds_type == DS_TYPE_ABSOLUTE)
     se->vl.values[ds_index].absolute = value.absolute;
   else
-    assert (23 == 42);
+    assert(23 == 42);
 
   se->flags |= (0x01 << ds_index);
 
   /* Check if all data sources have been set. If not, return here. */
-  if (se->flags != ((0x01 << se->vl.values_len) - 1))
-  {
-    pthread_mutex_unlock (&staging_lock);
+  if (se->flags != ((0x01 << se->vl.values_len) - 1)) {
+    pthread_mutex_unlock(&staging_lock);
     return (0);
   }
 
   /* Check if the interval of this metric is known. If not, request meta data
    * and return. */
-  if (se->vl.interval == 0)
-  {
+  if (se->vl.interval == 0) {
     /* No meta data has been received for this metric yet. */
     se->flags = 0;
-    pthread_mutex_unlock (&staging_lock);
+    pthread_mutex_unlock(&staging_lock);
 
-    request_meta_data (host, name);
+    request_meta_data(host, name);
     return (0);
   }
 
-  plugin_dispatch_values (&se->vl);
+  plugin_dispatch_values(&se->vl);
 
   se->flags = 0;
-  pthread_mutex_unlock (&staging_lock);
+  pthread_mutex_unlock(&staging_lock);
 
   return (0);
 } /* }}} int staging_entry_update */
 
-static int mc_handle_value_msg (Ganglia_value_msg *msg) /* {{{ */
+static int mc_handle_value_msg(Ganglia_value_msg *msg) /* {{{ */
 {
   const char *host;
   const char *name;
@@ -592,297 +542,268 @@ static int mc_handle_value_msg (Ganglia_value_msg *msg) /* {{{ */
    * the value type, or return with an error. */
   switch (msg->id) /* {{{ */
   {
-    case gmetric_uint:
-    {
-      Ganglia_gmetric_uint msg_uint;
+  case gmetric_uint: {
+    Ganglia_gmetric_uint msg_uint;
 
-      msg_uint = msg->Ganglia_value_msg_u.gu_int;
+    msg_uint = msg->Ganglia_value_msg_u.gu_int;
 
-      host = msg_uint.metric_id.host;
-      name = msg_uint.metric_id.name;
-      value_counter.counter = (counter_t) msg_uint.ui;
-      value_gauge.gauge = (gauge_t) msg_uint.ui;
-      value_derive.derive = (derive_t) msg_uint.ui;
-      break;
-    }
+    host = msg_uint.metric_id.host;
+    name = msg_uint.metric_id.name;
+    value_counter.counter = (counter_t)msg_uint.ui;
+    value_gauge.gauge = (gauge_t)msg_uint.ui;
+    value_derive.derive = (derive_t)msg_uint.ui;
+    break;
+  }
 
-    case gmetric_string:
-    {
-      Ganglia_gmetric_string msg_string;
-      int status;
+  case gmetric_string: {
+    Ganglia_gmetric_string msg_string;
+    int status;
 
-      msg_string = msg->Ganglia_value_msg_u.gstr;
+    msg_string = msg->Ganglia_value_msg_u.gstr;
 
-      host = msg_string.metric_id.host;
-      name = msg_string.metric_id.name;
+    host = msg_string.metric_id.host;
+    name = msg_string.metric_id.name;
 
-      status = parse_value (msg_string.str, &value_derive, DS_TYPE_DERIVE);
-      if (status != 0)
-        value_derive.derive = -1;
+    status = parse_value(msg_string.str, &value_derive, DS_TYPE_DERIVE);
+    if (status != 0)
+      value_derive.derive = -1;
 
-      status = parse_value (msg_string.str, &value_gauge, DS_TYPE_GAUGE);
-      if (status != 0)
-        value_gauge.gauge = NAN;
+    status = parse_value(msg_string.str, &value_gauge, DS_TYPE_GAUGE);
+    if (status != 0)
+      value_gauge.gauge = NAN;
 
-      status = parse_value (msg_string.str, &value_counter, DS_TYPE_COUNTER);
-      if (status != 0)
-        value_counter.counter = 0;
+    status = parse_value(msg_string.str, &value_counter, DS_TYPE_COUNTER);
+    if (status != 0)
+      value_counter.counter = 0;
 
-      break;
-    }
+    break;
+  }
 
-    case gmetric_float:
-    {
-      Ganglia_gmetric_float msg_float;
+  case gmetric_float: {
+    Ganglia_gmetric_float msg_float;
 
-      msg_float = msg->Ganglia_value_msg_u.gf;
+    msg_float = msg->Ganglia_value_msg_u.gf;
 
-      host = msg_float.metric_id.host;
-      name = msg_float.metric_id.name;
-      value_counter.counter = (counter_t) msg_float.f;
-      value_gauge.gauge = (gauge_t) msg_float.f;
-      value_derive.derive = (derive_t) msg_float.f;
-      break;
-    }
+    host = msg_float.metric_id.host;
+    name = msg_float.metric_id.name;
+    value_counter.counter = (counter_t)msg_float.f;
+    value_gauge.gauge = (gauge_t)msg_float.f;
+    value_derive.derive = (derive_t)msg_float.f;
+    break;
+  }
 
-    case gmetric_double:
-    {
-      Ganglia_gmetric_double msg_double;
+  case gmetric_double: {
+    Ganglia_gmetric_double msg_double;
 
-      msg_double = msg->Ganglia_value_msg_u.gd;
+    msg_double = msg->Ganglia_value_msg_u.gd;
 
-      host = msg_double.metric_id.host;
-      name = msg_double.metric_id.name;
-      value_counter.counter = (counter_t) msg_double.d;
-      value_gauge.gauge = (gauge_t) msg_double.d;
-      value_derive.derive = (derive_t) msg_double.d;
-      break;
-    }
-    default:
-      DEBUG ("gmond plugin: Value type not handled: %i", msg->id);
-      return (-1);
+    host = msg_double.metric_id.host;
+    name = msg_double.metric_id.name;
+    value_counter.counter = (counter_t)msg_double.d;
+    value_gauge.gauge = (gauge_t)msg_double.d;
+    value_derive.derive = (derive_t)msg_double.d;
+    break;
+  }
+  default:
+    DEBUG("gmond plugin: Value type not handled: %i", msg->id);
+    return (-1);
   } /* }}} switch (msg->id) */
 
-  assert (host != NULL);
-  assert (name != NULL);
+  assert(host != NULL);
+  assert(name != NULL);
 
-  map = metric_lookup (name);
-  if (map != NULL)
-  {
+  map = metric_lookup(name);
+  if (map != NULL) {
     value_t val_copy;
 
-    if ((map->ds_type == DS_TYPE_COUNTER)
-        || (map->ds_type == DS_TYPE_ABSOLUTE))
+    if ((map->ds_type == DS_TYPE_COUNTER) || (map->ds_type == DS_TYPE_ABSOLUTE))
       val_copy = value_counter;
     else if (map->ds_type == DS_TYPE_GAUGE)
       val_copy = value_gauge;
     else if (map->ds_type == DS_TYPE_DERIVE)
       val_copy = value_derive;
     else
-      assert (23 == 42);
+      assert(23 == 42);
 
-    return (staging_entry_update (host, name,
-          map->type, map->type_instance,
-          map->ds_index, map->ds_type,
-          val_copy));
+    return (staging_entry_update(host, name, map->type, map->type_instance,
+                                 map->ds_index, map->ds_type, val_copy));
   }
 
-  DEBUG ("gmond plugin: Cannot find a translation for %s.", name);
+  DEBUG("gmond plugin: Cannot find a translation for %s.", name);
   return (-1);
 } /* }}} int mc_handle_value_msg */
 
-static int mc_handle_metadata_msg (Ganglia_metadata_msg *msg) /* {{{ */
+static int mc_handle_metadata_msg(Ganglia_metadata_msg *msg) /* {{{ */
 {
-  switch (msg->id)
-  {
-    case gmetadata_full:
-    {
-      Ganglia_metadatadef msg_meta;
-      staging_entry_t *se;
-      const data_set_t *ds;
-      metric_map_t *map;
-
-      msg_meta = msg->Ganglia_metadata_msg_u.gfull;
-
-      if (msg_meta.metric.tmax == 0)
-        return (-1);
-
-      map = metric_lookup (msg_meta.metric_id.name);
-      if (map == NULL)
-      {
-        DEBUG ("gmond plugin: Not handling meta data %s.",
-            msg_meta.metric_id.name);
-        return (0);
-      }
+  switch (msg->id) {
+  case gmetadata_full: {
+    Ganglia_metadatadef msg_meta;
+    staging_entry_t *se;
+    const data_set_t *ds;
+    metric_map_t *map;
 
-      ds = plugin_get_ds (map->type);
-      if (ds == NULL)
-      {
-        WARNING ("gmond plugin: Could not find data set %s.", map->type);
-        return (-1);
-      }
+    msg_meta = msg->Ganglia_metadata_msg_u.gfull;
 
-      DEBUG ("gmond plugin: Received meta data for %s/%s.",
-          msg_meta.metric_id.host, msg_meta.metric_id.name);
+    if (msg_meta.metric.tmax == 0)
+      return (-1);
 
-      pthread_mutex_lock (&staging_lock);
-      se = staging_entry_get (msg_meta.metric_id.host,
-          msg_meta.metric_id.name,
-          map->type, map->type_instance,
-          ds->ds_num);
-      if (se != NULL)
-        se->vl.interval = TIME_T_TO_CDTIME_T (msg_meta.metric.tmax);
-      pthread_mutex_unlock (&staging_lock);
-
-      if (se == NULL)
-      {
-        ERROR ("gmond plugin: staging_entry_get failed.");
-        return (-1);
-      }
+    map = metric_lookup(msg_meta.metric_id.name);
+    if (map == NULL) {
+      DEBUG("gmond plugin: Not handling meta data %s.",
+            msg_meta.metric_id.name);
+      return (0);
+    }
 
-      break;
+    ds = plugin_get_ds(map->type);
+    if (ds == NULL) {
+      WARNING("gmond plugin: Could not find data set %s.", map->type);
+      return (-1);
     }
 
-    default:
-    {
+    DEBUG("gmond plugin: Received meta data for %s/%s.",
+          msg_meta.metric_id.host, msg_meta.metric_id.name);
+
+    pthread_mutex_lock(&staging_lock);
+    se = staging_entry_get(msg_meta.metric_id.host, msg_meta.metric_id.name,
+                           map->type, map->type_instance, ds->ds_num);
+    if (se != NULL)
+      se->vl.interval = TIME_T_TO_CDTIME_T(msg_meta.metric.tmax);
+    pthread_mutex_unlock(&staging_lock);
+
+    if (se == NULL) {
+      ERROR("gmond plugin: staging_entry_get failed.");
       return (-1);
     }
+
+    break;
+  }
+
+  default: { return (-1); }
   }
 
   return (0);
 } /* }}} int mc_handle_metadata_msg */
 
-static int mc_handle_metric (void *buffer, size_t buffer_size) /* {{{ */
+static int mc_handle_metric(void *buffer, size_t buffer_size) /* {{{ */
 {
   XDR xdr;
   Ganglia_msg_formats format;
 
-  xdrmem_create (&xdr, buffer, buffer_size, XDR_DECODE);
+  xdrmem_create(&xdr, buffer, buffer_size, XDR_DECODE);
 
-  xdr_Ganglia_msg_formats (&xdr, &format);
-  xdr_setpos (&xdr, 0);
+  xdr_Ganglia_msg_formats(&xdr, &format);
+  xdr_setpos(&xdr, 0);
 
-  switch (format)
-  {
-    case gmetric_ushort:
-    case gmetric_short:
-    case gmetric_int:
-    case gmetric_uint:
-    case gmetric_string:
-    case gmetric_float:
-    case gmetric_double:
-    {
-      Ganglia_value_msg msg = { 0 };
-
-      if (xdr_Ganglia_value_msg (&xdr, &msg))
-        mc_handle_value_msg (&msg);
-      break;
-    }
+  switch (format) {
+  case gmetric_ushort:
+  case gmetric_short:
+  case gmetric_int:
+  case gmetric_uint:
+  case gmetric_string:
+  case gmetric_float:
+  case gmetric_double: {
+    Ganglia_value_msg msg = {0};
 
-    case gmetadata_full:
-    case gmetadata_request:
-    {
-      Ganglia_metadata_msg msg = { 0 };
-      if (xdr_Ganglia_metadata_msg (&xdr, &msg))
-        mc_handle_metadata_msg (&msg);
-      break;
-    }
+    if (xdr_Ganglia_value_msg(&xdr, &msg))
+      mc_handle_value_msg(&msg);
+    break;
+  }
 
-    default:
-      DEBUG ("gmond plugin: Unknown format: %i", format);
-      return (-1);
-  } /* switch (format) */
+  case gmetadata_full:
+  case gmetadata_request: {
+    Ganglia_metadata_msg msg = {0};
+    if (xdr_Ganglia_metadata_msg(&xdr, &msg))
+      mc_handle_metadata_msg(&msg);
+    break;
+  }
 
+  default:
+    DEBUG("gmond plugin: Unknown format: %i", format);
+    return (-1);
+  } /* switch (format) */
 
   return (0);
 } /* }}} int mc_handle_metric */
 
-static int mc_handle_socket (struct pollfd *p) /* {{{ */
+static int mc_handle_socket(struct pollfd *p) /* {{{ */
 {
   char buffer[BUFF_SIZE];
   ssize_t buffer_size;
 
-  if ((p->revents & (POLLIN | POLLPRI)) == 0)
-  {
+  if ((p->revents & (POLLIN | POLLPRI)) == 0) {
     p->revents = 0;
     return (-1);
   }
 
-  buffer_size = recv (p->fd, buffer, sizeof (buffer), /* flags = */ 0);
-  if (buffer_size <= 0)
-  {
+  buffer_size = recv(p->fd, buffer, sizeof(buffer), /* flags = */ 0);
+  if (buffer_size <= 0) {
     char errbuf[1024];
-    ERROR ("gmond plugin: recv failed: %s",
-        sstrerror (errno, errbuf, sizeof (errbuf)));
+    ERROR("gmond plugin: recv failed: %s",
+          sstrerror(errno, errbuf, sizeof(errbuf)));
     p->revents = 0;
     return (-1);
   }
 
-  mc_handle_metric (buffer, (size_t) buffer_size);
+  mc_handle_metric(buffer, (size_t)buffer_size);
   return (0);
 } /* }}} int mc_handle_socket */
 
-static void *mc_receive_thread (void *arg) /* {{{ */
+static void *mc_receive_thread(void *arg) /* {{{ */
 {
   socket_entry_t *mc_receive_socket_entries;
   int status;
 
   mc_receive_socket_entries = NULL;
-  status = create_sockets (&mc_receive_socket_entries, &mc_receive_sockets_num,
+  status = create_sockets(
+      &mc_receive_socket_entries, &mc_receive_sockets_num,
       (mc_receive_group != NULL) ? mc_receive_group : MC_RECEIVE_GROUP_DEFAULT,
       (mc_receive_port != NULL) ? mc_receive_port : MC_RECEIVE_PORT_DEFAULT,
       /* listen = */ 1);
-  if (status != 0)
-  {
-    ERROR ("gmond plugin: create_sockets failed.");
-    return ((void *) -1);
+  if (status != 0) {
+    ERROR("gmond plugin: create_sockets failed.");
+    return ((void *)-1);
   }
 
-  mc_receive_sockets = (struct pollfd *) calloc (mc_receive_sockets_num,
-      sizeof (*mc_receive_sockets));
-  if (mc_receive_sockets == NULL)
-  {
-    ERROR ("gmond plugin: calloc failed.");
+  mc_receive_sockets = (struct pollfd *)calloc(mc_receive_sockets_num,
+                                               sizeof(*mc_receive_sockets));
+  if (mc_receive_sockets == NULL) {
+    ERROR("gmond plugin: calloc failed.");
     for (size_t i = 0; i < mc_receive_sockets_num; i++)
-      close (mc_receive_socket_entries[i].fd);
-    free (mc_receive_socket_entries);
+      close(mc_receive_socket_entries[i].fd);
+    free(mc_receive_socket_entries);
     mc_receive_socket_entries = NULL;
     mc_receive_sockets_num = 0;
-    return ((void *) -1);
+    return ((void *)-1);
   }
 
-  for (size_t i = 0; i < mc_receive_sockets_num; i++)
-  {
+  for (size_t i = 0; i < mc_receive_sockets_num; i++) {
     mc_receive_sockets[i].fd = mc_receive_socket_entries[i].fd;
     mc_receive_sockets[i].events = POLLIN | POLLPRI;
     mc_receive_sockets[i].revents = 0;
   }
 
-  while (mc_receive_thread_loop != 0)
-  {
-    status = poll (mc_receive_sockets, mc_receive_sockets_num, -1);
-    if (status <= 0)
-    {
+  while (mc_receive_thread_loop != 0) {
+    status = poll(mc_receive_sockets, mc_receive_sockets_num, -1);
+    if (status <= 0) {
       char errbuf[1024];
       if (errno == EINTR)
         continue;
-      ERROR ("gmond plugin: poll failed: %s",
-          sstrerror (errno, errbuf, sizeof (errbuf)));
+      ERROR("gmond plugin: poll failed: %s",
+            sstrerror(errno, errbuf, sizeof(errbuf)));
       break;
     }
 
-    for (size_t i = 0; i < mc_receive_sockets_num; i++)
-    {
+    for (size_t i = 0; i < mc_receive_sockets_num; i++) {
       if (mc_receive_sockets[i].revents != 0)
-        mc_handle_socket (mc_receive_sockets + i);
+        mc_handle_socket(mc_receive_sockets + i);
     }
   } /* while (mc_receive_thread_loop != 0) */
 
-  free (mc_receive_socket_entries);
-  return ((void *) 0);
+  free(mc_receive_socket_entries);
+  return ((void *)0);
 } /* }}} void *mc_receive_thread */
 
-static int mc_receive_thread_start (void) /* {{{ */
+static int mc_receive_thread_start(void) /* {{{ */
 {
   int status;
 
@@ -891,11 +812,11 @@ static int mc_receive_thread_start (void) /* {{{ */
 
   mc_receive_thread_loop = 1;
 
-  status = plugin_thread_create (&mc_receive_thread_id, /* attr = */ NULL,
-      mc_receive_thread, /* args = */ NULL, "gmond recv");
-  if (status != 0)
-  {
-    ERROR ("gmond plugin: Starting receive thread failed.");
+  status =
+      plugin_thread_create(&mc_receive_thread_id, /* attr = */ NULL,
+                           mc_receive_thread, /* args = */ NULL, "gmond recv");
+  if (status != 0) {
+    ERROR("gmond plugin: Starting receive thread failed.");
     mc_receive_thread_loop = 0;
     return (-1);
   }
@@ -904,17 +825,17 @@ static int mc_receive_thread_start (void) /* {{{ */
   return (0);
 } /* }}} int start_receive_thread */
 
-static int mc_receive_thread_stop (void) /* {{{ */
+static int mc_receive_thread_stop(void) /* {{{ */
 {
   if (mc_receive_thread_running == 0)
     return (-1);
 
   mc_receive_thread_loop = 0;
 
-  INFO ("gmond plugin: Stopping receive thread.");
-  pthread_kill (mc_receive_thread_id, SIGTERM);
-  pthread_join (mc_receive_thread_id, /* return value = */ NULL);
-  memset (&mc_receive_thread_id, 0, sizeof (mc_receive_thread_id));
+  INFO("gmond plugin: Stopping receive thread.");
+  pthread_kill(mc_receive_thread_id, SIGTERM);
+  pthread_join(mc_receive_thread_id, /* return value = */ NULL);
+  memset(&mc_receive_thread_id, 0, sizeof(mc_receive_thread_id));
 
   mc_receive_thread_running = 0;
 
@@ -933,85 +854,77 @@ static int mc_receive_thread_stop (void) /* {{{ */
  *   </Metric>
  * </Plugin>
  */
-static int gmond_config_set_string (oconfig_item_t *ci, char **str) /* {{{ */
+static int gmond_config_set_string(oconfig_item_t *ci, char **str) /* {{{ */
 {
   char *tmp;
 
-  if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING))
-  {
-    WARNING ("gmond plugin: The `%s' option needs "
-        "exactly one string argument.", ci->key);
+  if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING)) {
+    WARNING("gmond plugin: The `%s' option needs "
+            "exactly one string argument.",
+            ci->key);
     return (-1);
   }
 
-  tmp = strdup (ci->values[0].value.string);
-  if (tmp == NULL)
-  {
-    ERROR ("gmond plugin: strdup failed.");
+  tmp = strdup(ci->values[0].value.string);
+  if (tmp == NULL) {
+    ERROR("gmond plugin: strdup failed.");
     return (-1);
   }
 
-  sfree (*str);
+  sfree(*str);
   *str = tmp;
   return (0);
 } /* }}} int gmond_config_set_string */
 
-static int gmond_config_add_metric (oconfig_item_t *ci) /* {{{ */
+static int gmond_config_add_metric(oconfig_item_t *ci) /* {{{ */
 {
   metric_map_t *map;
 
-  if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING))
-  {
-    WARNING ("gmond plugin: `Metric' blocks need "
-        "exactly one string argument.");
+  if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING)) {
+    WARNING("gmond plugin: `Metric' blocks need "
+            "exactly one string argument.");
     return (-1);
   }
 
-  map = realloc (metric_map, (metric_map_len + 1) * sizeof (*metric_map));
-  if (map == NULL)
-  {
-    ERROR ("gmond plugin: realloc failed.");
+  map = realloc(metric_map, (metric_map_len + 1) * sizeof(*metric_map));
+  if (map == NULL) {
+    ERROR("gmond plugin: realloc failed.");
     return (-1);
   }
   metric_map = map;
   map = metric_map + metric_map_len;
 
-  memset (map, 0, sizeof (*map));
+  memset(map, 0, sizeof(*map));
   map->type = NULL;
   map->type_instance = NULL;
   map->ds_name = NULL;
   map->ds_type = -1;
   map->ds_index = -1;
 
-  map->ganglia_name = strdup (ci->values[0].value.string);
-  if (map->ganglia_name == NULL)
-  {
-    ERROR ("gmond plugin: strdup failed.");
+  map->ganglia_name = strdup(ci->values[0].value.string);
+  if (map->ganglia_name == NULL) {
+    ERROR("gmond plugin: strdup failed.");
     return (-1);
   }
 
-  for (int i = 0; i < ci->children_num; i++)
-  {
+  for (int i = 0; i < ci->children_num; i++) {
     oconfig_item_t *child = ci->children + i;
-    if (strcasecmp ("Type", child->key) == 0)
-      gmond_config_set_string (child, &map->type);
-    else if (strcasecmp ("TypeInstance", child->key) == 0)
-      gmond_config_set_string (child, &map->type_instance);
-    else if (strcasecmp ("DataSource", child->key) == 0)
-      gmond_config_set_string (child, &map->ds_name);
-    else
-    {
-      WARNING ("gmond plugin: Unknown configuration option `%s' ignored.",
-          child->key);
+    if (strcasecmp("Type", child->key) == 0)
+      gmond_config_set_string(child, &map->type);
+    else if (strcasecmp("TypeInstance", child->key) == 0)
+      gmond_config_set_string(child, &map->type_instance);
+    else if (strcasecmp("DataSource", child->key) == 0)
+      gmond_config_set_string(child, &map->ds_name);
+    else {
+      WARNING("gmond plugin: Unknown configuration option `%s' ignored.",
+              child->key);
     }
   }
 
-  if (map->type == NULL)
-  {
-    ERROR ("gmond plugin: No type is set for metric %s.",
-        map->ganglia_name);
-    sfree (map->ganglia_name);
-    sfree (map->type_instance);
+  if (map->type == NULL) {
+    ERROR("gmond plugin: No type is set for metric %s.", map->ganglia_name);
+    sfree(map->ganglia_name);
+    sfree(map->type_instance);
     return (-1);
   }
 
@@ -1019,45 +932,40 @@ static int gmond_config_add_metric (oconfig_item_t *ci) /* {{{ */
   return (0);
 } /* }}} int gmond_config_add_metric */
 
-static int gmond_config_set_address (oconfig_item_t *ci, /* {{{ */
-    char **ret_addr, char **ret_port)
-{
+static int gmond_config_set_address(oconfig_item_t *ci, /* {{{ */
+                                    char **ret_addr, char **ret_port) {
   char *addr;
   char *port;
 
-  if ((ci->values_num != 1) && (ci->values_num != 2))
-  {
-    WARNING ("gmond plugin: The `%s' config option needs "
-        "one or two string arguments.",
-        ci->key);
+  if ((ci->values_num != 1) && (ci->values_num != 2)) {
+    WARNING("gmond plugin: The `%s' config option needs "
+            "one or two string arguments.",
+            ci->key);
     return (-1);
   }
-  if ((ci->values[0].type != OCONFIG_TYPE_STRING)
-      || ((ci->values_num == 2)
-        && (ci->values[1].type != OCONFIG_TYPE_STRING)))
-  {
-    WARNING ("gmond plugin: The `%s' config option needs "
-        "one or two string arguments.",
-        ci->key);
+  if ((ci->values[0].type != OCONFIG_TYPE_STRING) ||
+      ((ci->values_num == 2) && (ci->values[1].type != OCONFIG_TYPE_STRING))) {
+    WARNING("gmond plugin: The `%s' config option needs "
+            "one or two string arguments.",
+            ci->key);
     return (-1);
   }
 
-  addr = strdup (ci->values[0].value.string);
+  addr = strdup(ci->values[0].value.string);
   if (ci->values_num == 2)
-    port = strdup (ci->values[1].value.string);
+    port = strdup(ci->values[1].value.string);
   else
     port = NULL;
 
-  if ((addr == NULL) || ((ci->values_num == 2) && (port == NULL)))
-  {
-    ERROR ("gmond plugin: strdup failed.");
-    sfree (addr);
-    sfree (port);
+  if ((addr == NULL) || ((ci->values_num == 2) && (port == NULL))) {
+    ERROR("gmond plugin: strdup failed.");
+    sfree(addr);
+    sfree(port);
     return (-1);
   }
 
-  sfree (*ret_addr);
-  sfree (*ret_port);
+  sfree(*ret_addr);
+  sfree(*ret_port);
 
   *ret_addr = addr;
   *ret_port = port;
@@ -1065,67 +973,62 @@ static int gmond_config_set_address (oconfig_item_t *ci, /* {{{ */
   return (0);
 } /* }}} int gmond_config_set_address */
 
-static int gmond_config (oconfig_item_t *ci) /* {{{ */
+static int gmond_config(oconfig_item_t *ci) /* {{{ */
 {
-  for (int i = 0; i < ci->children_num; i++)
-  {
+  for (int i = 0; i < ci->children_num; i++) {
     oconfig_item_t *child = ci->children + i;
-    if (strcasecmp ("MCReceiveFrom", child->key) == 0)
-      gmond_config_set_address (child, &mc_receive_group, &mc_receive_port);
-    else if (strcasecmp ("Metric", child->key) == 0)
-      gmond_config_add_metric (child);
-    else
-    {
-      WARNING ("gmond plugin: Unknown configuration option `%s' ignored.",
-          child->key);
+    if (strcasecmp("MCReceiveFrom", child->key) == 0)
+      gmond_config_set_address(child, &mc_receive_group, &mc_receive_port);
+    else if (strcasecmp("Metric", child->key) == 0)
+      gmond_config_add_metric(child);
+    else {
+      WARNING("gmond plugin: Unknown configuration option `%s' ignored.",
+              child->key);
     }
   }
 
   return (0);
 } /* }}} int gmond_config */
 
-static int gmond_init (void) /* {{{ */
+static int gmond_init(void) /* {{{ */
 {
-  create_sockets (&mc_send_sockets, &mc_send_sockets_num,
+  create_sockets(
+      &mc_send_sockets, &mc_send_sockets_num,
       (mc_receive_group != NULL) ? mc_receive_group : MC_RECEIVE_GROUP_DEFAULT,
       (mc_receive_port != NULL) ? mc_receive_port : MC_RECEIVE_PORT_DEFAULT,
       /* listen = */ 0);
 
-  staging_tree = c_avl_create ((int (*) (const void *, const void *)) strcmp);
-  if (staging_tree == NULL)
-  {
-    ERROR ("gmond plugin: c_avl_create failed.");
+  staging_tree = c_avl_create((int (*)(const void *, const void *))strcmp);
+  if (staging_tree == NULL) {
+    ERROR("gmond plugin: c_avl_create failed.");
     return (-1);
   }
 
-  mc_receive_thread_start ();
+  mc_receive_thread_start();
 
   return (0);
 } /* }}} int gmond_init */
 
-static int gmond_shutdown (void) /* {{{ */
+static int gmond_shutdown(void) /* {{{ */
 {
-  mc_receive_thread_stop ();
+  mc_receive_thread_stop();
 
-  pthread_mutex_lock (&mc_send_sockets_lock);
-  for (size_t i = 0; i < mc_send_sockets_num; i++)
-  {
-    close (mc_send_sockets[i].fd);
+  pthread_mutex_lock(&mc_send_sockets_lock);
+  for (size_t i = 0; i < mc_send_sockets_num; i++) {
+    close(mc_send_sockets[i].fd);
     mc_send_sockets[i].fd = -1;
   }
-  sfree (mc_send_sockets);
+  sfree(mc_send_sockets);
   mc_send_sockets_num = 0;
-  pthread_mutex_unlock (&mc_send_sockets_lock);
-
+  pthread_mutex_unlock(&mc_send_sockets_lock);
 
   return (0);
 } /* }}} int gmond_shutdown */
 
-void module_register (void)
-{
-  plugin_register_complex_config ("gmond", gmond_config);
-  plugin_register_init ("gmond", gmond_init);
-  plugin_register_shutdown ("gmond", gmond_shutdown);
+void module_register(void) {
+  plugin_register_complex_config("gmond", gmond_config);
+  plugin_register_init("gmond", gmond_init);
+  plugin_register_shutdown("gmond", gmond_shutdown);
 }
 
 /* vim: set sw=2 sts=2 et fdm=marker : */