Browse Source

TCP socket debug messages "sending <flags>" should be at DEBUG level.

They're not exceptional.
whitequark 7 years ago
parent
commit
b6f7529fd7
1 changed files with 31 additions and 10 deletions
  1. 31 10
      src/socket/tcp.rs

+ 31 - 10
src/socket/tcp.rs

@@ -1202,7 +1202,11 @@ impl<'a> TcpSocket<'a> {
             return Err(Error::Exhausted)
         }
 
-        if net_log_enabled!(debug) {
+        if repr.payload.len() > 0 {
+            net_trace!("[{}]{}:{}: tx buffer: peeking at {} octets (from {})",
+                       self.debug_id, self.local_endpoint, self.remote_endpoint,
+                       repr.payload.len(), self.remote_last_seq - self.local_seq_no);
+        } else {
             let flags =
                 match (repr.control, repr.ack_number) {
                     (TcpControl::Syn,  None)    => "SYN",
@@ -1213,15 +1217,9 @@ impl<'a> TcpSocket<'a> {
                     (TcpControl::None, Some(_)) => "ACK",
                     _ => unreachable!()
                 };
-            if repr.payload.len() > 0 {
-                net_trace!("[{}]{}:{}: tx buffer: peeking at {} octets (from {})",
-                           self.debug_id, self.local_endpoint, self.remote_endpoint,
-                           repr.payload.len(), self.remote_last_seq - self.local_seq_no);
-            } else {
-                net_debug!("[{}]{}:{}: sending {}",
-                           self.debug_id, self.local_endpoint, self.remote_endpoint,
-                           flags);
-            }
+            net_trace!("[{}]{}:{}: sending {}",
+                       self.debug_id, self.local_endpoint, self.remote_endpoint,
+                       flags);
         }
 
         // Remember the header length before enabling the MSS option, since that option
@@ -3006,4 +3004,27 @@ mod test {
             ..RECV_TEMPL
         }));
     }
+
+    #[test]
+    fn test_fill_peer_window() {
+        let mut s = socket_established();
+        s.remote_mss = 6;
+        s.send_slice(b"abcdef123456!@#$%^").unwrap();
+        recv!(s, [TcpRepr {
+            seq_number: LOCAL_SEQ + 1,
+            ack_number: Some(REMOTE_SEQ + 1),
+            payload:    &b"abcdef"[..],
+            ..RECV_TEMPL
+        }, TcpRepr {
+            seq_number: LOCAL_SEQ + 1 + 6,
+            ack_number: Some(REMOTE_SEQ + 1),
+            payload:    &b"123456"[..],
+            ..RECV_TEMPL
+        }, TcpRepr {
+            seq_number: LOCAL_SEQ + 1 + 6 + 6,
+            ack_number: Some(REMOTE_SEQ + 1),
+            payload:    &b"!@#$%^"[..],
+            ..RECV_TEMPL
+        }]);
+    }
 }