Skip to content

Commit 7912cad

Browse files
Clarify logging during MRP retransmit backoff computation. (#37369)
The log sounded like a retransmit was happening, but it's actually saying that a retransmit is scheduled. The retransmit might or might not happen, depending on whether we get an ack before the scheduled time.
1 parent 0d27f42 commit 7912cad

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/messaging/ReliableMessageMgr.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -513,8 +513,8 @@ void ReliableMessageMgr::CalculateNextRetransTime(RetransTableEntry & entry)
513513

514514
ChipLogProgress(ExchangeManager,
515515
"??%d [E:" ChipLogFormatExchange " S:%u M:" ChipLogFormatMessageCounter
516-
"] (%s) Msg Retransmission to %u:" ChipLogFormatX64 " in %" PRIu32 "ms [State:%s II:%" PRIu32 " AI:%" PRIu32
517-
" AT:%u]",
516+
"] (%s) Msg Retransmission to %u:" ChipLogFormatX64 " scheduled for %" PRIu32
517+
"ms from now [State:%s II:%" PRIu32 " AI:%" PRIu32 " AT:%u]",
518518
entry.sendCount + 1, ChipLogValueExchange(&entry.ec.Get()), sessionHandle->SessionIdForLogging(),
519519
messageCounter, Transport::GetSessionTypeString(sessionHandle), fabricIndex, ChipLogValueX64(destination),
520520
backoff.count(), peerIsActive ? "Active" : "Idle", config.mIdleRetransTimeout.count(),

0 commit comments

Comments
 (0)