Merge pull request #234 from draios/token-bucket-external-time

Allow for an external clock in token bucket.
This commit is contained in:
Mark Stemm 2017-04-25 17:40:50 -07:00 committed by GitHub
commit 76876bc3ae
2 changed files with 13 additions and 9 deletions

View File

@ -31,7 +31,7 @@ token_bucket::~token_bucket()
{ {
} }
void token_bucket::init(uint32_t rate, uint32_t max_tokens) void token_bucket::init(double rate, double max_tokens)
{ {
m_rate = rate; m_rate = rate;
m_max_tokens = max_tokens; m_max_tokens = max_tokens;
@ -39,12 +39,16 @@ void token_bucket::init(uint32_t rate, uint32_t max_tokens)
m_last_seen = sinsp_utils::get_current_time_ns(); m_last_seen = sinsp_utils::get_current_time_ns();
} }
bool token_bucket::claim() bool token_bucket::claim(uint64_t now)
{ {
// Determine the number of tokens gained. Delta between // Determine the number of tokens gained. Delta between
// last_seen and now, divided by the rate. // last_seen and now, divided by the rate.
uint64_t now = sinsp_utils::get_current_time_ns(); if(now == 0)
uint64_t tokens_gained = (now - m_last_seen) / (m_rate * 1000000000); {
now = sinsp_utils::get_current_time_ns();
}
double tokens_gained = (now - m_last_seen) / (m_rate * 1000000000);
m_last_seen = now; m_last_seen = now;
m_tokens += tokens_gained; m_tokens += tokens_gained;

View File

@ -31,30 +31,30 @@ public:
// //
// Initialize the token bucket and start accumulating tokens // Initialize the token bucket and start accumulating tokens
// //
void init(uint32_t rate, uint32_t max_tokens); void init(double rate, double max_tokens);
// //
// Returns true if a token can be claimed. Also updates // Returns true if a token can be claimed. Also updates
// internal metrics. // internal metrics.
// //
bool claim(); bool claim(uint64_t now = 0);
private: private:
// //
// The number of tokens generated per second. // The number of tokens generated per second.
// //
uint64_t m_rate; double m_rate;
// //
// The maximum number of tokens that can be banked for future // The maximum number of tokens that can be banked for future
// claim()s. // claim()s.
// //
uint64_t m_max_tokens; double m_max_tokens;
// //
// The current number of tokens // The current number of tokens
// //
uint64_t m_tokens; double m_tokens;
// //
// The last time claim() was called (or the object was created). // The last time claim() was called (or the object was created).