Merge branch 'collectd-5.7' into collectd-5.8
[collectd.git] / src / memory.c
index 16b8e09..e49fe84 100644 (file)
 #include "plugin.h"
 
 #ifdef HAVE_SYS_SYSCTL_H
-# include <sys/sysctl.h>
+#include <sys/sysctl.h>
 #endif
 #ifdef HAVE_SYS_VMMETER_H
-# include <sys/vmmeter.h>
+#include <sys/vmmeter.h>
 #endif
 
 #ifdef HAVE_MACH_KERN_RETURN_H
-# include <mach/kern_return.h>
+#include <mach/kern_return.h>
 #endif
 #ifdef HAVE_MACH_MACH_INIT_H
-# include <mach/mach_init.h>
+#include <mach/mach_init.h>
 #endif
 #ifdef HAVE_MACH_MACH_HOST_H
-# include <mach/mach_host.h>
+#include <mach/mach_host.h>
 #endif
 #ifdef HAVE_MACH_HOST_PRIV_H
-# include <mach/host_priv.h>
+#include <mach/host_priv.h>
 #endif
 #ifdef HAVE_MACH_VM_STATISTICS_H
-# include <mach/vm_statistics.h>
+#include <mach/vm_statistics.h>
 #endif
 
 #if HAVE_STATGRAB_H
-# include <statgrab.h>
+#include <statgrab.h>
 #endif
 
 #if HAVE_PERFSTAT
-# include <sys/protosw.h>
-# include <libperfstat.h>
+#include <libperfstat.h>
+#include <sys/protosw.h>
 #endif /* HAVE_PERFSTAT */
 
 /* vm_statistics_data_t */
@@ -91,34 +91,33 @@ static int pagesize;
 static int pagesize;
 /* endif HAVE_PERFSTAT */
 #else
-# error "No applicable input method."
+#error "No applicable input method."
 #endif
 
 static _Bool values_absolute = 1;
 static _Bool values_percentage = 0;
 
-static int memory_config (oconfig_item_t *ci) /* {{{ */
+static int memory_config(oconfig_item_t *ci) /* {{{ */
 {
-       for (int i = 0; i < ci->children_num; i++)
-       {
-               oconfig_item_t *child = ci->children + i;
-               if (strcasecmp ("ValuesAbsolute", child->key) == 0)
-                       cf_util_get_boolean (child, &values_absolute);
-               else if (strcasecmp ("ValuesPercentage", child->key) == 0)
-                       cf_util_get_boolean (child, &values_percentage);
-               else
-                       ERROR ("memory plugin: Invalid configuration option: "
-                                       "\"%s\".", child->key);
-       }
-
-       return (0);
+  for (int i = 0; i < ci->children_num; i++) {
+    oconfig_item_t *child = ci->children + i;
+    if (strcasecmp("ValuesAbsolute", child->key) == 0)
+      cf_util_get_boolean(child, &values_absolute);
+    else if (strcasecmp("ValuesPercentage", child->key) == 0)
+      cf_util_get_boolean(child, &values_percentage);
+    else
+      ERROR("memory plugin: Invalid configuration option: "
+            "\"%s\".",
+            child->key);
+  }
+
+  return 0;
 } /* }}} int memory_config */
 
-static int memory_init (void)
-{
+static int memory_init(void) {
 #if HAVE_HOST_STATISTICS
-       port_host = mach_host_self ();
-       host_page_size (port_host, &pagesize);
+  port_host = mach_host_self();
+  host_page_size(port_host, &pagesize);
 /* #endif HAVE_HOST_STATISTICS */
 
 #elif HAVE_SYSCTLBYNAME
@@ -130,28 +129,25 @@ static int memory_init (void)
 /* #endif KERNEL_LINUX */
 
 #elif defined(HAVE_LIBKSTAT)
-       /* getpagesize(3C) tells me this does not fail.. */
-       pagesize = getpagesize ();
-       if (get_kstat (&ksp, "unix", 0, "system_pages") != 0)
-       {
-               ksp = NULL;
-               return (-1);
-       }
-       if (get_kstat (&ksz, "zfs", 0, "arcstats") != 0)
-       {
-               ksz = NULL;
-               return (-1);
-       }
+  /* getpagesize(3C) tells me this does not fail.. */
+  pagesize = getpagesize();
+  if (get_kstat(&ksp, "unix", 0, "system_pages") != 0) {
+    ksp = NULL;
+    return -1;
+  }
+  if (get_kstat(&ksz, "zfs", 0, "arcstats") != 0) {
+    ksz = NULL;
+    return -1;
+  }
 
 /* #endif HAVE_LIBKSTAT */
 
 #elif HAVE_SYSCTL
-       pagesize = getpagesize ();
-       if (pagesize <= 0)
-       {
-               ERROR ("memory plugin: Invalid pagesize: %i", pagesize);
-               return (-1);
-       }
+  pagesize = getpagesize();
+  if (pagesize <= 0) {
+    ERROR("memory plugin: Invalid pagesize: %i", pagesize);
+    return -1;
+  }
 /* #endif HAVE_SYSCTL */
 
 #elif HAVE_LIBSTATGRAB
@@ -159,385 +155,343 @@ static int memory_init (void)
 /* #endif HAVE_LIBSTATGRAB */
 
 #elif HAVE_PERFSTAT
-       pagesize = getpagesize ();
+  pagesize = getpagesize();
 #endif /* HAVE_PERFSTAT */
-       return (0);
+  return 0;
 } /* int memory_init */
 
-#define MEMORY_SUBMIT(...) do { \
-       if (values_absolute) \
-               plugin_dispatch_multivalue (vl, 0, DS_TYPE_GAUGE, __VA_ARGS__, NULL); \
-       if (values_percentage) \
-               plugin_dispatch_multivalue (vl, 1, DS_TYPE_GAUGE, __VA_ARGS__, NULL); \
-} while (0)
+#define MEMORY_SUBMIT(...)                                                     \
+  do {                                                                         \
+    if (values_absolute)                                                       \
+      plugin_dispatch_multivalue(vl, 0, DS_TYPE_GAUGE, __VA_ARGS__, NULL);     \
+    if (values_percentage)                                                     \
+      plugin_dispatch_multivalue(vl, 1, DS_TYPE_GAUGE, __VA_ARGS__, NULL);     \
+  } while (0)
 
-static int memory_read_internal (value_list_t *vl)
-{
+static int memory_read_internal(value_list_t *vl) {
 #if HAVE_HOST_STATISTICS
-       kern_return_t status;
-       vm_statistics_data_t   vm_data;
-       mach_msg_type_number_t vm_data_len;
-
-       gauge_t wired;
-       gauge_t active;
-       gauge_t inactive;
-       gauge_t free;
-
-       if (!port_host || !pagesize)
-               return (-1);
-
-       vm_data_len = sizeof (vm_data) / sizeof (natural_t);
-       if ((status = host_statistics (port_host, HOST_VM_INFO,
-                                       (host_info_t) &vm_data,
-                                       &vm_data_len)) != KERN_SUCCESS)
-       {
-               ERROR ("memory-plugin: host_statistics failed and returned the value %i", (int) status);
-               return (-1);
-       }
-
-       /*
-        * From <http://docs.info.apple.com/article.html?artnum=107918>:
-        *
-        * Wired memory
-        *   This information can't be cached to disk, so it must stay in RAM.
-        *   The amount depends on what applications you are using.
-        *
-        * Active memory
-        *   This information is currently in RAM and actively being used.
-        *
-        * Inactive memory
-        *   This information is no longer being used and has been cached to
-        *   disk, but it will remain in RAM until another application needs
-        *   the space. Leaving this information in RAM is to your advantage if
-        *   you (or a client of your computer) come back to it later.
-        *
-        * Free memory
-        *   This memory is not being used.
-        */
-
-       wired    = (gauge_t) (((uint64_t) vm_data.wire_count)     * ((uint64_t) pagesize));
-       active   = (gauge_t) (((uint64_t) vm_data.active_count)   * ((uint64_t) pagesize));
-       inactive = (gauge_t) (((uint64_t) vm_data.inactive_count) * ((uint64_t) pagesize));
-       free     = (gauge_t) (((uint64_t) vm_data.free_count)     * ((uint64_t) pagesize));
-
-       MEMORY_SUBMIT ("wired",    wired,
-                      "active",   active,
-                      "inactive", inactive,
-                      "free",     free);
+  kern_return_t status;
+  vm_statistics_data_t vm_data;
+  mach_msg_type_number_t vm_data_len;
+
+  gauge_t wired;
+  gauge_t active;
+  gauge_t inactive;
+  gauge_t free;
+
+  if (!port_host || !pagesize)
+    return -1;
+
+  vm_data_len = sizeof(vm_data) / sizeof(natural_t);
+  if ((status = host_statistics(port_host, HOST_VM_INFO, (host_info_t)&vm_data,
+                                &vm_data_len)) != KERN_SUCCESS) {
+    ERROR("memory-plugin: host_statistics failed and returned the value %i",
+          (int)status);
+    return -1;
+  }
+
+  /*
+   * From <http://docs.info.apple.com/article.html?artnum=107918>:
+   *
+   * Wired memory
+   *   This information can't be cached to disk, so it must stay in RAM.
+   *   The amount depends on what applications you are using.
+   *
+   * Active memory
+   *   This information is currently in RAM and actively being used.
+   *
+   * Inactive memory
+   *   This information is no longer being used and has been cached to
+   *   disk, but it will remain in RAM until another application needs
+   *   the space. Leaving this information in RAM is to your advantage if
+   *   you (or a client of your computer) come back to it later.
+   *
+   * Free memory
+   *   This memory is not being used.
+   */
+
+  wired = (gauge_t)(((uint64_t)vm_data.wire_count) * ((uint64_t)pagesize));
+  active = (gauge_t)(((uint64_t)vm_data.active_count) * ((uint64_t)pagesize));
+  inactive =
+      (gauge_t)(((uint64_t)vm_data.inactive_count) * ((uint64_t)pagesize));
+  free = (gauge_t)(((uint64_t)vm_data.free_count) * ((uint64_t)pagesize));
+
+  MEMORY_SUBMIT("wired", wired, "active", active, "inactive", inactive, "free",
+                free);
 /* #endif HAVE_HOST_STATISTICS */
 
 #elif HAVE_SYSCTLBYNAME
-       /*
-        * vm.stats.vm.v_page_size: 4096
-        * vm.stats.vm.v_page_count: 246178
-        * vm.stats.vm.v_free_count: 28760
-        * vm.stats.vm.v_wire_count: 37526
-        * vm.stats.vm.v_active_count: 55239
-        * vm.stats.vm.v_inactive_count: 113730
-        * vm.stats.vm.v_cache_count: 10809
-        */
-       const char *sysctl_keys[8] =
-       {
-               "vm.stats.vm.v_page_size",
-               "vm.stats.vm.v_page_count",
-               "vm.stats.vm.v_free_count",
-               "vm.stats.vm.v_wire_count",
-               "vm.stats.vm.v_active_count",
-               "vm.stats.vm.v_inactive_count",
-               "vm.stats.vm.v_cache_count",
-               NULL
-       };
-       double sysctl_vals[8];
-
-       for (int i = 0; sysctl_keys[i] != NULL; i++)
-       {
-               int value;
-               size_t value_len = sizeof (value);
-
-               if (sysctlbyname (sysctl_keys[i], (void *) &value, &value_len,
-                                       NULL, 0) == 0)
-               {
-                       sysctl_vals[i] = value;
-                       DEBUG ("memory plugin: %26s: %g", sysctl_keys[i], sysctl_vals[i]);
-               }
-               else
-               {
-                       sysctl_vals[i] = NAN;
-               }
-       } /* for (sysctl_keys) */
-
-       /* multiply all all page counts with the pagesize */
-       for (int i = 1; sysctl_keys[i] != NULL; i++)
-               if (!isnan (sysctl_vals[i]))
-                       sysctl_vals[i] *= sysctl_vals[0];
-
-       MEMORY_SUBMIT ("free",     (gauge_t) sysctl_vals[2],
-                      "wired",    (gauge_t) sysctl_vals[3],
-                      "active",   (gauge_t) sysctl_vals[4],
-                      "inactive", (gauge_t) sysctl_vals[5],
-                      "cache",    (gauge_t) sysctl_vals[6]);
+  /*
+   * vm.stats.vm.v_page_size: 4096
+   * vm.stats.vm.v_page_count: 246178
+   * vm.stats.vm.v_free_count: 28760
+   * vm.stats.vm.v_wire_count: 37526
+   * vm.stats.vm.v_active_count: 55239
+   * vm.stats.vm.v_inactive_count: 113730
+   * vm.stats.vm.v_cache_count: 10809
+   */
+  const char *sysctl_keys[8] = {
+      "vm.stats.vm.v_page_size",    "vm.stats.vm.v_page_count",
+      "vm.stats.vm.v_free_count",   "vm.stats.vm.v_wire_count",
+      "vm.stats.vm.v_active_count", "vm.stats.vm.v_inactive_count",
+      "vm.stats.vm.v_cache_count",  NULL};
+  double sysctl_vals[8];
+
+  for (int i = 0; sysctl_keys[i] != NULL; i++) {
+    int value;
+    size_t value_len = sizeof(value);
+
+    if (sysctlbyname(sysctl_keys[i], (void *)&value, &value_len, NULL, 0) ==
+        0) {
+      sysctl_vals[i] = value;
+      DEBUG("memory plugin: %26s: %g", sysctl_keys[i], sysctl_vals[i]);
+    } else {
+      sysctl_vals[i] = NAN;
+    }
+  } /* for (sysctl_keys) */
+
+  /* multiply all all page counts with the pagesize */
+  for (int i = 1; sysctl_keys[i] != NULL; i++)
+    if (!isnan(sysctl_vals[i]))
+      sysctl_vals[i] *= sysctl_vals[0];
+
+  MEMORY_SUBMIT("free", (gauge_t)sysctl_vals[2], "wired",
+                (gauge_t)sysctl_vals[3], "active", (gauge_t)sysctl_vals[4],
+                "inactive", (gauge_t)sysctl_vals[5], "cache",
+                (gauge_t)sysctl_vals[6]);
 /* #endif HAVE_SYSCTLBYNAME */
 
 #elif KERNEL_LINUX
-       FILE *fh;
-       char buffer[1024];
-
-       char *fields[8];
-       int numfields;
-
-       _Bool detailed_slab_info = 0;
-
-       gauge_t mem_total = 0;
-       gauge_t mem_used = 0;
-       gauge_t mem_buffered = 0;
-       gauge_t mem_cached = 0;
-       gauge_t mem_free = 0;
-       gauge_t mem_slab_total = 0;
-       gauge_t mem_slab_reclaimable = 0;
-       gauge_t mem_slab_unreclaimable = 0;
-
-       if ((fh = fopen ("/proc/meminfo", "r")) == NULL)
-       {
-               char errbuf[1024];
-               WARNING ("memory: fopen: %s",
-                               sstrerror (errno, errbuf, sizeof (errbuf)));
-               return (-1);
-       }
-
-       while (fgets (buffer, sizeof (buffer), fh) != NULL)
-       {
-               gauge_t *val = NULL;
-
-               if (strncasecmp (buffer, "MemTotal:", 9) == 0)
-                       val = &mem_total;
-               else if (strncasecmp (buffer, "MemFree:", 8) == 0)
-                       val = &mem_free;
-               else if (strncasecmp (buffer, "Buffers:", 8) == 0)
-                       val = &mem_buffered;
-               else if (strncasecmp (buffer, "Cached:", 7) == 0)
-                       val = &mem_cached;
-               else if (strncasecmp (buffer, "Slab:", 5) == 0)
-                       val = &mem_slab_total;
-               else if (strncasecmp (buffer, "SReclaimable:", 13) == 0) {
-                       val = &mem_slab_reclaimable;
-                       detailed_slab_info = 1;
-               }
-               else if (strncasecmp (buffer, "SUnreclaim:", 11) == 0) {
-                       val = &mem_slab_unreclaimable;
-                       detailed_slab_info = 1;
-               }
-               else
-                       continue;
-
-               numfields = strsplit (buffer, fields, STATIC_ARRAY_SIZE (fields));
-               if (numfields < 2)
-                       continue;
-
-               *val = 1024.0 * atof (fields[1]);
-       }
-
-       if (fclose (fh))
-       {
-               char errbuf[1024];
-               WARNING ("memory: fclose: %s",
-                               sstrerror (errno, errbuf, sizeof (errbuf)));
-       }
-
-       if (mem_total < (mem_free + mem_buffered + mem_cached + mem_slab_total))
-               return (-1);
-
-       mem_used = mem_total - (mem_free + mem_buffered + mem_cached + mem_slab_total);
-
-       /* SReclaimable and SUnreclaim were introduced in kernel 2.6.19
-        * They sum up to the value of Slab, which is available on older & newer
-        * kernels. So SReclaimable/SUnreclaim are submitted if available, and Slab
-        * if not. */
-       if (detailed_slab_info)
-               MEMORY_SUBMIT ("used",        mem_used,
-                              "buffered",    mem_buffered,
-                              "cached",      mem_cached,
-                              "free",        mem_free,
-                              "slab_unrecl", mem_slab_unreclaimable,
-                              "slab_recl",   mem_slab_reclaimable);
-       else
-               MEMORY_SUBMIT ("used",     mem_used,
-                              "buffered", mem_buffered,
-                              "cached",   mem_cached,
-                              "free",     mem_free,
-                              "slab",     mem_slab_total);
+  FILE *fh;
+  char buffer[1024];
+
+  char *fields[8];
+  int numfields;
+
+  _Bool detailed_slab_info = 0;
+
+  gauge_t mem_total = 0;
+  gauge_t mem_used = 0;
+  gauge_t mem_buffered = 0;
+  gauge_t mem_cached = 0;
+  gauge_t mem_free = 0;
+  gauge_t mem_slab_total = 0;
+  gauge_t mem_slab_reclaimable = 0;
+  gauge_t mem_slab_unreclaimable = 0;
+
+  if ((fh = fopen("/proc/meminfo", "r")) == NULL) {
+    char errbuf[1024];
+    WARNING("memory: fopen: %s", sstrerror(errno, errbuf, sizeof(errbuf)));
+    return -1;
+  }
+
+  while (fgets(buffer, sizeof(buffer), fh) != NULL) {
+    gauge_t *val = NULL;
+
+    if (strncasecmp(buffer, "MemTotal:", 9) == 0)
+      val = &mem_total;
+    else if (strncasecmp(buffer, "MemFree:", 8) == 0)
+      val = &mem_free;
+    else if (strncasecmp(buffer, "Buffers:", 8) == 0)
+      val = &mem_buffered;
+    else if (strncasecmp(buffer, "Cached:", 7) == 0)
+      val = &mem_cached;
+    else if (strncasecmp(buffer, "Slab:", 5) == 0)
+      val = &mem_slab_total;
+    else if (strncasecmp(buffer, "SReclaimable:", 13) == 0) {
+      val = &mem_slab_reclaimable;
+      detailed_slab_info = 1;
+    } else if (strncasecmp(buffer, "SUnreclaim:", 11) == 0) {
+      val = &mem_slab_unreclaimable;
+      detailed_slab_info = 1;
+    } else
+      continue;
+
+    numfields = strsplit(buffer, fields, STATIC_ARRAY_SIZE(fields));
+    if (numfields < 2)
+      continue;
+
+    *val = 1024.0 * atof(fields[1]);
+  }
+
+  if (fclose(fh)) {
+    char errbuf[1024];
+    WARNING("memory: fclose: %s", sstrerror(errno, errbuf, sizeof(errbuf)));
+  }
+
+  if (mem_total < (mem_free + mem_buffered + mem_cached + mem_slab_total))
+    return -1;
+
+  mem_used =
+      mem_total - (mem_free + mem_buffered + mem_cached + mem_slab_total);
+
+  /* SReclaimable and SUnreclaim were introduced in kernel 2.6.19
+   * They sum up to the value of Slab, which is available on older & newer
+   * kernels. So SReclaimable/SUnreclaim are submitted if available, and Slab
+   * if not. */
+  if (detailed_slab_info)
+    MEMORY_SUBMIT("used", mem_used, "buffered", mem_buffered, "cached",
+                  mem_cached, "free", mem_free, "slab_unrecl",
+                  mem_slab_unreclaimable, "slab_recl", mem_slab_reclaimable);
+  else
+    MEMORY_SUBMIT("used", mem_used, "buffered", mem_buffered, "cached",
+                  mem_cached, "free", mem_free, "slab", mem_slab_total);
 /* #endif KERNEL_LINUX */
 
 #elif HAVE_LIBKSTAT
-       /* Most of the additions here were taken as-is from the k9toolkit from
-        * Brendan Gregg and are subject to change I guess */
-       long long mem_used;
-       long long mem_free;
-       long long mem_lock;
-       long long mem_kern;
-       long long mem_unus;
-       long long arcsize;
-
-
-       long long pp_kernel;
-       long long physmem;
-       long long availrmem;
-
-       if (ksp == NULL)
-               return (-1);
-       if (ksz == NULL)
-               return (-1);
-
-       mem_used = get_kstat_value (ksp, "pagestotal");
-       mem_free = get_kstat_value (ksp, "pagesfree");
-       mem_lock = get_kstat_value (ksp, "pageslocked");
-       arcsize = get_kstat_value (ksz, "size");
-       pp_kernel = get_kstat_value (ksp, "pp_kernel");
-       physmem = get_kstat_value (ksp, "physmem");
-       availrmem = get_kstat_value (ksp, "availrmem");
-
-       mem_kern = 0;
-       mem_unus = 0;
-
-       if ((mem_used < 0LL) || (mem_free < 0LL) || (mem_lock < 0LL))
-       {
-               WARNING ("memory plugin: one of used, free or locked is negative.");
-               return (-1);
-       }
-
-       mem_unus = physmem - mem_used;
-
-       if (mem_used < (mem_free + mem_lock))
-       {
-               /* source: http://wesunsolve.net/bugid/id/4909199
-                * this seems to happen when swap space is small, e.g. 2G on a 32G system
-                * we will make some assumptions here
-                * educated solaris internals help welcome here */
-               DEBUG ("memory plugin: pages total is smaller than \"free\" "
-                               "+ \"locked\". This is probably due to small "
-                               "swap space");
-               mem_free = availrmem;
-               mem_used = 0;
-       }
-       else
-       {
-               mem_used -= mem_free + mem_lock;
-       }
-
-       /* mem_kern is accounted for in mem_lock */
-       if (pp_kernel < mem_lock)
-       {
-               mem_kern = pp_kernel;
-               mem_lock -= pp_kernel;
-       }
-       else
-       {
-               mem_kern = mem_lock;
-               mem_lock = 0;
-       }
-
-       mem_used *= pagesize; /* If this overflows you have some serious */
-       mem_free *= pagesize; /* memory.. Why not call me up and give me */
-       mem_lock *= pagesize; /* some? ;) */
-       mem_kern *= pagesize; /* it's 2011 RAM is cheap */
-       mem_unus *= pagesize;
-       mem_kern -= arcsize;
-
-
-       MEMORY_SUBMIT ("used",     (gauge_t) mem_used,
-                      "free",     (gauge_t) mem_free,
-                      "locked",   (gauge_t) mem_lock,
-                      "kernel",   (gauge_t) mem_kern,
-                      "arc",      (gauge_t) arcsize,
-                      "unusable", (gauge_t) mem_unus);
+  /* Most of the additions here were taken as-is from the k9toolkit from
+   * Brendan Gregg and are subject to change I guess */
+  long long mem_used;
+  long long mem_free;
+  long long mem_lock;
+  long long mem_kern;
+  long long mem_unus;
+  long long arcsize;
+
+  long long pp_kernel;
+  long long physmem;
+  long long availrmem;
+
+  if (ksp == NULL)
+    return -1;
+  if (ksz == NULL)
+    return -1;
+
+  mem_used = get_kstat_value(ksp, "pagestotal");
+  mem_free = get_kstat_value(ksp, "pagesfree");
+  mem_lock = get_kstat_value(ksp, "pageslocked");
+  arcsize = get_kstat_value(ksz, "size");
+  pp_kernel = get_kstat_value(ksp, "pp_kernel");
+  physmem = get_kstat_value(ksp, "physmem");
+  availrmem = get_kstat_value(ksp, "availrmem");
+
+  mem_kern = 0;
+  mem_unus = 0;
+
+  if ((mem_used < 0LL) || (mem_free < 0LL) || (mem_lock < 0LL)) {
+    WARNING("memory plugin: one of used, free or locked is negative.");
+    return -1;
+  }
+
+  mem_unus = physmem - mem_used;
+
+  if (mem_used < (mem_free + mem_lock)) {
+    /* source: http://wesunsolve.net/bugid/id/4909199
+     * this seems to happen when swap space is small, e.g. 2G on a 32G system
+     * we will make some assumptions here
+     * educated solaris internals help welcome here */
+    DEBUG("memory plugin: pages total is smaller than \"free\" "
+          "+ \"locked\". This is probably due to small "
+          "swap space");
+    mem_free = availrmem;
+    mem_used = 0;
+  } else {
+    mem_used -= mem_free + mem_lock;
+  }
+
+  /* mem_kern is accounted for in mem_lock */
+  if (pp_kernel < mem_lock) {
+    mem_kern = pp_kernel;
+    mem_lock -= pp_kernel;
+  } else {
+    mem_kern = mem_lock;
+    mem_lock = 0;
+  }
+
+  mem_used *= pagesize; /* If this overflows you have some serious */
+  mem_free *= pagesize; /* memory.. Why not call me up and give me */
+  mem_lock *= pagesize; /* some? ;) */
+  mem_kern *= pagesize; /* it's 2011 RAM is cheap */
+  mem_unus *= pagesize;
+  mem_kern -= arcsize;
+
+  MEMORY_SUBMIT("used", (gauge_t)mem_used, "free", (gauge_t)mem_free, "locked",
+                (gauge_t)mem_lock, "kernel", (gauge_t)mem_kern, "arc",
+                (gauge_t)arcsize, "unusable", (gauge_t)mem_unus);
 /* #endif HAVE_LIBKSTAT */
 
 #elif HAVE_SYSCTL
-       int mib[] = {CTL_VM, VM_METER};
-       struct vmtotal vmtotal = { 0 };
-       gauge_t mem_active;
-       gauge_t mem_inactive;
-       gauge_t mem_free;
-       size_t size;
-
-       size = sizeof (vmtotal);
-
-       if (sysctl (mib, 2, &vmtotal, &size, NULL, 0) < 0) {
-               char errbuf[1024];
-               WARNING ("memory plugin: sysctl failed: %s",
-                       sstrerror (errno, errbuf, sizeof (errbuf)));
-               return (-1);
-       }
-
-       assert (pagesize > 0);
-       mem_active   = (gauge_t) (vmtotal.t_arm * pagesize);
-       mem_inactive = (gauge_t) ((vmtotal.t_rm - vmtotal.t_arm) * pagesize);
-       mem_free     = (gauge_t) (vmtotal.t_free * pagesize);
-
-       MEMORY_SUBMIT ("active",   mem_active,
-                      "inactive", mem_inactive,
-                      "free",     mem_free);
+  int mib[] = {CTL_VM, VM_METER};
+  struct vmtotal vmtotal = {0};
+  gauge_t mem_active;
+  gauge_t mem_inactive;
+  gauge_t mem_free;
+  size_t size;
+
+  size = sizeof(vmtotal);
+
+  if (sysctl(mib, 2, &vmtotal, &size, NULL, 0) < 0) {
+    char errbuf[1024];
+    WARNING("memory plugin: sysctl failed: %s",
+            sstrerror(errno, errbuf, sizeof(errbuf)));
+    return -1;
+  }
+
+  assert(pagesize > 0);
+  mem_active = (gauge_t)(vmtotal.t_arm * pagesize);
+  mem_inactive = (gauge_t)((vmtotal.t_rm - vmtotal.t_arm) * pagesize);
+  mem_free = (gauge_t)(vmtotal.t_free * pagesize);
+
+  MEMORY_SUBMIT("active", mem_active, "inactive", mem_inactive, "free",
+                mem_free);
 /* #endif HAVE_SYSCTL */
 
 #elif HAVE_LIBSTATGRAB
-       sg_mem_stats *ios;
+  sg_mem_stats *ios;
 
-       ios = sg_get_mem_stats ();
-       if (ios == NULL)
-               return (-1);
+  ios = sg_get_mem_stats();
+  if (ios == NULL)
+    return -1;
 
-       MEMORY_SUBMIT ("used",   (gauge_t) ios->used,
-                      "cached", (gauge_t) ios->cache,
-                      "free",   (gauge_t) ios->free);
+  MEMORY_SUBMIT("used", (gauge_t)ios->used, "cached", (gauge_t)ios->cache,
+                "free", (gauge_t)ios->free);
 /* #endif HAVE_LIBSTATGRAB */
 
 #elif HAVE_PERFSTAT
-       perfstat_memory_total_t pmemory = { 0 };
-
-       if (perfstat_memory_total(NULL, &pmemory, sizeof(pmemory), 1) < 0)
-       {
-               char errbuf[1024];
-               WARNING ("memory plugin: perfstat_memory_total failed: %s",
-                       sstrerror (errno, errbuf, sizeof (errbuf)));
-               return (-1);
-       }
-
-       /* Unfortunately, the AIX documentation is not very clear on how these
-        * numbers relate to one another. The only thing is states explcitly
-        * is:
-        *   real_total = real_process + real_free + numperm + real_system
-        *
-        * Another segmentation, which would be closer to the numbers reported
-        * by the "svmon" utility, would be:
-        *   real_total = real_free + real_inuse
-        *   real_inuse = "active" + real_pinned + numperm
-        */
-       MEMORY_SUBMIT ("free",   (gauge_t) (pmemory.real_free    * pagesize),
-                      "cached", (gauge_t) (pmemory.numperm      * pagesize),
-                      "system", (gauge_t) (pmemory.real_system  * pagesize),
-                      "user",   (gauge_t) (pmemory.real_process * pagesize));
+  perfstat_memory_total_t pmemory = {0};
+
+  if (perfstat_memory_total(NULL, &pmemory, sizeof(pmemory), 1) < 0) {
+    char errbuf[1024];
+    WARNING("memory plugin: perfstat_memory_total failed: %s",
+            sstrerror(errno, errbuf, sizeof(errbuf)));
+    return -1;
+  }
+
+  /* Unfortunately, the AIX documentation is not very clear on how these
+   * numbers relate to one another. The only thing is states explcitly
+   * is:
+   *   real_total = real_process + real_free + numperm + real_system
+   *
+   * Another segmentation, which would be closer to the numbers reported
+   * by the "svmon" utility, would be:
+   *   real_total = real_free + real_inuse
+   *   real_inuse = "active" + real_pinned + numperm
+   */
+  MEMORY_SUBMIT("free", (gauge_t)(pmemory.real_free * pagesize), "cached",
+                (gauge_t)(pmemory.numperm * pagesize), "system",
+                (gauge_t)(pmemory.real_system * pagesize), "user",
+                (gauge_t)(pmemory.real_process * pagesize));
 #endif /* HAVE_PERFSTAT */
 
-       return (0);
+  return 0;
 } /* }}} int memory_read_internal */
 
-static int memory_read (void) /* {{{ */
+static int memory_read(void) /* {{{ */
 {
-       value_t v[1];
-       value_list_t vl = VALUE_LIST_INIT;
+  value_t v[1];
+  value_list_t vl = VALUE_LIST_INIT;
 
-       vl.values = v;
-       vl.values_len = STATIC_ARRAY_SIZE (v);
-       sstrncpy (vl.host, hostname_g, sizeof (vl.host));
-       sstrncpy (vl.plugin, "memory", sizeof (vl.plugin));
-       sstrncpy (vl.type, "memory", sizeof (vl.type));
-       vl.time = cdtime ();
+  vl.values = v;
+  vl.values_len = STATIC_ARRAY_SIZE(v);
+  sstrncpy(vl.plugin, "memory", sizeof(vl.plugin));
+  sstrncpy(vl.type, "memory", sizeof(vl.type));
+  vl.time = cdtime();
 
-       return (memory_read_internal (&vl));
+  return memory_read_internal(&vl);
 } /* }}} int memory_read */
 
-void module_register (void)
-{
-       plugin_register_complex_config ("memory", memory_config);
-       plugin_register_init ("memory", memory_init);
-       plugin_register_read ("memory", memory_read);
+void module_register(void) {
+  plugin_register_complex_config("memory", memory_config);
+  plugin_register_init("memory", memory_init);
+  plugin_register_read("memory", memory_read);
 } /* void module_register */