]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Input.c
1)Global variables have been moved backward ahead of functions.
[mirror_edk2.git] / MdeModulePkg / Universal / Network / Tcp4Dxe / Tcp4Input.c
index b636016207679c9c21cb5e8d63841d12e9b98fbd..2e436249f23b14b71a01977feb624f8da43892ae 100644 (file)
@@ -72,7 +72,7 @@ TcpFastRecover (
     //\r
     FlightSize = TCP_SUB_SEQ (Tcb->SndNxt, Tcb->SndUna);\r
 \r
-    Tcb->Ssthresh     = NET_MAX (FlightSize >> 1, (UINT32) (2 * Tcb->SndMss));\r
+    Tcb->Ssthresh     = MAX (FlightSize >> 1, (UINT32) (2 * Tcb->SndMss));\r
     Tcb->Recover      = Tcb->SndNxt;\r
 \r
     Tcb->CongestState = TCP_CONGEST_RECOVER;\r
@@ -84,8 +84,8 @@ TcpFastRecover (
     TcpRetransmit (Tcb, Tcb->SndUna);\r
     Tcb->CWnd = Tcb->Ssthresh + 3 * Tcb->SndMss;\r
 \r
-    TCP4_DEBUG_TRACE (("TcpFastRecover: enter fast retransmission"\r
-      " for TCB %x, recover point is %d\n", Tcb, Tcb->Recover));\r
+    DEBUG ((EFI_D_INFO, "TcpFastRecover: enter fast retransmission"\r
+      " for TCB %p, recover point is %d\n", Tcb, Tcb->Recover));\r
     return;\r
   }\r
 \r
@@ -103,8 +103,8 @@ TcpFastRecover (
     // by TcpToSendData\r
     //\r
     Tcb->CWnd += Tcb->SndMss;\r
-    TCP4_DEBUG_TRACE (("TcpFastRecover: received another"\r
-      " duplicated ACK (%d) for TCB %x\n", Seg->Ack, Tcb));\r
+    DEBUG ((EFI_D_INFO, "TcpFastRecover: received another"\r
+      " duplicated ACK (%d) for TCB %p\n", Seg->Ack, Tcb));\r
 \r
   } else {\r
 \r
@@ -120,11 +120,11 @@ TcpFastRecover (
       //\r
       FlightSize = TCP_SUB_SEQ (Tcb->SndNxt, Tcb->SndUna);\r
 \r
-      Tcb->CWnd         = NET_MIN (Tcb->Ssthresh, FlightSize + Tcb->SndMss);\r
+      Tcb->CWnd         = MIN (Tcb->Ssthresh, FlightSize + Tcb->SndMss);\r
 \r
       Tcb->CongestState = TCP_CONGEST_OPEN;\r
-      TCP4_DEBUG_TRACE (("TcpFastRecover: received a full ACK(%d)"\r
-        " for TCB %x, exit fast recovery\n", Seg->Ack, Tcb));\r
+      DEBUG ((EFI_D_INFO, "TcpFastRecover: received a full ACK(%d)"\r
+        " for TCB %p, exit fast recovery\n", Seg->Ack, Tcb));\r
 \r
     } else {\r
 \r
@@ -148,8 +148,8 @@ TcpFastRecover (
 \r
       Tcb->CWnd -= Acked;\r
 \r
-      TCP4_DEBUG_TRACE (("TcpFastRecover: received a partial"\r
-        " ACK(%d) for TCB %x\n", Seg->Ack, Tcb));\r
+      DEBUG ((EFI_D_INFO, "TcpFastRecover: received a partial"\r
+        " ACK(%d) for TCB %p\n", Seg->Ack, Tcb));\r
 \r
     }\r
   }\r
@@ -185,8 +185,8 @@ TcpFastLossRecover (
       Tcb->LossTimes    = 0;\r
       Tcb->CongestState = TCP_CONGEST_OPEN;\r
 \r
-      TCP4_DEBUG_TRACE (("TcpFastLossRecover: received a "\r
-        "full ACK(%d) for TCB %x\n", Seg->Ack, Tcb));\r
+      DEBUG ((EFI_D_INFO, "TcpFastLossRecover: received a "\r
+        "full ACK(%d) for TCB %p\n", Seg->Ack, Tcb));\r
 \r
     } else {\r
 \r
@@ -195,8 +195,8 @@ TcpFastLossRecover (
       // fast retransmit the first unacknowledge field.\r
       //\r
       TcpRetransmit (Tcb, Seg->Ack);\r
-      TCP4_DEBUG_TRACE (("TcpFastLossRecover: received a "\r
-        "partial ACK(%d) for TCB %x\n", Seg->Ack, Tcb));\r
+      DEBUG ((EFI_D_INFO, "TcpFastLossRecover: received a "\r
+        "partial ACK(%d) for TCB %p\n", Seg->Ack, Tcb));\r
     }\r
   }\r
 }\r
@@ -241,7 +241,7 @@ TcpComputeRtt (
     Tcb->RttVar = Measure << (TCP_RTT_SHIFT - 1);\r
   }\r
 \r
-  Tcb->Rto = (Tcb->SRtt + NET_MAX (8, 4 * Tcb->RttVar)) >> TCP_RTT_SHIFT;\r
+  Tcb->Rto = (Tcb->SRtt + MAX (8, 4 * Tcb->RttVar)) >> TCP_RTT_SHIFT;\r
 \r
   //\r
   // Step 2.4: Limit the RTO to at least 1 second\r
@@ -256,7 +256,7 @@ TcpComputeRtt (
 \r
   }\r
 \r
-  TCP4_DEBUG_TRACE (("TcpComputeRtt: new RTT for TCB %x"\r
+  DEBUG ((EFI_D_INFO, "TcpComputeRtt: new RTT for TCB %p"\r
     " computed SRTT: %d RTTVAR: %d RTO: %d\n",\r
     Tcb, Tcb->SRtt, Tcb->RttVar, Tcb->Rto));\r
 \r
@@ -274,7 +274,6 @@ TcpComputeRtt (
   @return 0, the data is successfully trimmed.\r
 \r
 **/\r
-STATIC\r
 INTN\r
 TcpTrimSegment (\r
   IN NET_BUF   *Nbuf,\r
@@ -329,7 +328,7 @@ TcpTrimSegment (
       }\r
     }\r
 \r
-    if (Drop) {\r
+    if (Drop != 0) {\r
       NetbufTrim (Nbuf, Drop, NET_BUF_HEAD);\r
     }\r
   }\r
@@ -347,7 +346,7 @@ TcpTrimSegment (
       Drop--;\r
     }\r
 \r
-    if (Drop) {\r
+    if (Drop != 0) {\r
       NetbufTrim (Nbuf, Drop, NET_BUF_TAIL);\r
     }\r
   }\r
@@ -392,7 +391,7 @@ TcpDeliverData (
   IN TCP_CB *Tcb\r
   )\r
 {\r
-  NET_LIST_ENTRY  *Entry;\r
+  LIST_ENTRY      *Entry;\r
   NET_BUF         *Nbuf;\r
   TCP_SEQNO       Seq;\r
   TCP_SEG         *Seg;\r
@@ -404,7 +403,7 @@ TcpDeliverData (
   // make sure there is some data queued,\r
   // and TCP is in a proper state\r
   //\r
-  if (NetListIsEmpty (&Tcb->RcvQue) || !TCP_CONNECTED (Tcb->State)) {\r
+  if (IsListEmpty (&Tcb->RcvQue) || !TCP_CONNECTED (Tcb->State)) {\r
 \r
     return 0;\r
   }\r
@@ -430,7 +429,7 @@ TcpDeliverData (
     Seq         = Seg->End;\r
     Tcb->RcvNxt = Seq;\r
 \r
-    NetListRemoveEntry (&Nbuf->List);\r
+    RemoveEntryList (&Nbuf->List);\r
 \r
     //\r
     // RFC793 Eighth step: process FIN in sequence\r
@@ -441,16 +440,16 @@ TcpDeliverData (
       // The peer sends to us junky data after FIN,\r
       // reset the connection.\r
       //\r
-      if (!NetListIsEmpty (&Tcb->RcvQue)) {\r
-        TCP4_DEBUG_ERROR (("TcpDeliverData: data received after"\r
-          " FIN from peer of TCB %x, reset connection\n", Tcb));\r
+      if (!IsListEmpty (&Tcb->RcvQue)) {\r
+        DEBUG ((EFI_D_ERROR, "TcpDeliverData: data received after"\r
+          " FIN from peer of TCB %p, reset connection\n", Tcb));\r
 \r
         NetbufFree (Nbuf);\r
         return -1;\r
       }\r
 \r
-      TCP4_DEBUG_TRACE (("TcpDeliverData: processing FIN "\r
-        "from peer of TCB %x\n", Tcb));\r
+      DEBUG ((EFI_D_INFO, "TcpDeliverData: processing FIN "\r
+        "from peer of TCB %p\n", Tcb));\r
 \r
       switch (Tcb->State) {\r
       case TCP_SYN_RCVD:\r
@@ -480,8 +479,8 @@ TcpDeliverData (
           TcpSetTimer (Tcb, TCP_TIMER_2MSL, Tcb->TimeWaitTimeout);\r
         } else {\r
 \r
-          TCP4_DEBUG_WARN (("Connection closed immediately "\r
-            "because app disables TIME_WAIT timer for %x\n", Tcb));\r
+          DEBUG ((EFI_D_WARN, "Connection closed immediately "\r
+            "because app disables TIME_WAIT timer for %p\n", Tcb));\r
 \r
           TcpSendAck (Tcb);\r
           TcpClose (Tcb);\r
@@ -558,9 +557,9 @@ TcpQueueData (
   )\r
 {\r
   TCP_SEG         *Seg;\r
-  NET_LIST_ENTRY  *Head;\r
-  NET_LIST_ENTRY  *Prev;\r
-  NET_LIST_ENTRY  *Cur;\r
+  LIST_ENTRY      *Head;\r
+  LIST_ENTRY      *Prev;\r
+  LIST_ENTRY      *Cur;\r
   NET_BUF         *Node;\r
 \r
   ASSERT (Tcb && Nbuf && (Nbuf->Tcp == NULL));\r
@@ -574,9 +573,9 @@ TcpQueueData (
   // Fast path to process normal case. That is,\r
   // no out-of-order segments are received.\r
   //\r
-  if (NetListIsEmpty (Head)) {\r
+  if (IsListEmpty (Head)) {\r
 \r
-    NetListInsertTail (Head, &Nbuf->List);\r
+    InsertTailList (Head, &Nbuf->List);\r
     return ;\r
   }\r
 \r
@@ -613,7 +612,7 @@ TcpQueueData (
     }\r
   }\r
 \r
-  NetListInsertHead (Prev, &Nbuf->List);\r
+  InsertHeadList (Prev, &Nbuf->List);\r
 \r
   TCP_SET_FLG (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW);\r
 \r
@@ -627,7 +626,7 @@ TcpQueueData (
 \r
       Cur = Cur->ForwardLink;\r
 \r
-      NetListRemoveEntry (&Node->List);\r
+      RemoveEntryList (&Node->List);\r
       NetbufFree (Node);\r
       continue;\r
     }\r
@@ -636,7 +635,7 @@ TcpQueueData (
 \r
       if (TCP_SEQ_LEQ (TCPSEG_NETBUF (Node)->Seq, Seg->Seq)) {\r
 \r
-        NetListRemoveEntry (&Nbuf->List);\r
+        RemoveEntryList (&Nbuf->List);\r
         NetbufFree (Nbuf);\r
         return ;\r
       }\r
@@ -665,8 +664,8 @@ TcpAdjustSndQue (
   IN TCP_SEQNO Ack\r
   )\r
 {\r
-  NET_LIST_ENTRY  *Head;\r
-  NET_LIST_ENTRY  *Cur;\r
+  LIST_ENTRY      *Head;\r
+  LIST_ENTRY      *Cur;\r
   NET_BUF         *Node;\r
   TCP_SEG         *Seg;\r
 \r
@@ -687,7 +686,7 @@ TcpAdjustSndQue (
     if (TCP_SEQ_LEQ (Seg->End, Ack)) {\r
       Cur = Cur->ForwardLink;\r
 \r
-      NetListRemoveEntry (&Node->List);\r
+      RemoveEntryList (&Node->List);\r
       NetbufFree (Node);\r
       continue;\r
     }\r
@@ -736,9 +735,9 @@ TcpInput (
   Len     = Nbuf->TotalSize - (Head->HeadLen << 2);\r
 \r
   if ((Head->HeadLen < 5) || (Len < 0) ||\r
-      TcpChecksum (Nbuf, NetPseudoHeadChecksum (Src, Dst, 6, 0))) {\r
+      (TcpChecksum (Nbuf, NetPseudoHeadChecksum (Src, Dst, 6, 0)) != 0)) {\r
 \r
-    TCP4_DEBUG_TRACE (("TcpInput: received an mal-formated packet\n"));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: received an mal-formated packet\n"));\r
     goto DISCARD;\r
   }\r
 \r
@@ -759,7 +758,7 @@ TcpInput (
           );\r
 \r
   if ((Tcb == NULL) || (Tcb->State == TCP_CLOSED)) {\r
-    TCP4_DEBUG_TRACE (("TcpInput: send reset because no TCB find\n"));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: send reset because no TCB find\n"));\r
 \r
     Tcb = NULL;\r
     goto SEND_RESET;\r
@@ -772,8 +771,8 @@ TcpInput (
   // (in fact, an illegal option length) is reset.\r
   //\r
   if (TcpParseOption (Nbuf->Tcp, &Option) == -1) {\r
-    TCP4_DEBUG_ERROR (("TcpInput: reset the peer because"\r
-      " of mal-format option for Tcb %x\n", Tcb));\r
+    DEBUG ((EFI_D_ERROR, "TcpInput: reset the peer because"\r
+      " of mal-format option for Tcb %p\n", Tcb));\r
 \r
     goto SEND_RESET;\r
   }\r
@@ -796,8 +795,8 @@ TcpInput (
     // First step: Check RST\r
     //\r
     if (TCP_FLG_ON (Seg->Flag, TCP_FLG_RST)) {\r
-      TCP4_DEBUG_WARN (("TcpInput: discard a reset segment "\r
-        "for TCB %x in listening\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: discard a reset segment "\r
+        "for TCB %p in listening\n", Tcb));\r
 \r
       goto DISCARD;\r
     }\r
@@ -807,8 +806,8 @@ TcpInput (
     // Any ACK sent to TCP in LISTEN is reseted.\r
     //\r
     if (TCP_FLG_ON (Seg->Flag, TCP_FLG_ACK)) {\r
-      TCP4_DEBUG_WARN (("TcpInput: send reset because of"\r
-        " segment with ACK for TCB %x in listening\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: send reset because of"\r
+        " segment with ACK for TCB %p in listening\n", Tcb));\r
 \r
       goto SEND_RESET;\r
     }\r
@@ -824,13 +823,13 @@ TcpInput (
 \r
       Tcb     = TcpCloneTcb (Parent);\r
       if (Tcb == NULL) {\r
-        TCP4_DEBUG_ERROR (("TcpInput: discard a segment because"\r
+        DEBUG ((EFI_D_ERROR, "TcpInput: discard a segment because"\r
           "failed to clone a child for TCB%x\n", Tcb));\r
 \r
         goto DISCARD;\r
       }\r
 \r
-      TCP4_DEBUG_TRACE (("TcpInput: create a child for TCB %x"\r
+      DEBUG ((EFI_D_INFO, "TcpInput: create a child for TCB %p"\r
         " in listening\n", Tcb));\r
 \r
       //\r
@@ -859,8 +858,8 @@ TcpInput (
     //\r
     if (TCP_FLG_ON (Seg->Flag, TCP_FLG_ACK) && (Seg->Ack != Tcb->Iss + 1)) {\r
 \r
-      TCP4_DEBUG_WARN (("TcpInput: send reset because of "\r
-        "wrong ACK received for TCB %x in SYN_SENT\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: send reset because of "\r
+        "wrong ACK received for TCB %p in SYN_SENT\n", Tcb));\r
 \r
       goto SEND_RESET;\r
     }\r
@@ -872,14 +871,14 @@ TcpInput (
 \r
       if (TCP_FLG_ON (Seg->Flag, TCP_FLG_ACK)) {\r
 \r
-        TCP4_DEBUG_WARN (("TcpInput: connection reset by"\r
+        DEBUG ((EFI_D_WARN, "TcpInput: connection reset by"\r
           " peer for TCB%x in SYN_SENT\n", Tcb));\r
 \r
         SOCK_ERROR (Tcb->Sk, EFI_CONNECTION_RESET);\r
         goto DROP_CONNECTION;\r
       } else {\r
 \r
-        TCP4_DEBUG_WARN (("TcpInput: discard a reset segment "\r
+        DEBUG ((EFI_D_WARN, "TcpInput: discard a reset segment "\r
           "because of no ACK for TCB%x in SYN_SENT\n", Tcb));\r
 \r
         goto DISCARD;\r
@@ -922,8 +921,8 @@ TcpInput (
 \r
         TCP_SET_FLG (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW);\r
 \r
-        TCP4_DEBUG_TRACE (("TcpInput: connection established"\r
-          " for TCB %x in SYN_SENT\n", Tcb));\r
+        DEBUG ((EFI_D_INFO, "TcpInput: connection established"\r
+          " for TCB %p in SYN_SENT\n", Tcb));\r
 \r
         goto StepSix;\r
       } else {\r
@@ -937,8 +936,8 @@ TcpInput (
 \r
         TcpTrimInWnd (Tcb, Nbuf);\r
 \r
-        TCP4_DEBUG_WARN (("TcpInput: simultanous open "\r
-          "for TCB %x in SYN_SENT\n", Tcb));\r
+        DEBUG ((EFI_D_WARN, "TcpInput: simultanous open "\r
+          "for TCB %p in SYN_SENT\n", Tcb));\r
 \r
         goto StepSix;\r
       }\r
@@ -954,9 +953,9 @@ TcpInput (
   //\r
   // First step: Check whether SEG.SEQ is acceptable\r
   //\r
-  if (!TcpSeqAcceptable (Tcb, Seg)) {\r
-    TCP4_DEBUG_WARN (("TcpInput: sequence acceptance"\r
-      " test failed for segment of TCB %x\n", Tcb));\r
+  if (TcpSeqAcceptable (Tcb, Seg) == 0) {\r
+    DEBUG ((EFI_D_WARN, "TcpInput: sequence acceptance"\r
+      " test failed for segment of TCB %p\n", Tcb));\r
 \r
     if (!TCP_FLG_ON (Seg->Flag, TCP_FLG_RST)) {\r
       TcpSendAck (Tcb);\r
@@ -977,7 +976,7 @@ TcpInput (
   //\r
   if (TCP_FLG_ON (Seg->Flag, TCP_FLG_RST)) {\r
 \r
-    TCP4_DEBUG_WARN (("TcpInput: connection reset for TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_WARN, "TcpInput: connection reset for TCB %p\n", Tcb));\r
 \r
     if (Tcb->State == TCP_SYN_RCVD) {\r
 \r
@@ -1020,8 +1019,8 @@ TcpInput (
   //\r
   if (TCP_FLG_ON (Seg->Flag, TCP_FLG_SYN)) {\r
 \r
-    TCP4_DEBUG_WARN (("TcpInput: connection reset "\r
-      "because received extra SYN for TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_WARN, "TcpInput: connection reset "\r
+      "because received extra SYN for TCB %p\n", Tcb));\r
 \r
     SOCK_ERROR (Tcb->Sk, EFI_CONNECTION_RESET);\r
     goto RESET_THEN_DROP;\r
@@ -1031,8 +1030,8 @@ TcpInput (
   // Fifth step: Check the ACK\r
   //\r
   if (!TCP_FLG_ON (Seg->Flag, TCP_FLG_ACK)) {\r
-    TCP4_DEBUG_WARN (("TcpInput: segment discard because"\r
-      " of no ACK for connected TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_WARN, "TcpInput: segment discard because"\r
+      " of no ACK for connected TCB %p\n", Tcb));\r
 \r
     goto DISCARD;\r
 \r
@@ -1044,7 +1043,7 @@ TcpInput (
         TCP_SEQ_LEQ (Seg->Ack, Tcb->SndNxt)) {\r
 \r
       Tcb->SndWnd     = Seg->Wnd;\r
-      Tcb->SndWndMax  = NET_MAX (Tcb->SndWnd, Tcb->SndWndMax);\r
+      Tcb->SndWndMax  = MAX (Tcb->SndWnd, Tcb->SndWndMax);\r
       Tcb->SndWl1     = Seg->Seq;\r
       Tcb->SndWl2     = Seg->Ack;\r
       TcpSetState (Tcb, TCP_ESTABLISHED);\r
@@ -1052,15 +1051,15 @@ TcpInput (
       TcpClearTimer (Tcb, TCP_TIMER_CONNECT);\r
       TcpDeliverData (Tcb);\r
 \r
-      TCP4_DEBUG_TRACE (("TcpInput: connection established "\r
-        " for TCB %x in SYN_RCVD\n", Tcb));\r
+      DEBUG ((EFI_D_INFO, "TcpInput: connection established "\r
+        " for TCB %p in SYN_RCVD\n", Tcb));\r
 \r
       //\r
       // Continue the process as ESTABLISHED state\r
       //\r
     } else {\r
-      TCP4_DEBUG_WARN (("TcpInput: send reset because of"\r
-        " wrong ACK for TCB %x in SYN_RCVD\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: send reset because of"\r
+        " wrong ACK for TCB %p in SYN_RCVD\n", Tcb));\r
 \r
       goto SEND_RESET;\r
     }\r
@@ -1068,15 +1067,15 @@ TcpInput (
 \r
   if (TCP_SEQ_LT (Seg->Ack, Tcb->SndUna)) {\r
 \r
-    TCP4_DEBUG_WARN (("TcpInput: ignore the out-of-data"\r
-      " ACK for connected TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_WARN, "TcpInput: ignore the out-of-data"\r
+      " ACK for connected TCB %p\n", Tcb));\r
 \r
     goto StepSix;\r
 \r
   } else if (TCP_SEQ_GT (Seg->Ack, Tcb->SndNxt)) {\r
 \r
-    TCP4_DEBUG_WARN (("TcpInput: discard segment for "\r
-      "future ACK for connected TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_WARN, "TcpInput: discard segment for "\r
+      "future ACK for connected TCB %p\n", Tcb));\r
 \r
     TcpSendAck (Tcb);\r
     goto DISCARD;\r
@@ -1143,10 +1142,10 @@ TcpInput (
         Tcb->CWnd += Tcb->SndMss;\r
       } else {\r
 \r
-        Tcb->CWnd += NET_MAX (Tcb->SndMss * Tcb->SndMss / Tcb->CWnd, 1);\r
+        Tcb->CWnd += MAX (Tcb->SndMss * Tcb->SndMss / Tcb->CWnd, 1);\r
       }\r
 \r
-      Tcb->CWnd = NET_MIN (Tcb->CWnd, TCP_MAX_WIN << Tcb->SndWndScale);\r
+      Tcb->CWnd = MIN (Tcb->CWnd, TCP_MAX_WIN << Tcb->SndWndScale);\r
     }\r
 \r
     if (Tcb->CongestState == TCP_CONGEST_LOSS) {\r
@@ -1186,8 +1185,8 @@ TcpInput (
         goto NO_UPDATE;\r
       }\r
 \r
-      TCP4_DEBUG_WARN (("TcpInput: peer shrinks the"\r
-        " window  for connected TCB %x\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: peer shrinks the"\r
+        " window  for connected TCB %p\n", Tcb));\r
 \r
       if ((Tcb->CongestState == TCP_CONGEST_RECOVER) &&\r
           (TCP_SEQ_LT (Right, Tcb->Recover))) {\r
@@ -1214,7 +1213,7 @@ TcpInput (
     }\r
 \r
     Tcb->SndWnd     = Seg->Wnd;\r
-    Tcb->SndWndMax  = NET_MAX (Tcb->SndWnd, Tcb->SndWndMax);\r
+    Tcb->SndWndMax  = MAX (Tcb->SndWnd, Tcb->SndWndMax);\r
     Tcb->SndWl1     = Seg->Seq;\r
     Tcb->SndWl2     = Seg->Ack;\r
   }\r
@@ -1224,8 +1223,8 @@ NO_UPDATE:
   if (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_FIN_SENT) &&\r
       (Tcb->SndUna == Tcb->SndNxt)) {\r
 \r
-    TCP4_DEBUG_TRACE (("TcpInput: local FIN is ACKed by"\r
-      " peer for connected TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: local FIN is ACKed by"\r
+      " peer for connected TCB %p\n", Tcb));\r
 \r
     TCP_SET_FLG (Tcb->CtrlFlag, TCP_CTRL_FIN_ACKED);\r
   }\r
@@ -1264,8 +1263,8 @@ NO_UPDATE:
         TcpSetTimer (Tcb, TCP_TIMER_2MSL, Tcb->TimeWaitTimeout);\r
       } else {\r
 \r
-        TCP4_DEBUG_WARN (("Connection closed immediately "\r
-          "because app disables TIME_WAIT timer for %x\n", Tcb));\r
+        DEBUG ((EFI_D_WARN, "Connection closed immediately "\r
+          "because app disables TIME_WAIT timer for %p\n", Tcb));\r
 \r
         TcpClose (Tcb);\r
       }\r
@@ -1290,8 +1289,8 @@ NO_UPDATE:
       TcpSetTimer (Tcb, TCP_TIMER_2MSL, Tcb->TimeWaitTimeout);\r
     } else {\r
 \r
-      TCP4_DEBUG_WARN (("Connection closed immediately "\r
-        "because app disables TIME_WAIT timer for %x\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "Connection closed immediately "\r
+        "because app disables TIME_WAIT timer for %p\n", Tcb));\r
 \r
       TcpClose (Tcb);\r
     }\r
@@ -1315,8 +1314,8 @@ StepSix:
   if (TCP_FLG_ON (Seg->Flag, TCP_FLG_URG) &&\r
       !TCP_FIN_RCVD (Tcb->State)) {\r
 \r
-    TCP4_DEBUG_TRACE (("TcpInput: received urgent data "\r
-      "from peer for connected TCB %x\n", Tcb));\r
+    DEBUG ((EFI_D_INFO, "TcpInput: received urgent data "\r
+      "from peer for connected TCB %p\n", Tcb));\r
 \r
     Urg = Seg->Seq + Seg->Urg;\r
 \r
@@ -1338,15 +1337,15 @@ StepSix:
 \r
     if (TCP_FIN_RCVD (Tcb->State)) {\r
 \r
-      TCP4_DEBUG_WARN (("TcpInput: connection reset because"\r
-        " data is lost for connected TCB %x\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: connection reset because"\r
+        " data is lost for connected TCB %p\n", Tcb));\r
 \r
       goto RESET_THEN_DROP;\r
     }\r
 \r
     if (TCP_LOCAL_CLOSED (Tcb->State) && (Nbuf->TotalSize != 0)) {\r
-      TCP4_DEBUG_WARN (("TcpInput: connection reset because"\r
-        " data is lost for connected TCB %x\n", Tcb));\r
+      DEBUG ((EFI_D_WARN, "TcpInput: connection reset because"\r
+        " data is lost for connected TCB %p\n", Tcb));\r
 \r
       goto RESET_THEN_DROP;\r
     }\r
@@ -1356,7 +1355,7 @@ StepSix:
       goto RESET_THEN_DROP;\r
     }\r
 \r
-    if (!NetListIsEmpty (&Tcb->RcvQue)) {\r
+    if (!IsListEmpty (&Tcb->RcvQue)) {\r
       TCP_SET_FLG (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW);\r
     }\r
   }\r
@@ -1372,13 +1371,13 @@ StepSix:
   // Tcb is a new child of the listening Parent,\r
   // commit it.\r
   //\r
-  if (Parent) {\r
+  if (Parent != NULL) {\r
     Tcb->Parent = Parent;\r
     TcpInsertTcb (Tcb);\r
   }\r
 \r
   if ((Tcb->State != TCP_CLOSED) &&\r
-      (!TcpToSendData (Tcb, 0)) &&\r
+      (TcpToSendData (Tcb, 0) == 0) &&\r
       (TCP_FLG_ON (Tcb->CtrlFlag, TCP_CTRL_ACK_NOW) || Nbuf->TotalSize)) {\r
 \r
     TcpToSendAck (Tcb);\r
@@ -1407,10 +1406,10 @@ DISCARD:
   //\r
   // Tcb is a child of Parent, and it doesn't survive\r
   //\r
-  TCP4_DEBUG_WARN (("Tcp4Input: Discard a packet\n"));\r
+  DEBUG ((EFI_D_WARN, "Tcp4Input: Discard a packet\n"));\r
   NetbufFree (Nbuf);\r
 \r
-  if (Parent && Tcb) {\r
+  if ((Parent != NULL) && (Tcb != NULL)) {\r
 \r
     ASSERT (Tcb->Sk);\r
     TcpClose (Tcb);\r