Made some Code Optimization changes per review.

fixed build issues

fixed format issues

Refactored getting effective group id from group name part of fork_child

fixed a missing substitution from merge from master

fix issue 2696 to dynamically allocate grname buffer in increments of _SC_GETGR_R_SIZE_MAX/_SC_PAGESIZE

Merge pull request #2924 from kevin-laatz-intel/dpdk_path_fix

dpdk: fix dpdk runtime config file path

dpdk: fix dpdk runtime config file path

The config path was changed in DPDK by the commit 'adf1d86  eal: move
runtime config file to new location’. As a result, the dpdk plugins won't
be able to link to the DPDK primary application after this change.

This commit adds a compile time check for the DPDK verison being used and
sets the DPDK runtime config file location accordingly.

Suggested-by: Michael Luo <michael.luo@intel.com>
Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

Merge pull request #2922 from octo/rootfs_prefix

contrib/docker/rootfs_prefix: Add MIT license header.

contrib/docker/rootfs_prefix: Add MIT license header.

On 2018-09-06 Marc Fournier wrote:

Hello Florian !

Yes, sure, sorry to have omitted this! I'm fine with MIT. Glad this is
useful to others :-)

+1 for adding a default "fallback" licence to COPYING. In case this has
unexpected implications, let's just add the MIT licence to root_prefix/* so
Baruch can move forward with his project.

Cheers,
Marc

Merge branch 'collectd-5.8'

Merge pull request #2919 from rubenk/virt-fix-segfault

Fix segfault in libvirt plugin

virt plugin: fix segfault on error path

The libvirt plugin refreshes its list of domains every RefreshInterval
seconds. If a domain crashes after getting the list, getting the stats
for its block devices returns an error. We then try to call
virDomainGetName on an invalid pointer. Fix this.

fix c_avl_iterator_prev (#2917)

utils_avltree: fix 'c_avl_iterator_prev()' and cover it by tests

Merge pull request #2913 from StarryInternet/sensors-humidity

Add support for humidity in sensors plugin

add humidity for lm_sensors version >= 0x431

Auto-Merge pull request #2916 from rpv-tomsk/collectd-collectd-5.8

Automatically merged due to "Automerge" label

nfs: Fix number of fields for NFSv4

The previous buffer of 64 fields was too short for the current number of
72+2 fields in NFS 4.2, causing the message

  nfs plugin: Unexpected number of fields for NFSv4 server statistics: 62.

This change, originally proposed in [1], fixes that issue in a
forward-compatible manner by reusing the already known number of fields
from the static arrays.

[1]: https://github.com/collectd/collectd/issues/2076#issuecomment-350466628

Merge pull request #2915 from jplitza/fix-2076

nfs: Fix number of fields for NFSv4

nfs: Fix number of fields for NFSv4

The previous buffer of 64 fields was too short for the current number of
72+2 fields in NFS 4.2, causing the message

  nfs plugin: Unexpected number of fields for NFSv4 server statistics: 62.

This change, originally proposed in [1], fixes that issue in a
forward-compatible manner by reusing the already known number of fields
from the static arrays.

[1]: https://github.com/collectd/collectd/issues/2076#issuecomment-350466628

add humidity for lm_sensors version >= 0x402

add support for humidity in sensors plugin

Fix Solaris builds

Trying to fix error:

./build.sh: syntax error at line 78: `TOP_SRCDIR=$' unexpected

Merge pull request #2864 from jimis/libxml2_configure_fix

Add --with-libxml2 option to configure.

Merge pull request #2907 from Stackdriver/pkg-fix

Fix warning that pkgdatadir and pkglibdir were previously defined.

Add --with-libxml2 option to configure.

It's now valid to disable building with libxml2, using --without-libxml2. It's
also possible to set a custom prefix --with-libxml2=/custom/path.

Merge pull request #2896 from mlichvar/chrony

chrony: Ignore late responses

chrony: Ignore late responses

If a response from chronyd was not received before the timeout (e.g. on a
busy system), it would wait in the receive queue and be processed as a
response of the next request. It would fail the sequence check and cause
the plugin to return with an error. The latest response would wait in
the receive queue and this would repeat forever.

Flush the receive queue before sending a new batch of requests to
prevent this situation from happening.

Fix warning that pkgdatadir and pkglibdir were previously defined.

Merge pull request #2904 from rpv-tomsk/redis-connect-unixsock

redis plugin: Added option for connecting via UNIX socket

redis plugin: Added option for connecting via UNIX socket

redis plugin: Fix coredump when stopping Collectd with inaccessible Redis server

Minor codestyle fix

Merge pull request #2898 from romanek-adam/bugfix/memory-leak-in-plugin-dispatch-multivalue

daemon: make plugin_dispatch_multivalue() obey write queue limits

daemon: make plugin_dispatch_multivalue() obey write queue limits (#2897)

dbi plugin: Removed excessive error reporting

Latest libdbi code sets BADIDX error when no more rows is
in 'dbi_result_next_row()' (similar to 'dbi_result_prev_row()'
and to match library documentation).

There is confusing errors in Collectd logs due to this change.
Added a 'dbi_result_has_next_row()' check to solve.

Closes: #2894

Merge pull request #2882 from Stackdriver/gnulib

Add GNULIB_DIR to LDFLAGS in configure.ac on Windows

Merge pull request #2885 from Stackdriver/plugins

Enable additional plugins on Windows.

Enable additional plugins on Windows.

Add GNULIB_DIR to LDFLAGS in configure.ac on Windows

Base port to Windows (#2810)

* Update configure.ac / Makefile.am to build for Windows using Cygwin.
* Update build.sh to build for Windows.
* Base port of the daemon.
* Include gnulib as a submodule.

configure.ac: Split libxml2 and libvirt mix

Merge pull request #2878 from nhuff/freebsd_pending_io

Report number of in progess disk IO requests on FreeBSD.

Fix formating

Report number of in progess disk IO requests on FreeBSD.

Removed parentheses around return arguments

amqp1: clang-format

Merge pull request #2876 from abays/fix_amqp1_typo

Fix typo in amqp1 JSON format error message

Fix typo in amqp1 JSON format error message

processes: Fix conversion of 'delay_rate' values. Now they really have seconds per second unit.

utils_latency: Use plugin name from context to do error reporting

utils_tail: Added plugin name to error reporting

tail_csv: Fix typo in Synopsis

Merge branch 'collectd-5.8' into collectd-master

Merge pull request #2854 from rpv-tomsk/collectd-master

routeros: Use MAC-address when Radio-name is missing

Fix parsing option for avoiding making BaseDir

Parsing is broken since:
386ecab4 Reading configuration file is now done outside main().
So after mentioned commit we've got:

collectd: invalid option -- 'B'

This commit fixes this.

Merge pull request #2856 from manio/master

Fix parsing option for avoiding making BaseDir

Fix parsing option for avoiding making BaseDir

Parsing is broken since:
386ecab4 Reading configuration file is now done outside main().
So after mentioned commit we've got:

collectd: invalid option -- 'B'

This commit fixes this.

routeros: Use MAC-address when Radio-name is missing

routeros: Fix codestyle of latest change

Merge branch 'collectd-5.8' into master

Merge pull request #2851 from rpv-tomsk/external-manio-master

routeros: Add support for temperature and voltage data, fixed typo in 'sectors_written' metric name

routeros: Fixed typo in 'sectors_written' metric name

routeros: Use 'voltage' and 'temperature' as reported types

routeros: Fixed check of 'Collect*' options

routeros: Added conditionals around collect_health code

doc: add information about routeros CollectHealth option

routeros: add support for collecting health data

It includes the voltage and temperature data on supported hardware.

Auto-Merge pull request #2850 from rpv-tomsk/collectd-collectd-5.8

Automatically merged due to "Automerge" label

write_prometheus: Added MHD_USE_INTERNAL_POLLING_THREAD flag

Since 0.9.53, libmicrohttpd requires explicit setting of this flag.

Closes: #2849

turbostat plugin: show error details

Merge pull request #2733 from elfiesmelfie/feat_pcie_aer

New plugin to read PCIe errors

pcie_errors: Remove 'pcie_error' from types.db

Plugin does not send metrics, type is not needed to send notifications.

Merge pull request #2631 from DanCech/tagged-carbon

Graphite 1.1+ tag support

Merge pull request #2847 from rpv-tomsk/collectd-master

tree-wide: Use interval value from plugin context, do not set 'vl->interval' in plugins more.

Merge pull request #2845 from rpv-tomsk/tmpfix

redis plugin: Remove `operations_per_second` metric.

write_kafka: Added Graphite tag support

write_graphite: Documented 'UseTags' option

utils_format_graphite: add tests for graphite tag flag

utils_format_graphite: Add graphite tag support

curl plugin: Added 'Interval' option

curl_xml plugin: Added 'Interval' option

core: Added plugin name to (some) error reporting

tree-wide: Use interval value from plugin context

core: Cleanup in vl->interval assignment

core: Set plugin name in context when plugin is autoloaded

core: Set 'interval' value in plugin context to match read callback interval

postgresql: Set correct value to 'interval' query parameter (bugfix)

utils_tail_match: Updated formatting

utils_tail_match: Removed unused variables

utils_tail_match: Fix formatting

Merge pull request #2840 from elfiesmelfie/fix_deprecated_func

dpdk: fix deprecation warning

Merge branch 'master' into fix_deprecated_func

statsd: Improve error reporting

redis plugin: Remove `operations_per_second` metric.

The `operations_per_second` metric removed as duplicate of other metric and as
it has wrong value also. Collectd should not report wrong values.

Metric `operations_per_second` is based on `instantaneous_ops_per_sec` field
from Redis `INFO` output. That field is calculated based on buffer of 16 values,
updated from `stat_numcommands` once per 100ms.
So, `instantaneous_ops_per_sec` value has average only for last 1.6 seconds.
While Collectd poll period is 10 seconds, statistics for 8.4 seconds is lost.

Herewith, Collectd already has correct metric `total_operations`, based on
`total_commands_processed` field taken from same `stat_numcommands` internally
in Redis. Value of this metric should be used instead of removed metric.

Merge pull request #2844 from elfiesmelfie/fix_snmp_agent

SNMP Agent plugin: Fix coverity scan issues #2814

dpdk: fix clang issue for deprecated function patch

Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>

cleanup: Remove excessive error reporting

These messages already reported in parse_value()

Auto-Merge pull request #2838 from rpv-tomsk/collectd-master

Automatically merged due to "Automerge" label

Merge pull request #2831 from elieyal/patch-3

network plugin: Add BindAddress option