zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update TQ pacer holdback window defaults.

Browse Source 
Default holdback-window for non-prio packets is now 5ms, or the expected
pacing time for 3 packets if lower.
This brings wakeup frequency in line with legacy pacer at medium to low
packet rates.

Bug: webrtc:10809
Change-Id: I4045c40ae6b6d50f1ea049f3a26768023f77ec3c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/257301
Auto-Submit: Erik Språng <sprang@webrtc.org>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#36397}
This commit is contained in:
Erik Språng
2022-03-31 14:47:34 +02:00
committed by WebRTC LUCI CQ
parent 8beccd5c47
commit 128addb1a2
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
call/rtp_transport_controller_send.cc
View File

@ -76,9 +76,8 @@ bool IsRelayed(const rtc::NetworkRoute& route) {
RtpTransportControllerSend::PacerSettings::PacerSettings( RtpTransportControllerSend::PacerSettings::PacerSettings(
const FieldTrialsView& trials) const FieldTrialsView& trials)
: tq_disabled("Disabled"), : tq_disabled("Disabled"),
holdback_window("holdback_window", PacingController::kMinSleepTime), holdback_window("holdback_window", TimeDelta::Millis(5)),
holdback_packets("holdback_packets", holdback_packets("holdback_packets", 3) {
TaskQueuePacedSender::kNoPacketHoldback) {
ParseFieldTrial({&tq_disabled, &holdback_window, &holdback_packets}, ParseFieldTrial({&tq_disabled, &holdback_window, &holdback_packets},
trials.Lookup("WebRTC-TaskQueuePacer")); trials.Lookup("WebRTC-TaskQueuePacer"));
} }