Updated to actually honor the boolean and clarified documentation per request

diff --git a/src/collectd.conf.pod b/src/collectd.conf.pod
index 26da4a1fcfd744fba97593b56ffe5c4ec897ea73..21458e03a102ceba2f62998f4627173de14364d8 100644 (file)
--- 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 735ee100cf98b87b1da42eac5205b884c065fe52..c242b6eeb67db6e30ee2a4a97d306b3fdd8107c5 100644 (file)
--- 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) {