From: Brock Johnson <brock@brotay.net>
Date: Mon, 28 Nov 2016 02:20:02 +0000 (-0600)
Subject: Updated to actually honor the boolean and clarified documentation per request
X-Git-Tag: collectd-5.7.0~12^2~1
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=247e0a2c3e1b323684b5deec0311393dbdc2d00e;p=collectd.git

Updated to actually honor the boolean and clarified documentation per request
---

diff --git a/src/collectd.conf.pod b/src/collectd.conf.pod
index 26da4a1f..21458e03 100644
--- a/src/collectd.conf.pod
+++ b/src/collectd.conf.pod
@@ -7515,9 +7515,10 @@ by this plugin are:
 =item B<LogicalCoreNames> I<true>|I<false>
 
 Boolean enabling the use of logical core numbering for per core 
-statistics. This mirrors the in-kernel per core numbering. On
-multi-socket systems, the original per-cpu numbering behavior
-stomps on values on subsequent sockets.
+statistics. When enabled, cpu<n> is used as plugin instance, where n
+is a sequential number assigned by the kernel. Otherwise, core<n> is
+used where n is the n-th core of the socket, causing name conflicts
+when there is more than one socket.
 
 =over 4
 
diff --git a/src/turbostat.c b/src/turbostat.c
index 735ee100..c242b6ee 100644
--- a/src/turbostat.c
+++ b/src/turbostat.c
@@ -1579,7 +1579,7 @@ turbostat_config(const char *key, const char *value)
 		config_ptm = IS_TRUE(value);
 		apply_config_ptm = 1;
 	} else if (strcasecmp("LogicalCoreNames", key) == 0) {
-		config_lcn = 1;
+		config_lcn = IS_TRUE(value);
 	} else if (strcasecmp("RunningAveragePowerLimit", key) == 0) {
 		tmp_val = strtoul(value, &end, 0);
 		if (*end != '\0' || tmp_val > UINT_MAX) {