Turbostat: init_counter cannot fail anymore
[collectd.git] / src / turbostat.c
index 06f15cd..d8e31c1 100644 (file)
@@ -262,6 +262,7 @@ enum return_values {
        ERR_CPU_ALLOC,
        ERR_NOT_ROOT,
        UNSUPPORTED_CPU,
+       ERR_PATH_TOO_LONG,
 };
 
 
@@ -290,7 +291,7 @@ open_msr(int cpu, _Bool multiple_read)
                CPU_ZERO_S(cpu_affinity_setsize, cpu_affinity_set);
                CPU_SET_S(cpu, cpu_affinity_setsize, cpu_affinity_set);
                if (sched_setaffinity(0, cpu_affinity_setsize, cpu_affinity_set) == -1) {
-                       ERROR("Could not migrate to CPU %d", cpu);
+                       ERROR("Turbostat plugin: Could not migrate to CPU %d", cpu);
                        return -ERR_CPU_MIGRATE;
                }
        }
@@ -313,7 +314,8 @@ read_msr(int fd, off_t offset, unsigned long long *msr)
        retval = pread(fd, msr, sizeof *msr, offset);
 
        if (retval != sizeof *msr) {
-               ERROR("MSR offset 0x%llx read failed", (unsigned long long)offset);
+               ERROR("Turbostat plugin: MSR offset 0x%llx read failed",
+                     (unsigned long long)offset);
                return -1;
        }
        return 0;
@@ -402,20 +404,20 @@ do {                                              \
                goto out;
        }
 
-        if (do_pkg_cstate & (1 << 2))
-                READ_MSR(MSR_PKG_C2_RESIDENCY, &p->pc2);
-        if (do_pkg_cstate & (1 << 3))
-                READ_MSR(MSR_PKG_C3_RESIDENCY, &p->pc3);
-        if (do_pkg_cstate & (1 << 6))
-                READ_MSR(MSR_PKG_C6_RESIDENCY, &p->pc6);
-        if (do_pkg_cstate & (1 << 7))
-                READ_MSR(MSR_PKG_C7_RESIDENCY, &p->pc7);
-        if (do_pkg_cstate & (1 << 8))
-                READ_MSR(MSR_PKG_C8_RESIDENCY, &p->pc8);
-        if (do_pkg_cstate & (1 << 9))
-                READ_MSR(MSR_PKG_C9_RESIDENCY, &p->pc9);
-        if (do_pkg_cstate & (1 << 10))
-                READ_MSR(MSR_PKG_C10_RESIDENCY, &p->pc10);
+       if (do_pkg_cstate & (1 << 2))
+               READ_MSR(MSR_PKG_C2_RESIDENCY, &p->pc2);
+       if (do_pkg_cstate & (1 << 3))
+               READ_MSR(MSR_PKG_C3_RESIDENCY, &p->pc3);
+       if (do_pkg_cstate & (1 << 6))
+               READ_MSR(MSR_PKG_C6_RESIDENCY, &p->pc6);
+       if (do_pkg_cstate & (1 << 7))
+               READ_MSR(MSR_PKG_C7_RESIDENCY, &p->pc7);
+       if (do_pkg_cstate & (1 << 8))
+               READ_MSR(MSR_PKG_C8_RESIDENCY, &p->pc8);
+       if (do_pkg_cstate & (1 << 9))
+               READ_MSR(MSR_PKG_C9_RESIDENCY, &p->pc9);
+       if (do_pkg_cstate & (1 << 10))
+               READ_MSR(MSR_PKG_C10_RESIDENCY, &p->pc10);
 
        if (do_rapl & RAPL_PKG) {
                READ_MSR(MSR_PKG_ENERGY_STATUS, &msr);
@@ -515,9 +517,10 @@ delta_thread(struct thread_data *delta, const struct thread_data *new, const str
 
        /* check for TSC < 1 Mcycles over interval */
        if (delta->tsc < (1000 * 1000)) {
-               WARNING("Insanely slow TSC rate, TSC stops in idle? ");
-               WARNING("You can disable all c-states by booting with \"idle=poll\" ");
-               WARNING("or just the deep ones with \"processor.max_cstate=1\"");
+               WARNING("Turbostat plugin: Insanely slow TSC rate, TSC stops "
+                       "in idle? You can disable all c-states by booting with"
+                       " 'idle=poll' or just the deep ones with"
+                       " 'processor.max_cstate=1'");
                return -1;
        }
 
@@ -528,9 +531,10 @@ delta_thread(struct thread_data *delta, const struct thread_data *new, const str
                delta->mperf = new->mperf - old->mperf;
        } else {
                if (!aperf_mperf_unstable) {
-                       WARNING(" APERF or MPERF went backwards * ");
-                       WARNING("* Frequency results do not cover entire interval *");
-                       WARNING("* fix this by running Linux-2.6.30 or later *");
+                       WARNING("Turbostat plugin: APERF or MPERF went "
+                               "backwards. Frequency results do not cover "
+                               "the entire interval. Fix this by running "
+                               "Linux-2.6.30 or later.");
 
                        aperf_mperf_unstable = 1;
                }
@@ -550,7 +554,7 @@ delta_thread(struct thread_data *delta, const struct thread_data *new, const str
        }
 
        if (delta->mperf == 0) {
-               WARNING("cpu%d MPERF 0!", old->cpu_id);
+               WARNING("Turbostat plugin: cpu%d MPERF 0!", old->cpu_id);
                delta->mperf = 1;       /* divide by 0 protection */
        }
 
@@ -597,7 +601,7 @@ turbostat_submit (const char *plugin_instance,
 static int
 submit_counters(struct thread_data *t, struct core_data *c, struct pkg_data *p)
 {
-       char name[12];
+       char name[DATA_MAX_NAME_LEN];
        double interval_float;
 
        interval_float = tv_delta.tv_sec + tv_delta.tv_usec/1000000.0;
@@ -836,7 +840,8 @@ set_temperature_target(struct thread_data *t, struct core_data *c, struct pkg_da
 
 guess:
        p->tcc_activation_temp = TJMAX_DEFAULT;
-       WARNING("cpu%d: Guessing tjMax %d C, Please use TCCActivationTemp to specify",
+       WARNING("Turbostat plugin: cpu%d: Guessing tjMax %d C,"
+               " Please use TCCActivationTemp to specify it.",
                t->cpu_id, p->tcc_activation_temp);
 
        return 0;
@@ -855,25 +860,25 @@ probe_cpu()
         * - EAX: Maximum Input Value for Basic CPUID Information
         * - EBX: "Genu" (0x756e6547)
         * - EDX: "ineI" (0x49656e69)
-         * - ECX: "ntel" (0x6c65746e)
-         */
+        * - ECX: "ntel" (0x6c65746e)
+        */
        max_level = ebx = ecx = edx = 0;
        __get_cpuid(0, &max_level, &ebx, &ecx, &edx);
        if (ebx != 0x756e6547 && edx != 0x49656e69 && ecx != 0x6c65746e) {
-               ERROR("Unsupported CPU");
+               ERROR("Turbostat plugin: Unsupported CPU (not Intel)");
                return -UNSUPPORTED_CPU;
        }
 
        /* CPUID(1):
         * - EAX: Version Information: Type, Family, Model, and Stepping ID
-         *  + 4-7:   Model ID
-         *  + 8-11:  Family ID
+        *  + 4-7:   Model ID
+        *  + 8-11:  Family ID
         *  + 12-13: Processor type
         *  + 16-19: Extended Model ID
         *  + 20-27: Extended Family ID
         * - EDX: Feature Information:
         *  + 5: Support for MSR read/write operations
-         */
+        */
        fms = ebx = ecx = edx = 0;
        __get_cpuid(1, &fms, &ebx, &ecx, &edx);
        family = (fms >> 8) & 0xf;
@@ -883,7 +888,7 @@ probe_cpu()
        if (family == 6 || family == 0xf)
                model += ((fms >> 16) & 0xf) << 4;
        if (!(edx & (1 << 5))) {
-               ERROR("CPUID: no MSR");
+               ERROR("Turbostat plugin: Unsupported CPU (no MSR support)");
                return -ERR_NO_MSR;
        }
 
@@ -898,7 +903,8 @@ probe_cpu()
        max_level = ebx = ecx = edx = 0;
        __get_cpuid(0x80000000, &max_level, &ebx, &ecx, &edx);
        if (max_level < 0x80000007) {
-               ERROR("CPUID: no invariant TSC (max_level 0x%x)", max_level);
+               ERROR("Turbostat plugin: Unsupported CPU (no invariant TSC, "
+                     " Maximum Extended Function: 0x%x)", max_level);
                return -ERR_NO_INVARIANT_TSC;
        }
 
@@ -912,7 +918,7 @@ probe_cpu()
        eax = ebx = ecx = edx = 0;
        __get_cpuid(0x80000007, &eax, &ebx, &ecx, &edx);
        if (!(edx & (1 << 8))) {
-               ERROR("No invariant TSC");
+               ERROR("Turbostat plugin: Unsupported CPU (No invariant TSC)");
                return -ERR_NO_INVARIANT_TSC;
        }
 
@@ -933,7 +939,7 @@ probe_cpu()
        do_dts = eax & (1 << 0);
        do_ptm = eax & (1 << 6);
        if (!(ecx & (1 << 0))) {
-               ERROR("No APERF");
+               ERROR("Turbostat plugin: Unsupported CPU (No APERF)");
                return -ERR_NO_APERF;
        }
 
@@ -1005,7 +1011,8 @@ probe_cpu()
                        do_pkg_cstate = (1 << 2) | (1 << 3) | (1 << 6) | (1 << 7) | (1 << 8) | (1 << 9) | (1 << 10);
                        break;
                default:
-                       ERROR("Unsupported CPU");
+                       ERROR("Turbostat plugin: Unsupported CPU (family: %#x,"
+                             " model: %#x)", family, model);
                }
                switch (model) {
                case 0x2A:
@@ -1030,7 +1037,8 @@ probe_cpu()
                        do_rapl = 0;
                }
        } else {
-               ERROR("Unsupported CPU");
+               ERROR("Turbostat plugin: Unsupported CPU (family: %#x, "
+                     "model: %#x)", family, model);
                return -UNSUPPORTED_CPU;
        }
 
@@ -1062,18 +1070,23 @@ parse_int_file(const char *fmt, ...)
        va_list args;
        char path[PATH_MAX];
        FILE *filep;
-       int value;
+       int len, value;
 
        va_start(args, fmt);
-       vsnprintf(path, sizeof(path), fmt, args);
+       len = vsnprintf(path, sizeof(path), fmt, args);
        va_end(args);
+       if (len < 0 || len >= PATH_MAX) {
+               ERROR("Turbostat plugin: path truncated: '%s'", path);
+               return -ERR_PATH_TOO_LONG;
+       }
+
        filep = fopen(path, "r");
        if (!filep) {
-               ERROR("%s: open failed", path);
+               ERROR("Turbostat plugin: Failed to open '%s'", path);
                return -ERR_CANT_OPEN_FILE;
        }
        if (fscanf(filep, "%d", &value) != 1) {
-               ERROR("%s: failed to parse number from file", path);
+               ERROR("Turbostat plugin: Failed to parse number from '%s'", path);
                return -ERR_CANT_READ_NUMBER;
        }
        fclose(filep);
@@ -1091,10 +1104,10 @@ get_threads_on_core(int cpu)
 
        ssnprintf(path, sizeof(path), "/sys/devices/system/cpu/cpu%d/topology/thread_siblings_list", cpu);
        filep = fopen(path, "r");
-        if (!filep) {
-                ERROR("%s: open failed", path);
-                return -ERR_CANT_OPEN_FILE;
-        }
+       if (!filep) {
+               ERROR("Turbostat plugin: Failed to open '%s'", path);
+               return -ERR_CANT_OPEN_FILE;
+       }
        /*
         * file format:
         * if a pair of number with a character between: 2 siblings (eg. 1-2, or 1,4)
@@ -1122,14 +1135,14 @@ for_all_proc_cpus(int (func)(int))
        int retval;
 
        fp = fopen("/proc/stat", "r");
-        if (!fp) {
-                ERROR("Failed to open /proc/stat");
-                return -ERR_CANT_OPEN_FILE;
-        }
+       if (!fp) {
+               ERROR("Turbostat plugin: Failed to open /proc/stat");
+               return -ERR_CANT_OPEN_FILE;
+       }
 
        retval = fscanf(fp, "cpu %*d %*d %*d %*d %*d %*d %*d %*d %*d %*d\n");
        if (retval != 0) {
-               ERROR("Failed to parse /proc/stat");
+               ERROR("Turbostat plugin: Failed to parse /proc/stat");
                fclose(fp);
                return -ERR_CANT_READ_PROC_STAT;
        }
@@ -1171,7 +1184,7 @@ static int __attribute__((warn_unused_result))
 allocate_cpu_set(cpu_set_t ** set, size_t * size) {
        *set = CPU_ALLOC(topology.max_cpu_id  + 1);
        if (*set == NULL) {
-               ERROR("Unable to allocate CPU state");
+               ERROR("Turbostat plugin: Unable to allocate CPU state");
                return -ERR_CPU_ALLOC;
        }
        *size = CPU_ALLOC_SIZE(topology.max_cpu_id  + 1);
@@ -1200,7 +1213,7 @@ topology_probe()
 
        topology.cpus = calloc(1, (topology.max_cpu_id  + 1) * sizeof(struct cpu_topology));
        if (topology.cpus == NULL) {
-               ERROR("Unable to allocate memory for cpu topology");
+               ERROR("Turbostat plugin: Unable to allocate memory for CPU topology");
                return -ERR_CALLOC;
        }
 
@@ -1227,7 +1240,7 @@ topology_probe()
                struct cpu_topology *cpu = &topology.cpus[i];
 
                if (cpu_is_not_present(i)) {
-                       WARNING("cpu%d NOT PRESENT", i);
+                       WARNING("Turbostat plugin: cpu%d NOT PRESENT", i);
                        continue;
                }
 
@@ -1265,7 +1278,7 @@ topology_probe()
                else if (ret == i)
                        cpu->first_thread_in_core = 1;
 
-               DEBUG("cpu %d pkg %d core %d\n",
+               DEBUG("Turbostat plugin: cpu %d pkg %d core %d\n",
                        i, cpu->package_id, cpu->core_id);
        }
        /* Num is max + 1 (need to count 0) */
@@ -1320,11 +1333,11 @@ err_clean_cores:
 err_clean_threads:
        free(*threads);
 err:
-       ERROR("calloc counters");
+       ERROR("Turbostat plugin: Failled to allocate memory for counters");
        return -ERR_CALLOC;
 }
 
-static int
+static void
 init_counter(struct thread_data *thread_base, struct core_data *core_base,
        struct pkg_data *pkg_base, int cpu_id)
 {
@@ -1345,32 +1358,20 @@ init_counter(struct thread_data *thread_base, struct core_data *core_base,
 
        c->core_id = cpu->core_id;
        p->package_id = cpu->package_id;
-
-       return 0;
 }
 
-static int
+static void
 initialize_counters(void)
 {
-       int ret;
        int cpu_id;
 
        for (cpu_id = 0; cpu_id <= topology.max_cpu_id; ++cpu_id) {
-               if (cpu_is_not_present(cpu_id)) {
+               if (cpu_is_not_present(cpu_id))
                        continue;
-               }
-
-               ret = init_counter(EVEN_COUNTERS, cpu_id);
-               if (ret < 0)
-                       return ret;
-               ret = init_counter(ODD_COUNTERS, cpu_id);
-               if (ret < 0)
-                       return ret;
-               ret = init_counter(DELTA_COUNTERS, cpu_id);
-               if (ret < 0)
-                       return ret;
+               init_counter(EVEN_COUNTERS, cpu_id);
+               init_counter(ODD_COUNTERS, cpu_id);
+               init_counter(DELTA_COUNTERS, cpu_id);
        }
-       return 0;
 }
 
 
@@ -1424,10 +1425,10 @@ free_all_buffers(void)
  **********************/
 
 #define DO_OR_GOTO_ERR(something) \
-do {                         \
-       ret = (something);     \
-       if (ret < 0)         \
-               goto err;    \
+do {                             \
+       ret = (something);        \
+       if (ret < 0)              \
+               goto err;         \
 } while (0)
 
 static int setup_all_buffers(void)
@@ -1438,7 +1439,7 @@ static int setup_all_buffers(void)
        DO_OR_GOTO_ERR(allocate_counters(&thread_even, &core_even, &package_even));
        DO_OR_GOTO_ERR(allocate_counters(&thread_odd, &core_odd, &package_odd));
        DO_OR_GOTO_ERR(allocate_counters(&thread_delta, &core_delta, &package_delta));
-       DO_OR_GOTO_ERR(initialize_counters());
+       initialize_counters();
        DO_OR_GOTO_ERR(for_all_cpus(set_temperature_target, EVEN_COUNTERS));
        DO_OR_GOTO_ERR(for_all_cpus(set_temperature_target, ODD_COUNTERS));
 
@@ -1450,7 +1451,7 @@ err:
 }
 
 static int
-turbostat_read(user_data_t * not_used)
+turbostat_read(void)
 {
        int ret;
 
@@ -1519,20 +1520,25 @@ turbostat_init(void)
        int ret;
 
        if (getuid() != 0) {
-               ERROR("must be root");
+               ERROR("Turbostat plugin: Initialization failed: this plugin "
+                     "requires collectd to run as root in order to read "
+                     "special CPU registers");
                return -ERR_NOT_ROOT;
        }
 
        DO_OR_GOTO_ERR(probe_cpu());
 
        if (stat("/dev/cpu/0/msr", &sb)) {
-               ERROR("no /dev/cpu/0/msr, try \"# modprobe msr\"");
+               ERROR("Turbostat plugin: Initialization failed: /dev/cpu/0/msr"
+                     " does not exist while the CPU supports MSR. You may be "
+                     "missing the corresponding kernel module, please try '# "
+                     "modprobe msr'");
                return -ERR_NO_MSR;
        }
 
        DO_OR_GOTO_ERR(setup_all_buffers());
 
-       plugin_register_complex_read(NULL, PLUGIN_NAME, turbostat_read, NULL, NULL);
+       plugin_register_read(PLUGIN_NAME, turbostat_read);
 
        return 0;
 err:
@@ -1563,7 +1569,6 @@ turbostat_config(const char *key, const char *value)
        return 0;
 }
 
-void module_register(void);
 void module_register(void)
 {
        plugin_register_init(PLUGIN_NAME, turbostat_init);