projects
/
collectd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
s/readyness/readiness/
[collectd.git]
/
src
/
intel_pmu.c
diff --git
a/src/intel_pmu.c
b/src/intel_pmu.c
index
351f81b
..
957004d
100644
(file)
--- a/
src/intel_pmu.c
+++ b/
src/intel_pmu.c
@@
-192,7
+192,8
@@
static void pmu_dump_config(void) {
DEBUG(PMU_PLUGIN ": software_events : %d", g_ctx.sw_events);
for (size_t i = 0; i < g_ctx.hw_events_count; i++) {
DEBUG(PMU_PLUGIN ": software_events : %d", g_ctx.sw_events);
for (size_t i = 0; i < g_ctx.hw_events_count; i++) {
- DEBUG(PMU_PLUGIN ": hardware_events[%zu]: %s", i, g_ctx.hw_events[i]);
+ DEBUG(PMU_PLUGIN ": hardware_events[%" PRIsz "]: %s", i,
+ g_ctx.hw_events[i]);
}
}
}
}
@@
-289,18
+290,20
@@
meta_data_t *pmu_meta_data_create(const struct efd *efd) {
meta_data_t *meta = NULL;
/* create meta data only if value was scaled */
meta_data_t *meta = NULL;
/* create meta data only if value was scaled */
- if (efd->val[1] != efd->val[2] && efd->val[2]) {
- meta = meta_data_create();
- if (meta == NULL) {
- ERROR(PMU_PLUGIN ": meta_data_create failed.");
- return NULL;
- }
+ if (efd->val[1] == efd->val[2] || !efd->val[2]) {
+ return NULL;
+ }
- meta_data_add_unsigned_int(meta, "intel_pmu:raw_count", efd->val[0]);
- meta_data_add_unsigned_int(meta, "intel_pmu:time_enabled", efd->val[1]);
- meta_data_add_unsigned_int(meta, "intel_pmu:time_running", efd->val[2]);
+ meta = meta_data_create();
+ if (meta == NULL) {
+ ERROR(PMU_PLUGIN ": meta_data_create failed.");
+ return NULL;
}
}
+ meta_data_add_unsigned_int(meta, "intel_pmu:raw_count", efd->val[0]);
+ meta_data_add_unsigned_int(meta, "intel_pmu:time_enabled", efd->val[1]);
+ meta_data_add_unsigned_int(meta, "intel_pmu:time_running", efd->val[2]);
+
return meta;
}
return meta;
}
@@
-332,10
+335,7
@@
static void pmu_dispatch_data(void) {
/* dispatch per CPU value */
pmu_submit_counter(i, e->event, value, meta);
/* dispatch per CPU value */
pmu_submit_counter(i, e->event, value, meta);
- if (meta) {
- meta_data_destroy(meta);
- meta = NULL;
- }
+ meta_data_destroy(meta);
}
if (event_enabled > 0) {
}
if (event_enabled > 0) {