]> arthur.barton.de Git - netdata.git/commitdiff
Separate kern.devstat module with simple patterns adopted
authorVladimir Kobal <vlad@prokk.net>
Thu, 16 Mar 2017 20:02:08 +0000 (22:02 +0200)
committerVladimir Kobal <vlad@prokk.net>
Thu, 16 Mar 2017 20:02:08 +0000 (22:02 +0200)
src/freebsd_sysctl.c

index dcad74346023fe96371501c9d165fabaa147d74b..174be91f69851a3187128ac4447addcb2e7e26bf 100644 (file)
@@ -2750,6 +2750,8 @@ int do_getmntinfo(int update_every, usec_t dt) {
 
                 size_t collected; // the number of times this has been collected
 
+                // charts and dimensions
+
                 RRDSET *st_space;
                 RRDDIM *rd_space_used;
                 RRDDIM *rd_space_avail;
@@ -3057,6 +3059,8 @@ int do_getifaddrs(int update_every, usec_t dt) {
                 int do_drops;
                 int do_events;
 
+                // charts and dimensions
+
                 RRDSET *st_bandwidth;
                 RRDDIM *rd_bandwidth_in;
                 RRDDIM *rd_bandwidth_out;
@@ -3326,137 +3330,398 @@ int do_getifaddrs(int update_every, usec_t dt) {
 // kern.devstat
 
 int do_kern_devstat(int update_every, usec_t dt) {
-    static int do_disk_io = -1;
 
-    if (unlikely(do_disk_io == -1)) {
-        do_disk_io = config_get_boolean("plugin:freebsd:kern.devstat", "stats for all disks", 1);
+#define DELAULT_EXLUDED_DISKS ""
+#define CONFIG_SECTION_KERN_DEVSTAT "plugin:freebsd:kern.devstat"
+#define BINTIME_SCALE 5.42101086242752217003726400434970855712890625e-17 // this is 1000/2^64
+
+    static int enable_pass_devices = -1, do_system_io = -1, do_io = -1, do_ops = -1, do_qops = -1, do_util = -1,
+               do_iotime = -1, do_await = -1, do_avagsz = -1, do_svctm = -1;
+
+    if (unlikely(enable_pass_devices == -1)) {
+        enable_pass_devices = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT,
+                                                          "performance metrics for pass devices", CONFIG_BOOLEAN_AUTO);
+
+        do_system_io = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "total bandwidth for all disks",
+                                                   CONFIG_BOOLEAN_YES);
+
+        do_io     = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "bandwidth for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_ops    = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "operations for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_qops   = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "queued operations for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_util   = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "utilization percentage for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_iotime = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "i/o time for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_await  = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "average completed i/o time for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_avagsz = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "average completed i/o bandwidth for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
+        do_svctm  = config_get_boolean_ondemand(CONFIG_SECTION_KERN_DEVSTAT, "average service time for all disks",
+                                                CONFIG_BOOLEAN_AUTO);
     }
 
-    RRDSET *st;
-
-    int i;
-
-    #define RRD_TYPE_DISK "disk"
-    #define BINTIME_SCALE 5.42101086242752217003726400434970855712890625e-17 // this is 1000/2^64
-    int numdevs;
-    static void *devstat_data = NULL;
-    struct devstat *dstat;
-    char disk[DEVSTAT_NAME_LEN + MAX_INT_DIGITS + 1];
-    struct cur_dstat {
-        collected_number duration_read_ms;
-        collected_number duration_write_ms;
-        collected_number busy_time_ms;
-    } cur_dstat;
-    struct prev_dstat {
-        collected_number bytes_read;
-        collected_number bytes_write;
-        collected_number operations_read;
-        collected_number operations_write;
-        collected_number duration_read_ms;
-        collected_number duration_write_ms;
-        collected_number busy_time_ms;
-    } prev_dstat;
+    if (likely(do_system_io || do_io || do_ops || do_qops || do_util || do_iotime || do_await || do_avagsz || do_svctm)) {
+        static int mib_numdevs[3] = {0, 0, 0};
+        int numdevs;
+        int common_error = 0;
 
-    // --------------------------------------------------------------------
-
-    if (likely(do_disk_io)) {
-        if (unlikely(GETSYSCTL_BY_NAME("kern.devstat.numdevs", numdevs))) {
-            do_disk_io = 0;
-            error("DISABLED: disk.io");
+        if (unlikely(GETSYSCTL_SIMPLE("kern.devstat.numdevs", mib_numdevs, numdevs))) {
+            common_error = 1;
         } else {
+            static int mib_devstat[3] = {0, 0, 0};
+            static void *devstat_data = NULL;
+
             devstat_data = reallocz(devstat_data, sizeof(long) + sizeof(struct devstat) * numdevs); // there is generation number before devstat structures
-            if (unlikely(
-                    getsysctl_by_name("kern.devstat.all", devstat_data, sizeof(long) + sizeof(struct devstat) * numdevs))) {
-                do_disk_io = 0;
-                error("DISABLED: disk.io");
+            if (unlikely(GETSYSCTL_WSIZE("kern.devstat.all", mib_devstat, devstat_data,
+                                         sizeof(long) + sizeof(struct devstat) * numdevs))) {
+                common_error = 1;
             } else {
-                dstat = devstat_data + sizeof(long); // skip generation number
+                struct devstat *dstat;
+                int i;
                 collected_number total_disk_kbytes_read = 0;
                 collected_number total_disk_kbytes_write = 0;
 
-                for (i = 0; i < numdevs; i++) {
-                    if (((dstat[i].device_type & DEVSTAT_TYPE_MASK) == DEVSTAT_TYPE_DIRECT) || ((dstat[i].device_type & DEVSTAT_TYPE_MASK) == DEVSTAT_TYPE_STORARRAY)) {
-                        sprintf(disk, "%s%d", dstat[i].device_name, dstat[i].unit_number);
+                // Data to be stored in DICTIONARY disks.
+                // This DICTIONARY is used to lookup the settings of the disks on each iteration.
+                struct disks_metadata {
+                    int do_io;
+                    int do_ops;
+                    int do_qops;
+                    int do_util;
+                    int do_iotime;
+                    int do_await;
+                    int do_avagsz;
+                    int do_svctm;
 
-                        // --------------------------------------------------------------------
 
-                        st = rrdset_find_bytype_localhost(RRD_TYPE_DISK, disk);
-                        if (unlikely(!st)) {
-                            st = rrdset_create_localhost(RRD_TYPE_DISK, disk, NULL, disk, "disk.io", "Disk I/O Bandwidth", "kilobytes/s", 2000, update_every, RRDSET_TYPE_AREA);
+                    // data for differential charts
 
-                            rrddim_add(st, "reads", NULL, 1, 1024, RRD_ALGORITHM_INCREMENTAL);
-                            rrddim_add(st, "writes", NULL, -1, 1024, RRD_ALGORITHM_INCREMENTAL);
-                        }
-                        else rrdset_next(st);
+                    struct prev_dstat {
+                        collected_number bytes_read;
+                        collected_number bytes_write;
+                        collected_number operations_read;
+                        collected_number operations_write;
+                        collected_number duration_read_ms;
+                        collected_number duration_write_ms;
+                        collected_number busy_time_ms;
+                    } prev_dstat;
 
-                        total_disk_kbytes_read += dstat[i].bytes[DEVSTAT_READ]/KILO_FACTOR;
-                        total_disk_kbytes_write += dstat[i].bytes[DEVSTAT_WRITE]/KILO_FACTOR;
-                        prev_dstat.bytes_read = rrddim_set(st, "reads", dstat[i].bytes[DEVSTAT_READ]);
-                        prev_dstat.bytes_write = rrddim_set(st, "writes", dstat[i].bytes[DEVSTAT_WRITE]);
-                        rrdset_done(st);
+                    // charts and dimensions
 
-                        // --------------------------------------------------------------------
+                    RRDSET *st_io;
+                    RRDDIM *rd_io_in;
+                    RRDDIM *rd_io_out;
+
+                    RRDSET *st_ops;
+                    RRDDIM *rd_ops_in;
+                    RRDDIM *rd_ops_out;
+
+                    RRDSET *st_qops;
+                    RRDDIM *rd_qops;
 
-                        st = rrdset_find_bytype_localhost("disk_ops", disk);
-                        if (unlikely(!st)) {
-                            st = rrdset_create_localhost("disk_ops", disk, NULL, disk, "disk.ops", "Disk Completed I/O Operations", "operations/s", 2001, update_every, RRDSET_TYPE_LINE);
-                            rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
+                    RRDSET *st_util;
+                    RRDDIM *rd_util;
 
-                            rrddim_add(st, "reads", NULL, 1, 1, RRD_ALGORITHM_INCREMENTAL);
-                            rrddim_add(st, "writes", NULL, -1, 1, RRD_ALGORITHM_INCREMENTAL);
+                    RRDSET *st_iotime;
+                    RRDDIM *rd_iotime_in;
+                    RRDDIM *rd_iotime_out;
+
+                    RRDSET *st_await;
+                    RRDDIM *rd_await_in;
+                    RRDDIM *rd_await_out;
+
+                    RRDSET *st_avagsz;
+                    RRDDIM *rd_avagsz_in;
+                    RRDDIM *rd_avagsz_out;
+
+                    RRDSET *st_svctm;
+                    RRDDIM *rd_svctm;
+
+                };
+                static DICTIONARY *disks = NULL;
+                static SIMPLE_PATTERN *excluded_disks = NULL;
+
+                if(unlikely(!disks)) {
+
+                    excluded_disks = simple_pattern_create(
+                            config_get(CONFIG_SECTION_KERN_DEVSTAT, "disable by default disks matching",
+                                       DELAULT_EXLUDED_DISKS)
+                            , SIMPLE_PATTERN_EXACT
+                    );
+
+                    disks = dictionary_create(DICTIONARY_FLAG_SINGLE_THREADED);
+                }
+
+                dstat = devstat_data + sizeof(long); // skip generation number
+
+                for (i = 0; i < numdevs; i++) {
+                    if (unlikely(!do_system_io)) {
+                        if (((dstat[i].device_type & DEVSTAT_TYPE_MASK) == DEVSTAT_TYPE_DIRECT) || ((dstat[i].device_type & DEVSTAT_TYPE_MASK) == DEVSTAT_TYPE_STORARRAY)) {
+                            total_disk_kbytes_read += dstat[i].bytes[DEVSTAT_READ] / KILO_FACTOR;
+                            total_disk_kbytes_write += dstat[i].bytes[DEVSTAT_WRITE] / KILO_FACTOR;
                         }
-                        else rrdset_next(st);
+                    }
 
-                        prev_dstat.operations_read = rrddim_set(st, "reads", dstat[i].operations[DEVSTAT_READ]);
-                        prev_dstat.operations_write = rrddim_set(st, "writes", dstat[i].operations[DEVSTAT_WRITE]);
-                        rrdset_done(st);
+                    if (unlikely(!enable_pass_devices))
+                        if (dstat[i].device_type & DEVSTAT_TYPE_PASS == DEVSTAT_TYPE_PASS)
+                            continue;
 
-                        // --------------------------------------------------------------------
+                    if (((dstat[i].device_type & DEVSTAT_TYPE_MASK) == DEVSTAT_TYPE_DIRECT) || ((dstat[i].device_type & DEVSTAT_TYPE_MASK) == DEVSTAT_TYPE_STORARRAY)) {
+                        char disk[DEVSTAT_NAME_LEN + MAX_INT_DIGITS + 1];
+                        int def_io, def_ops, def_qops, def_util, def_iotime, def_await, def_avagsz, def_svctm,
+                            iter_io, iter_ops, iter_qops, iter_util, iter_iotime, iter_await, iter_avagsz, iter_svctm;
+                        struct cur_dstat {
+                            collected_number duration_read_ms;
+                            collected_number duration_write_ms;
+                            collected_number busy_time_ms;
+                        } cur_dstat;
+
+                        sprintf(disk, "%s%d", dstat[i].device_name, dstat[i].unit_number);
 
-                        st = rrdset_find_bytype_localhost("disk_qops", disk);
-                        if (unlikely(!st)) {
-                            st = rrdset_create_localhost("disk_qops", disk, NULL, disk, "disk.qops", "Disk Current I/O Operations", "operations", 2002, update_every, RRDSET_TYPE_LINE);
-                            rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
+                        struct disks_metadata *dm = dictionary_get(disks, disk);
+                        if(unlikely(!dm)) {
+                            char var_name[4096 + 1];
+                            snprintfz(var_name, 4096, "%s:%s", CONFIG_SECTION_KERN_DEVSTAT, disk);
+
+                            def_io     = do_io;
+                            def_ops    = do_ops;
+                            def_qops   = do_qops;
+                            def_util   = do_util;
+                            def_iotime = do_iotime;
+                            def_await  = do_await;
+                            def_avagsz = do_avagsz;
+                            def_svctm  = do_svctm;
+
+                            if(unlikely(simple_pattern_matches(excluded_disks, disk))) {
+                                def_io     = CONFIG_BOOLEAN_NO;
+                                def_ops    = CONFIG_BOOLEAN_NO;
+                                def_qops   = CONFIG_BOOLEAN_NO;
+                                def_util   = CONFIG_BOOLEAN_NO;
+                                def_iotime = CONFIG_BOOLEAN_NO;
+                                def_await  = CONFIG_BOOLEAN_NO;
+                                def_avagsz = CONFIG_BOOLEAN_NO;
+                                def_svctm  = CONFIG_BOOLEAN_NO;
+                            }
 
-                            rrddim_add(st, "operations", NULL, 1, 1, RRD_ALGORITHM_ABSOLUTE);
+                            iter_io     = config_get_boolean_ondemand(var_name, "bandwidth",                  def_io);
+                            iter_ops    = config_get_boolean_ondemand(var_name, "operations",                 def_ops);
+                            iter_qops   = config_get_boolean_ondemand(var_name, "queued operations",          def_qops);
+                            iter_util   = config_get_boolean_ondemand(var_name, "utilization percentage",     def_util);
+                            iter_iotime = config_get_boolean_ondemand(var_name, "i/o time",                   def_iotime);
+                            iter_await  = config_get_boolean_ondemand(var_name, "average completed i/o time", def_await);
+                            iter_avagsz = config_get_boolean_ondemand(var_name, "average completed i/o bandwidth",
+                                                                                                              def_avagsz);
+                            iter_svctm  = config_get_boolean_ondemand(var_name, "average service time",       def_svctm);
+
+                            struct disks_metadata dmp = {
+                                    .do_io     = iter_io,
+                                    .do_ops    = iter_ops,
+                                    .do_qops   = iter_qops,
+                                    .do_util   = iter_util,
+                                    .do_iotime = iter_iotime,
+                                    .do_await  = iter_await,
+                                    .do_avagsz = iter_avagsz,
+                                    .do_svctm  = iter_svctm,
+
+                                    // initialise data for differential charts
+
+                                    .prev_dstat.bytes_read        = dstat[i].bytes[DEVSTAT_READ],
+                                    .prev_dstat.bytes_write       = dstat[i].bytes[DEVSTAT_WRITE],
+                                    .prev_dstat.operations_read   = dstat[i].operations[DEVSTAT_READ],
+                                    .prev_dstat.operations_write  = dstat[i].operations[DEVSTAT_WRITE],
+                                    .prev_dstat.duration_read_ms  = dstat[i].duration[DEVSTAT_READ].sec * 1000
+                                                                    + dstat[i].duration[DEVSTAT_READ].frac * BINTIME_SCALE,
+                                    .prev_dstat.duration_write_ms = dstat[i].duration[DEVSTAT_WRITE].sec * 1000
+                                                                    + dstat[i].duration[DEVSTAT_READ].frac * BINTIME_SCALE,
+                                    .prev_dstat.busy_time_ms      = dstat[i].busy_time.sec * 1000
+                                                                    + dstat[i].busy_time.frac * BINTIME_SCALE,
+
+                                    .st_io = NULL,
+                                    .rd_io_in = NULL,
+                                    .rd_io_out = NULL,
+
+                                    .st_ops = NULL,
+                                    .rd_ops_in = NULL,
+                                    .rd_ops_out = NULL,
+
+                                    .st_qops = NULL,
+                                    .rd_qops = NULL,
+
+                                    .st_util = NULL,
+                                    .rd_util = NULL,
+
+                                    .st_iotime = NULL,
+                                    .rd_iotime_in = NULL,
+                                    .rd_iotime_out = NULL,
+
+                                    .st_await = NULL,
+                                    .rd_await_in = NULL,
+                                    .rd_await_out = NULL,
+
+                                    .st_avagsz = NULL,
+                                    .rd_avagsz_in = NULL,
+                                    .rd_avagsz_out = NULL,
+
+                                    .st_svctm = NULL,
+                                    .rd_svctm = NULL,
+                            };
+
+                            dm = dictionary_set(disks, disk, &dmp, sizeof(struct disks_metadata));
                         }
-                        else rrdset_next(st);
 
-                        rrddim_set(st, "operations", dstat[i].start_count - dstat[i].end_count);
-                        rrdset_done(st);
+                        cur_dstat.duration_read_ms  = dstat[i].duration[DEVSTAT_READ].sec * 1000
+                                                      + dstat[i].duration[DEVSTAT_READ].frac * BINTIME_SCALE;
+                        cur_dstat.duration_write_ms = dstat[i].duration[DEVSTAT_WRITE].sec * 1000
+                                                      + dstat[i].duration[DEVSTAT_READ].frac * BINTIME_SCALE;
+                        cur_dstat.busy_time_ms = dstat[i].busy_time.sec * 1000 + dstat[i].busy_time.frac * BINTIME_SCALE;
 
                         // --------------------------------------------------------------------
 
-                        st = rrdset_find_bytype_localhost("disk_util", disk);
-                        if (unlikely(!st)) {
-                            st = rrdset_create_localhost("disk_util", disk, NULL, disk, "disk.util", "Disk Utilization Time", "% of time working", 2004, update_every, RRDSET_TYPE_AREA);
-                            rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
+                        if(dm->do_io == CONFIG_BOOLEAN_YES || (dm->do_io == CONFIG_BOOLEAN_AUTO &&
+                                (dstat[i].bytes[DEVSTAT_READ] || dstat[i].bytes[DEVSTAT_WRITE]))) {
+                            if (unlikely(!dm->st_io)) {
+                                dm->st_io = rrdset_create_localhost("disk",
+                                                                    disk,
+                                                                    NULL,
+                                                                    disk,
+                                                                    "disk.io",
+                                                                    "Disk I/O Bandwidth",
+                                                                    "kilobytes/s",
+                                                                    2000,
+                                                                    update_every,
+                                                                    RRDSET_TYPE_AREA
+                                );
+
+                                dm->rd_io_in  = rrddim_add(dm->st_io, "reads",  NULL,  1, KILO_FACTOR,
+                                                          RRD_ALGORITHM_INCREMENTAL);
+                                dm->rd_io_out = rrddim_add(dm->st_io, "writes", NULL, -1, KILO_FACTOR,
+                                                           RRD_ALGORITHM_INCREMENTAL);
+                            } else
+                                rrdset_next(dm->st_io);
 
-                            rrddim_add(st, "utilization", NULL, 1, 10, RRD_ALGORITHM_INCREMENTAL);
+                            rrddim_set_by_pointer(dm->st_io, dm->rd_io_in,  dstat[i].bytes[DEVSTAT_READ]);
+                            rrddim_set_by_pointer(dm->st_io, dm->rd_io_out, dstat[i].bytes[DEVSTAT_WRITE]);
+                            rrdset_done(dm->st_io);
                         }
-                        else rrdset_next(st);
 
-                        cur_dstat.busy_time_ms = dstat[i].busy_time.sec * 1000 + dstat[i].busy_time.frac * BINTIME_SCALE;
-                        prev_dstat.busy_time_ms = rrddim_set(st, "utilization", cur_dstat.busy_time_ms);
-                        rrdset_done(st);
+                        // --------------------------------------------------------------------
+
+                        if(dm->do_ops == CONFIG_BOOLEAN_YES || (dm->do_ops == CONFIG_BOOLEAN_AUTO &&
+                                (dstat[i].operations[DEVSTAT_READ] || dstat[i].operations[DEVSTAT_WRITE]))) {
+                            if (unlikely(!dm->st_ops)) {
+                                dm->st_ops = rrdset_create_localhost("disk_ops",
+                                                                     disk,
+                                                                     NULL,
+                                                                     disk,
+                                                                     "disk.ops",
+                                                                     "Disk Completed I/O Operations",
+                                                                     "operations/s",
+                                                                     2001,
+                                                                     update_every,
+                                                                     RRDSET_TYPE_LINE
+                                );
+
+                                rrdset_flag_set(dm->st_ops, RRDSET_FLAG_DETAIL);
+
+                                dm->rd_ops_in = rrddim_add(dm->st_ops, "reads", NULL, 1, 1, RRD_ALGORITHM_INCREMENTAL);
+                                dm->rd_ops_out = rrddim_add(dm->st_ops, "writes", NULL, -1, 1,
+                                                            RRD_ALGORITHM_INCREMENTAL);
+                            } else
+                                rrdset_next(dm->st_ops);
+
+                            rrddim_set_by_pointer(dm->st_ops, dm->rd_ops_in, dstat[i].operations[DEVSTAT_READ]);
+                            rrddim_set_by_pointer(dm->st_ops, dm->rd_ops_out, dstat[i].operations[DEVSTAT_WRITE]);
+                            rrdset_done(dm->st_ops);
+                        }
 
                         // --------------------------------------------------------------------
 
-                        st = rrdset_find_bytype_localhost("disk_iotime", disk);
-                        if (unlikely(!st)) {
-                            st = rrdset_create_localhost("disk_iotime", disk, NULL, disk, "disk.iotime", "Disk Total I/O Time", "milliseconds/s", 2022, update_every, RRDSET_TYPE_LINE);
-                            rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
+                        if(dm->do_qops == CONFIG_BOOLEAN_YES || (dm->do_qops == CONFIG_BOOLEAN_AUTO &&
+                                (dstat[i].start_count || dstat[i].end_count))) {
+                            if (unlikely(!dm->st_qops)) {
+                                dm->st_qops = rrdset_create_localhost("disk_qops",
+                                                                      disk,
+                                                                      NULL,
+                                                                      disk,
+                                                                      "disk.qops",
+                                                                      "Disk Current I/O Operations",
+                                                                      "operations",
+                                                                      2002,
+                                                                      update_every,
+                                                                      RRDSET_TYPE_LINE
+                                );
+
+                                rrdset_flag_set(dm->st_qops, RRDSET_FLAG_DETAIL);
+
+                                dm->rd_qops = rrddim_add(dm->st_qops, "operations", NULL, 1, 1, RRD_ALGORITHM_ABSOLUTE);
+                            } else
+                                rrdset_next(dm->st_qops);
+
+                            rrddim_set_by_pointer(dm->st_qops, dm->rd_qops, dstat[i].start_count - dstat[i].end_count);
+                            rrdset_done(dm->st_qops);
+                        }
+
+                        // --------------------------------------------------------------------
+
+                        if(dm->do_util == CONFIG_BOOLEAN_YES || (dm->do_util == CONFIG_BOOLEAN_AUTO &&
+                                                                 cur_dstat.busy_time_ms)) {
+                            if (unlikely(!dm->st_util)) {
+                                dm->st_util = rrdset_create_localhost("disk_util",
+                                                                      disk,
+                                                                      NULL,
+                                                                      disk,
+                                                                      "disk.util",
+                                                                      "Disk Utilization Time",
+                                                                      "% of time working",
+                                                                      2004,
+                                                                      update_every,
+                                                                      RRDSET_TYPE_AREA
+                                );
+
+                                rrdset_flag_set(dm->st_util, RRDSET_FLAG_DETAIL);
+
+                                dm->rd_util = rrddim_add(dm->st_util, "utilization", NULL, 1, 10,
+                                                         RRD_ALGORITHM_INCREMENTAL);
+                            } else
+                                rrdset_next(dm->st_util);
 
-                            rrddim_add(st, "reads", NULL, 1, 1, RRD_ALGORITHM_INCREMENTAL);
-                            rrddim_add(st, "writes", NULL, -1, 1, RRD_ALGORITHM_INCREMENTAL);
+                            rrddim_set_by_pointer(dm->st_util, dm->rd_util, cur_dstat.busy_time_ms);
+                            rrdset_done(dm->st_util);
                         }
-                        else rrdset_next(st);
 
-                        cur_dstat.duration_read_ms = dstat[i].duration[DEVSTAT_READ].sec * 1000 + dstat[i].duration[DEVSTAT_READ].frac * BINTIME_SCALE;
-                        cur_dstat.duration_write_ms = dstat[i].duration[DEVSTAT_WRITE].sec * 1000 + dstat[i].duration[DEVSTAT_READ].frac * BINTIME_SCALE;
-                        prev_dstat.duration_read_ms = rrddim_set(st, "reads", cur_dstat.duration_read_ms);
-                        prev_dstat.duration_write_ms = rrddim_set(st, "writes", cur_dstat.duration_write_ms);
-                        rrdset_done(st);
+                        // --------------------------------------------------------------------
+
+                        if(dm->do_iotime == CONFIG_BOOLEAN_YES || (dm->do_iotime == CONFIG_BOOLEAN_AUTO &&
+                                (cur_dstat.duration_read_ms || cur_dstat.duration_write_ms))) {
+                            if (unlikely(!dm->st_iotime)) {
+                                dm->st_iotime = rrdset_create_localhost("disk_iotime",
+                                                                        disk,
+                                                                        NULL,
+                                                                        disk,
+                                                                        "disk.iotime",
+                                                                        "Disk Total I/O Time",
+                                                                        "milliseconds/s",
+                                                                        2022,
+                                                                        update_every,
+                                                                        RRDSET_TYPE_LINE
+                                );
+
+                                rrdset_flag_set(dm->st_iotime, RRDSET_FLAG_DETAIL);
+
+                                dm->rd_iotime_in  = rrddim_add(dm->st_iotime, "reads",  NULL,  1, 1,
+                                                               RRD_ALGORITHM_INCREMENTAL);
+                                dm->rd_iotime_out = rrddim_add(dm->st_iotime, "writes", NULL, -1, 1,
+                                                               RRD_ALGORITHM_INCREMENTAL);
+                            } else
+                                rrdset_next(dm->st_iotime);
+
+                            rrddim_set_by_pointer(dm->st_iotime, dm->rd_iotime_in,  cur_dstat.duration_read_ms);
+                            rrddim_set_by_pointer(dm->st_iotime, dm->rd_iotime_out, cur_dstat.duration_write_ms);
+                            rrdset_done(dm->st_iotime);
+                        }
 
                         // --------------------------------------------------------------------
                         // calculate differential charts
@@ -3466,73 +3731,193 @@ int do_kern_devstat(int update_every, usec_t dt) {
 
                             // --------------------------------------------------------------------
 
-                            st = rrdset_find_bytype_localhost("disk_await", disk);
-                            if (unlikely(!st)) {
-                                st = rrdset_create_localhost("disk_await", disk, NULL, disk, "disk.await", "Average Completed I/O Operation Time", "ms per operation", 2005, update_every, RRDSET_TYPE_LINE);
-                                rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
-
-                                rrddim_add(st, "reads", NULL, 1, 1, RRD_ALGORITHM_ABSOLUTE);
-                                rrddim_add(st, "writes", NULL, -1, 1, RRD_ALGORITHM_ABSOLUTE);
+                            if(dm->do_await == CONFIG_BOOLEAN_YES || (dm->do_await == CONFIG_BOOLEAN_AUTO &&
+                                    (dstat[i].operations[DEVSTAT_READ] || dstat[i].operations[DEVSTAT_WRITE]))) {
+                                if (unlikely(!dm->st_await)) {
+                                    dm->st_await = rrdset_create_localhost("disk_await",
+                                                                           disk,
+                                                                           NULL,
+                                                                           disk,
+                                                                           "disk.await",
+                                                                           "Average Completed I/O Operation Time",
+                                                                           "ms per operation",
+                                                                           2005,
+                                                                           update_every,
+                                                                           RRDSET_TYPE_LINE
+                                    );
+
+                                    rrdset_flag_set(dm->st_await, RRDSET_FLAG_DETAIL);
+
+                                    dm->rd_await_in  = rrddim_add(dm->st_await, "reads",  NULL,  1, 1,
+                                                                  RRD_ALGORITHM_ABSOLUTE);
+                                    dm->rd_await_out = rrddim_add(dm->st_await, "writes", NULL, -1, 1,
+                                                                  RRD_ALGORITHM_ABSOLUTE);
+                                } else
+                                    rrdset_next(dm->st_await);
+
+                                rrddim_set_by_pointer(dm->st_await, dm->rd_await_in,
+                                                      (dstat[i].operations[DEVSTAT_READ] -
+                                                       dm->prev_dstat.operations_read) ?
+                                                      (cur_dstat.duration_read_ms - dm->prev_dstat.duration_read_ms) /
+                                                      (dstat[i].operations[DEVSTAT_READ] -
+                                                       dm->prev_dstat.operations_read) :
+                                                      0);
+                                rrddim_set_by_pointer(dm->st_await, dm->rd_await_out,
+                                                      (dstat[i].operations[DEVSTAT_WRITE] -
+                                                       dm->prev_dstat.operations_write) ?
+                                                      (cur_dstat.duration_write_ms - dm->prev_dstat.duration_write_ms) /
+                                                      (dstat[i].operations[DEVSTAT_WRITE] -
+                                                       dm->prev_dstat.operations_write) :
+                                                      0);
+                                rrdset_done(dm->st_await);
                             }
-                            else rrdset_next(st);
-
-                            rrddim_set(st, "reads", (dstat[i].operations[DEVSTAT_READ] - prev_dstat.operations_read) ?
-                                (cur_dstat.duration_read_ms - prev_dstat.duration_read_ms) / (dstat[i].operations[DEVSTAT_READ] - prev_dstat.operations_read) : 0);
-                            rrddim_set(st, "writes", (dstat[i].operations[DEVSTAT_WRITE] - prev_dstat.operations_write) ?
-                                (cur_dstat.duration_write_ms - prev_dstat.duration_write_ms) / (dstat[i].operations[DEVSTAT_WRITE] - prev_dstat.operations_write) : 0);
-                            rrdset_done(st);
 
                             // --------------------------------------------------------------------
 
-                            st = rrdset_find_bytype_localhost("disk_avgsz", disk);
-                            if (unlikely(!st)) {
-                                st = rrdset_create_localhost("disk_avgsz", disk, NULL, disk, "disk.avgsz", "Average Completed I/O Operation Bandwidth", "kilobytes per operation", 2006, update_every, RRDSET_TYPE_AREA);
-                                rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
-
-                                rrddim_add(st, "reads", NULL, 1, 1024, RRD_ALGORITHM_ABSOLUTE);
-                                rrddim_add(st, "writes", NULL, -1, 1024, RRD_ALGORITHM_ABSOLUTE);
+                            if(dm->do_avagsz == CONFIG_BOOLEAN_YES || (dm->do_avagsz == CONFIG_BOOLEAN_AUTO &&
+                                    (dstat[i].operations[DEVSTAT_READ] || dstat[i].operations[DEVSTAT_WRITE]))) {
+                                if (unlikely(!dm->st_avagsz)) {
+                                    dm->st_avagsz = rrdset_create_localhost("disk_avgsz",
+                                                                            disk,
+                                                                            NULL,
+                                                                            disk,
+                                                                            "disk.avgsz",
+                                                                            "Average Completed I/O Operation Bandwidth",
+                                                                            "kilobytes per operation",
+                                                                            2006,
+                                                                            update_every,
+                                                                            RRDSET_TYPE_AREA
+                                    );
+
+                                    rrdset_flag_set(dm->st_avagsz, RRDSET_FLAG_DETAIL);
+
+                                    dm->rd_avagsz_in  = rrddim_add(dm->st_avagsz, "reads",  NULL,  1, KILO_FACTOR,
+                                                                  RRD_ALGORITHM_ABSOLUTE);
+                                    dm->rd_avagsz_out = rrddim_add(dm->st_avagsz, "writes", NULL, -1, KILO_FACTOR,
+                                                                   RRD_ALGORITHM_ABSOLUTE);
+                                } else
+                                    rrdset_next(dm->st_avagsz);
+
+                                rrddim_set_by_pointer(dm->st_avagsz, dm->rd_avagsz_in,
+                                                      (dstat[i].operations[DEVSTAT_READ] -
+                                                       dm->prev_dstat.operations_read) ?
+                                                      (dstat[i].bytes[DEVSTAT_READ] - dm->prev_dstat.bytes_read) /
+                                                      (dstat[i].operations[DEVSTAT_READ] -
+                                                       dm->prev_dstat.operations_read) :
+                                                      0);
+                                rrddim_set_by_pointer(dm->st_avagsz, dm->rd_avagsz_out,
+                                                      (dstat[i].operations[DEVSTAT_WRITE] -
+                                                       dm->prev_dstat.operations_write) ?
+                                                      (dstat[i].bytes[DEVSTAT_WRITE] - dm->prev_dstat.bytes_write) /
+                                                      (dstat[i].operations[DEVSTAT_WRITE] -
+                                                       dm->prev_dstat.operations_write) :
+                                                      0);
+                                rrdset_done(dm->st_avagsz);
                             }
-                            else rrdset_next(st);
-
-                            rrddim_set(st, "reads", (dstat[i].operations[DEVSTAT_READ] - prev_dstat.operations_read) ?
-                                (dstat[i].bytes[DEVSTAT_READ] - prev_dstat.bytes_read) / (dstat[i].operations[DEVSTAT_READ] - prev_dstat.operations_read) : 0);
-                            rrddim_set(st, "writes", (dstat[i].operations[DEVSTAT_WRITE] - prev_dstat.operations_write) ?
-                                (dstat[i].bytes[DEVSTAT_WRITE] - prev_dstat.bytes_write) / (dstat[i].operations[DEVSTAT_WRITE] - prev_dstat.operations_write) : 0);
-                            rrdset_done(st);
 
                             // --------------------------------------------------------------------
 
-                            st = rrdset_find_bytype_localhost("disk_svctm", disk);
-                            if (unlikely(!st)) {
-                                st = rrdset_create_localhost("disk_svctm", disk, NULL, disk, "disk.svctm", "Average Service Time", "ms per operation", 2007, update_every, RRDSET_TYPE_LINE);
-                                rrdset_flag_set(st, RRDSET_FLAG_DETAIL);
-
-                                rrddim_add(st, "svctm", NULL, 1, 1, RRD_ALGORITHM_ABSOLUTE);
+                            if(dm->do_svctm == CONFIG_BOOLEAN_YES || (dm->do_svctm == CONFIG_BOOLEAN_AUTO &&
+                                    (dstat[i].operations[DEVSTAT_READ] || dstat[i].operations[DEVSTAT_WRITE]))) {
+                                if (unlikely(!dm->st_svctm)) {
+                                    dm->st_svctm = rrdset_create_localhost("disk_svctm",
+                                                                           disk,
+                                                                           NULL,
+                                                                           disk,
+                                                                           "disk.svctm",
+                                                                           "Average Service Time",
+                                                                           "ms per operation",
+                                                                           2007,
+                                                                           update_every,
+                                                                           RRDSET_TYPE_LINE
+                                    );
+
+                                    rrdset_flag_set(dm->st_svctm, RRDSET_FLAG_DETAIL);
+
+                                    dm->rd_svctm = rrddim_add(dm->st_svctm, "svctm", NULL, 1, 1,
+                                                              RRD_ALGORITHM_ABSOLUTE);
+                                } else
+                                    rrdset_next(dm->st_svctm);
+
+                                rrddim_set_by_pointer(dm->st_svctm, dm->rd_svctm,
+                                    ((dstat[i].operations[DEVSTAT_READ] - dm->prev_dstat.operations_read) +
+                                     (dstat[i].operations[DEVSTAT_WRITE] - dm->prev_dstat.operations_write)) ?
+                                    (cur_dstat.busy_time_ms - dm->prev_dstat.busy_time_ms) /
+                                    ((dstat[i].operations[DEVSTAT_READ] - dm->prev_dstat.operations_read) +
+                                     (dstat[i].operations[DEVSTAT_WRITE] - dm->prev_dstat.operations_write)) :
+                                    0);
+                                rrdset_done(dm->st_svctm);
                             }
-                            else rrdset_next(st);
 
-                            rrddim_set(st, "svctm", ((dstat[i].operations[DEVSTAT_READ] - prev_dstat.operations_read) + (dstat[i].operations[DEVSTAT_WRITE] - prev_dstat.operations_write)) ?
-                                (cur_dstat.busy_time_ms - prev_dstat.busy_time_ms) / ((dstat[i].operations[DEVSTAT_READ] - prev_dstat.operations_read) + (dstat[i].operations[DEVSTAT_WRITE] - prev_dstat.operations_write)) : 0);
-                            rrdset_done(st);
+                            // --------------------------------------------------------------------
+
+                            dm->prev_dstat.bytes_read        = dstat[i].bytes[DEVSTAT_READ];
+                            dm->prev_dstat.bytes_write       = dstat[i].bytes[DEVSTAT_WRITE];
+                            dm->prev_dstat.operations_read   = dstat[i].operations[DEVSTAT_READ];
+                            dm->prev_dstat.operations_write  = dstat[i].operations[DEVSTAT_WRITE];
+                            dm->prev_dstat.duration_read_ms  = cur_dstat.duration_read_ms;
+                            dm->prev_dstat.duration_write_ms = cur_dstat.duration_write_ms;
+                            dm->prev_dstat.busy_time_ms      = cur_dstat.busy_time_ms;
                         }
                     }
                 }
 
                 // --------------------------------------------------------------------
 
-                st = rrdset_find_bytype_localhost("system", "io");
-                if (unlikely(!st)) {
-                    st = rrdset_create_localhost("system", "io", NULL, "disk", NULL, "Disk I/O", "kilobytes/s", 150, update_every, RRDSET_TYPE_AREA);
-                    rrddim_add(st, "in",  NULL,  1, 1, RRD_ALGORITHM_INCREMENTAL);
-                    rrddim_add(st, "out", NULL, -1, 1, RRD_ALGORITHM_INCREMENTAL);
-                }
-                else rrdset_next(st);
+                if (unlikely(!do_system_io)) {
+                    static RRDSET *st = NULL;
+                    static RRDDIM *rd_in = NULL, *rd_out = NULL;
+
+                    st = rrdset_find_bytype_localhost("system", "io");
+                    if (unlikely(!st)) {
+                        st = rrdset_create_localhost("system",
+                                                     "io",
+                                                     NULL,
+                                                     "disk",
+                                                     NULL,
+                                                     "Disk I/O",
+                                                     "kilobytes/s",
+                                                     150,
+                                                     update_every,
+                                                     RRDSET_TYPE_AREA
+                        );
 
-                rrddim_set(st, "in", total_disk_kbytes_read);
-                rrddim_set(st, "out", total_disk_kbytes_write);
-                rrdset_done(st);
+                        rd_in  = rrddim_add(st, "in",  NULL,  1, 1, RRD_ALGORITHM_INCREMENTAL);
+                        rd_out = rrddim_add(st, "out", NULL, -1, 1, RRD_ALGORITHM_INCREMENTAL);
+                    } else
+                        rrdset_next(st);
+
+                    rrddim_set_by_pointer(st, rd_in,  total_disk_kbytes_read);
+                    rrddim_set_by_pointer(st, rd_out, total_disk_kbytes_write);
+                    rrdset_done(st);
+                }
             }
         }
+        if (unlikely(common_error)) {
+            do_system_io = 0;
+            error("DISABLED: system.io chart");
+            do_io = 0;
+            error("DISABLED: disk.* charts");
+            do_ops = 0;
+            error("DISABLED: disk_ops.* charts");
+            do_qops = 0;
+            error("DISABLED: disk_qops.* charts");
+            do_util = 0;
+            error("DISABLED: disk_util.* charts");
+            do_iotime = 0;
+            error("DISABLED: disk_iotime.* charts");
+            do_await = 0;
+            error("DISABLED: disk_await.* charts");
+            do_avagsz = 0;
+            error("DISABLED: disk_avgsz.* charts");
+            do_svctm = 0;
+            error("DISABLED: disk_svctm.* charts");
+            error("DISABLED: kern.devstat module");
+            return 1;
+        }
+    } else {
+        error("DISABLED: kern.devstat module");
+        return 1;
     }
 
     return 0;