]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Include/Library/NetLib.h
Retire NetLibQueueDpc() and NetLibDispatchDpc() and use QueueDpc() and DispatchDpc...
[mirror_edk2.git] / MdeModulePkg / Include / Library / NetLib.h
index c317f7689920ab3c875725e30440e23db6fb68e4..7addeeef79d8dc67d456866e0c9ff4f6d53f2209 100644 (file)
@@ -1,5 +1,6 @@
 /** @file\r
-  This library provides basic function for UEFI network stack.\r
+  Ihis library is only intended to be used by UEFI network stack modules.\r
+  It provides basic functions for the UEFI network stack.\r
 \r
 Copyright (c) 2005 - 2008, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
@@ -15,14 +16,6 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #ifndef _NET_LIB_H_\r
 #define _NET_LIB_H_\r
 \r
-#include <Library/BaseMemoryLib.h>\r
-#include <Library/MemoryAllocationLib.h>\r
-#include <Protocol/DriverBinding.h>\r
-#include <Protocol/ComponentName.h>\r
-#include <Protocol/DriverConfiguration.h>\r
-#include <Protocol/DriverDiagnostics.h>\r
-#include <Protocol/Dpc.h>\r
-\r
 typedef UINT32          IP4_ADDR;\r
 typedef UINT32          TCP_SEQNO;\r
 typedef UINT16          TCP_PORTNO;\r
@@ -81,7 +74,7 @@ typedef struct {
 \r
 \r
 //\r
-// ICMP head definition. ICMP message is categoried as either an error\r
+// ICMP head definition. Each ICMP message is categorized as either an error\r
 // message or query message. Two message types have their own head format.\r
 //\r
 typedef struct {\r
@@ -166,13 +159,15 @@ typedef struct {
 #define EFI_IP4_EQUAL(Ip1, Ip2)  (CompareMem ((Ip1), (Ip2), sizeof (EFI_IPv4_ADDRESS)) == 0)\r
 \r
 /**\r
-  Return the length of the mask. If the mask is invalid,\r
-  return the invalid length 33, which is IP4_MASK_NUM.\r
+  Return the length of the mask. \r
+  \r
+  Return the length of the mask. Valid values are 0 to 32.\r
+  If the mask is invalid, return the invalid length 33, which is IP4_MASK_NUM.\r
   NetMask is in the host byte order.\r
 \r
   @param[in]  NetMask              The netmask to get the length from.\r
 \r
-  @return The length of the netmask, IP4_MASK_NUM if the mask isn't.\r
+  @return The length of the netmask, or IP4_MASK_NUM (33) if the mask is invalid.\r
   \r
 **/\r
 INTN\r
@@ -182,9 +177,21 @@ NetGetMaskLength (
   );\r
 \r
 /**\r
-  Return the class of the address, such as class a, b, c.\r
+  Return the class of the IP address, such as class A, B, C.\r
   Addr is in host byte order.\r
+  \r
+  The address of class A  starts with 0.\r
+  If the address belong to class A, return IP4_ADDR_CLASSA.\r
+  The address of class B  starts with 10. \r
+  If the address belong to class B, return IP4_ADDR_CLASSB.\r
+  The address of class C  starts with 110. \r
+  If the address belong to class C, return IP4_ADDR_CLASSC.\r
+  The address of class D  starts with 1110. \r
+  If the address belong to class D, return IP4_ADDR_CLASSD.\r
+  The address of class E  starts with 1111.\r
+  If the address belong to class E, return IP4_ADDR_CLASSE.\r
 \r
+  \r
   @param[in]   Addr                  The address to get the class from.\r
 \r
   @return IP address class, such as IP4_ADDR_CLASSA.\r
@@ -198,13 +205,17 @@ NetGetIpClass (
 \r
 /**\r
   Check whether the IP is a valid unicast address according to\r
-  the netmask. If NetMask is zero, use the IP address's class to\r
-  get the default mask.\r
+  the netmask. If NetMask is zero, use the IP address's class to get the default mask.\r
+  \r
+  If Ip is 0, IP is not a valid unicast address.\r
+  Class D address is used for multicasting and class E address is reserved for future. If Ip\r
+  belongs to class D or class E, Ip is not a valid unicast address. \r
+  If all bits of the host address of Ip are 0 or 1, Ip is not a valid unicast address.\r
 \r
   @param[in]  Ip                    The IP to check against.\r
   @param[in]  NetMask               The mask of the IP.\r
 \r
-  @return TRUE if IP is a valid unicast address on the network, otherwise FALSE.\r
+  @return TRUE if Ip is a valid unicast address on the network, otherwise FALSE.\r
 \r
 **/\r
 BOOLEAN\r
@@ -214,7 +225,7 @@ Ip4IsUnicast (
   IN IP4_ADDR               NetMask\r
   );\r
 \r
-extern IP4_ADDR gIp4AllMasks [IP4_MASK_NUM];\r
+extern IP4_ADDR gIp4AllMasks[IP4_MASK_NUM];\r
 \r
 \r
 extern EFI_IPv4_ADDRESS  mZeroIp4Addr;\r
@@ -230,8 +241,10 @@ extern EFI_IPv4_ADDRESS  mZeroIp4Addr;
 #define NET_RANDOM(Seed)        ((UINT32) ((UINT32) (Seed) * 1103515245UL + 12345) % 4294967295UL)\r
 \r
 /**\r
-  Extract a UINT32 from a byte stream, then convert it to host\r
-  byte order. Use this function to avoid alignment error.\r
+  Extract a UINT32 from a byte stream.\r
+  \r
+  This function copies a UINT32 from a byte stream, and then converts it from Network \r
+  byte order to host byte order. Use this function to avoid alignment error.\r
 \r
   @param[in]  Buf                 The buffer to extract the UINT32.\r
 \r
@@ -245,8 +258,10 @@ NetGetUint32 (
   );\r
 \r
 /**\r
-  Put a UINT32 to the byte stream. Convert it from host byte order\r
-  to network byte order before putting.\r
+  Puts a UINT32 into the byte stream in network byte order. \r
+  \r
+  Converts a UINT32 from host byte order to network byte order, and then copies it to the \r
+  byte stream.\r
 \r
   @param[in, out]  Buf          The buffer to put the UINT32.\r
   @param[in]      Data          The data to put.\r
@@ -261,8 +276,12 @@ NetPutUint32 (
 \r
 /**\r
   Initialize a random seed using current time.\r
-\r
-  @return The random seed initialized with current time.\r
+  \r
+  Get current time first. Then initialize a random seed based on some basic \r
+  mathematical operations on the hour, day, minute, second, nanosecond and year \r
+  of the current time.\r
+  \r
+  @return The random seed, initialized with current time.\r
 \r
 **/\r
 UINT32\r
@@ -279,13 +298,13 @@ NetRandomInitSeed (
           CR(Entry, Type, Field, Sig)\r
 \r
 //\r
-// Iterate through the doule linked list. It is NOT delete safe\r
+// Iterate through the double linked list. It is NOT delete safe\r
 //\r
 #define NET_LIST_FOR_EACH(Entry, ListHead) \\r
   for(Entry = (ListHead)->ForwardLink; Entry != (ListHead); Entry = Entry->ForwardLink)\r
 \r
 //\r
-// Iterate through the doule linked list. This is delete-safe.\r
+// Iterate through the double linked list. This is delete-safe.\r
 // Don't touch NextEntry. Also, don't use this macro if list\r
 // entries other than the Entry may be deleted when processing\r
 // the current Entry.\r
@@ -297,7 +316,7 @@ NetRandomInitSeed (
      )\r
 \r
 //\r
-// Make sure the list isn't empty before get the frist/last record.\r
+// Make sure the list isn't empty before getting the first/last record.\r
 //\r
 #define NET_LIST_HEAD(ListHead, Type, Field)  \\r
           NET_LIST_USER_STRUCT((ListHead)->ForwardLink, Type, Field)\r
@@ -307,11 +326,21 @@ NetRandomInitSeed (
 \r
 \r
 /**\r
-  Remove the first entry on the list.\r
+  Remove the first node entry on the list, and return the removed node entry.\r
+  \r
+  Removes the first node entry from a doubly linked list. It is up to the caller of\r
+  this function to release the memory used by the first node, if that is required. On\r
+  exit, the removed node is returned. \r
+\r
+  If Head is NULL, then ASSERT().\r
+  If Head was not initialized, then ASSERT().\r
+  If PcdMaximumLinkedListLength is not zero, and the number of nodes in the\r
+  linked list including the head node is greater than or equal to PcdMaximumLinkedListLength,\r
+  then ASSERT().    \r
 \r
   @param[in, out]  Head                  The list header.\r
 \r
-  @return The entry that is removed from the list, NULL if the list is empty.\r
+  @return The first node entry that is removed from the list, NULL if the list is empty.\r
 \r
 **/\r
 LIST_ENTRY *\r
@@ -321,11 +350,21 @@ NetListRemoveHead (
   );\r
 \r
 /**\r
-  Remove the last entry on the list.\r
+  Remove the last node entry on the list and return the removed node entry.\r
+\r
+  Removes the last node entry from a doubly linked list. It is up to the caller of\r
+  this function to release the memory used by the first node, if that is required. On\r
+  exit, the removed node is returned. \r
 \r
+  If Head is NULL, then ASSERT().\r
+  If Head was not initialized, then ASSERT().\r
+  If PcdMaximumLinkedListLength is not zero, and the number of nodes in the\r
+  linked list including the head node is greater than or equal to PcdMaximumLinkedListLength,\r
+  then ASSERT(). \r
+  \r
   @param[in, out]  Head                  The list head.\r
 \r
-  @return The entry that is removed from the list, NULL if the list is empty.\r
+  @return The last node entry that is removed from the list, NULL if the list is empty.\r
 \r
 **/\r
 LIST_ENTRY *\r
@@ -335,9 +374,12 @@ NetListRemoveTail (
   );\r
 \r
 /**\r
-  Insert the NewEntry after the PrevEntry.\r
-\r
-  @param[in, out]  PrevEntry             The previous entry to insert after.\r
+  Insert a new node entry after a designated node entry of a doubly linked list.\r
+  \r
+  Inserts a new node entry designated by NewEntry after the node entry designated by PrevEntry\r
+  of the doubly linked list.\r
\r
+  @param[in, out]  PrevEntry             The entry after which to insert. \r
   @param[in, out]  NewEntry              The new entry to insert.\r
 \r
 **/\r
@@ -349,8 +391,11 @@ NetListInsertAfter (
   );\r
 \r
 /**\r
-  Insert the NewEntry before the PostEntry.\r
-\r
+  Insert a new node entry before a designated node entry of a doubly linked list.\r
+  \r
+  Inserts a new node entry designated by NewEntry before the node entry designated by PostEntry\r
+  of the doubly linked list.\r
\r
   @param[in, out]  PostEntry             The entry to insert before.\r
   @param[in, out]  NewEntry              The new entry to insert.\r
 \r
@@ -383,7 +428,15 @@ typedef struct {
 \r
 /**\r
   Initialize the netmap. Netmap is a reposity to keep the <Key, Value> pairs.\r
-\r
\r
+  Initialize the forward and backward links of two head nodes donated by Map->Used \r
+  and Map->Recycled of two doubly linked lists.\r
+  Initializes the count of the <Key, Value> pairs in the netmap to zero.\r
+   \r
+  If Map is NULL, then ASSERT().\r
+  If the address of Map->Used is NULL, then ASSERT().\r
+  If the address of Map->Recycled is NULl, then ASSERT().\r
\r
   @param[in, out]  Map                   The netmap to initialize.\r
 \r
 **/\r
@@ -395,7 +448,13 @@ NetMapInit (
 \r
 /**\r
   To clean up the netmap, that is, release allocated memories.\r
-\r
+  \r
+  Removes all nodes of the Used doubly linked list and frees memory of all related netmap items.\r
+  Removes all nodes of the Recycled doubly linked list and free memory of all related netmap items.\r
+  The number of the <Key, Value> pairs in the netmap is set to zero.\r
+  \r
+  If Map is NULL, then ASSERT().\r
+  \r
   @param[in, out]  Map                   The netmap to clean up.\r
 \r
 **/\r
@@ -406,8 +465,13 @@ NetMapClean (
   );\r
 \r
 /**\r
-  Test whether the netmap is empty.\r
-\r
+  Test whether the netmap is empty and return true if it is.\r
+  \r
+  If the number of the <Key, Value> pairs in the netmap is zero, return TRUE.\r
+   \r
+  If Map is NULL, then ASSERT().\r
\r
+  \r
   @param[in]  Map                   The net map to test.\r
 \r
   @return TRUE if the netmap is empty, otherwise FALSE.\r
@@ -435,7 +499,13 @@ NetMapGetCount (
 \r
 /**\r
   Allocate an item to save the <Key, Value> pair to the head of the netmap.\r
+  \r
+  Allocate an item to save the <Key, Value> pair and add corresponding node entry\r
+  to the beginning of the Used doubly linked list. The number of the <Key, Value> \r
+  pairs in the netmap increase by 1.\r
 \r
+  If Map is NULL, then ASSERT().\r
+  \r
   @param[in, out]  Map                   The netmap to insert into.\r
   @param[in]       Key                   The user's key.\r
   @param[in]       Value                 The user's value for the key.\r
@@ -455,6 +525,12 @@ NetMapInsertHead (
 /**\r
   Allocate an item to save the <Key, Value> pair to the tail of the netmap.\r
 \r
+  Allocate an item to save the <Key, Value> pair and add corresponding node entry\r
+  to the tail of the Used doubly linked list. The number of the <Key, Value> \r
+  pairs in the netmap increase by 1.\r
+\r
+  If Map is NULL, then ASSERT().\r
+  \r
   @param[in, out]  Map                   The netmap to insert into.\r
   @param[in]       Key                   The user's key.\r
   @param[in]       Value                 The user's value for the key.\r
@@ -472,8 +548,13 @@ NetMapInsertTail (
   );\r
 \r
 /**\r
-  Find the key in the netmap.\r
+  Finds the key in the netmap and returns the point to the item containing the Key.\r
+  \r
+  Iterate the Used doubly linked list of the netmap to get every item. Compare the key of every \r
+  item with the key to search. It returns the point to the item contains the Key if found.\r
 \r
+  If Map is NULL, then ASSERT().\r
+  \r
   @param[in]  Map                   The netmap to search within.\r
   @param[in]  Key                   The key to search.\r
 \r
@@ -488,8 +569,17 @@ NetMapFindKey (
   );\r
 \r
 /**\r
-  Remove the item from the netmap.\r
-\r
+  Remove the node entry of the item from the netmap and return the key of the removed item.\r
+  \r
+  Remove the node entry of the item from the Used doubly linked list of the netmap. \r
+  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node \r
+  entry of the item to the Recycled doubly linked list of the netmap. If Value is not NULL,\r
+  Value will point to the value of the item. It returns the key of the removed item.\r
+  \r
+  If Map is NULL, then ASSERT().\r
+  If Item is NULL, then ASSERT().\r
+  if item in not in the netmap, then ASSERT().\r
+  \r
   @param[in, out]  Map                   The netmap to remove the item from.\r
   @param[in, out]  Item                  The item to remove.\r
   @param[out]      Value                 The variable to receive the value if not NULL.\r
@@ -506,8 +596,16 @@ NetMapRemoveItem (
   );\r
 \r
 /**\r
-  Remove the first entry on the netmap.\r
+  Remove the first node entry on the netmap and return the key of the removed item.\r
 \r
+  Remove the first node entry from the Used doubly linked list of the netmap. \r
+  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node \r
+  entry to the Recycled doubly linked list of the netmap. If parameter Value is not NULL,\r
+  parameter Value will point to the value of the item. It returns the key of the removed item.\r
+  \r
+  If Map is NULL, then ASSERT().\r
+  If the Used doubly linked list is empty, then ASSERT().\r
+  \r
   @param[in, out]  Map                   The netmap to remove the head from.\r
   @param[out]      Value                 The variable to receive the value if not NULL.\r
 \r
@@ -522,8 +620,16 @@ NetMapRemoveHead (
   );\r
 \r
 /**\r
-  Remove the last entry on the netmap.\r
+  Remove the last node entry on the netmap and return the key of the removed item.\r
 \r
+  Remove the last node entry from the Used doubly linked list of the netmap. \r
+  The number of the <Key, Value> pairs in the netmap decrease by 1. Then add the node \r
+  entry to the Recycled doubly linked list of the netmap. If parameter Value is not NULL,\r
+  parameter Value will point to the value of the item. It returns the key of the removed item.\r
+  \r
+  If Map is NULL, then ASSERT().\r
+  If the Used doubly linked list is empty, then ASSERT().\r
+  \r
   @param[in, out]  Map                   The netmap to remove the tail from.\r
   @param[out]      Value                 The variable to receive the value if not NULL.\r
 \r
@@ -546,11 +652,15 @@ EFI_STATUS
   );\r
 \r
 /**\r
-  Iterate through the netmap and call CallBack for each item. It will\r
-  contiue the traverse if CallBack returns EFI_SUCCESS, otherwise, break\r
-  from the loop. It returns the CallBack's last return value. This\r
-  function is delete safe for the current item.\r
+  Iterate through the netmap and call CallBack for each item.\r
+  \r
+  It will contiue the traverse if CallBack returns EFI_SUCCESS, otherwise, break\r
+  from the loop. It returns the CallBack's last return value. This function is \r
+  delete safe for the current item.\r
 \r
+  If Map is NULL, then ASSERT().\r
+  If CallBack is NULL, then ASSERT().\r
+  \r
   @param[in]  Map                   The Map to iterate through.\r
   @param[in]  CallBack              The callback function to call for each item.\r
   @param[in]  Arg                   The opaque parameter to the callback.\r
@@ -574,13 +684,18 @@ NetMapIterate (
 //\r
 /**\r
   Create a child of the service that is identified by ServiceBindingGuid.\r
+  \r
+  Get the ServiceBinding Protocol first, then use it to create a child.\r
 \r
+  If ServiceBindingGuid is NULL, then ASSERT().\r
+  If ChildHandle is NULL, then ASSERT().\r
+  \r
   @param[in]       Controller            The controller which has the service installed.\r
   @param[in]       Image                 The image handle used to open service.\r
   @param[in]       ServiceBindingGuid    The service's Guid.\r
-  @param[in, out]  ChildHandle           The handle to receive the create child\r
+  @param[in, out]  ChildHandle           The handle to receive the created child.\r
 \r
-  @retval EFI_SUCCESS           The child is successfully created.\r
+  @retval EFI_SUCCESS           The child was successfully created.\r
   @retval Others                Failed to create the child.\r
 \r
 **/\r
@@ -594,15 +709,19 @@ NetLibCreateServiceChild (
   );\r
 \r
 /**\r
-  Destory a child of the service that is identified by ServiceBindingGuid.\r
-\r
+  Destroy a child of the service that is identified by ServiceBindingGuid.\r
+  \r
+  Get the ServiceBinding Protocol first, then use it to destroy a child.\r
+  \r
+  If ServiceBindingGuid is NULL, then ASSERT().\r
+  \r
   @param[in]   Controller            The controller which has the service installed.\r
   @param[in]   Image                 The image handle used to open service.\r
   @param[in]   ServiceBindingGuid    The service's Guid.\r
-  @param[in]   ChildHandle           The child to destory\r
+  @param[in]   ChildHandle           The child to destroy.\r
 \r
-  @retval EFI_SUCCESS           The child is successfully destoried.\r
-  @retval Others                Failed to destory the child.\r
+  @retval EFI_SUCCESS           The child is successfully destroyed.\r
+  @retval Others                Failed to destroy the child.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -619,15 +738,20 @@ NetLibDestroyServiceChild (
   SnpHandle to a unicode string. Callers are responsible for freeing the\r
   string storage.\r
 \r
-  @param[in]   SnpHandle             The handle where the simple network protocol is\r
-                                     installed on.\r
-  @param[in]   ImageHandle           The image handle used to act as the agent handle to\r
+  Get the mac address of the Simple Network protocol from the SnpHandle. Then convert\r
+  the mac address into a unicode string. It takes 2 unicode characters to represent \r
+  a 1 byte binary buffer, plus one unicode character for the null terminator.\r
+\r
+\r
+  @param[in]   SnpHandle             The handle on which the simple network protocol is\r
+                                     installed.\r
+  @param[in]   ImageHandle           The image handle to act as the agent handle to\r
                                      get the simple network protocol.\r
   @param[out]  MacString             The pointer to store the address of the string\r
                                      representation of  the mac address.\r
   \r
-  @retval EFI_SUCCESS           Convert the mac address a unicode string successfully.\r
-  @retval EFI_OUT_OF_RESOURCES  There are not enough memory resource.\r
+  @retval EFI_SUCCESS           Converted the mac address a unicode string successfully.\r
+  @retval EFI_OUT_OF_RESOURCES  There are not enough memory resources.\r
   @retval Others                Failed to open the simple network protocol.\r
 \r
 **/\r
@@ -641,6 +765,11 @@ NetLibGetMacString (
 \r
 /**\r
   Create an IPv4 device path node.\r
+  \r
+  The header type of IPv4 device path node is MESSAGING_DEVICE_PATH.\r
+  The header subtype of IPv4 device path node is MSG_IPv4_DP.\r
+  The length of the IPv4 device path node in bytes is 19.\r
+  Get other info from parameters to make up the whole IPv4 device path node.\r
 \r
   @param[in, out]  Node                  Pointer to the IPv4 device path node.\r
   @param[in]       Controller            The handle where the NIC IP4 config protocol resides.\r
@@ -667,8 +796,9 @@ NetLibCreateIPv4DPathNode (
 \r
 /**\r
   Find the UNDI/SNP handle from controller and protocol GUID.\r
-  For example, IP will open a MNP child to transmit/receive\r
-  packets, when MNP is stopped, IP should also be stopped. IP\r
+  \r
+  For example, IP will open an MNP child to transmit/receive\r
+  packets. When MNP is stopped, IP should also be stopped. IP\r
   needs to find its own private data which is related the IP's\r
   service binding instance that is install on UNDI/SNP handle.\r
   Now, the controller is either a MNP or ARP child handle. But\r
@@ -688,48 +818,12 @@ NetLibGetNicHandle (
   IN EFI_GUID               *ProtocolGuid\r
   );\r
 \r
-/**\r
-  Add a Deferred Procedure Call to the end of the DPC queue.\r
-\r
-  @param[in]  DpcTpl           The EFI_TPL that the DPC should be invoked.\r
-  @param[in]  DpcProcedure     Pointer to the DPC's function.\r
-  @param[in]  DpcContext       Pointer to the DPC's context.  Passed to DpcProcedure\r
-                               when DpcProcedure is invoked.\r
-\r
-  @retval  EFI_SUCCESS              The DPC was queued.\r
-  @retval  EFI_INVALID_PARAMETER    DpcTpl is not a valid EFI_TPL, or DpcProcedure\r
-                                    is NULL.\r
-  @retval  EFI_OUT_OF_RESOURCES     There are not enough resources available to\r
-                                    add the DPC to the queue.\r
-\r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-NetLibQueueDpc (\r
-  IN EFI_TPL            DpcTpl,\r
-  IN EFI_DPC_PROCEDURE  DpcProcedure,\r
-  IN VOID               *DpcContext    OPTIONAL\r
-  );\r
-\r
-/**\r
-  Dispatch the queue of DPCs. ALL DPCs that have been queued with a DpcTpl\r
-  value greater than or equal to the current TPL are invoked in the order that\r
-  they were queued.  DPCs with higher DpcTpl values are invoked before DPCs with\r
-  lower DpcTpl values.\r
-\r
-  @retval  EFI_SUCCESS              One or more DPCs were invoked.\r
-  @retval  EFI_NOT_FOUND            No DPCs were invoked.\r
-\r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-NetLibDispatchDpc (\r
-  VOID\r
-  );\r
-\r
 /**\r
   This is the default unload handle for all the network drivers.\r
 \r
+  Disconnect the driver specified by ImageHandle from all the devices in the handle database.\r
+  Uninstall all the protocols installed in the driver entry point.\r
+  \r
   @param[in]  ImageHandle       The drivers' driver image.\r
 \r
   @retval EFI_SUCCESS           The image is unloaded.\r
@@ -791,8 +885,8 @@ typedef struct {
 } NET_VECTOR;\r
 \r
 //\r
-//NET_BLOCK_OP operate on the NET_BLOCK, It specifies\r
-//where the actual fragment begins and where it ends\r
+//NET_BLOCK_OP operates on the NET_BLOCK. It specifies\r
+//where the actual fragment begins and ends\r
 //\r
 typedef struct {\r
   UINT8               *BlockHead;   // Block's head, or the smallest valid Head\r
@@ -805,13 +899,12 @@ typedef struct {
 \r
 //\r
 //NET_BUF is the buffer manage structure used by the\r
-//network stack. Every network packet may be fragmented,\r
-//and contains multiple fragments. The Vector points to\r
-//memory blocks used by the each fragment, and BlockOp\r
+//network stack. Every network packet may be fragmented. The Vector points to\r
+//memory blocks used by each fragment, and BlockOp\r
 //specifies where each fragment begins and ends.\r
 //\r
-//It also contains a opaque area for protocol to store\r
-//per-packet informations. Protocol must be caution not\r
+//It also contains an opaque area for the protocol to store\r
+//per-packet information. Protocol must be careful not\r
 //to overwrite the members after that.\r
 //\r
 typedef struct {\r
@@ -832,7 +925,7 @@ typedef struct {
 \r
 \r
 //\r
-//A queue of NET_BUFs, It is just a thin extension of\r
+//A queue of NET_BUFs. It is a thin extension of\r
 //NET_BUF functions.\r
 //\r
 typedef struct {\r
@@ -908,8 +1001,8 @@ NetbufAlloc (
  \r
   Decrease the reference count of the net buffer by one. Free the associated net\r
   vector and itself if the reference count of the net buffer is decreased to 0. \r
-  The net vector free operation just decrease the reference count of the net \r
-  vector by one and do the real resource free operation when the reference count\r
+  The net vector free operation decreases the reference count of the net \r
+  vector by one, and performs the resource free operation when the reference count\r
   of the net vector is 0. \r
  \r
   @param[in]  Nbuf                  Pointer to the NET_BUF to be freed.\r
@@ -925,8 +1018,8 @@ NetbufFree (
   Get the index of NET_BLOCK_OP that contains the byte at Offset in the net \r
   buffer. \r
   \r
-  This can be used to, for example, retrieve the IP header in the packet. It \r
-  also can be used to get the fragment that contains the byte which is used \r
+  For example, this function can be used to retrieve the IP header in the packet. It \r
+  also can be used to get the fragment that contains the byte used \r
   mainly by the library implementation itself. \r
 \r
   @param[in]   Nbuf      Pointer to the net buffer.\r
@@ -1017,7 +1110,7 @@ NetbufGetFragment (
 /**\r
   Reserve some space in the header room of the net buffer.\r
 \r
-  Upon allocation, all the space are in the tail room of the buffer. Call this \r
+  Upon allocation, all the space is in the tail room of the buffer. Call this \r
   function to move some space to the header room. This function is quite limited\r
   in that it can only reserve space from the first block of an empty NET_BUF not \r
   built from the external. But it should be enough for the network stack. \r
@@ -1061,8 +1154,8 @@ NetbufAllocSpace (
   @param[in]      FromHead      The flag to indicate whether trim data from head \r
                                 (TRUE) or tail (FALSE).\r
 \r
-  @return    Length of the actually trimmed data, which is possible to be less \r
-             than Len because the TotalSize of Nbuf is less than Len.\r
+  @return    Length of the actually trimmed data, which may be less \r
+             than Len if the TotalSize of Nbuf is less than Len.\r
 \r
 **/\r
 UINT32\r
@@ -1077,7 +1170,7 @@ NetbufTrim (
   Copy Len bytes of data from the specific offset of the net buffer to the \r
   destination memory.\r
  \r
-  The Len bytes of data may cross the several fragments of the net buffer.\r
+  The Len bytes of data may cross several fragments of the net buffer.\r
  \r
   @param[in]   Nbuf         Pointer to the net buffer.\r
   @param[in]   Offset       The sequence number of the first byte to copy.\r
@@ -1085,7 +1178,7 @@ NetbufTrim (
   @param[in]   Dest         The destination of the data to copy to.\r
 \r
   @return           The length of the actual copied data, or 0 if the offset\r
-                    specified exceeds exceeds the total size of net buffer.\r
+                    specified exceeds the total size of net buffer.\r
 \r
 **/\r
 UINT32\r
@@ -1100,17 +1193,17 @@ NetbufCopy (
 /**\r
   Build a NET_BUF from external blocks. \r
    \r
-  A new NET_BUF structure will be created from external blocks. Additional block\r
+  A new NET_BUF structure will be created from external blocks. An additional block\r
   of memory will be allocated to hold reserved HeadSpace bytes of header room\r
-  and existing HeadLen bytes of header but the external blocks are shared by the\r
+  and existing HeadLen bytes of header, but the external blocks are shared by the\r
   net buffer to avoid data copying.\r
 \r
   @param[in]  ExtFragment           Pointer to the data block.\r
   @param[in]  ExtNum                The number of the data blocks.\r
   @param[in]  HeadSpace             The head space to be reserved.\r
-  @param[in]  HeadLen               The length of the protocol header, This function\r
-                                    will pull that number of data into a linear block.\r
-  @param[in]  ExtFree               Pointer to the caller provided free function.\r
+  @param[in]  HeadLen               The length of the protocol header. The function\r
+                                    pulls this amount of data into a linear block.\r
+  @param[in]  ExtFree               Pointer to the caller-provided free function.\r
   @param[in]  Arg                   The argument passed to ExtFree when ExtFree is\r
                                     called.\r
 \r
@@ -1138,7 +1231,7 @@ NetbufFromExt (
   @param[in, out]  ExtFragment           Pointer to the data block.\r
   @param[in, out]  ExtNum                The number of the data blocks.\r
 \r
-  @retval EFI_BUFFER_TOO_SMALL  The number of non-empty block is bigger than \r
+  @retval EFI_BUFFER_TOO_SMALL  The number of non-empty blocks is bigger than \r
                                 ExtNum.\r
   @retval EFI_SUCCESS           Fragment table is built successfully.\r
 \r
@@ -1159,8 +1252,8 @@ NetbufBuildExt (
    \r
   @param[in]   BufList    A List of the net buffer.\r
   @param[in]   HeadSpace  The head space to be reserved.\r
-  @param[in]   HeaderLen  The length of the protocol header, This function\r
-                          will pull that number of data into a linear block.\r
+  @param[in]   HeaderLen  The length of the protocol header. The function\r
+                          pulls this amount of data into a linear block.\r
   @param[in]   ExtFree    Pointer to the caller provided free function.\r
   @param[in]   Arg        The argument passed to ExtFree when ExtFree is called.\r
 \r
@@ -1286,8 +1379,8 @@ NetbufQueCopy (
   );\r
 \r
 /**\r
-  Trim Len bytes of data from the queue header, release any of the net buffer \r
-  whom is trimmed wholely.\r
+  Trim Len bytes of data from the queue header and release any net buffer \r
+  that is trimmed wholely.\r
    \r
   The trimming operation is the same as NetbufTrim but applies to the net buffer\r
   queue instead of the net buffer.\r