From 767235dda3546d76dfc16205c306dfa0759f45f3 Mon Sep 17 00:00:00 2001 From: Victor Boivie Date: Mon, 6 Sep 2021 12:10:14 +0200 Subject: [PATCH] media: sctp: Convert frequent logs to RTC_DLOG In benchmarks, each log statement represent 2% of the CPU usage. RTC_LOG is not very expensive, but not free either, and it's called for every received and sent packet. Bug: webrtc:12943 Change-Id: Id65baafb5e494091a3a7604687718fdd4f477d86 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/231223 Reviewed-by: Florent Castelli Commit-Queue: Victor Boivie Cr-Commit-Position: refs/heads/main@{#34929} --- media/sctp/dcsctp_transport.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/media/sctp/dcsctp_transport.cc b/media/sctp/dcsctp_transport.cc index 2f251980df..49cb4ef042 100644 --- a/media/sctp/dcsctp_transport.cc +++ b/media/sctp/dcsctp_transport.cc @@ -328,8 +328,8 @@ SendPacketStatus DcSctpTransport::SendPacketWithStatus( if (!transport_ || !transport_->writable()) return SendPacketStatus::kError; - RTC_LOG(LS_VERBOSE) << debug_name_ << "->SendPacket(length=" << data.size() - << ")"; + RTC_DLOG(LS_VERBOSE) << debug_name_ << "->SendPacket(length=" << data.size() + << ")"; auto result = transport_->SendPacket(reinterpret_cast(data.data()), @@ -510,8 +510,8 @@ void DcSctpTransport::OnTransportReadPacket( return; } - RTC_LOG(LS_VERBOSE) << debug_name_ - << "->OnTransportReadPacket(), length=" << length; + RTC_DLOG(LS_VERBOSE) << debug_name_ + << "->OnTransportReadPacket(), length=" << length; if (socket_) { socket_->ReceivePacket(rtc::ArrayView( reinterpret_cast(data), length));