]> arthur.barton.de Git - netdata.git/commitdiff
Merge pull request #1558 from ktsaou/master
authorCosta Tsaousis <costa@tsaousis.gr>
Thu, 12 Jan 2017 23:03:51 +0000 (01:03 +0200)
committerGitHub <noreply@github.com>
Thu, 12 Jan 2017 23:03:51 +0000 (01:03 +0200)
rewrite of /proc/net/dev for speed

conf.d/health.d/haproxy.conf
configs.signatures
src/proc_net_dev.c

index ee65b2bedb9dade9245e1b2cbafbef78d3f5534b..e49c70d485d9de4a3b3938dafce397f50ab89d32 100644 (file)
@@ -1,19 +1,19 @@
- template: haproxy_backend_server_status
+template: haproxy_backend_server_status
       on: haproxy_hs.down
    units: failed servers
    every: 10s
-    lookup: average -10s
+  lookup: average -10s
     crit: $this > 0
-    info: Some of your backend servers is down!
+    info: number of failed haproxy backend servers
       to: sysadmin
 
- template: haproxy_backend_status
+template: haproxy_backend_status
       on: haproxy_hb.down
    units: failed backend
    every: 10s
-    lookup: average -10s
+  lookup: average -10s
     crit: $this > 0
-    info: Some of your backends is down!
+    info: number of failed haproxy backends
       to: sysadmin
 
 template: haproxy_last_collected
@@ -25,5 +25,3 @@ template: haproxy_last_collected
     crit: $this > (($status == $CRITICAL) ? ($update_every) : (60 * $update_every))
     info: number of seconds since the last successful data collection
       to: sysadmin
-
-
index 198759f9cb193bb9868594c67d96836625191ec9..e83b90632e0946d04c2b2612f6941d2bae0eb1a5 100644 (file)
@@ -30,7 +30,9 @@ declare -A configs_signatures=(
   ['15d8401b56a74120f9f832873ec9c578']='health.d/postgres.conf'
   ['15e32114994b92be7853b88091e7c6fb']='python.d/exim.conf'
   ['174c21a6ce5de97bda83d502aa47a9f8']='health.d/apache.conf'
+  ['178281aa2241d4a3e6b798bb9c4ae577']='python.d/haproxy.conf'
   ['18ee1c6197a4381b1c1631ef6129824f']='apps_groups.conf'
+  ['1972e48345e6c3f0d65f94a03317622b']='health_alarm_notify.conf'
   ['1c12b678ab65f271a96da1bbd0a1ab1c']='health.d/softnet.conf'
   ['1ea8e8ef1fa8a3a0fcdfba236f4cb195']='python.d/mysql.conf'
   ['1ef0fd38e7969c023bc3fa6d89eaf6d6']='python.d/mdstat.conf'
@@ -243,8 +245,10 @@ declare -A configs_signatures=(
   ['a8bb4e1d0525f59692778ad8f675a77a']='python.d/example.conf'
   ['a8feb36776005bf419c90278787a1be8']='health.d/entropy.conf'
   ['a94af1c808aafdf00537d85ff2197ec8']='python.d/exim.conf'
+  ['a9ab68845db2fb695b7060273a6ac68e']='health_alarm_notify.conf'
   ['a9cd91675467c5426f5b51c47602c889']='apps_groups.conf'
   ['aa4bee249bfc0c4a88ac8c2ffb97aa0d']='health.d/squid.conf'
+  ['aa8b57a733c2035917acf81a8ebdfbe7']='health.d/haproxy.conf'
   ['abaf2e021f9f6ee5d1c4e4726f47348e']='health.d/ipc.conf'
   ['acaa6731a272f6d251afb357e99b518f']='apps_groups.conf'
   ['ade389c1b6efe0cff47c33e662731f0a']='python.d/squid.conf'
@@ -252,7 +256,9 @@ declare -A configs_signatures=(
   ['af44cc53aa2bc5cc8935667119567522']='python.d.conf'
   ['afdae4646c755ff2d117527fbf761c8e']='health.d/disks.conf'
   ['b07eebc6f58d19721ac069171b911d2a']='health_alarm_notify.conf'
+  ['b0c59b2bd7a10f6a3f2be6b4b27857db']='health.d/haproxy.conf'
   ['b0f0a0ac415e4b1a82187b80d211e83b']='python.d/mysql.conf'
+  ['b185914d4f795e1732273dc4c7a35845']='health.d/memory.conf'
   ['b27f10a38a95edbbec20f44a4728b7c4']='python.d.conf'
   ['b32164929eda7449a9677044e11151bf']='python.d.conf'
   ['b3fc4749b132e55ac0d3a0f92859237e']='health.d/tcp_resets.conf'
@@ -288,6 +294,7 @@ declare -A configs_signatures=(
   ['cd08e5534c94bf1f2cd28396c76b8bbc']='health.d/ram.conf'
   ['ce2e8768964a936f58c4c2144aee8a01']='health_alarm_notify.conf'
   ['ce3b65eac6c472b21905f7f72104f4c9']='python.d/nginx.conf'
+  ['cf48dfd828af70bea04db7a809f94358']='health.d/haproxy.conf'
   ['cfecf298bdafaa7e0a3a263548e82132']='python.d/sensors.conf'
   ['d11711b3647bc2bdd0292dd7deebbeb1']='health.d/net.conf'
   ['d1596fe068c8674efade49a4a8e22b5d']='health.d/isc_dhcpd.conf'
index 06182d44ff9559f8a15673f7c9cbee853d6ec859..99c77cd1ba398b58cb261e0acd1e3860c89b00f5 100644 (file)
 #include "common.h"
 
+struct netdev {
+    char *name;
+    uint32_t hash;
+    size_t len;
+
+    // flags
+    int configured;
+    int enabled;
+
+    int do_bandwidth;
+    int do_packets;
+    int do_errors;
+    int do_drops;
+    int do_fifo;
+    int do_compressed;
+    int do_events;
+
+    // data collected
+    unsigned long long rbytes;
+    unsigned long long rpackets;
+    unsigned long long rerrors;
+    unsigned long long rdrops;
+    unsigned long long rfifo;
+    unsigned long long rframe;
+    unsigned long long rcompressed;
+    unsigned long long rmulticast;
+
+    unsigned long long tbytes;
+    unsigned long long tpackets;
+    unsigned long long terrors;
+    unsigned long long tdrops;
+    unsigned long long tfifo;
+    unsigned long long tcollisions;
+    unsigned long long tcarrier;
+    unsigned long long tcompressed;
+
+    // charts
+    RRDSET *st_bandwidth;
+    RRDSET *st_packets;
+    RRDSET *st_errors;
+    RRDSET *st_drops;
+    RRDSET *st_fifo;
+    RRDSET *st_compressed;
+    RRDSET *st_events;
+
+    // dimensions
+    RRDDIM *rd_rbytes;
+    RRDDIM *rd_rpackets;
+    RRDDIM *rd_rerrors;
+    RRDDIM *rd_rdrops;
+    RRDDIM *rd_rfifo;
+    RRDDIM *rd_rframe;
+    RRDDIM *rd_rcompressed;
+    RRDDIM *rd_rmulticast;
+
+    RRDDIM *rd_tbytes;
+    RRDDIM *rd_tpackets;
+    RRDDIM *rd_terrors;
+    RRDDIM *rd_tdrops;
+    RRDDIM *rd_tfifo;
+    RRDDIM *rd_tcollisions;
+    RRDDIM *rd_tcarrier;
+    RRDDIM *rd_tcompressed;
+
+    struct netdev *next;
+};
+
+static struct netdev *netdev_root = NULL;
+
+static struct netdev *get_netdev(const char *name) {
+    static struct netdev *last = NULL;
+    struct netdev *d;
+
+    uint32_t hash = simple_hash(name);
+
+    // search it, from the last position to the end
+    for(d = last ; d ; d = d->next) {
+        if(unlikely(hash == d->hash && !strcmp(name, d->name))) {
+            last = d->next;
+            return d;
+        }
+    }
+
+    // search it from the beginning to the last position we used
+    for(d = netdev_root ; d != last ; d = d->next) {
+        if(unlikely(hash == d->hash && !strcmp(name, d->name))) {
+            last = d->next;
+            return d;
+        }
+    }
+
+    // create a new one
+    d = callocz(1, sizeof(struct netdev));
+    d->name = strdupz(name);
+    d->hash = simple_hash(d->name);
+    d->len = strlen(d->name);
+
+    // link it to the end
+    if(netdev_root) {
+        struct netdev *e;
+        for(e = netdev_root; e->next ; e = e->next) ;
+        e->next = d;
+    }
+    else
+        netdev_root = d;
+
+    return d;
+}
+
 int do_proc_net_dev(int update_every, usec_t dt) {
+    (void)dt;
+
     static procfile *ff = NULL;
-    static int enable_new_interfaces = -1, enable_ifb_interfaces = -1;
+    static int enable_new_interfaces = -1, enable_ifb_interfaces = -1, enable_veth_interfaces = -1;
     static int do_bandwidth = -1, do_packets = -1, do_errors = -1, do_drops = -1, do_fifo = -1, do_compressed = -1, do_events = -1;
 
-    if(dt) {};
+    if(unlikely(enable_new_interfaces == -1)) {
+        enable_new_interfaces = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "enable new interfaces detected at runtime", CONFIG_ONDEMAND_ONDEMAND);
+        enable_ifb_interfaces = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "enable ifb interfaces", CONFIG_ONDEMAND_NO);
+        enable_veth_interfaces = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "enable veth interfaces", enable_new_interfaces);
+
+        do_bandwidth    = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "bandwidth for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+        do_packets      = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "packets for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+        do_errors       = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "errors for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+        do_drops        = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "drops for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+        do_fifo         = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "fifo for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+        do_compressed   = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "compressed packets for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+        do_events       = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "frames, collisions, carrier counters for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+    }
 
-    if(!ff) {
+    if(unlikely(!ff)) {
         char filename[FILENAME_MAX + 1];
         snprintfz(filename, FILENAME_MAX, "%s%s", global_host_prefix, "/proc/net/dev");
         ff = procfile_open(config_get("plugin:proc:/proc/net/dev", "filename to monitor", filename), " \t,:|", PROCFILE_FLAG_DEFAULT);
+        if(unlikely(!ff)) return 1;
     }
-    if(!ff) return 1;
 
     ff = procfile_readall(ff);
-    if(!ff) return 0; // we return 0, so that we will retry to open it next time
-
-    if(enable_new_interfaces == -1) enable_new_interfaces = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "enable new interfaces detected at runtime", CONFIG_ONDEMAND_ONDEMAND);
-    if(enable_ifb_interfaces == -1) enable_ifb_interfaces = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "enable ifb interfaces", CONFIG_ONDEMAND_NO);
-
-    if(do_bandwidth == -1)  do_bandwidth    = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "bandwidth for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
-    if(do_packets == -1)    do_packets      = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "packets for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
-    if(do_errors == -1)     do_errors       = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "errors for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
-    if(do_drops == -1)      do_drops        = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "drops for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
-    if(do_fifo == -1)       do_fifo         = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "fifo for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
-    if(do_compressed == -1) do_compressed   = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "compressed packets for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
-    if(do_events == -1)     do_events       = config_get_boolean_ondemand("plugin:proc:/proc/net/dev", "frames, collisions, carrier counters for all interfaces", CONFIG_ONDEMAND_ONDEMAND);
+    if(unlikely(!ff)) return 0; // we return 0, so that we will retry to open it next time
 
     uint32_t lines = procfile_lines(ff), l;
-
-    char *iface;
-    unsigned long long rbytes, rpackets, rerrors, rdrops, rfifo, rframe, rcompressed, rmulticast;
-    unsigned long long tbytes, tpackets, terrors, tdrops, tfifo, tcollisions, tcarrier, tcompressed;
-
     for(l = 2; l < lines ;l++) {
-        uint32_t words = procfile_linewords(ff, l);
-        if(words < 17) continue;
-
-        iface       = procfile_lineword(ff, l, 0);
-
-        rbytes      = strtoull(procfile_lineword(ff, l, 1), NULL, 10);
-        rpackets    = strtoull(procfile_lineword(ff, l, 2), NULL, 10);
-        rerrors     = strtoull(procfile_lineword(ff, l, 3), NULL, 10);
-        rdrops      = strtoull(procfile_lineword(ff, l, 4), NULL, 10);
-        rfifo       = strtoull(procfile_lineword(ff, l, 5), NULL, 10);
-        rframe      = strtoull(procfile_lineword(ff, l, 6), NULL, 10);
-        rcompressed = strtoull(procfile_lineword(ff, l, 7), NULL, 10);
-        rmulticast  = strtoull(procfile_lineword(ff, l, 8), NULL, 10);
-
-        tbytes      = strtoull(procfile_lineword(ff, l, 9), NULL, 10);
-        tpackets    = strtoull(procfile_lineword(ff, l, 10), NULL, 10);
-        terrors     = strtoull(procfile_lineword(ff, l, 11), NULL, 10);
-        tdrops      = strtoull(procfile_lineword(ff, l, 12), NULL, 10);
-        tfifo       = strtoull(procfile_lineword(ff, l, 13), NULL, 10);
-        tcollisions = strtoull(procfile_lineword(ff, l, 14), NULL, 10);
-        tcarrier    = strtoull(procfile_lineword(ff, l, 15), NULL, 10);
-        tcompressed = strtoull(procfile_lineword(ff, l, 16), NULL, 10);
-
-        int ddo_bandwidth = do_bandwidth, ddo_packets = do_packets, ddo_errors = do_errors, ddo_drops = do_drops, ddo_fifo = do_fifo, ddo_compressed = do_compressed, ddo_events = do_events;
-
-        int default_enable = enable_new_interfaces;
-
-        // prevent unused interfaces from creating charts
-        if(strcmp(iface, "lo") == 0)
-            default_enable = 0;
-        else {
-            int len = strlen(iface);
-            if(len >= 4 && strcmp(&iface[len-4], "-ifb") == 0)
-                default_enable = enable_ifb_interfaces;
-        }
+        // require 17 words on each line
+        if(unlikely(procfile_linewords(ff, l) < 17)) continue;
+
+        struct netdev *d = get_netdev(procfile_lineword(ff, l, 0));
+
+        if(unlikely(!d->configured)) {
+            // this is the first time we see this interface
+
+            // remember we configured it
+            d->configured = 1;
+
+            // start with the default enabled flag
+            d->enabled = enable_new_interfaces;
+            if(d->enabled) {
+                if(unlikely(!strcmp(d->name, "lo")))
+                    d->enabled = CONFIG_ONDEMAND_NO;
+                else if(unlikely(!strncmp(d->name, "veth", 4)))
+                    d->enabled = enable_veth_interfaces;
+                else if(unlikely(d->len >= 4 && strcmp(&d->name[d->len - 4], "-ifb") == 0))
+                    d->enabled = enable_ifb_interfaces;
+            }
 
-        // check if the user wants it
-        {
             char var_name[512 + 1];
-            snprintfz(var_name, 512, "plugin:proc:/proc/net/dev:%s", iface);
-            default_enable = config_get_boolean_ondemand(var_name, "enabled", default_enable);
-            if(default_enable == CONFIG_ONDEMAND_NO) continue;
-            if(default_enable == CONFIG_ONDEMAND_ONDEMAND && !rbytes && !tbytes) continue;
-
-            ddo_bandwidth = config_get_boolean_ondemand(var_name, "bandwidth", ddo_bandwidth);
-            ddo_packets = config_get_boolean_ondemand(var_name, "packets", ddo_packets);
-            ddo_errors = config_get_boolean_ondemand(var_name, "errors", ddo_errors);
-            ddo_drops = config_get_boolean_ondemand(var_name, "drops", ddo_drops);
-            ddo_fifo = config_get_boolean_ondemand(var_name, "fifo", ddo_fifo);
-            ddo_compressed = config_get_boolean_ondemand(var_name, "compressed", ddo_compressed);
-            ddo_events = config_get_boolean_ondemand(var_name, "events", ddo_events);
-
-            if(ddo_bandwidth == CONFIG_ONDEMAND_ONDEMAND && rbytes == 0 && tbytes == 0) ddo_bandwidth = 0;
-            if(ddo_errors == CONFIG_ONDEMAND_ONDEMAND && rerrors == 0 && terrors == 0) ddo_errors = 0;
-            if(ddo_drops == CONFIG_ONDEMAND_ONDEMAND && rdrops == 0 && tdrops == 0) ddo_drops = 0;
-            if(ddo_fifo == CONFIG_ONDEMAND_ONDEMAND && rfifo == 0 && tfifo == 0) ddo_fifo = 0;
-            if(ddo_compressed == CONFIG_ONDEMAND_ONDEMAND && rcompressed == 0 && tcompressed == 0) ddo_compressed = 0;
-            if(ddo_events == CONFIG_ONDEMAND_ONDEMAND && rframe == 0 && tcollisions == 0 && tcarrier == 0) ddo_events = 0;
-
-            // for absolute values, we need to switch the setting to 'yes'
-            // to allow it refresh from now on
-            // if(ddo_fifo == CONFIG_ONDEMAND_ONDEMAND) config_set(var_name, "fifo", "yes");
+            snprintfz(var_name, 512, "plugin:proc:/proc/net/dev:%s", d->name);
+            d->enabled = config_get_boolean_ondemand(var_name, "enabled", d->enabled);
+
+            if(d->enabled == CONFIG_ONDEMAND_NO)
+                continue;
+
+            d->do_bandwidth = config_get_boolean_ondemand(var_name, "bandwidth", do_bandwidth);
+            d->do_packets = config_get_boolean_ondemand(var_name, "packets", do_packets);
+            d->do_errors = config_get_boolean_ondemand(var_name, "errors", do_errors);
+            d->do_drops = config_get_boolean_ondemand(var_name, "drops", do_drops);
+            d->do_fifo = config_get_boolean_ondemand(var_name, "fifo", do_fifo);
+            d->do_compressed = config_get_boolean_ondemand(var_name, "compressed", do_compressed);
+            d->do_events = config_get_boolean_ondemand(var_name, "events", do_events);
         }
 
-        RRDSET *st;
+        if(unlikely(!d->enabled))
+            continue;
+
+        d->rbytes      = strtoull(procfile_lineword(ff, l, 1), NULL, 10);
+        d->rpackets    = strtoull(procfile_lineword(ff, l, 2), NULL, 10);
+        d->rerrors     = strtoull(procfile_lineword(ff, l, 3), NULL, 10);
+        d->rdrops      = strtoull(procfile_lineword(ff, l, 4), NULL, 10);
+        d->rfifo       = strtoull(procfile_lineword(ff, l, 5), NULL, 10);
+        d->rframe      = strtoull(procfile_lineword(ff, l, 6), NULL, 10);
+        d->rcompressed = strtoull(procfile_lineword(ff, l, 7), NULL, 10);
+        d->rmulticast  = strtoull(procfile_lineword(ff, l, 8), NULL, 10);
+
+        d->tbytes      = strtoull(procfile_lineword(ff, l, 9), NULL, 10);
+        d->tpackets    = strtoull(procfile_lineword(ff, l, 10), NULL, 10);
+        d->terrors     = strtoull(procfile_lineword(ff, l, 11), NULL, 10);
+        d->tdrops      = strtoull(procfile_lineword(ff, l, 12), NULL, 10);
+        d->tfifo       = strtoull(procfile_lineword(ff, l, 13), NULL, 10);
+        d->tcollisions = strtoull(procfile_lineword(ff, l, 14), NULL, 10);
+        d->tcarrier    = strtoull(procfile_lineword(ff, l, 15), NULL, 10);
+        d->tcompressed = strtoull(procfile_lineword(ff, l, 16), NULL, 10);
 
         // --------------------------------------------------------------------
 
-        if(ddo_bandwidth) {
-            st = rrdset_find_bytype("net", iface);
-            if(!st) {
-                st = rrdset_create("net", iface, NULL, iface, "net.net", "Bandwidth", "kilobits/s", 7000, update_every, RRDSET_TYPE_AREA);
+        if(unlikely((d->do_bandwidth == CONFIG_ONDEMAND_ONDEMAND && (d->rbytes || d->tbytes))))
+            d->do_bandwidth = CONFIG_ONDEMAND_YES;
+
+        if(d->do_bandwidth == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_bandwidth)) {
+                d->st_bandwidth = rrdset_find_bytype("net", d->name);
 
-                rrddim_add(st, "received", NULL, 8, 1024, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "sent", NULL, -8, 1024, RRDDIM_INCREMENTAL);
+                if(!d->st_bandwidth) {
+                    d->st_bandwidth = rrdset_create("net", d->name, NULL, d->name, "net.net", "Bandwidth", "kilobits/s", 7000, update_every, RRDSET_TYPE_AREA);
+                    d->rd_rbytes = rrddim_add(d->st_bandwidth, "received", NULL, 8, 1024, RRDDIM_INCREMENTAL);
+                    d->rd_tbytes = rrddim_add(d->st_bandwidth, "sent", NULL, -8, 1024, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_bandwidth);
 
-            rrddim_set(st, "received", rbytes);
-            rrddim_set(st, "sent", tbytes);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_bandwidth, d->rd_rbytes, d->rbytes);
+            rrddim_set_by_pointer(d->st_bandwidth, d->rd_tbytes, d->tbytes);
+            rrdset_done(d->st_bandwidth);
         }
 
         // --------------------------------------------------------------------
 
-        if(ddo_packets) {
-            st = rrdset_find_bytype("net_packets", iface);
-            if(!st) {
-                st = rrdset_create("net_packets", iface, NULL, iface, "net.packets", "Packets", "packets/s", 7001, update_every, RRDSET_TYPE_LINE);
-                st->isdetail = 1;
+        if(unlikely((d->do_packets == CONFIG_ONDEMAND_ONDEMAND && (d->rpackets || d->tpackets || d->rmulticast))))
+            d->do_packets = CONFIG_ONDEMAND_YES;
 
-                rrddim_add(st, "received", NULL, 1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "sent", NULL, -1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "multicast", NULL, 1, 1, RRDDIM_INCREMENTAL);
+        if(d->do_packets == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_packets)) {
+                d->st_packets = rrdset_find_bytype("net_packets", d->name);
+
+                if(!d->st_packets) {
+                    d->st_packets = rrdset_create("net_packets", d->name, NULL, d->name, "net.packets", "Packets", "packets/s", 7001, update_every, RRDSET_TYPE_LINE);
+                    d->st_packets->isdetail = 1;
+
+                    d->rd_rpackets = rrddim_add(d->st_packets, "received", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_tpackets = rrddim_add(d->st_packets, "sent", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_rmulticast = rrddim_add(d->st_packets, "multicast", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_packets);
 
-            rrddim_set(st, "received", rpackets);
-            rrddim_set(st, "sent", tpackets);
-            rrddim_set(st, "multicast", rmulticast);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_packets, d->rd_rpackets, d->rpackets);
+            rrddim_set_by_pointer(d->st_packets, d->rd_tpackets, d->tpackets);
+            rrddim_set_by_pointer(d->st_packets, d->rd_rmulticast, d->rmulticast);
+            rrdset_done(d->st_packets);
         }
 
         // --------------------------------------------------------------------
 
-        if(ddo_errors) {
-            st = rrdset_find_bytype("net_errors", iface);
-            if(!st) {
-                st = rrdset_create("net_errors", iface, NULL, iface, "net.errors", "Interface Errors", "errors/s", 7002, update_every, RRDSET_TYPE_LINE);
-                st->isdetail = 1;
+        if(unlikely((d->do_errors == CONFIG_ONDEMAND_ONDEMAND && (d->rerrors || d->terrors))))
+            d->do_errors = CONFIG_ONDEMAND_YES;
+
+        if(d->do_errors == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_errors)) {
+                d->st_errors = rrdset_find_bytype("net_errors", d->name);
 
-                rrddim_add(st, "inbound", NULL, 1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "outbound", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                if(!d->st_errors) {
+                    d->st_errors = rrdset_create("net_errors", d->name, NULL, d->name, "net.errors", "Interface Errors", "errors/s", 7002, update_every, RRDSET_TYPE_LINE);
+                    d->st_errors->isdetail = 1;
+
+                    d->rd_rerrors = rrddim_add(d->st_errors, "inbound", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_terrors = rrddim_add(d->st_errors, "outbound", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_errors);
 
-            rrddim_set(st, "inbound", rerrors);
-            rrddim_set(st, "outbound", terrors);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_errors, d->rd_rerrors, d->rerrors);
+            rrddim_set_by_pointer(d->st_errors, d->rd_terrors, d->terrors);
+            rrdset_done(d->st_errors);
         }
 
         // --------------------------------------------------------------------
 
-        if(ddo_drops) {
-            st = rrdset_find_bytype("net_drops", iface);
-            if(!st) {
-                st = rrdset_create("net_drops", iface, NULL, iface, "net.drops", "Interface Drops", "drops/s", 7003, update_every, RRDSET_TYPE_LINE);
-                st->isdetail = 1;
+        if(unlikely((d->do_drops == CONFIG_ONDEMAND_ONDEMAND && (d->rdrops || d->tdrops))))
+            d->do_drops = CONFIG_ONDEMAND_YES;
+
+        if(d->do_drops == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_drops)) {
+                d->st_drops = rrdset_find_bytype("net_drops", d->name);
 
-                rrddim_add(st, "inbound", NULL, 1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "outbound", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                if(!d->st_drops) {
+                    d->st_drops = rrdset_create("net_drops", d->name, NULL, d->name, "net.drops", "Interface Drops", "drops/s", 7003, update_every, RRDSET_TYPE_LINE);
+                    d->st_drops->isdetail = 1;
+
+                    d->rd_rdrops = rrddim_add(d->st_drops, "inbound", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_tdrops = rrddim_add(d->st_drops, "outbound", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_drops);
 
-            rrddim_set(st, "inbound", rdrops);
-            rrddim_set(st, "outbound", tdrops);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_drops, d->rd_rdrops, d->rdrops);
+            rrddim_set_by_pointer(d->st_drops, d->rd_tdrops, d->tdrops);
+            rrdset_done(d->st_drops);
         }
 
         // --------------------------------------------------------------------
 
-        if(ddo_fifo) {
-            st = rrdset_find_bytype("net_fifo", iface);
-            if(!st) {
-                st = rrdset_create("net_fifo", iface, NULL, iface, "net.fifo", "Interface FIFO Buffer Errors", "errors", 7004, update_every, RRDSET_TYPE_LINE);
-                st->isdetail = 1;
+        if(unlikely((d->do_fifo == CONFIG_ONDEMAND_ONDEMAND && (d->rfifo || d->tfifo))))
+            d->do_fifo = CONFIG_ONDEMAND_YES;
+
+        if(d->do_fifo == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_fifo)) {
+                d->st_fifo = rrdset_find_bytype("net_fifo", d->name);
 
-                rrddim_add(st, "receive", NULL, 1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "transmit", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                if(!d->st_fifo) {
+                    d->st_fifo = rrdset_create("net_fifo", d->name, NULL, d->name, "net.fifo", "Interface FIFO Buffer Errors", "errors", 7004, update_every, RRDSET_TYPE_LINE);
+                    d->st_fifo->isdetail = 1;
+
+                    d->rd_rfifo = rrddim_add(d->st_fifo, "receive", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_tfifo = rrddim_add(d->st_fifo, "transmit", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_fifo);
 
-            rrddim_set(st, "receive", rfifo);
-            rrddim_set(st, "transmit", tfifo);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_fifo, d->rd_rfifo, d->rfifo);
+            rrddim_set_by_pointer(d->st_fifo, d->rd_tfifo, d->tfifo);
+            rrdset_done(d->st_fifo);
         }
 
         // --------------------------------------------------------------------
 
-        if(ddo_compressed) {
-            st = rrdset_find_bytype("net_compressed", iface);
-            if(!st) {
-                st = rrdset_create("net_compressed", iface, NULL, iface, "net.compressed", "Compressed Packets", "packets/s", 7005, update_every, RRDSET_TYPE_LINE);
-                st->isdetail = 1;
+        if(unlikely((d->do_compressed == CONFIG_ONDEMAND_ONDEMAND && (d->rcompressed || d->tcompressed))))
+            d->do_compressed = CONFIG_ONDEMAND_YES;
+
+        if(d->do_compressed == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_compressed)) {
+                d->st_compressed = rrdset_find_bytype("net_compressed", d->name);
+                if(!d->st_compressed) {
+                    d->st_compressed = rrdset_create("net_compressed", d->name, NULL, d->name, "net.compressed", "Compressed Packets", "packets/s", 7005, update_every, RRDSET_TYPE_LINE);
+                    d->st_compressed->isdetail = 1;
 
-                rrddim_add(st, "received", NULL, 1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "sent", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_rcompressed = rrddim_add(d->st_compressed, "received", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_tcompressed = rrddim_add(d->st_compressed, "sent", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_compressed);
 
-            rrddim_set(st, "received", rcompressed);
-            rrddim_set(st, "sent", tcompressed);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_compressed, d->rd_rcompressed, d->rcompressed);
+            rrddim_set_by_pointer(d->st_compressed, d->rd_tcompressed, d->tcompressed);
+            rrdset_done(d->st_compressed);
         }
 
         // --------------------------------------------------------------------
 
-        if(ddo_events) {
-            st = rrdset_find_bytype("net_events", iface);
-            if(!st) {
-                st = rrdset_create("net_events", iface, NULL, iface, "net.events", "Network Interface Events", "events/s", 7006, update_every, RRDSET_TYPE_LINE);
-                st->isdetail = 1;
+        if(unlikely((d->do_events == CONFIG_ONDEMAND_ONDEMAND && (d->rframe || d->tcollisions || d->tcarrier))))
+            d->do_events = CONFIG_ONDEMAND_YES;
+
+        if(d->do_events == CONFIG_ONDEMAND_YES) {
+            if(unlikely(!d->st_events)) {
+                d->st_events = rrdset_find_bytype("net_events", d->name);
+                if(!d->st_events) {
+                    d->st_events = rrdset_create("net_events", d->name, NULL, d->name, "net.events", "Network Interface Events", "events/s", 7006, update_every, RRDSET_TYPE_LINE);
+                    d->st_events->isdetail = 1;
 
-                rrddim_add(st, "frames", NULL, 1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "collisions", NULL, -1, 1, RRDDIM_INCREMENTAL);
-                rrddim_add(st, "carrier", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_rframe      = rrddim_add(d->st_events, "frames", NULL, 1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_tcollisions = rrddim_add(d->st_events, "collisions", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                    d->rd_tcarrier    = rrddim_add(d->st_events, "carrier", NULL, -1, 1, RRDDIM_INCREMENTAL);
+                }
             }
-            else rrdset_next(st);
+            else rrdset_next(d->st_events);
 
-            rrddim_set(st, "frames", rframe);
-            rrddim_set(st, "collisions", tcollisions);
-            rrddim_set(st, "carrier", tcarrier);
-            rrdset_done(st);
+            rrddim_set_by_pointer(d->st_events, d->rd_rframe,      d->rframe);
+            rrddim_set_by_pointer(d->st_events, d->rd_tcollisions, d->tcollisions);
+            rrddim_set_by_pointer(d->st_events, d->rd_tcarrier,    d->tcarrier);
+            rrdset_done(d->st_events);
         }
     }