Code

Merge branch 'collectd-4.2' into collectd-4.3
authorFlorian Forster <octo@huhu.verplant.org>
Mon, 25 Feb 2008 07:30:10 +0000 (08:30 +0100)
committerFlorian Forster <octo@huhu.verplant.org>
Mon, 25 Feb 2008 07:30:10 +0000 (08:30 +0100)
1  2 
src/exec.c
src/perl.c

diff --combined src/exec.c
index b25e76979ebb2322db79a5acdd746d28ecf73335,0ab259dcbb277031a4a4b26d9ed225fa46e85ed3..20e65df0a2656f5eb4204aef28cad181579f5ff5
@@@ -1,6 -1,6 +1,6 @@@
  /**
   * collectd - src/exec.c
 - * Copyright (C) 2007  Florian octo Forster
 + * Copyright (C) 2007,2008  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
@@@ -22,9 -22,7 +22,9 @@@
  #include "collectd.h"
  #include "common.h"
  #include "plugin.h"
 +
  #include "utils_cmd_putval.h"
 +#include "utils_cmd_putnotif.h"
  
  #include <sys/types.h>
  #include <pwd.h>
  
  #include <pthread.h>
  
 +#define PL_NORMAL        0x01
 +#define PL_NOTIF_ACTION  0x02
 +
 +#define PL_RUNNING       0x10
 +
  /*
   * Private data types
   */
 +/*
 + * Access to this structure is serialized using the `pl_lock' lock and the
 + * `PL_RUNNING' flag. The execution of notifications is *not* serialized, so
 + * all functions used to handle notifications MUST NOT write to this structure.
 + * The `pid' and `status' fields are thus unused if the `PL_NOTIF_ACTION' flag
 + * is set.
 + * The `PL_RUNNING' flag is set in `exec_read' and unset in `exec_read_one'.
 + */
  struct program_list_s;
  typedef struct program_list_s program_list_t;
  struct program_list_s
    char           *user;
    char           *group;
    char           *exec;
 +  char          **argv;
    int             pid;
 +  int             status;
 +  int             flags;
    program_list_t *next;
  };
  
 +typedef struct program_list_and_notification_s
 +{
 +  program_list_t *pl;
 +  notification_t n;
 +} program_list_and_notification_t;
 +
  /*
   * Private variables
   */
 -static const char *config_keys[] =
 -{
 -  "Exec"
 -};
 -static int config_keys_num = STATIC_ARRAY_SIZE (config_keys);
 -
  static program_list_t *pl_head = NULL;
 +static pthread_mutex_t pl_lock = PTHREAD_MUTEX_INITIALIZER;
  
  /*
   * Functions
   */
 -static int exec_config (const char *key, const char *value)
 +static void sigchld_handler (int signal) /* {{{ */
  {
 -  if (strcasecmp ("Exec", key) == 0)
 +  pid_t pid;
 +  int status;
 +  while ((pid = waitpid (-1, &status, WNOHANG)) > 0)
    {
      program_list_t *pl;
 -    pl = (program_list_t *) malloc (sizeof (program_list_t));
 -    if (pl == NULL)
 -      return (1);
 -    memset (pl, '\0', sizeof (program_list_t));
 +    for (pl = pl_head; pl != NULL; pl = pl->next)
 +      if (pl->pid == pid)
 +      break;
 +    if (pl != NULL)
 +      pl->status = status;
 +  } /* while (waitpid) */
 +} /* void sigchld_handler }}} */
 +
 +static int exec_config_exec (oconfig_item_t *ci) /* {{{ */
 +{
 +  program_list_t *pl;
 +  char buffer[128];
 +  int i;
  
 -    pl->user = strdup (value);
 -    if (pl->user == NULL)
 -    {
 -      sfree (pl);
 -      return (1);
 -    }
 +  if (ci->children_num != 0)
 +  {
 +    WARNING ("exec plugin: The config option `%s' may not be a block.",
 +      ci->key);
 +    return (-1);
 +  }
 +  if (ci->values_num < 2)
 +  {
 +    WARNING ("exec plugin: The config option `%s' needs at least two "
 +      "arguments.", ci->key);
 +    return (-1);
 +  }
 +  if ((ci->values[0].type != OCONFIG_TYPE_STRING)
 +      || (ci->values[1].type != OCONFIG_TYPE_STRING))
 +  {
 +    WARNING ("exec plugin: The first two arguments to the `%s' option must "
 +      "be string arguments.", ci->key);
 +    return (-1);
 +  }
 +
 +  pl = (program_list_t *) malloc (sizeof (program_list_t));
 +  if (pl == NULL)
 +  {
 +    ERROR ("exec plugin: malloc failed.");
 +    return (-1);
 +  }
 +  memset (pl, '\0', sizeof (program_list_t));
 +
 +  if (strcasecmp ("NotificationExec", ci->key) == 0)
 +    pl->flags |= PL_NOTIF_ACTION;
 +  else
 +    pl->flags |= PL_NORMAL;
 +
 +  pl->user = strdup (ci->values[0].value.string);
 +  if (pl->user == NULL)
 +  {
 +    ERROR ("exec plugin: strdup failed.");
 +    sfree (pl);
 +    return (-1);
 +  }
 +
 +  pl->group = strchr (pl->user, ':');
 +  if (pl->group != NULL)
 +  {
 +    *pl->group = '\0';
 +    pl->group++;
 +  }
 +
 +  pl->exec = strdup (ci->values[1].value.string);
 +  if (pl->exec == NULL)
 +  {
 +    ERROR ("exec plugin: strdup failed.");
 +    sfree (pl->user);
 +    sfree (pl);
 +    return (-1);
 +  }
  
 -    pl->exec = strchr (pl->user, ' ');
 -    if (pl->exec == NULL)
 +  pl->argv = (char **) malloc (ci->values_num * sizeof (char *));
 +  if (pl->argv == NULL)
 +  {
 +    ERROR ("exec plugin: malloc failed.");
 +    sfree (pl->exec);
 +    sfree (pl->user);
 +    sfree (pl);
 +    return (-1);
 +  }
 +  memset (pl->argv, '\0', ci->values_num * sizeof (char *));
 +
 +  {
 +    char *tmp = strrchr (ci->values[1].value.string, '/');
 +    if (tmp == NULL)
 +      strncpy (buffer, ci->values[1].value.string, sizeof (buffer));
 +    else
 +      strncpy (buffer, tmp + 1, sizeof (buffer));
 +    buffer[sizeof (buffer) - 1] = '\0';
 +  }
 +  pl->argv[0] = strdup (buffer);
 +  if (pl->argv[0] == NULL)
 +  {
 +    ERROR ("exec plugin: malloc failed.");
 +    sfree (pl->argv);
 +    sfree (pl->exec);
 +    sfree (pl->user);
 +    sfree (pl);
 +    return (-1);
 +  }
 +
 +  for (i = 1; i < (ci->values_num - 1); i++)
 +  {
 +    if (ci->values[i + 1].type == OCONFIG_TYPE_STRING)
      {
 -      sfree (pl->user);
 -      sfree (pl);
 -      return (1);
 +      pl->argv[i] = strdup (ci->values[i + 1].value.string);
      }
 -    while (*pl->exec == ' ')
 +    else
      {
 -      *pl->exec = '\0';
 -      pl->exec++;
 +      if (ci->values[i + 1].type == OCONFIG_TYPE_NUMBER)
 +      {
 +      snprintf (buffer, sizeof (buffer), "%lf",
 +          ci->values[i + 1].value.number);
 +      }
 +      else
 +      {
 +      if (ci->values[i + 1].value.boolean)
 +        strncpy (buffer, "true", sizeof (buffer));
 +      else
 +        strncpy (buffer, "false", sizeof (buffer));
 +      }
 +      buffer[sizeof (buffer) - 1] = '\0';
 +
 +      pl->argv[i] = strdup (buffer);
      }
  
 -    if (*pl->exec == '\0')
 +    if (pl->argv[i] == NULL)
      {
 -      sfree (pl->user);
 -      sfree (pl);
 -      return (1);
 +      ERROR ("exec plugin: strdup failed.");
 +      break;
      }
 +  } /* for (i) */
  
 -    pl->next = pl_head;
 -    pl_head = pl;
 -
 -    pl->group = strchr (pl->user, ':');
 -    if (NULL != pl->group) {
 -      *pl->group = '\0';
 -      pl->group++;
 +  if (i < (ci->values_num - 1))
 +  {
 +    while ((--i) >= 0)
 +    {
 +      sfree (pl->argv[i]);
      }
 +    sfree (pl->argv);
 +    sfree (pl->exec);
 +    sfree (pl->user);
 +    sfree (pl);
 +    return (-1);
    }
 -  else
 +
 +  for (i = 0; pl->argv[i] != NULL; i++)
    {
 -    return (-1);
 +    DEBUG ("exec plugin: argv[%i] = %s", i, pl->argv[i]);
    }
  
 +  pl->next = pl_head;
 +  pl_head = pl;
 +
 +  return (0);
 +} /* int exec_config_exec }}} */
 +
 +static int exec_config (oconfig_item_t *ci) /* {{{ */
 +{
 +  int i;
 +
 +  for (i = 0; i < ci->children_num; i++)
 +  {
 +    oconfig_item_t *child = ci->children + i;
 +    if ((strcasecmp ("Exec", child->key) == 0)
 +      || (strcasecmp ("NotificationExec", child->key) == 0))
 +      exec_config_exec (child);
 +    else
 +    {
 +      WARNING ("exec plugin: Unknown config option `%s'.", child->key);
 +    }
 +  } /* for (i) */
 +
    return (0);
 -} /* int exec_config */
 +} /* int exec_config }}} */
  
 -static void exec_child (program_list_t *pl)
 +static void exec_child (program_list_t *pl) /* {{{ */
  {
    int status;
    int uid;
    int gid;
    int egid;
 -  char *arg0;
  
    struct passwd *sp_ptr;
    struct passwd sp;
      glist[0] = gid;
      glist_len = 1;
  
-     if (gid != egid)
+     if ((gid != egid) && (egid != -1))
      {
        glist[1] = egid;
        glist_len = 2;
      exit (-1);
    }
  
 -  arg0 = strrchr (pl->exec, '/');
 -  if (arg0 != NULL)
 -    arg0++;
 -  if ((arg0 == NULL) || (*arg0 == '\0'))
 -    arg0 = pl->exec;
 -
 -  status = execlp (pl->exec, arg0, (char *) 0);
 +  status = execvp (pl->exec, pl->argv);
  
    ERROR ("exec plugin: exec failed: %s",
        sstrerror (errno, errbuf, sizeof (errbuf)));
    exit (-1);
 -} /* void exec_child */
 +} /* void exec_child }}} */
  
 -static int fork_child (program_list_t *pl)
 +/*
 + * Creates two pipes (one for reading, ong for writing), forks a child, sets up
 + * the pipes so that fd_in is connected to STDIN of the child and fd_out is
 + * connected to STDOUT and STDERR of the child. Then is calls `exec_child'.
 + */
 +static int fork_child (program_list_t *pl, int *fd_in, int *fd_out) /* {{{ */
  {
 -  int fd_pipe[2];
 +  int fd_pipe_in[2];
 +  int fd_pipe_out[2];
    int status;
 +  int pid;
  
    if (pl->pid != 0)
      return (-1);
  
 -  status = pipe (fd_pipe);
 +  status = pipe (fd_pipe_in);
    if (status != 0)
    {
      char errbuf[1024];
      return (-1);
    }
  
 -  pl->pid = fork ();
 -  if (pl->pid < 0)
 +  status = pipe (fd_pipe_out);
 +  if (status != 0)
 +  {
 +    char errbuf[1024];
 +    ERROR ("exec plugin: pipe failed: %s",
 +      sstrerror (errno, errbuf, sizeof (errbuf)));
 +    return (-1);
 +  }
 +
 +  pid = fork ();
 +  if (pid < 0)
    {
      char errbuf[1024];
      ERROR ("exec plugin: fork failed: %s",
        sstrerror (errno, errbuf, sizeof (errbuf)));
      return (-1);
    }
 -  else if (pl->pid == 0)
 +  else if (pid == 0)
    {
 -    close (fd_pipe[0]);
 +    close (fd_pipe_in[1]);
 +    close (fd_pipe_out[0]);
  
 -    /* Connect the pipe to STDOUT and STDERR */
 -    if (fd_pipe[1] != STDOUT_FILENO)
 -      dup2 (fd_pipe[1], STDOUT_FILENO);
 -    if (fd_pipe[1] != STDERR_FILENO)
 -      dup2 (fd_pipe[1], STDERR_FILENO);
 -    if ((fd_pipe[1] != STDOUT_FILENO) && (fd_pipe[1] != STDERR_FILENO))
 -      close (fd_pipe[1]);
 +    /* If the `out' pipe has the filedescriptor STDIN we have to be careful
 +     * with the `dup's below. So, if this is the case we have to handle the
 +     * `out' pipe first. */
 +    if (fd_pipe_out[1] == STDIN_FILENO)
 +    {
 +      int new_fileno = (fd_pipe_in[0] == STDOUT_FILENO)
 +      ? STDERR_FILENO : STDOUT_FILENO;
 +      dup2 (fd_pipe_out[1], new_fileno);
 +      close (fd_pipe_out[1]);
 +      fd_pipe_out[1] = new_fileno;
 +    }
 +    /* Now `fd_pipe_out[1]' is either `STDOUT' or `STDERR', but definitely not
 +     * `STDIN_FILENO'. */
 +
 +    /* Connect the `in' pipe to STDIN */
 +    if (fd_pipe_in[0] != STDIN_FILENO)
 +    {
 +      dup2 (fd_pipe_in[0], STDIN_FILENO);
 +      close (fd_pipe_in[0]);
 +      fd_pipe_in[0] = STDIN_FILENO;
 +    }
 +
 +    /* Now connect the `out' pipe to STDOUT and STDERR */
 +    if (fd_pipe_out[1] != STDOUT_FILENO)
 +      dup2 (fd_pipe_out[1], STDOUT_FILENO);
 +    if (fd_pipe_out[1] != STDERR_FILENO)
 +      dup2 (fd_pipe_out[1], STDERR_FILENO);
 +
 +    /* If the pipe has some FD that's something completely different, close it
 +     * now. */
 +    if ((fd_pipe_out[1] != STDOUT_FILENO) && (fd_pipe_out[1] != STDERR_FILENO))
 +    {
 +      close (fd_pipe_out[1]);
 +      fd_pipe_out[1] = STDOUT_FILENO;
 +    }
  
      exec_child (pl);
      /* does not return */
    }
  
 -  close (fd_pipe[1]);
 -  return (fd_pipe[0]);
 -} /* int fork_child */
 +  close (fd_pipe_in[0]);
 +  close (fd_pipe_out[1]);
  
 -static int parse_line (char *buffer)
 +  if (fd_in != NULL)
 +    *fd_in = fd_pipe_in[1];
 +  else
 +    close (fd_pipe_in[1]);
 +
 +  if (fd_out != NULL)
 +    *fd_out = fd_pipe_out[0];
 +  else
 +    close (fd_pipe_out[0]);
 +
 +  return (pid);
 +} /* int fork_child }}} */
 +
 +static int parse_line (char *buffer) /* {{{ */
  {
    char *fields[256];
    int fields_num;
  
    fields[0] = "PUTVAL";
 -  fields_num = strsplit (buffer, &fields[1], STATIC_ARRAY_SIZE(fields) - 1);
 +  fields_num = strsplit (buffer, fields + 1, STATIC_ARRAY_SIZE(fields) - 1);
  
 -  handle_putval (stdout, fields, fields_num + 1);
 -  return (0);
 -} /* int parse_line */
 +  if (strcasecmp (fields[1], "putval") == 0)
 +    return (handle_putval (stdout, fields + 1, fields_num));
 +  else if (strcasecmp (fields[1], "putnotif") == 0)
 +    return (handle_putnotif (stdout, fields + 1, fields_num));
  
 -static void *exec_read_one (void *arg)
 +  /* compatibility code */
 +  return (handle_putval (stdout, fields, fields_num + 1));
 +} /* int parse_line }}} */
 +
 +static void *exec_read_one (void *arg) /* {{{ */
  {
    program_list_t *pl = (program_list_t *) arg;
    int fd;
    FILE *fh;
    char buffer[1024];
 +  int status;
  
 -  fd = fork_child (pl);
 -  if (fd < 0)
 +  status = fork_child (pl, NULL, &fd);
 +  if (status < 0)
      pthread_exit ((void *) 1);
 +  pl->pid = status;
  
    assert (pl->pid != 0);
  
      pthread_exit ((void *) 1);
    }
  
 +  buffer[0] = '\0';
    while (fgets (buffer, sizeof (buffer), fh) != NULL)
    {
      int len;
    } /* while (fgets) */
  
    fclose (fh);
 +
 +  if (waitpid (pl->pid, &status, 0) > 0)
 +    pl->status = status;
 +
 +  DEBUG ("exec plugin: Child %i exited with status %i.",
 +      (int) pl->pid, pl->status);
 +
    pl->pid = 0;
  
 +  pthread_mutex_lock (&pl_lock);
 +  pl->flags &= ~PL_RUNNING;
 +  pthread_mutex_unlock (&pl_lock);
 +
    pthread_exit ((void *) 0);
    return (NULL);
 -} /* void *exec_read_one */
 +} /* void *exec_read_one }}} */
  
 -static int exec_read (void)
 +static void *exec_notification_one (void *arg) /* {{{ */
 +{
 +  program_list_t *pl = ((program_list_and_notification_t *) arg)->pl;
 +  const notification_t *n = &((program_list_and_notification_t *) arg)->n;
 +  int fd;
 +  FILE *fh;
 +  int pid;
 +  int status;
 +  const char *severity;
 +
 +  pid = fork_child (pl, &fd, NULL);
 +  if (pid < 0) {
 +    sfree (arg);
 +    pthread_exit ((void *) 1);
 +  }
 +
 +  fh = fdopen (fd, "w");
 +  if (fh == NULL)
 +  {
 +    char errbuf[1024];
 +    ERROR ("exec plugin: fdopen (%i) failed: %s", fd,
 +      sstrerror (errno, errbuf, sizeof (errbuf)));
 +    kill (pl->pid, SIGTERM);
 +    pl->pid = 0;
 +    close (fd);
 +    sfree (arg);
 +    pthread_exit ((void *) 1);
 +  }
 +
 +  severity = "FAILURE";
 +  if (n->severity == NOTIF_WARNING)
 +    severity = "WARNING";
 +  else if (n->severity == NOTIF_OKAY)
 +    severity = "OKAY";
 +
 +  fprintf (fh,
 +      "Severity: %s\n"
 +      "Time: %u\n",
 +      severity, (unsigned int) n->time);
 +
 +  /* Print the optional fields */
 +  if (strlen (n->host) > 0)
 +    fprintf (fh, "Host: %s\n", n->host);
 +  if (strlen (n->plugin) > 0)
 +    fprintf (fh, "Plugin: %s\n", n->plugin);
 +  if (strlen (n->plugin_instance) > 0)
 +    fprintf (fh, "PluginInstance: %s\n", n->plugin_instance);
 +  if (strlen (n->type) > 0)
 +    fprintf (fh, "Type: %s\n", n->type);
 +  if (strlen (n->type_instance) > 0)
 +    fprintf (fh, "TypeInstance: %s\n", n->type_instance);
 +
 +  fprintf (fh, "\n%s\n", n->message);
 +
 +  fflush (fh);
 +  fclose (fh);
 +
 +  waitpid (pid, &status, 0);
 +
 +  DEBUG ("exec plugin: Child %i exited with status %i.",
 +      pid, status);
 +
 +  sfree (arg);
 +  pthread_exit ((void *) 0);
 +  return (NULL);
 +} /* void *exec_notification_one }}} */
 +
 +static int exec_init (void) /* {{{ */
 +{
 +  struct sigaction sa;
 +
 +  memset (&sa, '\0', sizeof (sa));
 +  sa.sa_handler = sigchld_handler;
 +  sigaction (SIGCHLD, &sa, NULL);
 +
 +  return (0);
 +} /* int exec_init }}} */
 +
 +static int exec_read (void) /* {{{ */
  {
    program_list_t *pl;
  
      pthread_t t;
      pthread_attr_t attr;
  
 -    if (pl->pid != 0)
 +    /* Only execute `normal' style executables here. */
 +    if ((pl->flags & PL_NORMAL) == 0)
        continue;
  
 +    pthread_mutex_lock (&pl_lock);
 +    /* Skip if a child is already running. */
 +    if ((pl->flags & PL_RUNNING) != 0)
 +    {
 +      pthread_mutex_unlock (&pl_lock);
 +      continue;
 +    }
 +    pl->flags |= PL_RUNNING;
 +    pthread_mutex_unlock (&pl_lock);
 +
      pthread_attr_init (&attr);
      pthread_attr_setdetachstate (&attr, PTHREAD_CREATE_DETACHED);
      pthread_create (&t, &attr, exec_read_one, (void *) pl);
    } /* for (pl) */
  
    return (0);
 -} /* int exec_read */
 +} /* int exec_read }}} */
 +
 +static int exec_notification (const notification_t *n)
 +{
 +  program_list_t *pl;
 +  program_list_and_notification_t *pln;
 +
 +  for (pl = pl_head; pl != NULL; pl = pl->next)
 +  {
 +    pthread_t t;
 +    pthread_attr_t attr;
 +
 +    /* Only execute `notification' style executables here. */
 +    if ((pl->flags & PL_NOTIF_ACTION) == 0)
 +      continue;
 +
 +    /* Skip if a child is already running. */
 +    if (pl->pid != 0)
 +      continue;
 +
 +    pln = (program_list_and_notification_t *) malloc (sizeof
 +      (program_list_and_notification_t));
 +    if (pln == NULL)
 +    {
 +      ERROR ("exec plugin: malloc failed.");
 +      continue;
 +    }
 +
 +    pln->pl = pl;
 +    memcpy (&pln->n, n, sizeof (notification_t));
 +
 +    pthread_attr_init (&attr);
 +    pthread_attr_setdetachstate (&attr, PTHREAD_CREATE_DETACHED);
 +    pthread_create (&t, &attr, exec_notification_one, (void *) pln);
 +  } /* for (pl) */
 +
 +  return (0);
 +} /* int exec_notification */
  
 -static int exec_shutdown (void)
 +static int exec_shutdown (void) /* {{{ */
  {
    program_list_t *pl;
    program_list_t *next;
    pl_head = NULL;
  
    return (0);
 -} /* int exec_shutdown */
 +} /* int exec_shutdown }}} */
  
  void module_register (void)
  {
 -  plugin_register_config ("exec", exec_config, config_keys, config_keys_num);
 +  plugin_register_complex_config ("exec", exec_config);
 +  plugin_register_init ("exec", exec_init);
    plugin_register_read ("exec", exec_read);
 +  plugin_register_notification ("exec", exec_notification);
    plugin_register_shutdown ("exec", exec_shutdown);
  } /* void module_register */
  
  /*
 - * vim:shiftwidth=2:softtabstop=2:tabstop=8
 + * vim:shiftwidth=2:softtabstop=2:tabstop=8:fdm=marker
   */
diff --combined src/perl.c
index 7558a50638390fc7918b8d1b60420d14a2f8a3a6,50d189f8aaeb82490ff2d80d5d8a5eb37fdd4134..6d3326faebca7f89b4c5f02223c3bc5e2237ac50
@@@ -24,9 -24,6 +24,9 @@@
   * interface for collectd plugins written in perl.
   */
  
 +/* do not automatically get the thread specific perl interpreter */
 +#define PERL_NO_GET_CONTEXT
 +
  #include "collectd.h"
  
  #include "configfile.h"
  #include "plugin.h"
  #include "common.h"
  
 +#include <pthread.h>
 +
 +#if !defined(USE_ITHREADS)
 +# error "Perl does not support ithreads!"
 +#endif /* !defined(USE_ITHREADS) */
 +
 +/* clear the Perl sub's stack frame
 + * (this should only be used inside an XSUB) */
 +#define CLEAR_STACK_FRAME PL_stack_sp = PL_stack_base + *PL_markstack_ptr
 +
  #define PLUGIN_INIT     0
  #define PLUGIN_READ     1
  #define PLUGIN_WRITE    2
  #define PLUGIN_SHUTDOWN 3
  #define PLUGIN_LOG      4
 +#define PLUGIN_NOTIF    5
  
 -#define PLUGIN_TYPES    5
 +#define PLUGIN_TYPES    6
  
  #define PLUGIN_DATASET  255
  
@@@ -77,47 -63,15 +77,47 @@@ void boot_DynaLoader (PerlInterpreter *
  static XS (Collectd_plugin_register_ds);
  static XS (Collectd_plugin_unregister_ds);
  static XS (Collectd_plugin_dispatch_values);
 +static XS (Collectd_plugin_dispatch_notification);
  static XS (Collectd_plugin_log);
 +static XS (Collectd_call_by_name);
 +
 +/*
 + * private data types
 + */
 +
 +typedef struct c_ithread_s {
 +      /* the thread's Perl interpreter */
 +      PerlInterpreter *interp;
 +
 +      /* double linked list of threads */
 +      struct c_ithread_s *prev;
 +      struct c_ithread_s *next;
 +} c_ithread_t;
 +
 +typedef struct {
 +      c_ithread_t *head;
 +      c_ithread_t *tail;
 +
 +#if COLLECT_DEBUG
 +      /* some usage stats */
 +      int number_of_threads;
 +#endif /* COLLECT_DEBUG */
 +
 +      pthread_mutex_t mutex;
 +} c_ithread_list_t;
  
  /*
   * private variables
   */
  
 -static PerlInterpreter *perl = NULL;
 +/* if perl_threads != NULL perl_threads->head must
 + * point to the "base" thread */
 +static c_ithread_list_t *perl_threads = NULL;
 +
 +/* the key used to store each pthread's ithread */
 +static pthread_key_t perl_thr_key;
  
 -static int  perl_argc   = 0;
 +static int    perl_argc = 0;
  static char **perl_argv = NULL;
  
  static char base_name[DATA_MAX_NAME_LEN] = "";
@@@ -130,10 -84,7 +130,10 @@@ static struct 
        { "Collectd::plugin_register_data_set",   Collectd_plugin_register_ds },
        { "Collectd::plugin_unregister_data_set", Collectd_plugin_unregister_ds },
        { "Collectd::plugin_dispatch_values",     Collectd_plugin_dispatch_values },
 +      { "Collectd::plugin_dispatch_notification",
 +              Collectd_plugin_dispatch_notification },
        { "Collectd::plugin_log",                 Collectd_plugin_log },
 +      { "Collectd::call_by_name",               Collectd_call_by_name },
        { "", NULL }
  };
  
@@@ -147,7 -98,6 +147,7 @@@ struct 
        { "Collectd::TYPE_WRITE",      PLUGIN_WRITE },
        { "Collectd::TYPE_SHUTDOWN",   PLUGIN_SHUTDOWN },
        { "Collectd::TYPE_LOG",        PLUGIN_LOG },
 +      { "Collectd::TYPE_NOTIF",      PLUGIN_NOTIF },
        { "Collectd::TYPE_DATASET",    PLUGIN_DATASET },
        { "Collectd::DS_TYPE_COUNTER", DS_TYPE_COUNTER },
        { "Collectd::DS_TYPE_GAUGE",   DS_TYPE_GAUGE },
        { "Collectd::LOG_NOTICE",      LOG_NOTICE },
        { "Collectd::LOG_INFO",        LOG_INFO },
        { "Collectd::LOG_DEBUG",       LOG_DEBUG },
 +      { "Collectd::NOTIF_FAILURE",   NOTIF_FAILURE },
 +      { "Collectd::NOTIF_WARNING",   NOTIF_WARNING },
 +      { "Collectd::NOTIF_OKAY",      NOTIF_OKAY },
        { "", 0 }
  };
  
 +struct {
 +      char  name[64];
 +      char *var;
 +} g_strings[] =
 +{
 +      { "Collectd::hostname_g", hostname_g },
 +      { "", NULL }
 +};
 +
 +struct {
 +      char  name[64];
 +      int  *var;
 +} g_integers[] =
 +{
 +      { "Collectd::interval_g", &interval_g },
 +      { "", NULL }
 +};
 +
  /*
   * Helper functions for data type conversion.
   */
   *   ...
   * ]
   */
 -static int hv2data_source (HV *hash, data_source_t *ds)
 +static int hv2data_source (pTHX_ HV *hash, data_source_t *ds)
  {
        SV **tmp = NULL;
  
        if ((NULL == hash) || (NULL == ds))
                return -1;
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, hash, "name", 4, 0))) {
 +      if (NULL != (tmp = hv_fetch (hash, "name", 4, 0))) {
                strncpy (ds->name, SvPV_nolen (*tmp), DATA_MAX_NAME_LEN);
                ds->name[DATA_MAX_NAME_LEN - 1] = '\0';
        }
                return -1;
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, hash, "type", 4, 0))) {
 +      if (NULL != (tmp = hv_fetch (hash, "type", 4, 0))) {
                ds->type = SvIV (*tmp);
  
                if ((DS_TYPE_COUNTER != ds->type) && (DS_TYPE_GAUGE != ds->type)) {
                ds->type = DS_TYPE_COUNTER;
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, hash, "min", 3, 0)))
 +      if (NULL != (tmp = hv_fetch (hash, "min", 3, 0)))
                ds->min = SvNV (*tmp);
        else
                ds->min = NAN;
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, hash, "max", 3, 0)))
 +      if (NULL != (tmp = hv_fetch (hash, "max", 3, 0)))
                ds->max = SvNV (*tmp);
        else
                ds->max = NAN;
        return 0;
  } /* static data_source_t *hv2data_source (HV *) */
  
 -static int av2value (char *name, AV *array, value_t *value, int len)
 +static int av2value (pTHX_ char *name, AV *array, value_t *value, int len)
  {
        const data_set_t *ds;
  
        if ((NULL == name) || (NULL == array) || (NULL == value))
                return -1;
  
 -      if (Perl_av_len (perl, array) < len - 1)
 -              len = Perl_av_len (perl, array) + 1;
 +      if (av_len (array) < len - 1)
 +              len = av_len (array) + 1;
  
        if (0 >= len)
                return -1;
        }
  
        for (i = 0; i < len; ++i) {
 -              SV **tmp = Perl_av_fetch (perl, array, i, 0);
 +              SV **tmp = av_fetch (array, i, 0);
  
                if (NULL != tmp) {
                        if (DS_TYPE_COUNTER == ds->ds[i].type)
        return len;
  } /* static int av2value (char *, AV *, value_t *, int) */
  
 -static int data_set2av (data_set_t *ds, AV *array)
 +static int data_set2av (pTHX_ data_set_t *ds, AV *array)
  {
        int i = 0;
  
        if ((NULL == ds) || (NULL == array))
                return -1;
  
 -      Perl_av_extend (perl, array, ds->ds_num);
 +      av_extend (array, ds->ds_num);
  
        for (i = 0; i < ds->ds_num; ++i) {
 -              HV *source = Perl_newHV (perl);
 +              HV *source = newHV ();
  
 -              if (NULL == Perl_hv_store (perl, source, "name", 4,
 -                              Perl_newSVpv (perl, ds->ds[i].name, 0), 0))
 +              if (NULL == hv_store (source, "name", 4,
 +                              newSVpv (ds->ds[i].name, 0), 0))
                        return -1;
  
 -              if (NULL == Perl_hv_store (perl, source, "type", 4,
 -                              Perl_newSViv (perl, ds->ds[i].type), 0))
 +              if (NULL == hv_store (source, "type", 4, newSViv (ds->ds[i].type), 0))
                        return -1;
  
                if (! isnan (ds->ds[i].min))
 -                      if (NULL == Perl_hv_store (perl, source, "min", 3,
 -                                      Perl_newSVnv (perl, ds->ds[i].min), 0))
 +                      if (NULL == hv_store (source, "min", 3,
 +                                      newSVnv (ds->ds[i].min), 0))
                                return -1;
  
                if (! isnan (ds->ds[i].max))
 -                      if (NULL == Perl_hv_store (perl, source, "max", 3,
 -                                      Perl_newSVnv (perl, ds->ds[i].max), 0))
 +                      if (NULL == hv_store (source, "max", 3,
 +                                      newSVnv (ds->ds[i].max), 0))
                                return -1;
  
 -              if (NULL == Perl_av_store (perl, array, i,
 -                              Perl_newRV_noinc (perl, (SV *)source)))
 +              if (NULL == av_store (array, i, newRV_noinc ((SV *)source)))
                        return -1;
        }
        return 0;
  } /* static int data_set2av (data_set_t *, AV *) */
  
 -static int value_list2hv (value_list_t *vl, data_set_t *ds, HV *hash)
 +static int value_list2hv (pTHX_ value_list_t *vl, data_set_t *ds, HV *hash)
  {
        AV *values = NULL;
  
                len = ds->ds_num;
        }
  
 -      values = Perl_newAV (perl);
 -      Perl_av_extend (perl, values, len - 1);
 +      values = newAV ();
 +      av_extend (values, len - 1);
  
        for (i = 0; i < len; ++i) {
                SV *val = NULL;
  
                if (DS_TYPE_COUNTER == ds->ds[i].type)
 -                      val = Perl_newSViv (perl, vl->values[i].counter);
 +                      val = newSViv (vl->values[i].counter);
                else
 -                      val = Perl_newSVnv (perl, vl->values[i].gauge);
 +                      val = newSVnv (vl->values[i].gauge);
  
 -              if (NULL == Perl_av_store (perl, values, i, val)) {
 -                      Perl_av_undef (perl, values);
 +              if (NULL == av_store (values, i, val)) {
 +                      av_undef (values);
                        return -1;
                }
        }
  
 -      if (NULL == Perl_hv_store (perl, hash, "values", 6,
 -                      Perl_newRV_noinc (perl, (SV *)values), 0))
 +      if (NULL == hv_store (hash, "values", 6, newRV_noinc ((SV *)values), 0))
                return -1;
  
        if (0 != vl->time)
 -              if (NULL == Perl_hv_store (perl, hash, "time", 4,
 -                              Perl_newSViv (perl, vl->time), 0))
 +              if (NULL == hv_store (hash, "time", 4, newSViv (vl->time), 0))
                        return -1;
  
        if ('\0' != vl->host[0])
 -              if (NULL == Perl_hv_store (perl, hash, "host", 4,
 -                              Perl_newSVpv (perl, vl->host, 0), 0))
 +              if (NULL == hv_store (hash, "host", 4, newSVpv (vl->host, 0), 0))
                        return -1;
  
        if ('\0' != vl->plugin[0])
 -              if (NULL == Perl_hv_store (perl, hash, "plugin", 6,
 -                              Perl_newSVpv (perl, vl->plugin, 0), 0))
 +              if (NULL == hv_store (hash, "plugin", 6, newSVpv (vl->plugin, 0), 0))
                        return -1;
  
        if ('\0' != vl->plugin_instance[0])
 -              if (NULL == Perl_hv_store (perl, hash, "plugin_instance", 15,
 -                              Perl_newSVpv (perl, vl->plugin_instance, 0), 0))
 +              if (NULL == hv_store (hash, "plugin_instance", 15,
 +                              newSVpv (vl->plugin_instance, 0), 0))
                        return -1;
  
        if ('\0' != vl->type_instance[0])
 -              if (NULL == Perl_hv_store (perl, hash, "type_instance", 13,
 -                              Perl_newSVpv (perl, vl->type_instance, 0), 0))
 +              if (NULL == hv_store (hash, "type_instance", 13,
 +                              newSVpv (vl->type_instance, 0), 0))
                        return -1;
        return 0;
  } /* static int value2av (value_list_t *, data_set_t *, HV *) */
  
 +static int notification2hv (pTHX_ notification_t *n, HV *hash)
 +{
 +      if (NULL == hv_store (hash, "severity", 8, newSViv (n->severity), 0))
 +              return -1;
 +
 +      if (0 != n->time)
 +              if (NULL == hv_store (hash, "time", 4, newSViv (n->time), 0))
 +                      return -1;
 +
 +      if ('\0' != *n->message)
 +              if (NULL == hv_store (hash, "message", 7, newSVpv (n->message, 0), 0))
 +                      return -1;
 +
 +      if ('\0' != *n->host)
 +              if (NULL == hv_store (hash, "host", 4, newSVpv (n->host, 0), 0))
 +                      return -1;
 +
 +      if ('\0' != *n->plugin)
 +              if (NULL == hv_store (hash, "plugin", 6, newSVpv (n->plugin, 0), 0))
 +                      return -1;
 +
 +      if ('\0' != *n->plugin_instance)
 +              if (NULL == hv_store (hash, "plugin_instance", 15,
 +                              newSVpv (n->plugin_instance, 0), 0))
 +                      return -1;
 +
 +      if ('\0' != *n->type)
 +              if (NULL == hv_store (hash, "type", 4, newSVpv (n->type, 0), 0))
 +                      return -1;
 +
 +      if ('\0' != *n->type_instance)
 +              if (NULL == hv_store (hash, "type_instance", 13,
 +                              newSVpv (n->type_instance, 0), 0))
 +                      return -1;
 +      return 0;
 +} /* static int notification2hv (notification_t *, HV *) */
 +
  /*
   * Internal functions.
   */
@@@ -421,7 -319,7 +421,7 @@@ static char *get_module_name (char *buf
                status = snprintf (buf, buf_len, "%s", module);
        else
                status = snprintf (buf, buf_len, "%s::%s", base_name, module);
 -      if ((status < 0) || (status >= buf_len))
 +      if ((status < 0) || ((unsigned int)status >= buf_len))
                return (NULL);
        buf[buf_len - 1] = '\0';
        return (buf);
  /*
   * Add a plugin's data set definition.
   */
 -static int pplugin_register_data_set (char *name, AV *dataset)
 +static int pplugin_register_data_set (pTHX_ char *name, AV *dataset)
  {
        int len = -1;
 +      int ret = 0;
        int i   = 0;
  
        data_source_t *ds  = NULL;
        if ((NULL == name) || (NULL == dataset))
                return -1;
  
 -      len = Perl_av_len (perl, dataset);
 +      len = av_len (dataset);
  
        if (-1 == len)
                return -1;
        set = (data_set_t *)smalloc (sizeof (data_set_t));
  
        for (i = 0; i <= len; ++i) {
 -              SV **elem = Perl_av_fetch (perl, dataset, i, 0);
 +              SV **elem = av_fetch (dataset, i, 0);
  
                if (NULL == elem)
                        return -1;
                        return -1;
                }
  
 -              if (-1 == hv2data_source ((HV *)SvRV (*elem), &ds[i]))
 +              if (-1 == hv2data_source (aTHX_ (HV *)SvRV (*elem), &ds[i]))
                        return -1;
  
                log_debug ("pplugin_register_data_set: "
  
        set->ds_num = len + 1;
        set->ds = ds;
 -      return plugin_register_data_set (set);
 +
 +      ret = plugin_register_data_set (set);
 +
 +      free (ds);
 +      free (set);
 +      return ret;
  } /* static int pplugin_register_data_set (char *, SV *) */
  
  /*
@@@ -505,7 -397,7 +505,7 @@@ static int pplugin_unregister_data_set 
   *   type_instance   => $tinstance,
   * }
   */
 -static int pplugin_dispatch_values (char *name, HV *values)
 +static int pplugin_dispatch_values (pTHX_ char *name, HV *values)
  {
        value_list_t list = VALUE_LIST_INIT;
        value_t      *val = NULL;
        if ((NULL == name) || (NULL == values))
                return -1;
  
 -      if ((NULL == (tmp = Perl_hv_fetch (perl, values, "values", 6, 0)))
 +      if ((NULL == (tmp = hv_fetch (values, "values", 6, 0)))
                        || (! (SvROK (*tmp) && (SVt_PVAV == SvTYPE (SvRV (*tmp)))))) {
                log_err ("pplugin_dispatch_values: No valid values given.");
                return -1;
  
        {
                AV  *array = (AV *)SvRV (*tmp);
 -              int len    = Perl_av_len (perl, array) + 1;
 +              int len    = av_len (array) + 1;
  
                if (len <= 0)
                        return -1;
  
                val = (value_t *)smalloc (len * sizeof (value_t));
  
 -              list.values_len = av2value (name, (AV *)SvRV (*tmp), val, len);
 +              list.values_len = av2value (aTHX_ name, (AV *)SvRV (*tmp), val, len);
                list.values = val;
  
                if (-1 == list.values_len) {
                }
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, values, "time", 4, 0))) {
 +      if (NULL != (tmp = hv_fetch (values, "time", 4, 0))) {
                list.time = (time_t)SvIV (*tmp);
        }
        else {
                list.time = time (NULL);
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, values, "host", 4, 0))) {
 +      if (NULL != (tmp = hv_fetch (values, "host", 4, 0))) {
                strncpy (list.host, SvPV_nolen (*tmp), DATA_MAX_NAME_LEN);
                list.host[DATA_MAX_NAME_LEN - 1] = '\0';
        }
                strcpy (list.host, hostname_g);
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, values, "plugin", 6, 0))) {
 +      if (NULL != (tmp = hv_fetch (values, "plugin", 6, 0))) {
                strncpy (list.plugin, SvPV_nolen (*tmp), DATA_MAX_NAME_LEN);
                list.plugin[DATA_MAX_NAME_LEN - 1] = '\0';
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, values,
 -                      "plugin_instance", 15, 0))) {
 +      if (NULL != (tmp = hv_fetch (values, "plugin_instance", 15, 0))) {
                strncpy (list.plugin_instance, SvPV_nolen (*tmp), DATA_MAX_NAME_LEN);
                list.plugin_instance[DATA_MAX_NAME_LEN - 1] = '\0';
        }
  
 -      if (NULL != (tmp = Perl_hv_fetch (perl, values, "type_instance", 13, 0))) {
 +      if (NULL != (tmp = hv_fetch (values, "type_instance", 13, 0))) {
                strncpy (list.type_instance, SvPV_nolen (*tmp), DATA_MAX_NAME_LEN);
                list.type_instance[DATA_MAX_NAME_LEN - 1] = '\0';
        }
        return ret;
  } /* static int pplugin_dispatch_values (char *, HV *) */
  
 +/*
 + * Dispatch a notification.
 + *
 + * notification:
 + * {
 + *   severity => $severity,
 + *   time     => $time,
 + *   message  => $msg,
 + *   host     => $host,
 + *   plugin   => $plugin,
 + *   type     => $type,
 + *   plugin_instance => $instance,
 + *   type_instance   => $type_instance
 + * }
 + */
 +static int pplugin_dispatch_notification (pTHX_ HV *notif)
 +{
 +      notification_t n;
 +
 +      SV **tmp = NULL;
 +
 +      if (NULL == notif)
 +              return -1;
 +
 +      memset (&n, 0, sizeof (n));
 +
 +      if (NULL != (tmp = hv_fetch (notif, "severity", 8, 0)))
 +              n.severity = SvIV (*tmp);
 +      else
 +              n.severity = NOTIF_FAILURE;
 +
 +      if (NULL != (tmp = hv_fetch (notif, "time", 4, 0)))
 +              n.time = (time_t)SvIV (*tmp);
 +      else
 +              n.time = time (NULL);
 +
 +      if (NULL != (tmp = hv_fetch (notif, "message", 7, 0)))
 +              strncpy (n.message, SvPV_nolen (*tmp), sizeof (n.message));
 +      n.message[sizeof (n.message) - 1] = '\0';
 +
 +      if (NULL != (tmp = hv_fetch (notif, "host", 4, 0)))
 +              strncpy (n.host, SvPV_nolen (*tmp), sizeof (n.host));
 +      else
 +              strncpy (n.host, hostname_g, sizeof (n.host));
 +      n.host[sizeof (n.host) - 1] = '\0';
 +
 +      if (NULL != (tmp = hv_fetch (notif, "plugin", 6, 0)))
 +              strncpy (n.plugin, SvPV_nolen (*tmp), sizeof (n.plugin));
 +      n.plugin[sizeof (n.plugin) - 1] = '\0';
 +
 +      if (NULL != (tmp = hv_fetch (notif, "plugin_instance", 15, 0)))
 +              strncpy (n.plugin_instance, SvPV_nolen (*tmp),
 +                              sizeof (n.plugin_instance));
 +      n.plugin_instance[sizeof (n.plugin_instance) - 1] = '\0';
 +
 +      if (NULL != (tmp = hv_fetch (notif, "type", 4, 0)))
 +              strncpy (n.type, SvPV_nolen (*tmp), sizeof (n.type));
 +      n.type[sizeof (n.type) - 1] = '\0';
 +
 +      if (NULL != (tmp = hv_fetch (notif, "type_instance", 13, 0)))
 +              strncpy (n.type_instance, SvPV_nolen (*tmp), sizeof (n.type_instance));
 +      n.type_instance[sizeof (n.type_instance) - 1] = '\0';
 +      return plugin_dispatch_notification (&n);
 +} /* static int pplugin_dispatch_notification (HV *) */
 +
  /*
   * Call all working functions of the given type.
   */
 -static int pplugin_call_all (int type, ...)
 +static int pplugin_call_all (pTHX_ int type, ...)
  {
        int retvals = 0;
  
  
        PUSHMARK (SP);
  
 -      XPUSHs (sv_2mortal (Perl_newSViv (perl, (IV)type)));
 +      XPUSHs (sv_2mortal (newSViv ((IV)type)));
  
        if (PLUGIN_WRITE == type) {
                /*
                data_set_t   *ds;
                value_list_t *vl;
  
 -              AV *pds = Perl_newAV (perl);
 -              HV *pvl = Perl_newHV (perl);
 +              AV *pds = newAV ();
 +              HV *pvl = newHV ();
  
                ds = va_arg (ap, data_set_t *);
                vl = va_arg (ap, value_list_t *);
  
 -              if (-1 == data_set2av (ds, pds))
 -                      return -1;
 +              if (-1 == data_set2av (aTHX_ ds, pds)) {
 +                      av_clear (pds);
 +                      av_undef (pds);
 +                      pds = Nullav;
 +                      ret = -1;
 +              }
  
 -              if (-1 == value_list2hv (vl, ds, pvl))
 -                      return -1;
 +              if (-1 == value_list2hv (aTHX_ vl, ds, pvl)) {
 +                      hv_clear (pvl);
 +                      hv_undef (pvl);
 +                      pvl = Nullhv;
 +                      ret = -1;
 +              }
  
 -              XPUSHs (sv_2mortal (Perl_newSVpv (perl, ds->type, 0)));
 -              XPUSHs (sv_2mortal (Perl_newRV_noinc (perl, (SV *)pds)));
 -              XPUSHs (sv_2mortal (Perl_newRV_noinc (perl, (SV *)pvl)));
 +              XPUSHs (sv_2mortal (newSVpv (ds->type, 0)));
 +              XPUSHs (sv_2mortal (newRV_noinc ((SV *)pds)));
 +              XPUSHs (sv_2mortal (newRV_noinc ((SV *)pvl)));
        }
        else if (PLUGIN_LOG == type) {
                /*
                 *
                 * $_[1] = $message;
                 */
 -              XPUSHs (sv_2mortal (Perl_newSViv (perl, va_arg (ap, int))));
 -              XPUSHs (sv_2mortal (Perl_newSVpv (perl, va_arg (ap, char *), 0)));
 +              XPUSHs (sv_2mortal (newSViv (va_arg (ap, int))));
 +              XPUSHs (sv_2mortal (newSVpv (va_arg (ap, char *), 0)));
 +      }
 +      else if (PLUGIN_NOTIF == type) {
 +              /*
 +               * $_[0] =
 +               * {
 +               *   severity => $severity,
 +               *   time     => $time,
 +               *   message  => $msg,
 +               *   host     => $host,
 +               *   plugin   => $plugin,
 +               *   type     => $type,
 +               *   plugin_instance => $instance,
 +               *   type_instance   => $type_instance
 +               * };
 +               */
 +              notification_t *n;
 +              HV *notif = newHV ();
 +
 +              n = va_arg (ap, notification_t *);
 +
 +              if (-1 == notification2hv (aTHX_ n, notif)) {
 +                      hv_clear (notif);
 +                      hv_undef (notif);
 +                      notif = Nullhv;
 +                      ret = -1;
 +              }
 +
 +              XPUSHs (sv_2mortal (newRV_noinc ((SV *)notif)));
        }
  
        PUTBACK;
  
 -      retvals = Perl_call_pv (perl, "Collectd::plugin_call_all", G_SCALAR);
 +      retvals = call_pv ("Collectd::plugin_call_all", G_SCALAR);
  
        SPAGAIN;
        if (0 < retvals) {
@@@ -807,7 -599,7 +807,7 @@@ static XS (Collectd_plugin_register_ds
        data = ST (1);
  
        if (SvROK (data) && (SVt_PVAV == SvTYPE (SvRV (data)))) {
 -              ret = pplugin_register_data_set (SvPV_nolen (ST (0)),
 +              ret = pplugin_register_data_set (aTHX_ SvPV_nolen (ST (0)),
                                (AV *)SvRV (data));
        }
        else {
@@@ -839,7 -631,7 +839,7 @@@ static XS (Collectd_plugin_unregister_d
        log_debug ("Collectd::plugin_unregister_data_set: type = \"%s\"",
                        SvPV_nolen (ST (0)));
  
 -      if (0 == pplugin_unregister_data_set (SvPV_nolen (ST (1))))
 +      if (0 == pplugin_unregister_data_set (SvPV_nolen (ST (0))))
                XSRETURN_YES;
        else
                XSRETURN_EMPTY;
@@@ -881,8 -673,7 +881,8 @@@ static XS (Collectd_plugin_dispatch_val
        if ((NULL == ST (0)) || (NULL == values))
                XSRETURN_EMPTY;
  
 -      ret = pplugin_dispatch_values (SvPV_nolen (ST (0)), (HV *)SvRV (values));
 +      ret = pplugin_dispatch_values (aTHX_ SvPV_nolen (ST (0)),
 +                      (HV *)SvRV (values));
  
        if (0 == ret)
                XSRETURN_YES;
                XSRETURN_EMPTY;
  } /* static XS (Collectd_plugin_dispatch_values) */
  
 +/*
 + * Collectd::plugin_dispatch_notification (notif).
 + *
 + * notif:
 + *   notification to dispatch
 + */
 +static XS (Collectd_plugin_dispatch_notification)
 +{
 +      SV *notif = NULL;
 +
 +      int ret = 0;
 +
 +      dXSARGS;
 +
 +      if (1 != items) {
 +              log_err ("Usage: Collectd::plugin_dispatch_notification(notif)");
 +              XSRETURN_EMPTY;
 +      }
 +
 +      log_debug ("Collectd::plugin_dispatch_notification: notif = \"%s\"",
 +                      SvPV_nolen (ST (0)));
 +
 +      notif = ST (0);
 +
 +      if (! (SvROK (notif) && (SVt_PVHV == SvTYPE (SvRV (notif))))) {
 +              log_err ("Collectd::plugin_dispatch_notification: Invalid notif.");
 +              XSRETURN_EMPTY;
 +      }
 +
 +      ret = pplugin_dispatch_notification (aTHX_ (HV *)SvRV (notif));
 +
 +      if (0 == ret)
 +              XSRETURN_YES;
 +      else
 +              XSRETURN_EMPTY;
 +} /* static XS (Collectd_plugin_dispatch_notification) */
 +
  /*
   * Collectd::plugin_log (level, message).
   *
@@@ -949,305 -703,71 +949,305 @@@ static XS (Collectd_plugin_log
        XSRETURN_YES;
  } /* static XS (Collectd_plugin_log) */
  
 +/*
 + * Collectd::call_by_name (...).
 + *
 + * Call a Perl sub identified by its name passed through $Collectd::cb_name.
 + */
 +static XS (Collectd_call_by_name)
 +{
 +      SV   *tmp  = NULL;
 +      char *name = NULL;
 +
 +      if (NULL == (tmp = get_sv ("Collectd::cb_name", 0))) {
 +              sv_setpv (get_sv ("@", 1), "cb_name has not been set");
 +              CLEAR_STACK_FRAME;
 +              return;
 +      }
 +
 +      name = SvPV_nolen (tmp);
 +
 +      if (NULL == get_cv (name, 0)) {
 +              sv_setpvf (get_sv ("@", 1), "unknown callback \"%s\"", name);
 +              CLEAR_STACK_FRAME;
 +              return;
 +      }
 +
 +      /* simply pass on the subroutine call without touching the stack,
 +       * thus leaving any arguments and return values in place */
 +      call_pv (name, 0);
 +} /* static XS (Collectd_call_by_name) */
 +
 +/*
 + * collectd's perl interpreter based thread implementation.
 + *
 + * This has been inspired by Perl's ithreads introduced in version 5.6.0.
 + */
 +
 +/* must be called with perl_threads->mutex locked */
 +static void c_ithread_destroy (c_ithread_t *ithread)
 +{
 +      dTHXa (ithread->interp);
 +
 +      assert (NULL != perl_threads);
 +
 +      PERL_SET_CONTEXT (aTHX);
 +      log_debug ("Shutting down Perl interpreter %p...", aTHX);
 +
 +#if COLLECT_DEBUG
 +      sv_report_used ();
 +
 +      --perl_threads->number_of_threads;
 +#endif /* COLLECT_DEBUG */
 +
 +      perl_destruct (aTHX);
 +      perl_free (aTHX);
 +
 +      if (NULL == ithread->prev)
 +              perl_threads->head = ithread->next;
 +      else
 +              ithread->prev->next = ithread->next;
 +
 +      if (NULL == ithread->next)
 +              perl_threads->tail = ithread->prev;
 +      else
 +              ithread->next->prev = ithread->prev;
 +
 +      sfree (ithread);
 +      return;
 +} /* static void c_ithread_destroy (c_ithread_t *) */
 +
 +static void c_ithread_destructor (void *arg)
 +{
 +      c_ithread_t *ithread = (c_ithread_t *)arg;
 +      c_ithread_t *t = NULL;
 +
 +      if (NULL == perl_threads)
 +              return;
 +
 +      pthread_mutex_lock (&perl_threads->mutex);
 +
 +      for (t = perl_threads->head; NULL != t; t = t->next)
 +              if (t == ithread)
 +                      break;
 +
 +      /* the ithread no longer exists */
 +      if (NULL == t)
 +              return;
 +
 +      c_ithread_destroy (ithread);
 +
 +      pthread_mutex_unlock (&perl_threads->mutex);
 +      return;
 +} /* static void c_ithread_destructor (void *) */
 +
 +/* must be called with perl_threads->mutex locked */
 +static c_ithread_t *c_ithread_create (PerlInterpreter *base)
 +{
 +      c_ithread_t *t = NULL;
 +      dTHXa (NULL);
 +
 +      assert (NULL != perl_threads);
 +
 +      t = (c_ithread_t *)smalloc (sizeof (c_ithread_t));
 +      memset (t, 0, sizeof (c_ithread_t));
 +
 +      t->interp = (NULL == base)
 +              ? NULL
 +              : perl_clone (base, CLONEf_KEEP_PTR_TABLE);
 +
 +      aTHX = t->interp;
 +
 +      if (NULL != base) {
 +              av_clear (PL_endav);
 +              av_undef (PL_endav);
 +              PL_endav = Nullav;
 +      }
 +
 +#if COLLECT_DEBUG
 +      ++perl_threads->number_of_threads;
 +#endif /* COLLECT_DEBUG */
 +
 +      t->next = NULL;
 +
 +      if (NULL == perl_threads->tail) {
 +              perl_threads->head = t;
 +              t->prev = NULL;
 +      }
 +      else {
 +              perl_threads->tail->next = t;
 +              t->prev = perl_threads->tail;
 +      }
 +
 +      perl_threads->tail = t;
 +
 +      pthread_setspecific (perl_thr_key, (const void *)t);
 +      return t;
 +} /* static c_ithread_t *c_ithread_create (PerlInterpreter *) */
 +
  /*
   * Interface to collectd.
   */
  
  static int perl_init (void)
  {
 -      if (NULL == perl)
 +      dTHX;
 +
 +      if (NULL == perl_threads)
                return 0;
  
 -      PERL_SET_CONTEXT (perl);
 -      return pplugin_call_all (PLUGIN_INIT);
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +
 +      log_debug ("perl_init: c_ithread: interp = %p (active threads: %i)",
 +                      aTHX, perl_threads->number_of_threads);
 +      return pplugin_call_all (aTHX_ PLUGIN_INIT);
  } /* static int perl_init (void) */
  
  static int perl_read (void)
  {
 -      if (NULL == perl)
 +      dTHX;
 +
 +      if (NULL == perl_threads)
                return 0;
  
 -      PERL_SET_CONTEXT (perl);
 -      return pplugin_call_all (PLUGIN_READ);
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +
 +      log_debug ("perl_read: c_ithread: interp = %p (active threads: %i)",
 +                      aTHX, perl_threads->number_of_threads);
 +      return pplugin_call_all (aTHX_ PLUGIN_READ);
  } /* static int perl_read (void) */
  
  static int perl_write (const data_set_t *ds, const value_list_t *vl)
  {
 -      if (NULL == perl)
 +      dTHX;
 +
 +      if (NULL == perl_threads)
                return 0;
  
 -      PERL_SET_CONTEXT (perl);
 -      return pplugin_call_all (PLUGIN_WRITE, ds, vl);
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +
 +      log_debug ("perl_write: c_ithread: interp = %p (active threads: %i)",
 +                      aTHX, perl_threads->number_of_threads);
 +      return pplugin_call_all (aTHX_ PLUGIN_WRITE, ds, vl);
  } /* static int perl_write (const data_set_t *, const value_list_t *) */
  
  static void perl_log (int level, const char *msg)
  {
 -      if (NULL == perl)
 +      dTHX;
 +
 +      if (NULL == perl_threads)
                return;
  
 -      PERL_SET_CONTEXT (perl);
 -      pplugin_call_all (PLUGIN_LOG, level, msg);
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +
 +      pplugin_call_all (aTHX_ PLUGIN_LOG, level, msg);
        return;
  } /* static void perl_log (int, const char *) */
  
 +static int perl_notify (const notification_t *notif)
 +{
 +      dTHX;
 +
 +      if (NULL == perl_threads)
 +              return 0;
 +
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +      return pplugin_call_all (aTHX_ PLUGIN_NOTIF, notif);
 +} /* static int perl_notify (const notification_t *) */
 +
  static int perl_shutdown (void)
  {
 +      c_ithread_t *t = NULL;
 +
        int ret = 0;
  
 +      dTHX;
 +
        plugin_unregister_complex_config ("perl");
  
 -      if (NULL == perl)
 +      if (NULL == perl_threads)
                return 0;
  
 +      if (NULL == aTHX) {
 +              c_ithread_t *t = NULL;
 +
 +              pthread_mutex_lock (&perl_threads->mutex);
 +              t = c_ithread_create (perl_threads->head->interp);
 +              pthread_mutex_unlock (&perl_threads->mutex);
 +
 +              aTHX = t->interp;
 +      }
 +
 +      log_debug ("perl_shutdown: c_ithread: interp = %p (active threads: %i)",
 +                      aTHX, perl_threads->number_of_threads);
 +
        plugin_unregister_log ("perl");
 +      plugin_unregister_notification ("perl");
        plugin_unregister_init ("perl");
        plugin_unregister_read ("perl");
        plugin_unregister_write ("perl");
  
 -      PERL_SET_CONTEXT (perl);
 -      ret = pplugin_call_all (PLUGIN_SHUTDOWN);
 +      ret = pplugin_call_all (aTHX_ PLUGIN_SHUTDOWN);
  
 -#if COLLECT_DEBUG
 -      Perl_sv_report_used (perl);
 -#endif /* COLLECT_DEBUG */
 +      pthread_mutex_lock (&perl_threads->mutex);
 +      t = perl_threads->tail;
 +
 +      while (NULL != t) {
 +              c_ithread_t *thr = t;
 +
 +              /* the pointer has to be advanced before destroying
 +               * the thread as this will free the memory */
 +              t = t->prev;
 +
 +              c_ithread_destroy (thr);
 +      }
  
 -      perl_destruct (perl);
 -      perl_free (perl);
 -      perl = NULL;
 +      pthread_mutex_unlock (&perl_threads->mutex);
 +      pthread_mutex_destroy (&perl_threads->mutex);
 +
 +      sfree (perl_threads);
 +
 +      pthread_key_delete (perl_thr_key);
  
        PERL_SYS_TERM ();
  
        return ret;
  } /* static void perl_shutdown (void) */
  
 +/*
 + * Access functions for global variables.
 + *
 + * These functions implement the "magic" used to access
 + * the global variables from Perl.
 + */
 +
 +static int g_pv_get (pTHX_ SV *var, MAGIC *mg)
 +{
 +      char *pv = mg->mg_ptr;
 +      sv_setpv (var, pv);
 +      return 0;
 +} /* static int g_pv_get (pTHX_ SV *, MAGIC *) */
 +
 +static int g_pv_set (pTHX_ SV *var, MAGIC *mg)
 +{
 +      char *pv = mg->mg_ptr;
 +      strncpy (pv, SvPV_nolen (var), DATA_MAX_NAME_LEN);
 +      pv[DATA_MAX_NAME_LEN - 1] = '\0';
 +      return 0;
 +} /* static int g_pv_set (pTHX_ SV *, MAGIC *) */
 +
 +static int g_iv_get (pTHX_ SV *var, MAGIC *mg)
 +{
 +      int *iv = (int *)mg->mg_ptr;
 +      sv_setiv (var, *iv);
 +      return 0;
 +} /* static int g_iv_get (pTHX_ SV *, MAGIC *) */
 +
 +static int g_iv_set (pTHX_ SV *var, MAGIC *mg)
 +{
 +      int *iv = (int *)mg->mg_ptr;
 +      *iv = (int)SvIV (var);
 +      return 0;
 +} /* static int g_iv_set (pTHX_ SV *, MAGIC *) */
 +
 +static MGVTBL g_pv_vtbl = { g_pv_get, g_pv_set, NULL, NULL, NULL, NULL, NULL };
 +static MGVTBL g_iv_vtbl = { g_iv_get, g_iv_set, NULL, NULL, NULL, NULL, NULL };
 +
  /* bootstrap the Collectd module */
  static void xs_init (pTHX)
  {
        HV   *stash = NULL;
 +      SV   *tmp   = NULL;
        char *file  = __FILE__;
  
        int i = 0;
        dXSUB_SYS;
  
        /* enable usage of Perl modules using shared libraries */
 -      Perl_newXS (perl, "DynaLoader::boot_DynaLoader", boot_DynaLoader, file);
 +      newXS ("DynaLoader::boot_DynaLoader", boot_DynaLoader, file);
  
        /* register API */
        for (i = 0; NULL != api[i].f; ++i)
 -              Perl_newXS (perl, api[i].name, api[i].f, file);
 +              newXS (api[i].name, api[i].f, file);
  
 -      stash = Perl_gv_stashpv (perl, "Collectd", 1);
 +      stash = gv_stashpv ("Collectd", 1);
  
        /* export "constants" */
        for (i = 0; '\0' != constants[i].name[0]; ++i)
 -              Perl_newCONSTSUB (perl, stash, constants[i].name,
 -                              Perl_newSViv (perl, constants[i].value));
 +              newCONSTSUB (stash, constants[i].name, newSViv (constants[i].value));
 +
 +      /* export global variables
 +       * by adding "magic" to the SV's representing the globale variables
 +       * perl is able to automagically call the get/set function when
 +       * accessing any such variable (this is basically the same as using
 +       * tie() in Perl) */
 +      /* global strings */
 +      for (i = 0; '\0' != g_strings[i].name[0]; ++i) {
 +              tmp = get_sv (g_strings[i].name, 1);
 +              sv_magicext (tmp, NULL, PERL_MAGIC_ext, &g_pv_vtbl,
 +                              g_strings[i].var, 0);
 +      }
 +
 +      /* global integers */
 +      for (i = 0; '\0' != g_integers[i].name[0]; ++i) {
 +              tmp = get_sv (g_integers[i].name, 1);
 +              sv_magicext (tmp, NULL, PERL_MAGIC_ext, &g_iv_vtbl,
 +                              (char *)g_integers[i].var, 0);
 +      }
        return;
  } /* static void xs_init (pTHX) */
  
  /* Initialize the global Perl interpreter. */
  static int init_pi (int argc, char **argv)
  {
 -      if (NULL != perl)
 +      dTHXa (NULL);
 +
 +      if (NULL != perl_threads)
                return 0;
  
        log_info ("Initializing Perl interpreter...");
        }
  #endif /* COLLECT_DEBUG */
  
 +      if (0 != pthread_key_create (&perl_thr_key, c_ithread_destructor)) {
 +              log_err ("init_pi: pthread_key_create failed");
 +
 +              /* this must not happen - cowardly giving up if it does */
 +              exit (1);
 +      }
 +
        PERL_SYS_INIT3 (&argc, &argv, &environ);
  
 -      if (NULL == (perl = perl_alloc ())) {
 -              log_err ("module_register: Not enough memory.");
 +      perl_threads = (c_ithread_list_t *)smalloc (sizeof (c_ithread_list_t));
 +      memset (perl_threads, 0, sizeof (c_ithread_list_t));
 +
 +      pthread_mutex_init (&perl_threads->mutex, NULL);
 +      /* locking the mutex should not be necessary at this point
 +       * but let's just do it for the sake of completeness */
 +      pthread_mutex_lock (&perl_threads->mutex);
 +
 +      perl_threads->head = c_ithread_create (NULL);
 +      perl_threads->tail = perl_threads->head;
 +
 +      if (NULL == (perl_threads->head->interp = perl_alloc ())) {
 +              log_err ("init_pi: Not enough memory.");
                exit (3);
        }
 -      perl_construct (perl);
 +
 +      aTHX = perl_threads->head->interp;
 +      pthread_mutex_unlock (&perl_threads->mutex);
 +
 +      perl_construct (aTHX);
  
        PL_exit_flags |= PERL_EXIT_DESTRUCT_END;
  
 -      if (0 != perl_parse (perl, xs_init, argc, argv, NULL)) {
 -              log_err ("module_register: Unable to bootstrap Collectd.");
 +      if (0 != perl_parse (aTHX_ xs_init, argc, argv, NULL)) {
 +              log_err ("init_pi: Unable to bootstrap Collectd.");
                exit (1);
        }
  
        /* Set $0 to "collectd" because perl_parse() has to set it to "-e". */
 -      Perl_sv_setpv (perl, Perl_get_sv (perl, "0", 0), "collectd");
 +      sv_setpv (get_sv ("0", 0), "collectd");
  
 -      perl_run (perl);
 +      perl_run (aTHX);
  
        plugin_register_log ("perl", perl_log);
 +      plugin_register_notification ("perl", perl_notify);
        plugin_register_init ("perl", perl_init);
  
        plugin_register_read ("perl", perl_read);
  /*
   * LoadPlugin "<Plugin>"
   */
 -static int perl_config_loadplugin (oconfig_item_t *ci)
 +static int perl_config_loadplugin (pTHX_ oconfig_item_t *ci)
  {
        char module_name[DATA_MAX_NAME_LEN];
  
        char *value = NULL;
  
        if ((0 != ci->children_num) || (1 != ci->values_num)
 -                      || (OCONFIG_TYPE_STRING != ci->values[0].type))
 +                      || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
 +              log_err ("LoadPlugin expects a single string argument.");
                return 1;
 +      }
  
        value = ci->values[0].value.string;
  
        }
  
        init_pi (perl_argc, perl_argv);
 +      assert (NULL != perl_threads);
 +      assert (NULL != perl_threads->head);
 +
 +      aTHX = perl_threads->head->interp;
  
        log_debug ("perl_config: loading perl plugin \"%s\"", value);
 -      Perl_load_module (perl, PERL_LOADMOD_NOIMPORT,
 -                      Perl_newSVpv (perl, module_name, strlen (module_name)),
 -                      Nullsv);
 +      load_module (PERL_LOADMOD_NOIMPORT,
 +                      newSVpv (module_name, strlen (module_name)), Nullsv);
        return 0;
  } /* static int perl_config_loadplugin (oconfig_item_it *) */
  
  /*
   * BaseName "<Name>"
   */
 -static int perl_config_basename (oconfig_item_t *ci)
 +static int perl_config_basename (pTHX_ oconfig_item_t *ci)
  {
        char *value = NULL;
  
        if ((0 != ci->children_num) || (1 != ci->values_num)
 -                      || (OCONFIG_TYPE_STRING != ci->values[0].type))
 +                      || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
 +              log_err ("BaseName expects a single string argument.");
                return 1;
 +      }
  
        value = ci->values[0].value.string;
  
  /*
   * EnableDebugger "<Package>"|""
   */
 -static int perl_config_enabledebugger (oconfig_item_t *ci)
 +static int perl_config_enabledebugger (pTHX_ oconfig_item_t *ci)
  {
        char *value = NULL;
  
        if ((0 != ci->children_num) || (1 != ci->values_num)
 -                      || (OCONFIG_TYPE_STRING != ci->values[0].type))
 +                      || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
 +              log_err ("EnableDebugger expects a single string argument.");
                return 1;
 +      }
  
        value = ci->values[0].value.string;
  
  /*
   * IncludeDir "<Dir>"
   */
 -static int perl_config_includedir (oconfig_item_t *ci)
 +static int perl_config_includedir (pTHX_ oconfig_item_t *ci)
  {
        char *value = NULL;
  
        if ((0 != ci->children_num) || (1 != ci->values_num)
 -                      || (OCONFIG_TYPE_STRING != ci->values[0].type))
 +                      || (OCONFIG_TYPE_STRING != ci->values[0].type)) {
 +              log_err ("IncludeDir expects a single string argument.");
                return 1;
 +      }
  
+       if (NULL == aTHX) {
+               log_warn ("EnableDebugger has no effects if used after LoadPlugin.");
+               return 1;
+       }
        value = ci->values[0].value.string;
  
 -      if (NULL == perl) {
 +      if (NULL == aTHX) {
                perl_argv = (char **)realloc (perl_argv,
                                (++perl_argc + 1) * sizeof (char *));
  
        }
        else {
                /* prepend the directory to @INC */
 -              Perl_av_unshift (perl, GvAVn (PL_incgv), 1);
 -              Perl_av_store (perl, GvAVn (PL_incgv),
 -                              0, Perl_newSVpv (perl, value, strlen (value)));
 +              av_unshift (GvAVn (PL_incgv), 1);
 +              av_store (GvAVn (PL_incgv), 0, newSVpv (value, strlen (value)));
        }
        return 0;
  } /* static int perl_config_includedir (oconfig_item_it *) */
@@@ -1543,24 -975,17 +1548,24 @@@ static int perl_config (oconfig_item_t 
  {
        int i = 0;
  
 +      dTHX;
 +
 +      /* dTHX does not get any valid values in case Perl
 +       * has not been initialized */
 +      if (NULL == perl_threads)
 +              aTHX = NULL;
 +
        for (i = 0; i < ci->children_num; ++i) {
                oconfig_item_t *c = ci->children + i;
  
                if (0 == strcasecmp (c->key, "LoadPlugin"))
 -                      perl_config_loadplugin (c);
 +                      perl_config_loadplugin (aTHX_ c);
                else if (0 == strcasecmp (c->key, "BaseName"))
 -                      perl_config_basename (c);
 +                      perl_config_basename (aTHX_ c);
                else if (0 == strcasecmp (c->key, "EnableDebugger"))
 -                      perl_config_enabledebugger (c);
 +                      perl_config_enabledebugger (aTHX_ c);
                else if (0 == strcasecmp (c->key, "IncludeDir"))
 -                      perl_config_includedir (c);
 +                      perl_config_includedir (aTHX_ c);
                else
                        log_warn ("Ignoring unknown config key \"%s\".", c->key);
        }