hugepages: check for error if readdir returns NULL

Change-Id: Ib361b648407c7052218c1f6b9b624cb7aa799634
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

hugepages: typo fix in 2nd pathconf() check

Change-Id: Ie4ceb0ec4b55e60fcffb1d59d33a5711366eb2f7
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

hugepages: rework 2

Fix typo in .pod file.
Change char * strings to static const char [].
Change INFO() to DEBUG() in 1 place.
Removed unnecessary initialisations & casts.
Fixed ignored error returns from some funcs.
Actually use result returned from pathconf().

Change-Id: I0b59af4eb8afd29141abfa9233ca1d4a8d19d0d1
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

hugepages: rework

Changed hugepage metric type and type instances.
Made a single entry in types.db for "free" and
    "used" hugepages.
Calculate the used hugepages.
Removed read_syshugepage_dir; replaced with
    walk_directory.
Cleared up descriptions in .pod.
Made the .conf plugin option names less cryptic.
Linux-only plugin; changed configure.ac accordingly.
Stripped function declarations and header includes
    that are not used.
Tidied up globals and made some globals local.
Removed some hardcoded values and fixed spacing.
Other general tidy-ups.

Change-Id: Id0243397d3fd4ac1be90266b266341318c05c903
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>

hugpages: fix compilation error

Fix error: format '%lu' expects type 'long unsigned int', but
argument 6 has type 'derive_t'. For line 93.

Change-Id: I4f164d238cd4fa6f8a69d78b2906042aa3c5da72
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

hugepages plugin: a stats plugin for hugepages

Plugin that allows stats for hugepages to be collected.

Change-Id: Ide5ff31c54206cf5ee3d6155899bd53e8edae7bd
Signed-off-by: Jaroslav Safka <jaroslavx.safka@intel.com>
Signed-off-by: Kim Jones <kim-marie.jones@intel.com>
Signed-off-by: Maryam Tahhan <maryam.tahhan@intel.com>

Merge pull request #1796 from tokkee/sh/configure

configure: Use AC_MSG_RESULT to print the config summary.

configure: Use AC_MSG_RESULT to print the config summary.

This way, it will show up in config.log as well.

configure: Add missing end fold marker (for Vim).

Merge pull request #1783 from matwey/ntp

ntpd: Check STA_NANO in the server status

ntpd: Check STA_NANO in the server status

When STA_NANO is set in ik->status, then results are
expressed in nanoseconds (instead of microseconds by default).

Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>

zookeeper plugin: remove unneccesary cast

ntpd plugin: remove unneccesary casts

collectd-nagios: fix some whitespace

virt plugin: fix some whitespace

Merge pull request #1793 from rubenk/fix-riemann-plugin-on-32bit-solaris

Fix write_riemann plugin build on 32-bit Solaris

Fix write_riemann plugin build on 32-bit Solaris

Fixes #1782

CC       write_riemann_la-write_riemann.lo
In file included from ./daemon/collectd.h:31:0,
                 from write_riemann.c:34:
./config.h:1664:0: error: "_FILE_OFFSET_BITS" redefined [-Werror]
 #define _FILE_OFFSET_BITS 64
 ^
In file included from /usr/include/limits.h:17:0,
                 from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/limits.h:168,
                 from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/syslimits.h:7,
                 from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/limits.h:34,
                 from /opt/csw/include/protobuf-c/protobuf-c.h:200,
                 from /opt/csw/include/riemann/proto/riemann.pb-c.h:7,
                 from /opt/csw/include/riemann/attribute.h:21,
                 from /opt/csw/include/riemann/riemann-client.h:23,
                 from write_riemann.c:31:
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/sys/feature_tests.h:196:0:
note: this is the location of the previous definition
 #define _FILE_OFFSET_BITS 32
 ^
cc1: all warnings being treated as errors
Makefile:4645: recipe for target 'write_riemann_la-write_riemann.lo'
failed
gmake[3]: *** [write_riemann_la-write_riemann.lo] Error 1
gmake[3]: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/src'
Makefile:4879: recipe for target 'all-recursive' failed
gmake[2]: *** [all-recursive] Error 1
gmake[2]: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/src'
Makefile:3209: recipe for target 'all' failed
gmake[1]: *** [all] Error 2
gmake[1]: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/src'
Makefile:572: recipe for target 'all-recursive' failed
gmake: *** [all-recursive] Error 1
gmake: Leaving directory
'/home/dam/mgar/pkg/collectd/trunk/work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166'
/home/dam/mgar/pkg/.buildsys/v2/gar//gar.lib.mk:874: die Regel für Ziel
„build-work/solaris10-i386/build-isa-pentium_pro/collectd-5.5.1.1126.g37fe166/Makefile“
scheiterte

Merge pull request #1770 from swt2c/smart_ignore_sleep

smart plugin: add IgnoreSleepMode option to ignore sleeping disks

smart plugin: add IgnoreSleepMode option to ignore sleeping disks

This option enables the smart plugin to use disks that libatasmart mistakenly
reports as asleep.  This happens because libatasmart has not been updated to
incorporate support for newer idle states in the ATA spec.

Merge pull request #1773 from skob/master

Add slow queries statistics to mysql module

Add slow queries statistics to mysql module

Merge pull request #1774 from mmckinst/rdkafka_now_libkafka

rdkafka is called librdkafka

rdkafka is called librdkafka

The [spec file for
librdkafka](https://github.com/edenhill/librdkafka/blob/master/packaging/rpm/librdkafka.spec)
calls its package 'librdkafka-devel', not 'rdkafka-devel'. Searching the
web for 'rdkafka-devel RPM' turns up nothing, all the RPMs are named
'librdkafka-devel'.

README: add reference to libxen

Merge branch 'collectd-5.5'

README: mention 3 missing plugins

AUTHORS: mention new contributors and update plugin attributions

Merge branch 'collectd-5.4' into collectd-5.5

src/*.pod: fix minor warnings spotted by podchecker

collectd.conf.pod: fix minor typos in ntpd plugin section

doc: add chronyd section to manpage

Merge pull request #1004 from tokkee/sh/curl

Add support for detailed cURL statistics

Language and typo fixes.

utils_curl_stats: Drop the nonnull attribute from curl_stats_from_config.

It's not overly useful after all but generates false compiler warnings /
errors on Xenial using CLANG, since we also check the parameter in the
function (which is what we want in the end and there's no risk in actually
calling it with a NULL value).

curl stats: Use cf_util_get_boolean().

curl stats: Simplified some error checks and removed unused code.

configure / curl stats: Check for CURLINFO_APPCONNECT_TIME.

It's available since cURL 7.19 but RHEL5 ships an older version.

collectd.conf(5): Briefly explain MeasureResponseTime vs TotalTime.

curl stats: Use CamelCase identifiers instead of using underscores.

This is more consistent with the usual collectd style but still close enough
to the names used in cURL.

collectd.conf(5): Add documentation for cURL statistics.

Add request specific statistics to all CURL-based plugins.

All metrics supported by curl_easy_getinfo can be retrieved and dispatched for
each request done through CURL. All statistics are optional and disabled by
default.

configure.ac: indent and sort some conditionals

configure.ac: remove obsolete macro

We don't need it anymore. See
http://gnu-autoconf.7623.n7.nabble.com/Bug-389558-autoconf-AC-PROG-GCC-TRADITIONAL-throws-error-in-comparision-td1885.html

chrony plugin: use ANSI function definitions

chrony.c:392:1: warning: function declaration isn’t a prototype
[-Wstrict-prototypes]
 chrony_set_timeout()
 ^
chrony.c: In function ‘chrony_set_timeout’:
chrony.c:392:1: warning: old-style function definition
[-Wold-style-definition]
chrony.c: At top level:

Merge pull request #1758 from rubenk/README

README: use https for collectd.org link

Merge pull request #1759 from rubenk/ansi-function-declaration

uc_get_iterator: use ansi function declaration

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

Bug#1059 : Remove any white spaces in ethstats stat names (affects VMXNet3 interface)

uc_get_iterator: use ansi function declaration

In file included from common.c:37:0:
utils_cache.h:76:1: warning: function declaration isn’t a prototype
[-Wstrict-prototypes]
 uc_iter_t *uc_get_iterator ();
 ^

README: use https for collectd.org link

Merge pull request #1750 from tokkee/sh/cli

daemon: Let all command line options override global options from the config.

configfile: Log errors when trying to access unknown global options.

daemon: Let all command line options override global options from the config.

This replaces 09fb81e130ad70bf0547cec3d3c349ce8cb092cf which provided a
specific solution for the `PIDFile' option. The new solution is generic and
more nicely abstracted without the need for global variables across multiple
modules.

Fixes #366

Merge pull request #1747 from rubenk/write_graphite_check_fd

Write graphite check fd

wg_send_buffer: return error if fd < 0

CID 116310

write_graphite: remove dead assignment

status is immediately overwritten in the next line.

Merge pull request #1746 from rubenk/swrite-check-for-open-fd

swrite: return error if fd is negative

swrite: return error if fd is negative

Merge pull request #1743 from rubenk/apcups-coverity

apcups: ignore result of swrite in net_shutdown

apcups: ignore result of swrite in net_shutdown

CID 37945

Add test logs to .gitignore

automake: disable gzipped tarballs

We only use bz2.

Fixes make distcheck on FreeBSD-11-CURRENT

test -n ""  || find "collectd-5.5.1.1086.g41e7c34" -type d ! -perm -755  -exec chmod u+rwx,go+rx {} \; -o  ! -type d ! -perm -444 -links 1 -exec chmod a+r {} \; -o  ! -type d ! -perm -400 -exec chmod a+r {} \; -o  ! -type d ! -perm -444 -exec /bin/sh /home/ruben/src/collectd/libltdl/config/install-sh -c -m a+r {} {} \;  || chmod -R a+r "collectd-5.5.1.1086.g41e7c34"
tardir=collectd-5.5.1.1086.g41e7c34 && tar --format=posix -chf - "$tardir" | BZIP2=${BZIP2--9} bzip2 -c >collectd-5.5.1.1086.g41e7c34.tar.bz2
tardir=collectd-5.5.1.1086.g41e7c34 && tar --format=posix -chf - "$tardir" | GZIP=--best gzip -c >collectd-5.5.1.1086.g41e7c34.tar.gz
if test -d "collectd-5.5.1.1086.g41e7c34"; then  find "collectd-5.5.1.1086.g41e7c34" -type d ! -perm -200 -exec chmod u+w {} ';'  && rm -rf "collectd-5.5.1.1086.g41e7c34"  || { sleep 5 && rm -rf "collectd-5.5.1.1086.g41e7c34"; };  else :; fi
case 'collectd-5.5.1.1086.g41e7c34.tar.gz collectd-5.5.1.1086.g41e7c34.tar.bz2' in  *.tar.gz*)  GZIP=--best gzip -dc collectd-5.5.1.1086.g41e7c34.tar.gz | tar -xf - ;; *.tar.bz2*)  bzip2 -dc collectd-5.5.1.1086.g41e7c34.tar.bz2 | tar -xf - ;; *.tar.lz*)  lzip -dc collectd-5.5.1.1086.g41e7c34.tar.lz | tar -xf - ;; *.tar.xz*)  xz -dc collectd-5.5.1.1086.g41e7c34.tar.xz | tar -xf - ;; *.tar.Z*)  uncompress -c collectd-5.5.1.1086.g41e7c34.tar.Z | tar -xf - ;; *.shar.gz*)  GZIP=--best gzip -dc collectd-5.5.1.1086.g41e7c34.shar.gz | unshar ;; *.zip*)  unzip collectd-5.5.1.1086.g41e7c34.zip ;; esac
tar: Parse error: SCHILY.acl.default
tar: Parse error: SCHILY.acl.default
tar: Error exit delayed from previous errors.
*** Error code 1

Stop.
make: stopped in /usr/home/ruben/src/collectd

src/Makefile: Add valgrind.FreeBSD.suppress to the dist tarball.

Merge pull request #1742 from tokkee/sh/suppress

Valgrind: Suppress a bogus invalid read on FreeBSD.

Merge pull request #1741 from rubenk/fix-leak-in-disk-plugin

Plug a memory leak in disk plugin

Valgrind: Suppress a bogus invalid read on FreeBSD.

Valgrind reports an invalid read in common.c:parse_value. The error is
supposed to happen in strlen() on a just previously strdup()ed string.

Possibly, this is some sort of alignment issue. The read size is reported to
be 4 on a buffer of size 3.

https://bugzilla.redhat.com/show_bug.cgi?id=678518 may also be related.

Plug a memory leak in disk plugin

Introduced by bdbc066 so no need to backport.
CID 116311

Makefile: Add testwrapper.sh to dist tarballs.

Merge pull request #1740 from rubenk/correct-am_cflags

Append -Werror to the right flags

chrony plugin: constify 2 function arguments

Fixes a couple of dozen of these warnings:
chrony.c:889:20: warning: passing 'const char [14]' to parameter of type
'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
  chrony_push_data("clock_stratum", "chrony", (__extension__ ({ unsigned
short int __v, __x = (unsigned short int)
(chrony_resp.body.tracking.f_stratum); if (__builtin_constant_p (__x))
__v = ((unsigned short int) ((((__x) >> 8) & 0xff) | (((__x) & 0xff) <<
8))); else __asm__ ("rorw $8, %w0" : "=r" (__v) : "0" (__x) : "cc");
__v; })));
                   ^~~~~~~~~~~~~~~

chrony plugin: fix clang warning

chrony.c:745:11: warning: declaration shadows a local variable
[-Wshadow]
      int fh = open("/dev/random", 00);
          ^

collectd.spec: add missing %endif

configure.ac: fix autoreconf warnings

configure.ac:804: warning: AC_LANG_CONFTEST: no AC_LANG_SOURCE call
detected in body
../../lib/autoconf/lang.m4:193: AC_LANG_CONFTEST is expanded from...
configure.ac:804: the top level

Append -Werror to the right flags

testwrapper.sh: Use /bin/sh instead of /bin/bash.

We don't use any bashisms.

Merge pull request #1739 from tokkee/sh/memcheck

unit tests: Run all tests through Valgrind's memcheck, if available.

Actually add the testwrapper.sh script.

… it's now used for running all tests.

configure: Drop unneeded AC_SUBST() calls for AC_PATH_PROG() results.

AC_PATH_PROG calls AC_SUBST for us.

unit tests: Run all tests through Valgrind's memcheck, if available.

collectd.spec: add new gRPC plugin

Disable it for now as the version of protobuf-compiler
in RHEL and Fedora is too old.

ceph_test: Free the yajl handle.

utils_mount_test: Free dynamically allocated strings.

testing: Evaluate expressions only once in EXPECT_EQ_STR().

utils_vl_lookup: Free regexen when destroying class lists.

Else, we'll leak any memory used by the regex.

Merge pull request #1738 from tokkee/sh/silent

src/Makefile: Support silent build rules for pod2man.

src/Makefile: Support silent build rules for pod2man.

Merge pull request #1699 from tokkee/sh/grpc

gRPC server plugin

gRPC doc: Fixed typos and mention C++11 requirement.

src/Makefile: Don't unnecessarily set plugin specific CXXFLAGS.

Also, drop unneeded -lphread / -ldl from grpc's libraries.

src/Makefile: Don't set subdir-objects.

That's now done globally via configure.

collectd.spec: fix rpmlint warning

collectd.src: W: summary-not-capitalized C statistics collection and
monitoring daemon

collectd.spec: make scriptlets work on Fedora

collectd.spec: only require python26 on RHEL < 6

Otherwise the else clause in the condition matches on Fedora too.

This makes the spec build on Fedora.

collectd.spec: disable plugins on older distro

Instead of keeping a list per RHEL version of which plugins to enable,
invert the logic and keep a list of plugins to disable when the distro
is too old. This way we only have to specify a plugin once, and this
also opens up the possibility to build on Fedora.

Merge pull request #1737 from rubenk/pthread-cleanup

Pthread cleanup

collectd.spec: sort list

collectd.spec: EPEL7 has a recent ganglia-devel

collectd.spec: sort plugin lists

collectd.spec: turn TODO into changelog entry