From 278bb4b2c9a7f8ab4ed2b33d81f261a0247d56d2 Mon Sep 17 00:00:00 2001
From: whitequark <pz@m-labs.hk>
Date: Mon, 18 Dec 2017 12:47:42 +0000
Subject: [PATCH] Trace the TCP socket acknowledging incoming segment.

---
 src/socket/tcp.rs | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/socket/tcp.rs b/src/socket/tcp.rs
index 7b497e0..9f21297 100644
--- a/src/socket/tcp.rs
+++ b/src/socket/tcp.rs
@@ -1166,6 +1166,8 @@ impl<'a> TcpSocket<'a> {
             // Note that we change the transmitter state here.
             // This is fine because smoltcp assumes that it can always transmit zero or one
             // packets for every packet it receives.
+            net_trace!("{}:{}:{}: acknowledging incoming segment",
+                       self.meta.handle, self.local_endpoint, self.remote_endpoint);
             self.remote_last_ack = Some(self.remote_seq_no + self.rx_buffer.len());
             Ok(Some(self.ack_reply(ip_repr, &repr)))
         } else {
@@ -1246,19 +1248,19 @@ impl<'a> TcpSocket<'a> {
         // Decide whether we're sending a packet.
         if self.seq_to_transmit() {
             // If we have data to transmit and it fits into partner's window, do it.
-            net_trace!("{}:{}:{}: will send",
+            net_trace!("{}:{}:{}: outgoing segment will send data or flags",
                        self.meta.handle, self.local_endpoint, self.remote_endpoint);
         } else if self.ack_to_transmit() {
             // If we have data to acknowledge, do it.
-            net_trace!("{}:{}:{}: will acknowledge",
+            net_trace!("{}:{}:{}: outgoing segment will acknowledge",
                        self.meta.handle, self.local_endpoint, self.remote_endpoint);
         } else if self.window_to_update() {
             // If we have window length increase to advertise, do it.
-            net_trace!("{}:{}:{}: will update window",
+            net_trace!("{}:{}:{}: outgoing segment will update window",
                        self.meta.handle, self.local_endpoint, self.remote_endpoint);
         } else if self.state == State::Closed {
             // If we need to abort the connection, do it.
-            net_trace!("{}:{}:{}: will abort connection",
+            net_trace!("{}:{}:{}: outgoing segment will abort connection",
                        self.meta.handle, self.local_endpoint, self.remote_endpoint);
         } else if self.timer.should_retransmit(timestamp).is_some() {
             // If we have packets to retransmit, do it.