Updated to actually honor the boolean and clarified documentation per request
authorBrock Johnson <brock@brotay.net>
Mon, 28 Nov 2016 02:20:02 +0000 (20:20 -0600)
committerBrock Johnson <brock@brotay.net>
Mon, 28 Nov 2016 02:20:02 +0000 (20:20 -0600)
src/collectd.conf.pod
src/turbostat.c

index 26da4a1..21458e0 100644 (file)
@@ -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
 
index 735ee10..c242b6e 100644 (file)
@@ -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) {