From caf3367aab54591d2f6badc23418c70bcec97ff4 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Tue, 2 May 2017 21:11:35 -0500 Subject: [PATCH] common/perf_counters: histograms are counters The histogram is a histogram of counters (not guages), so set the COUNTER bit. (This only matters because we expose and document the bits of the type, for better for for worse.) Signed-off-by: Sage Weil --- src/common/perf_counters.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/common/perf_counters.cc b/src/common/perf_counters.cc index 62d9b57a4f859..d07501fb2c913 100644 --- a/src/common/perf_counters.cc +++ b/src/common/perf_counters.cc @@ -317,7 +317,7 @@ void PerfCounters::hinc(int idx, int64_t x, int64_t y) assert(idx < m_upper_bound); perf_counter_data_any_d& data(m_data[idx - m_lower_bound - 1]); - assert(data.type == (PERFCOUNTER_HISTOGRAM | PERFCOUNTER_U64)); + assert(data.type == (PERFCOUNTER_HISTOGRAM | PERFCOUNTER_COUNTER | PERFCOUNTER_U64)); assert(data.histogram); data.histogram->inc(x, y); @@ -407,7 +407,7 @@ void PerfCounters::dump_formatted_generic(Formatter *f, bool schema, } f->close_section(); } else if (d->type & PERFCOUNTER_HISTOGRAM) { - assert(d->type == (PERFCOUNTER_HISTOGRAM | PERFCOUNTER_U64)); + assert(d->type == (PERFCOUNTER_HISTOGRAM | PERFCOUNTER_COUNTER | PERFCOUNTER_U64)); assert(d->histogram); f->open_object_section(d->name); d->histogram->dump_formatted(f); @@ -504,7 +504,7 @@ void PerfCountersBuilder::add_histogram( const char *description, const char *nick, int prio) { add_impl(idx, name, description, nick, prio, - PERFCOUNTER_U64 | PERFCOUNTER_HISTOGRAM, + PERFCOUNTER_U64 | PERFCOUNTER_HISTOGRAM | PERFCOUNTER_COUNTER, unique_ptr>{new PerfHistogram<>{x_axis_config, y_axis_config}}); } -- 2.39.5