]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/IScsiDxe/IScsiProto.h
For network dynamic media support:
[mirror_edk2.git] / MdeModulePkg / Universal / Network / IScsiDxe / IScsiProto.h
index 43a8c25f96a211803b52c106f6e820fad57e75be..e11565e3015dd6b6cdd517d06218a55f99274178 100644 (file)
@@ -1,6 +1,7 @@
-/*++\r
+/** @file\r
+  The header file of iSCSI Protocol that defines many specific data structures.\r
 \r
-Copyright (c) 2004 - 2008, Intel Corporation\r
+Copyright (c) 2004 - 2010, Intel Corporation.<BR>\r
 All rights reserved. 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
@@ -9,15 +10,7 @@ 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
-  IScsiProto.h\r
-\r
-Abstract:\r
-\r
-  Protocol definitions for iSCSI driver, mainly from RFC3720.\r
-\r
---*/\r
+**/\r
 \r
 #ifndef _ISCSI_PROTO_H_\r
 #define _ISCSI_PROTO_H_\r
@@ -30,13 +23,13 @@ Abstract:
 #define ISCSI_SEQ_EQ(s1, s2)  ((s1) == (s2))\r
 #define ISCSI_SEQ_LT(s1, s2) \\r
     ( \\r
-      (((INT32) (s1) < (INT32) (s2)) && (s2 - s1) < (1 << 31)) || \\r
-      (((INT32) (s1) > (INT32) (s2)) && (s1 - s2) > (1 << 31)) \\r
+      (((INT32) (s1) < (INT32) (s2)) && (s2 - s1) < ((UINT32) 1 << 31)) || \\r
+      (((INT32) (s1) > (INT32) (s2)) && (s1 - s2) > ((UINT32) 1 << 31)) \\r
     )\r
 #define ISCSI_SEQ_GT(s1, s2) \\r
     ( \\r
-      (((INT32) (s1) < (INT32) (s2)) && (s2 - s1) > (1 << 31)) || \\r
-      (((INT32) (s1) > (INT32) (s2)) && (s1 - s2) < (1 << 31)) \\r
+      (((INT32) (s1) < (INT32) (s2)) && (s2 - s1) > ((UINT32) 1 << 31)) || \\r
+      (((INT32) (s1) > (INT32) (s2)) && (s1 - s2) < ((UINT32) 1 << 31)) \\r
     )\r
 \r
 #define ISCSI_WELL_KNOWN_PORT                   3260\r
@@ -79,28 +72,25 @@ Abstract:
 \r
 #define ISCSI_KEY_VALUE_NONE                    "None"\r
 \r
-//\r
-// connection state for initiator\r
-//\r
-typedef enum {\r
-  CONN_STATE_FREE,\r
-  CONN_STATE_XPT_WAIT,\r
-  CONN_STATE_IN_LOGIN,\r
-  CONN_STATE_LOGGED_IN,\r
-  CONN_STATE_IN_LOGOUT,\r
-  CONN_STATE_LOGOUT_REQUESTED,\r
-  CONN_STATE_CLEANUP_WAIT,\r
-  CONN_STATE_IN_CLEANUP\r
-} CONNECTION_STATE;\r
-\r
-//\r
-// session state for initiator\r
-//\r
-typedef enum {\r
-  SESSION_STATE_FREE,\r
-  SESSION_STATE_LOGGED_IN,\r
-  SESSION_STATE_FAILED\r
-} SESSION_STATE;\r
+///\r
+/// connection state for initiator\r
+///\r
+\r
+#define CONN_STATE_FREE                         0\r
+#define CONN_STATE_XPT_WAIT                     1\r
+#define CONN_STATE_IN_LOGIN                     2\r
+#define CONN_STATE_LOGGED_IN                    3\r
+#define CONN_STATE_IN_LOGOUT                    4\r
+#define CONN_STATE_LOGOUT_REQUESTED             5\r
+#define CONN_STATE_CLEANUP_WAIT                 6\r
+#define CONN_STATE_IN_CLEANUP                   7\r
+\r
+///\r
+/// session state for initiator\r
+///\r
+#define SESSION_STATE_FREE                      0\r
+#define SESSION_STATE_LOGGED_IN                 1\r
+#define SESSION_STATE_FAILED                    2\r
 \r
 typedef enum {\r
   DataIn  = 0,\r
@@ -173,9 +163,9 @@ typedef enum {
 \r
 #define ISCSI_BHS_FLAG_FINAL      0x80\r
 \r
-//\r
-// iSCSI Basic Header Segment\r
-//\r
+///\r
+/// iSCSI Basic Header Segment\r
+///\r
 typedef struct _ISCSI_BASIC_HEADER {\r
   UINT8   OpCode;\r
   UINT8   Flags;\r
@@ -220,9 +210,9 @@ typedef struct _ISCSI_BI_EXP_READ_DATA_LEN_AHS {
 #define ISCSI_TASK_ATTR_HOQ       0x03\r
 #define ISCSI_TASK_ATTR_ACA       0x04\r
 \r
-//\r
-// SCSI Command\r
-//\r
+///\r
+/// SCSI Command\r
+///\r
 typedef struct _SCSI_COMMAND {\r
   UINT8   OpCode;\r
   UINT8   Flags;\r
@@ -234,7 +224,7 @@ typedef struct _SCSI_COMMAND {
   UINT32  ExpDataXferLength;\r
   UINT32  CmdSN;\r
   UINT32  ExpStatSN;\r
-  UINT8   CDB[16];\r
+  UINT8   Cdb[16];\r
 } SCSI_COMMAND;\r
 \r
 //\r
@@ -251,9 +241,9 @@ typedef struct _SCSI_COMMAND {
 #define ISCSI_SERVICE_RSP_COMMAND_COMPLETE_AT_TARGET  0x00\r
 #define ISCSI_SERVICE_RSP_TARGET_FAILURE              0x01\r
 \r
-//\r
-// SCSI Response\r
-//\r
+///\r
+/// SCSI Response\r
+///\r
 typedef struct _SCSI_RESPONSE {\r
   UINT8   OpCode;\r
   UINT8   Flags;\r
@@ -277,9 +267,9 @@ typedef struct _ISCSI_SENSE_DATA {
   UINT8   Data[2];\r
 } ISCSI_SENSE_DATA;\r
 \r
-//\r
-// iSCSI Task Managment Function Request\r
-//\r
+///\r
+/// iSCSI Task Managment Function Request\r
+///\r
 typedef struct _ISCSI_TMF_REQUEST {\r
   UINT8   OpCode;\r
   UINT8   Fuction;\r
@@ -305,9 +295,9 @@ typedef struct _ISCSI_TMF_REQUEST {
 #define ISCSI_TMF_RSP_PDU_RSP_FUNCTION_AHTH_FAILED        6\r
 #define ISCSI_TMF_RSP_PDU_RSP_FUNCTION_REJECTED           255\r
 \r
-//\r
-// iSCSI Task Management Function Response\r
-//\r
+///\r
+/// iSCSI Task Management Function Response\r
+///\r
 typedef struct _ISCSI_TMF_RESPONSE {\r
   UINT8   OpCode;\r
   UINT8   Reserved1;\r
@@ -324,9 +314,9 @@ typedef struct _ISCSI_TMF_RESPONSE {
   UINT32  Reserved[3];\r
 } ISCSI_TMF_RESPONSE;\r
 \r
-//\r
-// SCSI Data-Out\r
-//\r
+///\r
+/// SCSI Data-Out\r
+///\r
 typedef struct _ISCSI_SCSI_DATA_OUT {\r
   UINT8   OpCode;\r
   UINT8   Reserved1[3];\r
@@ -347,9 +337,10 @@ typedef struct _ISCSI_SCSI_DATA_OUT {
 #define SCSI_DATA_IN_PDU_FLAG_OVERFLOW      SCSI_RSP_PDU_FLAG_OVERFLOW\r
 #define SCSI_DATA_IN_PDU_FLAG_UNDERFLOW     SCSI_RSP_PDU_FLAG_UNDERFLOW\r
 #define SCSI_DATA_IN_PDU_FLAG_STATUS_VALID  0x01\r
-//\r
-// SCSI Data-In\r
-//\r
+\r
+///\r
+/// SCSI Data-In\r
+///\r
 typedef struct _ISCSI_SCSI_DATA_IN {\r
   UINT8   OpCode;\r
   UINT8   Flags;\r
@@ -369,9 +360,10 @@ typedef struct _ISCSI_SCSI_DATA_IN {
 } ISCSI_SCSI_DATA_IN;\r
 \r
 #define ISCSI_GET_BUFFER_OFFSET(PduHdr) NTOHL (((ISCSI_SCSI_DATA_IN *) (PduHdr))->BufferOffset)\r
-//\r
-// Ready To Transfer\r
-//\r
+\r
+///\r
+/// Ready To Transfer\r
+///\r
 typedef struct _ISCSI_READY_TO_TRANSFER {\r
   UINT8   OpCode;\r
   UINT8   Reserved1[3];\r
@@ -383,7 +375,7 @@ typedef struct _ISCSI_READY_TO_TRANSFER {
   UINT32  StatSN;\r
   UINT32  ExpCmdSN;\r
   UINT32  MaxCmdSN;\r
-  UINT32  R2TSN;\r
+  UINT32  R2TSeqNum;\r
   UINT32  BufferOffset;\r
   UINT32  DesiredDataTransferLength;\r
 } ISCSI_READY_TO_TRANSFER;\r
@@ -410,9 +402,9 @@ typedef struct _ISCSI_ASYNC_MESSAGE {
 #define ISCSI_LOGIN_REQ_PDU_FLAG_TRANSIT  0x80\r
 #define ISCSI_LOGIN_REQ_PDU_FLAG_CONTINUE 0x40\r
 \r
-//\r
-// Login Request\r
-//\r
+///\r
+/// Login Request\r
+///\r
 typedef struct _ISCSI_LOGIN_REQUEST {\r
   UINT8   OpCode;\r
   UINT8   Flags;\r
@@ -420,10 +412,10 @@ typedef struct _ISCSI_LOGIN_REQUEST {
   UINT8   VersionMin;\r
   UINT8   TotalAHSLength;\r
   UINT8   DataSegmentLength[3];\r
-  UINT8   ISID[6];\r
-  UINT16  TSIH;\r
+  UINT8   Isid[6];\r
+  UINT16  Tsih;\r
   UINT32  InitiatorTaskTag;\r
-  UINT16  CID;\r
+  UINT16  Cid;\r
   UINT16  Reserved1;\r
   UINT32  CmdSN;\r
   UINT32  ExpStatSN;\r
@@ -438,9 +430,9 @@ typedef struct _ISCSI_LOGIN_REQUEST {
 #define ISCSI_LOGIN_STATUS_INITIATOR_ERROR  2\r
 #define ISCSI_LOGIN_STATUS_TARGET_ERROR     3\r
 \r
-//\r
-// Login Response\r
-//\r
+///\r
+/// Login Response\r
+///\r
 typedef struct _ISCSI_LOGIN_RESPONSE {\r
   UINT8   OpCode;\r
   UINT8   Flags;\r
@@ -448,8 +440,8 @@ typedef struct _ISCSI_LOGIN_RESPONSE {
   UINT8   VersionActive;\r
   UINT8   TotalAHSLength;\r
   UINT8   DataSegmentLength[3];\r
-  UINT8   ISID[6];\r
-  UINT16  TSIH;\r
+  UINT8   Isid[6];\r
+  UINT16  Tsih;\r
   UINT32  InitiatorTaskTag;\r
   UINT32  Reserved1;\r
   UINT32  StatSN;\r
@@ -464,9 +456,9 @@ typedef struct _ISCSI_LOGIN_RESPONSE {
 #define ISCSI_LOGOUT_REASON_CLOSE_CONNECTION                1\r
 #define ISCSI_LOGOUT_REASON_REMOVE_CONNECTION_FOR_RECOVERY  2\r
 \r
-//\r
-// Logout Request\r
-//\r
+///\r
+/// Logout Request\r
+///\r
 typedef struct _ISCSI_LOGOUT_REQUEST {\r
   UINT8   OpCode;\r
   UINT8   ReasonCode;\r
@@ -475,7 +467,7 @@ typedef struct _ISCSI_LOGOUT_REQUEST {
   UINT8   DataSegmentLength[3];\r
   UINT32  Reserved2[2];\r
   UINT32  InitiatorTaskTag;\r
-  UINT16  CID;\r
+  UINT16  Cid;\r
   UINT16  Reserved3;\r
   UINT32  CmdSN;\r
   UINT32  ExpStatSN;\r
@@ -487,9 +479,9 @@ typedef struct _ISCSI_LOGOUT_REQUEST {
 #define ISCSI_LOGOUT_RESPONSE_RECOVERY_NOT_SUPPORTED  2\r
 #define ISCSI_LOGOUT_RESPONSE_CLEANUP_FAILED          3\r
 \r
-//\r
-// Logout Response\r
-//\r
+///\r
+/// Logout Response\r
+///\r
 typedef struct _ISCSI_LOGOUT_RESPONSE {\r
   UINT8   OpCode;\r
   UINT8   Reserved1;\r
@@ -514,9 +506,9 @@ typedef struct _ISCSI_LOGOUT_RESPONSE {
 #define ISCSI_SNACK_REQUEST_TYPE_DATA_ACK     2\r
 #define ISCSI_SNACK_REQUEST_TYPE_RDATA        3\r
 \r
-//\r
-// SNACK Request\r
-//\r
+///\r
+/// SNACK Request\r
+///\r
 typedef struct _ISCSI_SNACK_REQUEST {\r
   UINT8   OpCode;\r
   UINT8   Type;\r
@@ -533,9 +525,9 @@ typedef struct _ISCSI_SNACK_REQUEST {
   UINT32  RunLength;\r
 } ISCSI_SNACK_REQUEST;\r
 \r
-//\r
-// Reject\r
-//\r
+///\r
+/// Reject\r
+///\r
 typedef struct _ISCSI_REJECT {\r
   UINT8   OpCode;\r
   UINT8   Reserved1;\r
@@ -553,9 +545,9 @@ typedef struct _ISCSI_REJECT {
   UINT32  Reserved5[2];\r
 } ISCSI_REJECT;\r
 \r
-//\r
-// NOP-Out\r
-//\r
+///\r
+/// NOP-Out\r
+///\r
 typedef struct _ISCSI_NOP_OUT {\r
   UINT8   OpCode;\r
   UINT8   Reserved1[3];\r
@@ -569,9 +561,9 @@ typedef struct _ISCSI_NOP_OUT {
   UINT32  Reserved2[4];\r
 } ISCSI_NOP_OUT;\r
 \r
-//\r
-// NOP-In\r
-//\r
+///\r
+/// NOP-In\r
+///\r
 typedef struct _ISCSI_NOP_IN {\r
   UINT8   OpCode;\r
   UINT8   Reserved1[3];\r
@@ -591,8 +583,8 @@ typedef struct _ISCSI_NOP_IN {
 #define ISCSI_FULL_FEATURE_PHASE            3\r
 \r
 typedef enum {\r
-  ISCSI_DIGEST_NONE,\r
-  ISCSI_DIGEST_CRC32\r
+  IScsiDigestNone,\r
+  IScsiDigestCRC32\r
 } ISCSI_DIGEST_TYPE;\r
 \r
 typedef struct _ISCSI_XFER_CONTEXT {\r
@@ -633,81 +625,213 @@ typedef struct _ISCSI_KEY_VALUE_PAIR {
   CHAR8           *Value;\r
 } ISCSI_KEY_VALUE_PAIR;\r
 \r
-//\r
-// function prototypes.\r
-//\r
+/**\r
+  Attach the iSCSI connection to the iSCSI session. \r
+\r
+  @param[in, out]  Session The iSCSI session.\r
+  @param[in, out]  Conn    The iSCSI connection.\r
+**/\r
 VOID\r
 IScsiAttatchConnection (\r
-  IN ISCSI_SESSION     *Session,\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_SESSION     *Session,\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  Detach the iSCSI connection from the session it belongs to. \r
+\r
+  @param[in, out]  Conn The iSCSI connection.\r
+**/\r
 VOID\r
 IScsiDetatchConnection (\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  This function does the iSCSI connection login.\r
+\r
+  @param[in, out]  Conn      The iSCSI connection to login.\r
+\r
+  @retval EFI_SUCCESS        The iSCSI connection is logged into the iSCSI target.\r
+  @retval EFI_TIMEOUT        Timeout happened during the login procedure.\r
+  @retval Others             Other errors as indicated.  \r
+**/\r
 EFI_STATUS\r
 IScsiConnLogin (\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   );\r
 \r
-ISCSI_CONNECTION  *\r
+/**\r
+  Create a TCP connection for the iSCSI session.\r
+\r
+  @param[in]  Private The iSCSI driver data.\r
+  @param[in]  Session Maximum CmdSN from the target.\r
+\r
+  @return The newly created iSCSI connection.\r
+**/\r
+ISCSI_CONNECTION *\r
 IScsiCreateConnection (\r
   IN ISCSI_DRIVER_DATA  *Private,\r
   IN ISCSI_SESSION      *Session\r
   );\r
 \r
+/**\r
+  Destroy an iSCSI connection.\r
+\r
+  @param[in]  Conn The connection to destroy.\r
+**/\r
 VOID\r
 IScsiDestroyConnection (\r
   IN ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  Login the iSCSI session.\r
+\r
+  @param[in]  Private          The iSCSI driver data.\r
+\r
+  @retval EFI_SUCCESS          The iSCSI session login procedure finished.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_NO_MEDIA         There was a media error.\r
+  @retval Others               Other errors as indicated.\r
+\r
+**/\r
 EFI_STATUS\r
 IScsiSessionLogin (\r
   IN ISCSI_DRIVER_DATA  *Private\r
   );\r
 \r
+/**\r
+  Build and send the iSCSI login request to the iSCSI target according to\r
+  the current login stage.\r
+\r
+  @param[in]  Conn             The connection in the iSCSI login phase.\r
+\r
+  @retval EFI_SUCCESS          The iSCSI login request PDU is built and sent on this\r
+                               connection.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_DEVICE_ERROR     Some kind of device error happened.\r
+**/\r
 EFI_STATUS\r
 IScsiSendLoginReq (\r
   IN ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  Receive and process the iSCSI login response.\r
+\r
+  @param[in]  Conn             The connection in the iSCSI login phase.\r
+  \r
+  @retval EFI_SUCCESS          The iSCSI login response PDU is received and processed.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiReceiveLoginRsp (\r
   IN ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  Add an iSCSI key-value pair as a string into the data segment of the Login Request PDU.\r
+  The DataSegmentLength and the actual size of the net buffer containing this PDU will be\r
+  updated.\r
+\r
+  @param[in, out]  Pdu         The iSCSI PDU whose data segment the key-value pair will\r
+                               be added to.\r
+  @param[in]       Key         The key name string.\r
+  @param[in]       Value       The value string.\r
+\r
+  @retval EFI_SUCCESS          The key-valu pair is added to the PDU's datasegment and\r
+                               the correspondence length fields are updated.\r
+  @retval EFI_OUT_OF_RESOURCES There is not enough space in the PDU to add the key-value\r
+                               pair.\r
+**/\r
 EFI_STATUS\r
 IScsiAddKeyValuePair (\r
-  IN NET_BUF          *Pdu,\r
+  IN OUT NET_BUF      *Pdu,\r
   IN CHAR8            *Key,\r
   IN CHAR8            *Value\r
   );\r
 \r
-NET_BUF           *\r
+/**\r
+  Prepare the iSCSI login request to be sent according to the current login status.\r
+\r
+  @param[in, out]  Conn The connection in the iSCSI login phase.\r
+\r
+  @return The pointer to the net buffer containing the iSCSI login request built.\r
+  @retval Others    Other errors as indicated.\r
+**/\r
+NET_BUF *\r
 IScsiPrepareLoginReq (\r
-  IN ISCSI_CONNECTION  *Conn\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  Process the iSCSI Login Response.\r
+\r
+  @param[in, out]  Conn The connection on which the iSCSI login response is received.\r
+  @param[in, out]  Pdu  The iSCSI login response PDU.\r
+\r
+  @retval EFI_SUCCESS        The iSCSI login response PDU is processed and all check are passed.\r
+  @retval EFI_PROTOCOL_ERROR Some kind of iSCSI protocol error happened.\r
+  @retval EFI_MEDIA_CHANGED  Target is redirected.\r
+  @retval Others             Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiProcessLoginRsp (\r
-  IN ISCSI_CONNECTION  *Conn,\r
-  IN NET_BUF           *Pdu\r
+  IN OUT ISCSI_CONNECTION  *Conn,\r
+  IN OUT NET_BUF           *Pdu\r
   );\r
 \r
+/**\r
+  Updated the target information according the data received in the iSCSI\r
+  login response with an target redirection status.\r
+\r
+  @param[in, out] Session      The iSCSI session.\r
+  @param[in]      Data         The data segment which should contain the\r
+                               TargetAddress key-value list.\r
+  @param[in]      Len          Length of the data.\r
+  \r
+  @retval EFI_SUCCESS          The target address is updated.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_NOT_FOUND        The TargetAddress key is not found.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiUpdateTargetAddress (\r
-  IN ISCSI_SESSION  *Session,\r
-  IN CHAR8          *Data,\r
-  IN UINT32         Len\r
+  IN OUT ISCSI_SESSION  *Session,\r
+  IN CHAR8              *Data,\r
+  IN UINT32             Len\r
   );\r
 \r
+/**\r
+  The callback function to free the net buffer list.\r
+\r
+  @param[in]  Arg The opaque parameter.\r
+**/\r
 VOID\r
 IScsiFreeNbufList (\r
   VOID *Arg\r
   );\r
 \r
+/**\r
+  Receive an iSCSI response PDU. An iSCSI response PDU contains an iSCSI PDU header and\r
+  an optional data segment. The two parts will be put into two blocks of buffers in the\r
+  net buffer. The digest check will be conducted in this function if needed and the digests\r
+  will be trimmed from the PDU buffer.\r
+\r
+  @param[in]   Conn        The iSCSI connection to receive data from.\r
+  @param[out]  Pdu         The received iSCSI pdu.\r
+  @param[in]   Context     The context used to describe information on the caller provided\r
+                           buffer to receive data segment of the iSCSI pdu, it's optional.\r
+  @param[in]  HeaderDigest Whether there will be header digest received.\r
+  @param[in]  DataDigest   Whether there will be data digest.\r
+  @param[in]  TimeoutEvent The timeout event, it's optional.\r
+\r
+  @retval EFI_SUCCESS          An iSCSI pdu is received.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval EFI_PROTOCOL_ERROR   Some kind of iSCSI protocol error happened.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiReceivePdu (\r
   IN ISCSI_CONNECTION                      *Conn,\r
@@ -718,47 +842,120 @@ IScsiReceivePdu (
   IN EFI_EVENT                             TimeoutEvent OPTIONAL\r
   );\r
 \r
+/**\r
+  Check and get the result of the prameter negotiation.\r
+\r
+  @param[in, out]  Conn          The connection in iSCSI login.\r
+\r
+  @retval EFI_SUCCESS          The parmeter check is passed and negotiation is finished.\r
+  @retval EFI_PROTOCOL_ERROR   Some kind of iSCSI protocol error happened.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+**/\r
 EFI_STATUS\r
 IScsiCheckOpParams (\r
-  IN ISCSI_CONNECTION  *Conn,\r
-  IN BOOLEAN           Transit\r
+  IN OUT ISCSI_CONNECTION  *Conn\r
   );\r
 \r
+/**\r
+  Fill the oprational prameters.\r
+\r
+  @param[in]       Conn        The connection in iSCSI login.\r
+  @param[in, out]  Pdu         The iSCSI login request PDU to fill the parameters.\r
+\r
+  @retval EFI_SUCCESS          The parmeters are filled into the iSCSI login request PDU.\r
+**/\r
 EFI_STATUS\r
 IScsiFillOpParams (\r
   IN ISCSI_CONNECTION  *Conn,\r
-  IN NET_BUF           *Pdu\r
+  IN OUT NET_BUF       *Pdu\r
   );\r
 \r
+/**\r
+  Pad the iSCSI AHS or data segment to an integer number of 4 byte words.\r
+\r
+  @param[in, out]  Pdu         The iSCSI pdu which contains segments to pad.\r
+  @param[in]       Len         The length of the last semgnet in the PDU.\r
+\r
+  @retval EFI_SUCCESS          The segment is padded or no need to pad it.\r
+  @retval EFI_OUT_OF_RESOURCES There is not enough remaining free space to add the\r
+                               padding bytes.\r
+**/\r
 EFI_STATUS\r
 IScsiPadSegment (\r
-  IN NET_BUF  *Pdu,\r
-  IN UINT32   Len\r
+  IN OUT NET_BUF  *Pdu,\r
+  IN UINT32       Len\r
   );\r
 \r
-LIST_ENTRY        *\r
+/**\r
+  Build a key-value list from the data segment.\r
+\r
+  @param[in]  Data The data segment containing the key-value pairs.\r
+  @param[in]  Len  Length of the data segment.\r
+\r
+  @return The key-value list.\r
+  @retval NULL Other errors as indicated.\r
+**/\r
+LIST_ENTRY *\r
 IScsiBuildKeyValueList (\r
   IN CHAR8  *Data,\r
   IN UINT32 Len\r
   );\r
 \r
-CHAR8             *\r
+/**\r
+  Get the value string by the key name from the key-value list. If found,\r
+  the key-value entry will be removed from the list.\r
+\r
+  @param[in, out]  KeyValueList  The key-value list.\r
+  @param[in]       Key           The key name to find.\r
+\r
+  @return The value string.\r
+**/\r
+CHAR8 *\r
 IScsiGetValueByKeyFromList (\r
-  IN LIST_ENTRY      *KeyValueList,\r
+  IN OUT LIST_ENTRY  *KeyValueList,\r
   IN CHAR8           *Key\r
   );\r
 \r
+/**\r
+  Free the key-value list.\r
+\r
+  @param[in]  KeyValueList The key-value list.\r
+**/\r
 VOID\r
 IScsiFreeKeyValueList (\r
   IN LIST_ENTRY      *KeyValueList\r
   );\r
 \r
+/**\r
+  Normalize the iSCSI name according to RFC.\r
+\r
+  @param[in, out]  Name       The iSCSI name.\r
+  @param[in]       Len        length of the iSCSI name.\r
+\r
+  @retval EFI_SUCCESS        The iSCSI name is valid and normalized.\r
+  @retval EFI_PROTOCOL_ERROR The iSCSI name is mal-formatted or not in the IQN format.\r
+**/\r
 EFI_STATUS\r
 IScsiNormalizeName (\r
-  IN CHAR8  *Name,\r
-  IN UINTN  Len\r
+  IN OUT CHAR8  *Name,\r
+  IN UINTN      Len\r
   );\r
 \r
+/**\r
+  Execute the SCSI command issued through the EXT SCSI PASS THRU protocol.\r
+\r
+  @param[in]       PassThru  The EXT SCSI PASS THRU protocol.\r
+  @param[in]       Target    The target ID.\r
+  @param[in]       Lun       The LUN.\r
+  @param[in, out]  Packet    The request packet containing IO request, SCSI command\r
+                             buffer and buffers to read/write.\r
+                             \r
+  @retval EFI_SUCCES           The SCSI command is executed and the result is updated to \r
+                               the Packet.\r
+  @retval EFI_DEVICE_ERROR     Session state was not as required.\r
+  @retval EFI_OUT_OF_RESOURCES Failed to allocate memory.\r
+  @retval Others               Other errors as indicated.\r
+**/\r
 EFI_STATUS\r
 IScsiExecuteScsiCommand (\r
   IN EFI_EXT_SCSI_PASS_THRU_PROTOCOL                 *PassThru,\r
@@ -767,20 +964,42 @@ IScsiExecuteScsiCommand (
   IN OUT EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *Packet\r
   );\r
 \r
+/**\r
+  Reinstate the session on some error.\r
+\r
+  @param[in, out]  Private The iSCSI driver data.\r
+\r
+  @retval EFI_SUCCES  The session is reinstated from some error.\r
+  @retval Other       Reinstatement failed.\r
+**/\r
 EFI_STATUS\r
 IScsiSessionReinstatement (\r
-  IN ISCSI_DRIVER_DATA  *Private\r
+  IN OUT ISCSI_DRIVER_DATA  *Private\r
   );\r
 \r
+/**\r
+  Initialize some session parameters before login.\r
+\r
+  @param[in, out]  Session  The iSCSI session.\r
+  @param[in]       Recovery Whether the request is from a fresh new start or recovery.\r
+**/\r
 VOID\r
 IScsiSessionInit (\r
-  IN ISCSI_SESSION  *Session,\r
-  IN BOOLEAN        Recovery\r
+  IN OUT ISCSI_SESSION  *Session,\r
+  IN BOOLEAN            Recovery\r
   );\r
+  \r
+/**\r
+  Abort the iSCSI session, that is, reset all the connection and free the\r
+  resources.\r
+\r
+  @param[in, out]  Session The iSCSI session.\r
 \r
+  @retval EFI_SUCCES  The session is aborted.\r
+**/\r
 EFI_STATUS\r
 IScsiSessionAbort (\r
-  IN ISCSI_SESSION  *Session\r
+  IN OUT ISCSI_SESSION  *Session\r
   );\r
 \r
 #endif\r