From be0a3a6eeb3655d5559abe768c4027889d1ae979 Mon Sep 17 00:00:00 2001
From: Marc Fournier <marc.fournier@camptocamp.com>
Date: Fri, 29 Aug 2014 17:07:18 +0200
Subject: [PATCH] improve CollectInternalStats's documentation

---
 src/collectd.conf.in  |  3 ++-
 src/collectd.conf.pod | 24 +++++++++++++++++-------
 2 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/src/collectd.conf.in b/src/collectd.conf.in
index 0875d50e..7c9860f0 100644
--- a/src/collectd.conf.in
+++ b/src/collectd.conf.in
@@ -25,7 +25,8 @@
 #AutoLoadPlugin false
 
 #----------------------------------------------------------------------------#
-# When enabled, some internal statistics are recorded as values              #
+# When enabled, internal statistics are collected, using "collectd" as the   #
+# plugin name.                                                               #
 # Disabled by default.                                                       #
 #----------------------------------------------------------------------------#
 #CollectInternalStats false
diff --git a/src/collectd.conf.pod b/src/collectd.conf.pod
index 2f75c148..cabbb526 100644
--- a/src/collectd.conf.pod
+++ b/src/collectd.conf.pod
@@ -146,6 +146,18 @@ B<E<lt>PluginE<nbsp>...E<gt>> block acts as if it was immediately preceded by a
 B<LoadPlugin> statement. B<LoadPlugin> statements are still required for
 plugins that don't provide any configuration, e.g. the I<Load plugin>.
 
+=item B<CollectInternalStats> B<false>|B<true>
+
+When set to B<true>, various statistics about the I<collectd> daemon will be
+collected, with "collectd" as the I<plugin name>. Defaults to B<false>.
+
+The "write_queue" I<plugin instance> reports the number of elements currently
+queued and the number of elements dropped off the queue by the
+B<WriteQueueLimitLow>/B<WriteQueueLimitHigh> mechanism.
+
+The "cache" I<plugin instance> reports the number of elements in the value list
+cache (the cache you can interact with using L<collectd-unixsock(5)>).
+
 =item B<Include> I<Path> [I<pattern>]
 
 If I<Path> points to a file, includes that file. If I<Path> points to a
@@ -236,11 +248,6 @@ on the I<Interval> information contained in each value list. This is used in
 the I<Threshold> configuration to dispatch notifications about missing values,
 see L<collectd-threshold(5)> for details.
 
-=item B<CollectInternalStats> I<true|false>
-
-Some internal statistics can be recorded to monitor Collectd itself.
-Default value : false.
-
 =item B<ReadThreads> I<Num>
 
 Number of threads to start for reading plugins. The default value is B<5>, but
@@ -281,8 +288,11 @@ If B<WriteQueueLimitHigh> is set to non-zero and B<WriteQueueLimitLow> is
 unset, the latter will default to half of B<WriteQueueLimitHigh>.
 
 If you do not want to randomly drop values when the queue size is between
-I<LowNum> and I<HighNum>, set If B<WriteQueueLimitHigh> and
-B<WriteQueueLimitLow> to same value.
+I<LowNum> and I<HighNum>, set B<WriteQueueLimitHigh> and B<WriteQueueLimitLow>
+to the same value.
+
+Enabling the B<CollectInternalStats> option is of great help to figure out the
+values to set B<WriteQueueLimitHigh> and B<WriteQueueLimitLow> to.
 
 =item B<Hostname> I<Name>
 
-- 
2.11.0