Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[common] add PerformanceTimer #893

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion common/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,8 @@ common_libswsscommon_la_SOURCES = \
common/zmqclient.cpp \
common/zmqserver.cpp \
common/asyncdbupdater.cpp \
common/redis_table_waiter.cpp
common/redis_table_waiter.cpp \
common/performancetimer.cpp

common_libswsscommon_la_CXXFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(LIBNL_CFLAGS) $(CODE_COVERAGE_CXXFLAGS)
common_libswsscommon_la_CPPFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(LIBNL_CPPFLAGS) $(CODE_COVERAGE_CPPFLAGS)
Expand Down
133 changes: 133 additions & 0 deletions common/performancetimer.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,133 @@
#include "performancetimer.h"

#include "logger.h"
#include <nlohmann/json.hpp>
#include <fstream>

using namespace swss;

bool PerformanceTimer::m_enable = true;
#define LIMIT 5
#define INDICATOR "/var/log/syslog_notice_flag"

PerformanceTimer::PerformanceTimer(
a114j0y marked this conversation as resolved.
Show resolved Hide resolved
_In_ std::string funcName,
_In_ uint64_t threshold,
_In_ bool verbose):
m_name(funcName),
m_threshold(threshold),
m_verbose(verbose)
{
reset();
m_stop = std::chrono::steady_clock::now();
}

void PerformanceTimer::reset()
{
SWSS_LOG_ENTER();

m_tasks = 0;
m_calls = 0;
m_busy = 0;
m_idle = 0;

m_intervals.clear();
m_gaps.clear();
m_incs.clear();
}

void PerformanceTimer::start()
{
SWSS_LOG_ENTER();

m_start = std::chrono::steady_clock::now();
// measures the gap between this start() and the last stop()
m_gaps.push_back(std::chrono::duration_cast<std::chrono::milliseconds>(m_start-m_stop).count());
}

void PerformanceTimer::stop()
{
SWSS_LOG_ENTER();
m_stop = std::chrono::steady_clock::now();
}

std::string PerformanceTimer::inc(uint64_t count)
{
SWSS_LOG_ENTER();

std::string output = "";

m_calls += 1;

m_tasks += count;

m_idle += m_gaps.back();

uint64_t interval = std::chrono::duration_cast<std::chrono::nanoseconds>(m_stop - m_start).count();

m_busy += interval;

if (count == 0) {
m_gaps.pop_back();
m_calls -= 1;
return output;
}

if (m_incs.size() <= LIMIT) {
m_incs.push_back(count);
m_intervals.push_back(interval/1000000);
} else {
m_gaps.pop_back();
}

if (m_tasks >= m_threshold)
{
uint64_t mseconds = m_busy/1000000;

if (m_enable && mseconds > 0)
{
output = getTimerState();
std::ifstream indicator(INDICATOR);
if (indicator.good()) {
SWSS_LOG_NOTICE("%s", output.c_str());
} else {
SWSS_LOG_INFO("%s", output.c_str());
}
}

reset();
}

return output;
}

std::string PerformanceTimer::getTimerState()
{
nlohmann::json data;
data["API"] = m_name;
data["Tasks"] = m_tasks;
data["busy[ms]"] = m_busy/1000000;
data["idle[ms]"] = m_idle;
data["Total[ms]"] = m_busy/1000000 + m_idle;
double ratio = static_cast<double>(m_tasks) / static_cast<double>(m_busy/1000000 + m_idle);
data["RPS[k]"] = std::round(ratio * 10.0) / 10.0f;
if (m_verbose) {
data["m_intervals"] = m_intervals;
data["m_gaps"] = m_gaps;
data["m_incs"] = m_incs;
}

return data.dump();
}

void PerformanceTimer::setTimerName(const std::string& funcName) {
m_name = funcName;
}

void PerformanceTimer::setTimerThreshold(uint64_t threshold) {
m_threshold = threshold;
}

void PerformanceTimer::setTimerVerbose(bool verbose) {
m_verbose = verbose;
}
63 changes: 63 additions & 0 deletions common/performancetimer.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,63 @@
#pragma once

#include "sal.h"
#include <cstdint>

#include <iostream>
#include <chrono>
#include <string>
#include <deque>
namespace swss
{
class PerformanceTimer
{
public:

PerformanceTimer(
_In_ std::string funcName = "",
_In_ uint64_t threshold = 10000,
_In_ bool verbose = false
);

~PerformanceTimer() = default;

public:

void start();

void stop();

std::string inc(uint64_t count = 1);

void reset();

std::string getTimerState();

static bool m_enable;

void setTimerName(const std::string& funcName);
void setTimerThreshold(uint64_t threshold);
void setTimerVerbose(bool verbose);

private:

std::string m_name; // records what this timer measures about
uint64_t m_threshold; // reset the timer when the m_tasks reachs m_threshold
bool m_verbose; // decides whether to print in verbose when m_threshold is reached

std::chrono::time_point<std::chrono::steady_clock> m_start;
std::chrono::time_point<std::chrono::steady_clock> m_stop;

/* records how long the timer has idled between last stop and this start */
std::deque<uint64_t> m_gaps;
/* records how long each call takes */
std::deque<uint64_t> m_intervals;
/* records how many tasks each call finishes */
std::deque<uint64_t> m_incs;

uint64_t m_tasks; // sum of m_incs
uint64_t m_calls; // how many times the timer is used
uint64_t m_busy; // sum of m_intervals
uint64_t m_idle; // sum of m_gaps
};
}
1 change: 1 addition & 0 deletions tests/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ tests_tests_SOURCES = tests/redis_ut.cpp \
tests/binary_serializer_ut.cpp \
tests/zmq_state_ut.cpp \
tests/profileprovider_ut.cpp \
tests/performancetimer_ut.cpp \
tests/main.cpp

tests_tests_CFLAGS = $(DBGFLAGS) $(AM_CFLAGS) $(CFLAGS_COMMON) $(CFLAGS_GTEST) $(LIBNL_CFLAGS)
Expand Down
43 changes: 43 additions & 0 deletions tests/performancetimer_ut.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
#include "common/performancetimer.h"
#include <nlohmann/json.hpp>
#include "gtest/gtest.h"
#include <thread>

using namespace std;

#define PRINT_ALL 1

TEST(PerformancetimerTest, basic)
{
std::string expected;

static swss::PerformanceTimer timer("basic", PRINT_ALL);
timer.start();
this_thread::sleep_for(chrono::milliseconds(100));
timer.stop();
std::string output = timer.inc(1000);

expected = R"({"API":"basic","RPS[k]":10.0,"Tasks":1000,"Total[ms]":100,"busy[ms]":100,"idle[ms]":0})";
EXPECT_EQ(output, expected);

timer.setTimerName("basic_set_name");
timer.setTimerVerbose(true);
timer.setTimerThreshold(3000);

timer.start();
this_thread::sleep_for(chrono::milliseconds(100));
timer.stop();
output = timer.inc(1000);
EXPECT_EQ(output, "");

this_thread::sleep_for(chrono::milliseconds(200));

timer.start();
this_thread::sleep_for(chrono::milliseconds(300));
timer.stop();
output = timer.inc(2000);

expected = R"({"API":"basic_set_name","RPS[k]":5.0,"Tasks":3000,"Total[ms]":600,"busy[ms]":400,"idle[ms]":200,"m_gaps":[0,200],"m_incs":[1000,2000],"m_intervals":[100,300]})";

EXPECT_EQ(output, expected);
}
Loading