Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Unified Diff: webrtc/base/ratetracker.cc

Issue 1279433006: Add a rate tracker that tracks rate over a given interval split up into buckets that accumulate uni… (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: New tests and readability fixes. Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/base/ratetracker.h ('k') | webrtc/base/ratetracker_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/ratetracker.cc
diff --git a/webrtc/base/ratetracker.cc b/webrtc/base/ratetracker.cc
index e03bfe0f295fd91614d5ba5435ad3208cc73c6ec..7dcdb9116918f5096554442880f04044414489c1 100644
--- a/webrtc/base/ratetracker.cc
+++ b/webrtc/base/ratetracker.cc
@@ -1,5 +1,5 @@
/*
- * Copyright 2004 The WebRTC Project Authors. All rights reserved.
+ * Copyright 2015 The WebRTC Project Authors. All rights reserved.
*
* Use of this source code is governed by a BSD-style license
* that can be found in the LICENSE file in the root of the source
@@ -9,56 +9,139 @@
*/
#include "webrtc/base/ratetracker.h"
+
+#include <stddef.h>
+
+#include <algorithm>
+
+#include "webrtc/base/checks.h"
#include "webrtc/base/timeutils.h"
namespace rtc {
-RateTracker::RateTracker()
- : total_units_(0), units_second_(0),
- last_units_second_time_(~0u),
- last_units_second_calc_(0) {
+RateTracker::RateTracker(
+ uint32 bucket_milliseconds, size_t bucket_count)
+ : bucket_milliseconds_(bucket_milliseconds),
+ bucket_count_(bucket_count),
+ sample_buckets_(new size_t[bucket_count + 1]),
+ total_sample_count_(0u),
+ bucket_start_time_milliseconds_(~0u) {
+ CHECK(bucket_milliseconds > 0u);
+ CHECK(bucket_count > 0u);
}
-size_t RateTracker::total_units() const {
- return total_units_;
+RateTracker::~RateTracker() {
+ delete[] sample_buckets_;
}
-size_t RateTracker::units_second() {
- // Snapshot units / second calculator. Determine how many seconds have
- // elapsed since our last reference point. If over 1 second, establish
- // a new reference point that is an integer number of seconds since the
- // last one, and compute the units over that interval.
+double RateTracker::ComputeRateForInterval(
+ uint32 interval_milliseconds) const {
+ if (bucket_start_time_milliseconds_ == ~0u) {
+ return 0.0;
+ }
uint32 current_time = Time();
- if (last_units_second_time_ == ~0u) {
- last_units_second_time_ = current_time;
- last_units_second_calc_ = total_units_;
+ // Calculate which buckets to sum up given the current time. If the time
+ // has passed to a new bucket then we have to skip some of the oldest buckets.
+ uint32 available_interval_milliseconds = std::min<uint32>(
+ interval_milliseconds,
+ bucket_milliseconds_ * static_cast<uint32>(bucket_count_));
+ // number of old buckets (i.e. after the current bucket in the ring buffer)
+ // that are expired given our current time interval.
+ size_t buckets_to_skip;
+ // Number of milliseconds of the first bucket that are not a portion of the
+ // current interval.
+ uint32 milliseconds_to_skip;
+ if (current_time >
+ initialization_time_milliseconds_ + available_interval_milliseconds) {
+ uint32 time_to_skip = current_time - bucket_start_time_milliseconds_ +
+ static_cast<uint32>(bucket_count_) * bucket_milliseconds_ -
+ available_interval_milliseconds;
+ buckets_to_skip = time_to_skip / bucket_milliseconds_;
+ milliseconds_to_skip = time_to_skip % bucket_milliseconds_;
} else {
- int delta = rtc::TimeDiff(current_time, last_units_second_time_);
- if (delta >= 1000) {
- int fraction_time = delta % 1000;
- int seconds = delta / 1000;
- int fraction_units =
- static_cast<int>(total_units_ - last_units_second_calc_) *
- fraction_time / delta;
- // Compute "units received during the interval" / "seconds in interval"
- units_second_ =
- (total_units_ - last_units_second_calc_ - fraction_units) / seconds;
- last_units_second_time_ = current_time - fraction_time;
- last_units_second_calc_ = total_units_ - fraction_units;
- }
+ buckets_to_skip = bucket_count_ - current_bucket_;
+ milliseconds_to_skip = 0u;
+ available_interval_milliseconds =
+ TimeDiff(current_time, initialization_time_milliseconds_);
+ }
+ // If we're skipping all buckets that means that there have been no samples
+ // within the sampling interval so report 0.
+ if (buckets_to_skip > bucket_count_ ||
+ available_interval_milliseconds == 0u) {
+ return 0.0;
}
+ size_t start_bucket = NextBucketIndex(current_bucket_ + buckets_to_skip);
+ // Only count a portion of the first bucket according to how much of the
+ // first bucket is within the current interval.
+ size_t total_samples = sample_buckets_[start_bucket] *
+ (bucket_milliseconds_ - milliseconds_to_skip) /
+ bucket_milliseconds_;
+ // All other buckets in the interval are counted in their entirety.
+ for (size_t i = NextBucketIndex(start_bucket);
+ i != NextBucketIndex(current_bucket_);
+ i = NextBucketIndex(i)) {
+ total_samples += sample_buckets_[i];
+ }
+ // Convert to samples per second.
+ return static_cast<double>(total_samples * 1000u) /
+ static_cast<double>(available_interval_milliseconds);
+}
- return units_second_;
+double RateTracker::ComputeTotalRate() const {
+ if (bucket_start_time_milliseconds_ == ~0u) {
+ return 0.0;
+ }
+ uint32 current_time = Time();
+ if (TimeIsLaterOrEqual(current_time, initialization_time_milliseconds_)) {
+ return 0.0;
+ }
+ return static_cast<double>(total_sample_count_ * 1000u) /
+ static_cast<double>(
+ TimeDiff(current_time, initialization_time_milliseconds_));
}
-void RateTracker::Update(size_t units) {
- if (last_units_second_time_ == ~0u)
- last_units_second_time_ = Time();
- total_units_ += units;
+size_t RateTracker::TotalSampleCount() const {
+ return total_sample_count_;
+}
+
+void RateTracker::AddSamples(size_t sample_count) {
+ EnsureInitialized();
+ uint32 current_time = Time();
+ // Advance the current bucket as needed for the current time, and reset
+ // bucket counts as we advance.
+ for (size_t i = 0u; i <= bucket_count_ &&
+ current_time >= bucket_start_time_milliseconds_ + bucket_milliseconds_;
+ ++i) {
+ bucket_start_time_milliseconds_ += bucket_milliseconds_;
+ current_bucket_ = NextBucketIndex(current_bucket_);
+ sample_buckets_[current_bucket_] = 0u;
+ }
+ // Ensure that bucket_start_time_milliseconds_ is updated appropriately if
+ // the entire buffer of samples has been expired.
+ bucket_start_time_milliseconds_ += bucket_milliseconds_ *
+ ((current_time - bucket_start_time_milliseconds_) / bucket_milliseconds_);
+ // Add all samples in the bucket that includes the current time.
+ sample_buckets_[current_bucket_] += sample_count;
+ total_sample_count_ += sample_count;
}
uint32 RateTracker::Time() const {
return rtc::Time();
}
+void RateTracker::EnsureInitialized() {
+ if (bucket_start_time_milliseconds_ == ~0u) {
+ initialization_time_milliseconds_ = Time();
+ bucket_start_time_milliseconds_ = initialization_time_milliseconds_;
+ current_bucket_ = 0u;
+ // We only need to initialize the first bucket because we reset buckets when
+ // current_bucket_ increments.
+ sample_buckets_[current_bucket_] = 0u;
+ }
+}
+
+size_t RateTracker::NextBucketIndex(size_t bucket_index) const {
+ return (bucket_index + 1u) % (bucket_count_ + 1u);
+}
+
} // namespace rtc
« no previous file with comments | « webrtc/base/ratetracker.h ('k') | webrtc/base/ratetracker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698