]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/Udp4Dxe/Udp4Impl.c
1. Mark the network volatile variables as deprecated in code comments and remove...
[mirror_edk2.git] / MdeModulePkg / Universal / Network / Udp4Dxe / Udp4Impl.c
index 18ba7f410390f19f52c0f7874389dc5d28f0c27e..ce952b1e2d19b148d41b88a607e682d79768b07e 100644 (file)
@@ -1,7 +1,8 @@
 /** @file\r
-\r
-Copyright (c) 2006 - 2008, Intel Corporation                                                         \r
-All rights reserved. This program and the accompanying materials\r
+  The implementation of the Udp4 protocol.\r
+  \r
+Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>\r
+This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
 http://opensource.org/licenses/bsd-license.php\r
@@ -9,15 +10,6 @@ http://opensource.org/licenses/bsd-license.php
 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
 WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
-Module Name:\r
-\r
-  Udp4Impl.c\r
-\r
-Abstract:\r
-\r
-  The implementation of the Udp4 protocol.\r
-\r
-\r
 **/\r
 \r
 \r
@@ -29,11 +21,9 @@ UINT16  mUdp4RandomPort;
   This function checks and timeouts the I/O datagrams holding by the corresponding\r
   service context.\r
 \r
-  @param  Event                  The event this function registered to.\r
-  @param  Context                The context data registered during the creation of\r
-                                 the Event.\r
-\r
-  @return None.\r
+  @param[in]  Event                  The event this function registered to.\r
+  @param[in]  Context                The context data registered during the creation of\r
+                                     the Event.\r
 \r
 **/\r
 VOID\r
@@ -46,10 +36,10 @@ Udp4CheckTimeout (
 /**\r
   This function finds the udp instance by the specified <Address, Port> pair.\r
 \r
-  @param  InstanceList           Pointer to the head of the list linking the udp\r
-                                 instances.\r
-  @param  Address                Pointer to the specified IPv4 address.\r
-  @param  Port                   The udp port number.\r
+  @param[in]  InstanceList           Pointer to the head of the list linking the udp\r
+                                     instances.\r
+  @param[in]  Address                Pointer to the specified IPv4 address.\r
+  @param[in]  Port                   The udp port number.\r
 \r
   @retval TRUE     The specified <Address, Port> pair is found.\r
   @retval FALSE    Otherwise.\r
@@ -67,40 +57,38 @@ Udp4FindInstanceByPort (
   interface. It's called to signal the udp TxToken when IpIo layer completes the\r
   transmitting of the udp datagram.\r
 \r
-  @param  Status                 The completion status of the output udp datagram.\r
-  @param  Context                Pointer to the context data.\r
-  @param  Sender                 Pointer to the Ip sender of the udp datagram.\r
-  @param  NotifyData             Pointer to the notify data.\r
-\r
-  @return None.\r
+  @param[in]  Status                 The completion status of the output udp datagram.\r
+  @param[in]  Context                Pointer to the context data.\r
+  @param[in]  Sender                 Specify a pointer of EFI_IP4_PROTOCOL for sending.\r
+  @param[in]  NotifyData             Pointer to the notify data.\r
 \r
 **/\r
 VOID\r
+EFIAPI\r
 Udp4DgramSent (\r
-  IN EFI_STATUS  Status,\r
-  IN VOID        *Context,\r
-  IN VOID        *Sender,\r
-  IN VOID        *NotifyData\r
+  IN EFI_STATUS        Status,\r
+  IN VOID              *Context,\r
+  IN IP_IO_IP_PROTOCOL Sender,\r
+  IN VOID              *NotifyData\r
   );\r
 \r
 /**\r
   This function processes the received datagram passed up by the IpIo layer.\r
 \r
-  @param  Status                 The status of this udp datagram.\r
-  @param  IcmpError              The IcmpError code, only available when Status is\r
+  @param[in]  Status             The status of this udp datagram.\r
+  @param[in]  IcmpError          The IcmpError code, only available when Status is\r
                                  EFI_ICMP_ERROR.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA.\r
-  @param  Packet                 Pointer to the NET_BUF containing the received udp\r
+  @param[in]  NetSession         Pointer to the EFI_NET_SESSION_DATA.\r
+  @param[in]  Packet             Pointer to the NET_BUF containing the received udp\r
                                  datagram.\r
-  @param  Context                Pointer to the context data.\r
-\r
-  @return None.\r
+  @param[in]  Context            Pointer to the context data.\r
 \r
 **/\r
 VOID\r
+EFIAPI\r
 Udp4DgramRcvd (\r
   IN EFI_STATUS            Status,\r
-  IN ICMP_ERROR            IcmpError,\r
+  IN UINT8                 IcmpError,\r
   IN EFI_NET_SESSION_DATA  *NetSession,\r
   IN NET_BUF               *Packet,\r
   IN VOID                  *Context\r
@@ -110,9 +98,9 @@ Udp4DgramRcvd (
   This function cancels the token specified by Arg in the Map. This is a callback\r
   used by Udp4InstanceCancelToken().\r
 \r
-  @param  Map                    Pointer to the NET_MAP.\r
-  @param  Item                   Pointer to the NET_MAP_ITEM.\r
-  @param  Arg                    Pointer to the token to be cancelled, if NULL,\r
+  @param[in]  Map                Pointer to the NET_MAP.\r
+  @param[in]  Item               Pointer to the NET_MAP_ITEM.\r
+  @param[in]  Arg                Pointer to the token to be cancelled, if NULL,\r
                                  the token specified by Item is cancelled.\r
 \r
   @retval EFI_SUCCESS            The token is cancelled if Arg is NULL or the token\r
@@ -123,6 +111,7 @@ Udp4DgramRcvd (
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 Udp4CancelTokens (\r
   IN NET_MAP       *Map,\r
   IN NET_MAP_ITEM  *Item,\r
@@ -132,8 +121,8 @@ Udp4CancelTokens (
 /**\r
   This function matches the received udp datagram with the Instance.\r
 \r
-  @param  Instance               Pointer to the udp instance context data.\r
-  @param  Udp4Session            Pointer to the EFI_UDP4_SESSION_DATA abstracted\r
+  @param[in]  Instance           Pointer to the udp instance context data.\r
+  @param[in]  Udp4Session        Pointer to the EFI_UDP4_SESSION_DATA abstracted\r
                                  from the received udp datagram.\r
 \r
   @retval TRUE       The udp datagram matches the receiving requirments of the\r
@@ -150,10 +139,8 @@ Udp4MatchDgram (
 /**\r
   This function removes the Wrap specified by Context and release relevant resources.\r
 \r
-  @param  Event                  The Event this notify function registered to.\r
-  @param  Context                Pointer to the context data.\r
-\r
-  @return None.\r
+  @param[in]  Event              The Event this notify function registered to.\r
+  @param[in]  Context            Pointer to the context data.\r
 \r
 **/\r
 VOID\r
@@ -166,11 +153,11 @@ Udp4RecycleRxDataWrap (
 /**\r
   This function wraps the Packet and the RxData.\r
 \r
-  @param  Instance               Pointer to the instance context data.\r
-  @param  Packet                 Pointer to the buffer containing the received\r
-                                 datagram.\r
-  @param  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
-                                 datagram.\r
+  @param[in]  Instance               Pointer to the instance context data.\r
+  @param[in]  Packet                 Pointer to the buffer containing the received\r
+                                     datagram.\r
+  @param[in]  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
+                                     datagram.\r
 \r
   @return Pointer to the structure wrapping the RxData and the Packet.\r
 \r
@@ -185,11 +172,11 @@ Udp4WrapRxData (
 /**\r
   This function enqueues the received datagram into the instances' receiving queues.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-  @param  Packet                 Pointer to the buffer containing the received\r
-                                 datagram.\r
-  @param  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
-                                 datagram.\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
+  @param[in]  Packet                 Pointer to the buffer containing the received\r
+                                     datagram.\r
+  @param[in]  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
+                                     datagram.\r
 \r
   @return The times this datagram is enqueued.\r
 \r
@@ -204,9 +191,7 @@ Udp4EnqueueDgram (
 /**\r
   This function delivers the datagrams enqueued in the instances.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-\r
-  @return None.\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
 \r
 **/\r
 VOID\r
@@ -217,13 +202,11 @@ Udp4DeliverDgram (
 /**\r
   This function demultiplexes the received udp datagram to the apropriate instances.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA abstrated from\r
-                                 the received datagram.\r
-  @param  Packet                 Pointer to the buffer containing the received udp\r
-                                 datagram.\r
-\r
-  @return None.\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
+  @param[in]  NetSession             Pointer to the EFI_NET_SESSION_DATA abstrated from\r
+                                     the received datagram.\r
+  @param[in]  Packet                 Pointer to the buffer containing the received udp\r
+                                     datagram.\r
 \r
 **/\r
 VOID\r
@@ -237,19 +220,17 @@ Udp4Demultiplex (
   This function handles the received Icmp Error message and demultiplexes it to the\r
   instance.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-  @param  IcmpError              The icmp error code.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA abstracted\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
+  @param[in]  IcmpError              The icmp error code.\r
+  @param[in]  NetSession             Pointer to the EFI_NET_SESSION_DATA abstracted\r
                                  from the received Icmp Error packet.\r
-  @param  Packet                 Pointer to the Icmp Error packet.\r
-\r
-  @return None.\r
+  @param[in]  Packet                 Pointer to the Icmp Error packet.\r
 \r
 **/\r
 VOID\r
 Udp4IcmpHandler (\r
   IN UDP4_SERVICE_DATA     *Udp4Service,\r
-  IN ICMP_ERROR            IcmpError,\r
+  IN UINT8                 IcmpError,\r
   IN EFI_NET_SESSION_DATA  *NetSession,\r
   IN NET_BUF               *Packet\r
   );\r
@@ -257,13 +238,11 @@ Udp4IcmpHandler (
 /**\r
   This function builds and sends out a icmp port unreachable message.\r
 \r
-  @param  IpIo                   Pointer to the IP_IO instance.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA of the packet\r
-                                 causes this icmp error message.\r
-  @param  Udp4Header             Pointer to the udp header of the datagram causes\r
-                                 this icmp error message.\r
-\r
-  @return None.\r
+  @param[in]  IpIo                   Pointer to the IP_IO instance.\r
+  @param[in]  NetSession             Pointer to the EFI_NET_SESSION_DATA of the packet\r
+                                     causes this icmp error message.\r
+  @param[in]  Udp4Header             Pointer to the udp header of the datagram causes\r
+                                     this icmp error message.\r
 \r
 **/\r
 VOID\r
@@ -277,14 +256,14 @@ Udp4SendPortUnreach (
 /**\r
   Create the Udp service context data.\r
 \r
-  @param  Udp4Service            Pointer to the UDP4_SERVICE_DATA.\r
-  @param  ImageHandle            The image handle of this udp4 driver.\r
-  @param  ControllerHandle       The controller handle this udp4 driver binds on.\r
+  @param[in, out] Udp4Service      Pointer to the UDP4_SERVICE_DATA.\r
+  @param[in]      ImageHandle      The image handle of this udp4 driver.\r
+  @param[in]      ControllerHandle The controller handle this udp4 driver binds on.\r
 \r
-  @retval EFI_SUCCESS            The udp4 service context data is created and\r
-                                 initialized.\r
-  @retval EFI_OUT_OF_RESOURCES   Cannot allocate memory.\r
-  @retval other                  Other error occurs.\r
+  @retval EFI_SUCCESS              The udp4 service context data is created and\r
+                                   initialized.\r
+  @retval EFI_OUT_OF_RESOURCES     Cannot allocate memory.\r
+  @retval other                    Other error occurs.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -294,8 +273,9 @@ Udp4CreateService (
   IN     EFI_HANDLE         ControllerHandle\r
   )\r
 {\r
-  EFI_STATUS       Status;\r
-  IP_IO_OPEN_DATA  OpenData;\r
+  EFI_STATUS          Status;\r
+  IP_IO_OPEN_DATA     OpenData;\r
+  EFI_IP4_CONFIG_DATA *Ip4ConfigData;\r
 \r
   ZeroMem (Udp4Service, sizeof (UDP4_SERVICE_DATA));\r
 \r
@@ -310,7 +290,7 @@ Udp4CreateService (
   //\r
   // Create the IpIo for this service context.\r
   //\r
-  Udp4Service->IpIo = IpIoCreate (ImageHandle, ControllerHandle);\r
+  Udp4Service->IpIo = IpIoCreate (ImageHandle, ControllerHandle, IP_VERSION_4);\r
   if (Udp4Service->IpIo == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -318,12 +298,13 @@ Udp4CreateService (
   //\r
   // Set the OpenData used to open the IpIo.\r
   //\r
-  CopyMem (&OpenData.IpConfigData, &mIpIoDefaultIpConfigData, sizeof (OpenData.IpConfigData));\r
-  OpenData.IpConfigData.AcceptBroadcast = TRUE;\r
-  OpenData.RcvdContext                  = (VOID *) Udp4Service;\r
-  OpenData.SndContext                   = NULL;\r
-  OpenData.PktRcvdNotify                = Udp4DgramRcvd;\r
-  OpenData.PktSentNotify                = Udp4DgramSent;\r
+  Ip4ConfigData = &OpenData.IpConfigData.Ip4CfgData;\r
+  CopyMem (Ip4ConfigData, &mIp4IoDefaultIpConfigData, sizeof (EFI_IP4_CONFIG_DATA));\r
+  Ip4ConfigData->AcceptBroadcast = TRUE;\r
+  OpenData.RcvdContext           = (VOID *) Udp4Service;\r
+  OpenData.SndContext            = NULL;\r
+  OpenData.PktRcvdNotify         = Udp4DgramRcvd;\r
+  OpenData.PktSentNotify         = Udp4DgramSent;\r
 \r
   //\r
   // Configure and start the IpIo.\r
@@ -376,9 +357,7 @@ ON_ERROR:
 /**\r
   Clean the Udp service context data.\r
 \r
-  @param  Udp4Service            Pointer to the UDP4_SERVICE_DATA.\r
-\r
-  @return None.\r
+  @param[in]  Udp4Service            Pointer to the UDP4_SERVICE_DATA.\r
 \r
 **/\r
 VOID\r
@@ -407,11 +386,9 @@ Udp4CleanService (
   This function checks and timeouts the I/O datagrams holding by the corresponding\r
   service context.\r
 \r
-  @param  Event                  The event this function registered to.\r
-  @param  Context                The context data registered during the creation of\r
-                                 the Event.\r
-\r
-  @return None.\r
+  @param[in]  Event                  The event this function registered to.\r
+  @param[in]  Context                The context data registered during the creation of\r
+                                     the Event.\r
 \r
 **/\r
 VOID\r
@@ -451,13 +428,16 @@ Udp4CheckTimeout (
       //\r
       Wrap = NET_LIST_USER_STRUCT (WrapEntry, UDP4_RXDATA_WRAP, Link);\r
 \r
-      if (Wrap->TimeoutTick <= UDP4_TIMEOUT_INTERVAL / 1000) {\r
+      //\r
+      // TimeoutTick unit is microsecond, MNP_TIMEOUT_CHECK_INTERVAL unit is 100ns.\r
+      //\r
+      if (Wrap->TimeoutTick < (UDP4_TIMEOUT_INTERVAL / 10)) {\r
         //\r
         // Remove this RxData if it timeouts.\r
         //\r
         Udp4RecycleRxDataWrap (NULL, (VOID *) Wrap);\r
       } else {\r
-        Wrap->TimeoutTick -= UDP4_TIMEOUT_INTERVAL / 1000;\r
+        Wrap->TimeoutTick -= (UDP4_TIMEOUT_INTERVAL / 10);\r
       }\r
     }\r
   }\r
@@ -467,10 +447,8 @@ Udp4CheckTimeout (
 /**\r
   This function intializes the new created udp instance.\r
 \r
-  @param  Udp4Service            Pointer to the UDP4_SERVICE_DATA.\r
-  @param  Instance               Pointer to the un-initialized UDP4_INSTANCE_DATA.\r
-\r
-  @return None.\r
+  @param[in]      Udp4Service       Pointer to the UDP4_SERVICE_DATA.\r
+  @param[in, out] Instance          Pointer to the un-initialized UDP4_INSTANCE_DATA.\r
 \r
 **/\r
 VOID\r
@@ -506,16 +484,14 @@ Udp4InitInstance (
   Instance->IcmpError   = EFI_SUCCESS;\r
   Instance->Configured  = FALSE;\r
   Instance->IsNoMapping = FALSE;\r
-  Instance->Destroyed   = FALSE;\r
+  Instance->InDestroy   = FALSE;\r
 }\r
 \r
 \r
 /**\r
   This function cleans the udp instance.\r
 \r
-  @param  Instance               Pointer to the UDP4_INSTANCE_DATA to clean.\r
-\r
-  @return None.\r
+  @param[in]  Instance               Pointer to the UDP4_INSTANCE_DATA to clean.\r
 \r
 **/\r
 VOID\r
@@ -532,10 +508,10 @@ Udp4CleanInstance (
 /**\r
   This function finds the udp instance by the specified <Address, Port> pair.\r
 \r
-  @param  InstanceList           Pointer to the head of the list linking the udp\r
-                                 instances.\r
-  @param  Address                Pointer to the specified IPv4 address.\r
-  @param  Port                   The udp port number.\r
+  @param[in]  InstanceList           Pointer to the head of the list linking the udp\r
+                                     instances.\r
+  @param[in]  Address                Pointer to the specified IPv4 address.\r
+  @param[in]  Port                   The udp port number.\r
 \r
   @retval TRUE     The specified <Address, Port> pair is found.\r
   @retval FALSE    Otherwise.\r
@@ -587,9 +563,9 @@ Udp4FindInstanceByPort (
   This function tries to bind the udp instance according to the configured port\r
   allocation strategy.\r
 \r
-  @param  InstanceList           Pointer to the head of the list linking the udp\r
+  @param[in]      InstanceList   Pointer to the head of the list linking the udp\r
                                  instances.\r
-  @param  ConfigData             Pointer to the ConfigData of the instance to be\r
+  @param[in, out] ConfigData     Pointer to the ConfigData of the instance to be\r
                                  bound. ConfigData->StationPort will be assigned\r
                                  with an available port value on success.\r
 \r
@@ -669,9 +645,9 @@ Udp4Bind (
   This function is used to check whether the NewConfigData has any un-reconfigurable\r
   parameters changed compared to the OldConfigData.\r
 \r
-  @param  OldConfigData          Pointer to the current ConfigData the udp instance\r
+  @param[in]  OldConfigData      Pointer to the current ConfigData the udp instance\r
                                  uses.\r
-  @param  NewConfigData          Pointer to the new ConfigData.\r
+  @param[in]  NewConfigData      Pointer to the new ConfigData.\r
 \r
   @retval TRUE     The instance is reconfigurable.\r
   @retval FALSE    Otherwise.\r
@@ -750,10 +726,8 @@ Udp4IsReconfigurable (
 /**\r
   This function builds the Ip4 configdata from the Udp4ConfigData.\r
 \r
-  @param  Udp4ConfigData         Pointer to the EFI_UDP4_CONFIG_DATA.\r
-  @param  Ip4ConfigData          Pointer to the EFI_IP4_CONFIG_DATA.\r
-\r
-  @return None.\r
+  @param[in]       Udp4ConfigData    Pointer to the EFI_UDP4_CONFIG_DATA.\r
+  @param[in, out]  Ip4ConfigData     Pointer to the EFI_IP4_CONFIG_DATA.\r
 \r
 **/\r
 VOID\r
@@ -762,7 +736,7 @@ Udp4BuildIp4ConfigData (
   IN OUT EFI_IP4_CONFIG_DATA   *Ip4ConfigData\r
   )\r
 {\r
-  CopyMem (Ip4ConfigData, &mIpIoDefaultIpConfigData, sizeof (*Ip4ConfigData));\r
+  CopyMem (Ip4ConfigData, &mIp4IoDefaultIpConfigData, sizeof (*Ip4ConfigData));\r
 \r
   Ip4ConfigData->DefaultProtocol   = EFI_IP_PROTO_UDP;\r
   Ip4ConfigData->AcceptBroadcast   = Udp4ConfigData->AcceptBroadcast;\r
@@ -781,8 +755,8 @@ Udp4BuildIp4ConfigData (
 /**\r
   This function validates the TxToken, it returns the error code according to the spec.\r
 \r
-  @param  Instance               Pointer to the udp instance context data.\r
-  @param  TxToken                Pointer to the token to be checked.\r
+  @param[in]  Instance           Pointer to the udp instance context data.\r
+  @param[in]  TxToken            Pointer to the token to be checked.\r
 \r
   @retval EFI_SUCCESS            The TxToken is valid.\r
   @retval EFI_INVALID_PARAMETER  One or more of the following are TRUE: This is\r
@@ -853,7 +827,7 @@ Udp4ValidateTxToken (
   if (TxData->GatewayAddress != NULL) {\r
     CopyMem (&GatewayAddress, TxData->GatewayAddress, sizeof (IP4_ADDR));\r
 \r
-    if (!Ip4IsUnicast (NTOHL (GatewayAddress), 0)) {\r
+    if (!NetIp4IsUnicast (NTOHL (GatewayAddress), 0)) {\r
       //\r
       // The specified GatewayAddress is not a unicast IPv4 address while it's not 0.\r
       //\r
@@ -868,7 +842,7 @@ Udp4ValidateTxToken (
 \r
     CopyMem (&SourceAddress, &UdpSessionData->SourceAddress, sizeof (IP4_ADDR));\r
 \r
-    if ((SourceAddress != 0) && !Ip4IsUnicast (HTONL (SourceAddress), 0)) {\r
+    if ((SourceAddress != 0) && !NetIp4IsUnicast (HTONL (SourceAddress), 0)) {\r
       //\r
       // Check whether SourceAddress is a valid IPv4 address in case it's not zero.\r
       // The configured station address is used if SourceAddress is zero.\r
@@ -908,10 +882,10 @@ Udp4ValidateTxToken (
 /**\r
   This function checks whether the specified Token duplicates with the one in the Map.\r
 \r
-  @param  Map                    Pointer to the NET_MAP.\r
-  @param  Item                   Pointer to the NET_MAP_ITEM contain the pointer to\r
+  @param[in]  Map                Pointer to the NET_MAP.\r
+  @param[in]  Item               Pointer to the NET_MAP_ITEM contain the pointer to\r
                                  the Token.\r
-  @param  Context                Pointer to the Token to be checked.\r
+  @param[in]  Context            Pointer to the Token to be checked.\r
 \r
   @retval EFI_SUCCESS            The Token specified by Context differs from the\r
                                  one in the Item.\r
@@ -919,6 +893,7 @@ Udp4ValidateTxToken (
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 Udp4TokenExist (\r
   IN NET_MAP       *Map,\r
   IN NET_MAP_ITEM  *Item,\r
@@ -947,11 +922,11 @@ Udp4TokenExist (
   This function calculates the checksum for the Packet, utilizing the pre-calculated\r
   pseudo HeadSum to reduce some overhead.\r
 \r
-  @param  Packet                 Pointer to the NET_BUF contains the udp datagram.\r
-  @param  HeadSum                Checksum of the pseudo header execpt the length\r
+  @param[in]  Packet             Pointer to the NET_BUF contains the udp datagram.\r
+  @param[in]  HeadSum            Checksum of the pseudo header execpt the length\r
                                  field.\r
 \r
-  @return The 16-bit checksum of this udp datagram.\r
+  @retval The 16-bit checksum of this udp datagram.\r
 \r
 **/\r
 UINT16\r
@@ -974,8 +949,8 @@ Udp4Checksum (
 /**\r
   This function removes the specified Token from the TokenMap.\r
 \r
-  @param  TokenMap               Pointer to the NET_MAP containing the tokens.\r
-  @param  Token                  Pointer to the Token to be removed.\r
+  @param[in, out] TokenMap       Pointer to the NET_MAP containing the tokens.\r
+  @param[in]      Token          Pointer to the Token to be removed.\r
 \r
   @retval EFI_SUCCESS            The specified Token is removed from the TokenMap.\r
   @retval EFI_NOT_FOUND          The specified Token is not found in the TokenMap.\r
@@ -1012,20 +987,19 @@ Udp4RemoveToken (
   interface. It's called to signal the udp TxToken when IpIo layer completes the\r
   transmitting of the udp datagram.\r
 \r
-  @param  Status                 The completion status of the output udp datagram.\r
-  @param  Context                Pointer to the context data.\r
-  @param  Sender                 Pointer to the Ip sender of the udp datagram.\r
-  @param  NotifyData             Pointer to the notify data.\r
-\r
-  @return None.\r
+  @param[in]  Status                 The completion status of the output udp datagram.\r
+  @param[in]  Context                Pointer to the context data.\r
+  @param[in]  Sender                 Specify a pointer of EFI_IP4_PROTOCOL for sending.\r
+  @param[in]  NotifyData             Pointer to the notify data.\r
 \r
 **/\r
 VOID\r
+EFIAPI\r
 Udp4DgramSent (\r
-  IN EFI_STATUS  Status,\r
-  IN VOID        *Context,\r
-  IN VOID        *Sender,\r
-  IN VOID        *NotifyData\r
+  IN EFI_STATUS        Status,\r
+  IN VOID              *Context,\r
+  IN IP_IO_IP_PROTOCOL Sender,\r
+  IN VOID              *NotifyData\r
   )\r
 {\r
   UDP4_INSTANCE_DATA         *Instance;\r
@@ -1040,7 +1014,7 @@ Udp4DgramSent (
     //\r
     Token->Status = Status;\r
     gBS->SignalEvent (Token->Event);\r
-    NetLibDispatchDpc ();\r
+    DispatchDpc ();\r
   }\r
 }\r
 \r
@@ -1048,21 +1022,20 @@ Udp4DgramSent (
 /**\r
   This function processes the received datagram passed up by the IpIo layer.\r
 \r
-  @param  Status                 The status of this udp datagram.\r
-  @param  IcmpError              The IcmpError code, only available when Status is\r
+  @param[in]  Status             The status of this udp datagram.\r
+  @param[in]  IcmpError          The IcmpError code, only available when Status is\r
                                  EFI_ICMP_ERROR.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA.\r
-  @param  Packet                 Pointer to the NET_BUF containing the received udp\r
+  @param[in]  NetSession         Pointer to the EFI_NET_SESSION_DATA.\r
+  @param[in]  Packet             Pointer to the NET_BUF containing the received udp\r
                                  datagram.\r
-  @param  Context                Pointer to the context data.\r
-\r
-  @return None.\r
+  @param[in]  Context            Pointer to the context data.\r
 \r
 **/\r
 VOID\r
+EFIAPI\r
 Udp4DgramRcvd (\r
   IN EFI_STATUS            Status,\r
-  IN ICMP_ERROR            IcmpError,\r
+  IN UINT8                 IcmpError,\r
   IN EFI_NET_SESSION_DATA  *NetSession,\r
   IN NET_BUF               *Packet,\r
   IN VOID                  *Context\r
@@ -1089,16 +1062,16 @@ Udp4DgramRcvd (
   // Dispatch the DPC queued by the NotifyFunction of the rx token's events\r
   // which are signaled with received data.\r
   //\r
-  NetLibDispatchDpc ();\r
+  DispatchDpc ();\r
 }\r
 \r
 \r
 /**\r
   This function removes the multicast group specified by Arg from the Map.\r
 \r
-  @param  Map                    Pointer to the NET_MAP.\r
-  @param  Item                   Pointer to the NET_MAP_ITEM.\r
-  @param  Arg                    Pointer to the Arg, it's the pointer to a\r
+  @param[in, out] Map            Pointer to the NET_MAP.\r
+  @param[in]      Item           Pointer to the NET_MAP_ITEM.\r
+  @param[in]      Arg            Pointer to the Arg, it's the pointer to a\r
                                  multicast IPv4 Address.\r
 \r
   @retval EFI_SUCCESS            The multicast address is removed.\r
@@ -1107,6 +1080,7 @@ Udp4DgramRcvd (
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 Udp4LeaveGroup (\r
   IN OUT NET_MAP       *Map,\r
   IN     NET_MAP_ITEM  *Item,\r
@@ -1145,9 +1119,9 @@ Udp4LeaveGroup (
   This function cancels the token specified by Arg in the Map. This is a callback\r
   used by Udp4InstanceCancelToken().\r
 \r
-  @param  Map                    Pointer to the NET_MAP.\r
-  @param  Item                   Pointer to the NET_MAP_ITEM.\r
-  @param  Arg                    Pointer to the token to be cancelled, if NULL,\r
+  @param[in]  Map                Pointer to the NET_MAP.\r
+  @param[in]  Item               Pointer to the NET_MAP_ITEM.\r
+  @param[in]  Arg                Pointer to the token to be cancelled, if NULL,\r
                                  the token specified by Item is cancelled.\r
 \r
   @retval EFI_SUCCESS            The token is cancelled if Arg is NULL or the token\r
@@ -1158,6 +1132,7 @@ Udp4LeaveGroup (
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 Udp4CancelTokens (\r
   IN NET_MAP       *Map,\r
   IN NET_MAP_ITEM  *Item,\r
@@ -1205,9 +1180,7 @@ Udp4CancelTokens (
 /**\r
   This function removes all the Wrap datas in the RcvdDgramQue.\r
 \r
-  @param  Instance           Pointer to the udp instance context data.\r
-\r
-  @return None.\r
+  @param[in]  Instance           Pointer to the udp instance context data.\r
 \r
 **/\r
 VOID\r
@@ -1235,8 +1208,8 @@ Udp4FlushRcvdDgram (
 /**\r
   Cancel Udp4 tokens from the Udp4 instance.\r
 \r
-  @param  Instance               Pointer to the udp instance context data.\r
-  @param  Token                  Pointer to the token to be canceled, if NULL, all\r
+  @param[in]  Instance           Pointer to the udp instance context data.\r
+  @param[in]  Token              Pointer to the token to be canceled, if NULL, all\r
                                  tokens in this instance will be cancelled.\r
 \r
   @retval EFI_SUCCESS            The Token is cancelled.\r
@@ -1288,8 +1261,8 @@ Udp4InstanceCancelToken (
 /**\r
   This function matches the received udp datagram with the Instance.\r
 \r
-  @param  Instance               Pointer to the udp instance context data.\r
-  @param  Udp4Session            Pointer to the EFI_UDP4_SESSION_DATA abstracted\r
+  @param[in]  Instance           Pointer to the udp instance context data.\r
+  @param[in]  Udp4Session        Pointer to the EFI_UDP4_SESSION_DATA abstracted\r
                                  from the received udp datagram.\r
 \r
   @retval TRUE       The udp datagram matches the receiving requirments of the\r
@@ -1368,10 +1341,8 @@ Udp4MatchDgram (
 /**\r
   This function removes the Wrap specified by Context and release relevant resources.\r
 \r
-  @param  Event                  The Event this notify function registered to.\r
-  @param  Context                Pointer to the context data.\r
-\r
-  @return None.\r
+  @param[in]  Event              The Event this notify function registered to.\r
+  @param[in]  Context            Pointer to the context data.\r
 \r
 **/\r
 VOID\r
@@ -1400,18 +1371,18 @@ Udp4RecycleRxDataWrap (
   //\r
   gBS->CloseEvent (Wrap->RxData.RecycleSignal);\r
 \r
-  gBS->FreePool (Wrap);\r
+  FreePool (Wrap);\r
 }\r
 \r
 \r
 /**\r
   This function wraps the Packet and the RxData.\r
 \r
-  @param  Instance               Pointer to the instance context data.\r
-  @param  Packet                 Pointer to the buffer containing the received\r
-                                 datagram.\r
-  @param  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
-                                 datagram.\r
+  @param[in]  Instance               Pointer to the instance context data.\r
+  @param[in]  Packet                 Pointer to the buffer containing the received\r
+                                     datagram.\r
+  @param[in]  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
+                                     datagram.\r
 \r
   @return Pointer to the structure wrapping the RxData and the Packet.\r
 \r
@@ -1450,7 +1421,7 @@ Udp4WrapRxData (
                   &Wrap->RxData.RecycleSignal\r
                   );\r
   if (EFI_ERROR (Status)) {\r
-    gBS->FreePool (Wrap);\r
+    FreePool (Wrap);\r
     return NULL;\r
   }\r
 \r
@@ -1464,11 +1435,11 @@ Udp4WrapRxData (
 /**\r
   This function enqueues the received datagram into the instances' receiving queues.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-  @param  Packet                 Pointer to the buffer containing the received\r
-                                 datagram.\r
-  @param  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
-                                 datagram.\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
+  @param[in]  Packet                 Pointer to the buffer containing the received\r
+                                     datagram.\r
+  @param[in]  RxData                 Pointer to the EFI_UDP4_RECEIVE_DATA of this\r
+                                     datagram.\r
 \r
   @return The times this datagram is enqueued.\r
 \r
@@ -1521,9 +1492,7 @@ Udp4EnqueueDgram (
 /**\r
   This function delivers the received datagrams for the specified instance.\r
 \r
-  @param  Instance               Pointer to the instance context data.\r
-\r
-  @return None.\r
+  @param[in]  Instance               Pointer to the instance context data.\r
 \r
 **/\r
 VOID\r
@@ -1587,9 +1556,7 @@ Udp4InstanceDeliverDgram (
 /**\r
   This function delivers the datagrams enqueued in the instances.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-\r
-  @return None.\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
 \r
 **/\r
 VOID\r
@@ -1621,13 +1588,11 @@ Udp4DeliverDgram (
 /**\r
   This function demultiplexes the received udp datagram to the apropriate instances.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA abstrated from\r
-                                 the received datagram.\r
-  @param  Packet                 Pointer to the buffer containing the received udp\r
-                                 datagram.\r
-\r
-  @return None.\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
+  @param[in]  NetSession             Pointer to the EFI_NET_SESSION_DATA abstrated from\r
+                                     the received datagram.\r
+  @param[in]  Packet                 Pointer to the buffer containing the received udp\r
+                                     datagram.\r
 \r
 **/\r
 VOID\r
@@ -1637,7 +1602,7 @@ Udp4Demultiplex (
   IN NET_BUF               *Packet\r
   )\r
 {\r
-  EFI_UDP4_HEADER        *Udp4Header;\r
+  EFI_UDP_HEADER        *Udp4Header;\r
   UINT16                 HeadSum;\r
   EFI_UDP4_RECEIVE_DATA  RxData;\r
   EFI_UDP4_SESSION_DATA  *Udp4Session;\r
@@ -1646,15 +1611,16 @@ Udp4Demultiplex (
   //\r
   // Get the datagram header from the packet buffer.\r
   //\r
-  Udp4Header = (EFI_UDP4_HEADER *) NetbufGetByte (Packet, 0, NULL);\r
+  Udp4Header = (EFI_UDP_HEADER *) NetbufGetByte (Packet, 0, NULL);\r
+  ASSERT (Udp4Header != NULL);\r
 \r
   if (Udp4Header->Checksum != 0) {\r
     //\r
     // check the checksum.\r
     //\r
     HeadSum = NetPseudoHeadChecksum (\r
-                NetSession->Source,\r
-                NetSession->Dest,\r
+                NetSession->Source.Addr[0],\r
+                NetSession->Dest.Addr[0],\r
                 EFI_IP_PROTO_UDP,\r
                 0\r
                 );\r
@@ -1667,8 +1633,6 @@ Udp4Demultiplex (
     }\r
   }\r
 \r
-  gRT->GetTime (&RxData.TimeStamp, NULL);\r
-\r
   Udp4Session                  = &RxData.UdpSession;\r
   Udp4Session->SourcePort      = NTOHS (Udp4Header->SrcPort);\r
   Udp4Session->DestinationPort = NTOHS (Udp4Header->DstPort);\r
@@ -1712,13 +1676,11 @@ Udp4Demultiplex (
 /**\r
   This function builds and sends out a icmp port unreachable message.\r
 \r
-  @param  IpIo                   Pointer to the IP_IO instance.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA of the packet\r
-                                 causes this icmp error message.\r
-  @param  Udp4Header             Pointer to the udp header of the datagram causes\r
-                                 this icmp error message.\r
-\r
-  @return None.\r
+  @param[in]  IpIo                   Pointer to the IP_IO instance.\r
+  @param[in]  NetSession             Pointer to the EFI_NET_SESSION_DATA of the packet\r
+                                     causes this icmp error message.\r
+  @param[in]  Udp4Header             Pointer to the udp header of the datagram causes\r
+                                     this icmp error message.\r
 \r
 **/\r
 VOID\r
@@ -1736,7 +1698,7 @@ Udp4SendPortUnreach (
   IP_IO_OVERRIDE       Override;\r
   IP_IO_IP_INFO        *IpSender;\r
 \r
-  IpSender = IpIoFindSender (&IpIo, NetSession->Dest);\r
+  IpSender = IpIoFindSender (&IpIo, NetSession->IpVersion, &NetSession->Dest);\r
   if (IpSender == NULL) {\r
     //\r
     // No apropriate sender, since we cannot send out the ICMP message through\r
@@ -1745,7 +1707,7 @@ Udp4SendPortUnreach (
     return;\r
   }\r
 \r
-  IpHdr = NetSession->IpHdr;\r
+  IpHdr = NetSession->IpHdr.Ip4Hdr;\r
 \r
   //\r
   // Calculate the requried length of the icmp error message.\r
@@ -1765,6 +1727,7 @@ Udp4SendPortUnreach (
   // Allocate space for the IP4_ICMP_ERROR_HEAD.\r
   //\r
   IcmpErrHdr = (IP4_ICMP_ERROR_HEAD *) NetbufAllocSpace (Packet, Len, FALSE);\r
+  ASSERT (IcmpErrHdr != NULL);\r
 \r
   //\r
   // Set the required fields for the icmp port unreachable message.\r
@@ -1793,18 +1756,18 @@ Udp4SendPortUnreach (
   //\r
   // Fill the override data.\r
   //\r
-  Override.DoNotFragment = FALSE;\r
-  Override.TypeOfService = 0;\r
-  Override.TimeToLive    = 255;\r
-  Override.Protocol      = EFI_IP_PROTO_ICMP;\r
+  Override.Ip4OverrideData.DoNotFragment = FALSE;\r
+  Override.Ip4OverrideData.TypeOfService = 0;\r
+  Override.Ip4OverrideData.TimeToLive    = 255;\r
+  Override.Ip4OverrideData.Protocol      = EFI_IP_PROTO_ICMP;\r
 \r
-  CopyMem (&Override.SourceAddress, &NetSession->Dest, sizeof (EFI_IPv4_ADDRESS));\r
-  ZeroMem (&Override.GatewayAddress, sizeof (EFI_IPv4_ADDRESS));\r
+  CopyMem (&Override.Ip4OverrideData.SourceAddress, &NetSession->Dest, sizeof (EFI_IPv4_ADDRESS));\r
+  ZeroMem (&Override.Ip4OverrideData.GatewayAddress, sizeof (EFI_IPv4_ADDRESS));\r
 \r
   //\r
   // Send out this icmp packet.\r
   //\r
-  IpIoSend (IpIo, Packet, IpSender, NULL, NULL, NetSession->Source, &Override);\r
+  IpIoSend (IpIo, Packet, IpSender, NULL, NULL, &NetSession->Source, &Override);\r
 \r
   NetbufFree (Packet);\r
 }\r
@@ -1814,29 +1777,28 @@ Udp4SendPortUnreach (
   This function handles the received Icmp Error message and demultiplexes it to the\r
   instance.\r
 \r
-  @param  Udp4Service            Pointer to the udp service context data.\r
-  @param  IcmpError              The icmp error code.\r
-  @param  NetSession             Pointer to the EFI_NET_SESSION_DATA abstracted\r
+  @param[in]  Udp4Service            Pointer to the udp service context data.\r
+  @param[in]  IcmpError              The icmp error code.\r
+  @param[in]  NetSession             Pointer to the EFI_NET_SESSION_DATA abstracted\r
                                  from the received Icmp Error packet.\r
-  @param  Packet                 Pointer to the Icmp Error packet.\r
-\r
-  @return None.\r
+  @param[in]  Packet                 Pointer to the Icmp Error packet.\r
 \r
 **/\r
 VOID\r
 Udp4IcmpHandler (\r
   IN UDP4_SERVICE_DATA     *Udp4Service,\r
-  IN ICMP_ERROR            IcmpError,\r
+  IN UINT8                 IcmpError,\r
   IN EFI_NET_SESSION_DATA  *NetSession,\r
   IN NET_BUF               *Packet\r
   )\r
 {\r
-  EFI_UDP4_HEADER        *Udp4Header;\r
+  EFI_UDP_HEADER        *Udp4Header;\r
   EFI_UDP4_SESSION_DATA  Udp4Session;\r
   LIST_ENTRY             *Entry;\r
   UDP4_INSTANCE_DATA     *Instance;\r
 \r
-  Udp4Header = (EFI_UDP4_HEADER *) NetbufGetByte (Packet, 0, NULL);\r
+  Udp4Header = (EFI_UDP_HEADER *) NetbufGetByte (Packet, 0, NULL);\r
+  ASSERT (Udp4Header != NULL);\r
 \r
   CopyMem (&Udp4Session.SourceAddress, &NetSession->Source, sizeof (EFI_IPv4_ADDRESS));\r
   CopyMem (&Udp4Session.DestinationAddress, &NetSession->Dest, sizeof (EFI_IPv4_ADDRESS));\r
@@ -1850,16 +1812,7 @@ Udp4IcmpHandler (
     //\r
     Instance = NET_LIST_USER_STRUCT (Entry, UDP4_INSTANCE_DATA, Link);\r
 \r
-    if (!Instance->Configured ||\r
-        Instance->ConfigData.AcceptPromiscuous ||\r
-        Instance->ConfigData.AcceptAnyPort ||\r
-        EFI_IP4_EQUAL (&Instance->ConfigData.StationAddress, &mZeroIp4Addr)\r
-        ) {\r
-      //\r
-      // Don't try to deliver the ICMP error to this instance if it is not configured,\r
-      // or it's configured to be promiscuous or accept any port or accept all the\r
-      // datagrams.\r
-      //\r
+    if (!Instance->Configured) {\r
       continue;\r
     }\r
 \r
@@ -1867,7 +1820,7 @@ Udp4IcmpHandler (
       //\r
       // Translate the Icmp Error code according to the udp spec.\r
       //\r
-      Instance->IcmpError = IpIoGetIcmpErrStatus (IcmpError, NULL, NULL);\r
+      Instance->IcmpError = IpIoGetIcmpErrStatus (IcmpError, IP_VERSION_4, NULL, NULL);\r
 \r
       if (IcmpError > ICMP_ERR_UNREACH_PORT) {\r
         Instance->IcmpError = EFI_ICMP_ERROR;\r
@@ -1889,9 +1842,7 @@ Udp4IcmpHandler (
 /**\r
   This function reports the received ICMP error.\r
 \r
-  @param  Instance               Pointer to the udp instance context data.\r
-\r
-  @return None.\r
+  @param[in]  Instance               Pointer to the udp instance context data.\r
 \r
 **/\r
 VOID\r
@@ -1934,178 +1885,13 @@ Udp4ReportIcmpError (
   This function is a dummy ext-free function for the NET_BUF created for the output\r
   udp datagram.\r
 \r
-  @param  Context                Pointer to the context data.\r
-\r
-  @return None.\r
+  @param[in]  Context                Pointer to the context data.\r
 \r
 **/\r
 VOID\r
+EFIAPI\r
 Udp4NetVectorExtFree (\r
   VOID  *Context\r
   )\r
 {\r
-}\r
-\r
-\r
-/**\r
-  Set the Udp4 variable data.\r
-\r
-  @param  Udp4Service            Udp4 service data.\r
-\r
-  @retval EFI_OUT_OF_RESOURCES   There are not enough resources to set the\r
-                                 variable.\r
-  @retval EFI_SUCCESS            Set variable successfully.\r
-  @retval other                  Set variable failed.\r
-\r
-**/\r
-EFI_STATUS\r
-Udp4SetVariableData (\r
-  IN UDP4_SERVICE_DATA  *Udp4Service\r
-  )\r
-{\r
-  UINT32                  NumConfiguredInstance;\r
-  LIST_ENTRY              *Entry;\r
-  UINTN                   VariableDataSize;\r
-  EFI_UDP4_VARIABLE_DATA  *Udp4VariableData;\r
-  EFI_UDP4_SERVICE_POINT  *Udp4ServicePoint;\r
-  UDP4_INSTANCE_DATA      *Udp4Instance;\r
-  CHAR16                  *NewMacString;\r
-  EFI_STATUS              Status;\r
-\r
-  NumConfiguredInstance = 0;\r
-\r
-  //\r
-  // Go through the children list to count the configured children.\r
-  //\r
-  NET_LIST_FOR_EACH (Entry, &Udp4Service->ChildrenList) {\r
-    Udp4Instance = NET_LIST_USER_STRUCT_S (\r
-                     Entry,\r
-                     UDP4_INSTANCE_DATA,\r
-                     Link,\r
-                     UDP4_INSTANCE_DATA_SIGNATURE\r
-                     );\r
-\r
-    if (Udp4Instance->Configured) {\r
-      NumConfiguredInstance++;\r
-    }\r
-  }\r
-\r
-  //\r
-  // Calculate the size of the Udp4VariableData. As there may be no Udp4 child,\r
-  // we should add extra buffer for the service points only if the number of configured\r
-  // children is more than 1.\r
-  //\r
-  VariableDataSize = sizeof (EFI_UDP4_VARIABLE_DATA);\r
-\r
-  if (NumConfiguredInstance > 1) {\r
-    VariableDataSize += sizeof (EFI_UDP4_SERVICE_POINT) * (NumConfiguredInstance - 1);\r
-  }\r
-\r
-  Udp4VariableData = AllocatePool (VariableDataSize);\r
-  if (Udp4VariableData == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  Udp4VariableData->DriverHandle = Udp4Service->ImageHandle;\r
-  Udp4VariableData->ServiceCount = NumConfiguredInstance;\r
-\r
-  Udp4ServicePoint = &Udp4VariableData->Services[0];\r
-\r
-  //\r
-  // Go through the children list to fill the configured children's address pairs.\r
-  //\r
-  NET_LIST_FOR_EACH (Entry, &Udp4Service->ChildrenList) {\r
-    Udp4Instance = NET_LIST_USER_STRUCT_S (\r
-                     Entry,\r
-                     UDP4_INSTANCE_DATA,\r
-                     Link,\r
-                     UDP4_INSTANCE_DATA_SIGNATURE\r
-                     );\r
-\r
-    if (Udp4Instance->Configured) {\r
-      Udp4ServicePoint->InstanceHandle = Udp4Instance->ChildHandle;\r
-      Udp4ServicePoint->LocalAddress   = Udp4Instance->ConfigData.StationAddress;\r
-      Udp4ServicePoint->LocalPort      = Udp4Instance->ConfigData.StationPort;\r
-      Udp4ServicePoint->RemoteAddress  = Udp4Instance->ConfigData.RemoteAddress;\r
-      Udp4ServicePoint->RemotePort     = Udp4Instance->ConfigData.RemotePort;\r
-\r
-      Udp4ServicePoint++;\r
-    }\r
-  }\r
-\r
-  //\r
-  // Get the mac string.\r
-  //\r
-  Status = NetLibGetMacString (\r
-             Udp4Service->ControllerHandle,\r
-             Udp4Service->ImageHandle,\r
-             &NewMacString\r
-             );\r
-  if (EFI_ERROR (Status)) {\r
-    goto ON_ERROR;\r
-  }\r
-\r
-  if (Udp4Service->MacString != NULL) {\r
-    //\r
-    // The variable is set already, we're going to update it.\r
-    //\r
-    if (StrCmp (Udp4Service->MacString, NewMacString) != 0) {\r
-      //\r
-      // The mac address is changed, delete the previous variable first.\r
-      //\r
-      gRT->SetVariable (\r
-             Udp4Service->MacString,\r
-             &gEfiUdp4ServiceBindingProtocolGuid,\r
-             EFI_VARIABLE_BOOTSERVICE_ACCESS,\r
-             0,\r
-             NULL\r
-             );\r
-    }\r
-\r
-    gBS->FreePool (Udp4Service->MacString);\r
-  }\r
-\r
-  Udp4Service->MacString = NewMacString;\r
-\r
-  Status = gRT->SetVariable (\r
-                  Udp4Service->MacString,\r
-                  &gEfiUdp4ServiceBindingProtocolGuid,\r
-                  EFI_VARIABLE_BOOTSERVICE_ACCESS,\r
-                  VariableDataSize,\r
-                  (VOID *) Udp4VariableData\r
-                  );\r
-\r
-ON_ERROR:\r
-\r
-  gBS->FreePool (Udp4VariableData);\r
-\r
-  return Status;\r
-}\r
-\r
-\r
-/**\r
-  Clear the variable and free the resource.\r
-\r
-  @param  Udp4Service            Udp4 service data.\r
-\r
-  @return None.\r
-\r
-**/\r
-VOID\r
-Udp4ClearVariableData (\r
-  IN UDP4_SERVICE_DATA  *Udp4Service\r
-  )\r
-{\r
-  ASSERT (Udp4Service->MacString != NULL);\r
-\r
-  gRT->SetVariable (\r
-         Udp4Service->MacString,\r
-         &gEfiUdp4ServiceBindingProtocolGuid,\r
-         EFI_VARIABLE_BOOTSERVICE_ACCESS,\r
-         0,\r
-         NULL\r
-         );\r
-\r
-  gBS->FreePool (Udp4Service->MacString);\r
-  Udp4Service->MacString = NULL;\r
-}\r
+}
\ No newline at end of file