From 247e0a2c3e1b323684b5deec0311393dbdc2d00e Mon Sep 17 00:00:00 2001 From: Brock Johnson Date: Sun, 27 Nov 2016 20:20:02 -0600 Subject: [PATCH] Updated to actually honor the boolean and clarified documentation per request --- src/collectd.conf.pod | 7 ++++--- src/turbostat.c | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-) 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 I|I 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 is used as plugin instance, where n +is a sequential number assigned by the kernel. Otherwise, core 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) { -- 2.30.2