]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/IScsiDxe/IScsiConfig.c
CryptoPkg/OpensslLib: introduce OpensslLibCrypto instance
[mirror_edk2.git] / NetworkPkg / IScsiDxe / IScsiConfig.c
index 3c299be94928354ce8c58ce3b3759f3d9b59bcb2..b1696205c4dbff9c29ffe191f68ee238d5502817 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   Helper functions for configuring or getting the parameters relating to iSCSI.\r
 \r
-Copyright (c) 2004 - 2011, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2017, 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
@@ -14,7 +14,6 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 #include "IScsiImpl.h"\r
 \r
-EFI_GUID        mVendorGuid              = ISCSI_CONFIG_GUID;\r
 CHAR16          mVendorStorageName[]     = L"ISCSI_CONFIG_IFR_NVDATA";\r
 BOOLEAN         mIScsiDeviceListUpdated  = FALSE;\r
 UINTN           mNumberOfIScsiDevices    = 0;\r
@@ -30,10 +29,7 @@ HII_VENDOR_DEVICE_PATH  mIScsiHiiVendorDevicePath = {
         (UINT8) ((sizeof (VENDOR_DEVICE_PATH)) >> 8)\r
       }\r
     },\r
-    //\r
-    // {49D7B73E-143D-4716-977B-C45F1CB038CC}\r
-    //\r
-    { 0x49d7b73e, 0x143d, 0x4716, { 0x97, 0x7b, 0xc4, 0x5f, 0x1c, 0xb0, 0x38, 0xcc } }\r
+    ISCSI_CONFIG_GUID\r
   },\r
   {\r
     END_DEVICE_PATH_TYPE,\r
@@ -168,7 +164,10 @@ IpIsUnicast (
   )\r
 {\r
   if (IpMode == IP_MODE_IP4) {\r
-    return NetIp4IsUnicast (NTOHL (Ip->Addr[0]), 0);\r
+    if (IP4_IS_UNSPECIFIED (NTOHL (Ip->Addr[0])) || IP4_IS_LOCAL_BROADCAST (NTOHL (Ip->Addr[0])))   {\r
+      return FALSE;\r
+    }\r
+    return TRUE;\r
   } else if (IpMode == IP_MODE_IP6) {\r
     return NetIp6IsValidUnicast (&Ip->v6);\r
   } else {\r
@@ -290,6 +289,98 @@ IScsiConvertIsIdToString (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Get the Offset value specified by the input String.\r
+\r
+  @param[in]  Configuration      A null-terminated Unicode string in\r
+                                 <ConfigString> format.\r
+  @param[in]  String             The string is "&OFFSET=".\r
+  @param[out] Value              The Offset value.\r
+\r
+  @retval EFI_OUT_OF_RESOURCES   Insufficient resources to store neccessary\r
+                                 structures.\r
+  @retval EFI_SUCCESS            Value of <Number> is outputted in Number\r
+                                 successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+IScsiGetValue (\r
+  IN  CONST EFI_STRING             Configuration,\r
+  IN  CHAR16                       *String,\r
+  OUT UINTN                        *Value\r
+  )\r
+{\r
+  CHAR16                           *StringPtr;\r
+  CHAR16                           *TmpPtr;\r
+  CHAR16                           *Str;\r
+  CHAR16                           TmpStr[2];\r
+  UINTN                            Length;\r
+  UINTN                            Len;\r
+  UINTN                            Index;\r
+  UINT8                            *Buf;\r
+  UINT8                            DigitUint8;\r
+  EFI_STATUS                       Status;\r
+\r
+  //\r
+  // Get Value.\r
+  //\r
+  Buf = NULL;\r
+  StringPtr = StrStr (Configuration, String);\r
+  ASSERT(StringPtr != NULL);\r
+  StringPtr += StrLen (String);\r
+  TmpPtr = StringPtr;\r
+\r
+  while (*StringPtr != L'\0' && *StringPtr != L'&') {\r
+    StringPtr ++;\r
+  }\r
+  Length = StringPtr - TmpPtr;\r
+  Len = Length + 1;\r
+\r
+  Str = AllocateZeroPool (Len * sizeof (CHAR16));\r
+  if (Str == NULL) {\r
+    Status = EFI_OUT_OF_RESOURCES;\r
+    goto Exit;\r
+  }\r
+\r
+  CopyMem (Str, TmpPtr, Len * sizeof (CHAR16));\r
+  *(Str + Length) = L'\0';\r
+\r
+  Len = (Len + 1) / 2;\r
+  Buf = (UINT8 *) AllocateZeroPool (Len);\r
+  if (Buf == NULL) {\r
+    Status = EFI_OUT_OF_RESOURCES;\r
+    goto Exit;\r
+  }\r
+\r
+  ZeroMem (TmpStr, sizeof (TmpStr));\r
+  for (Index = 0; Index < Length; Index ++) {\r
+    TmpStr[0] = Str[Length - Index - 1];\r
+    DigitUint8 = (UINT8) StrHexToUint64 (TmpStr);\r
+    if ((Index & 1) == 0) {\r
+      Buf [Index/2] = DigitUint8;\r
+    } else {\r
+      Buf [Index/2] = (UINT8) ((DigitUint8 << 4) + Buf [Index/2]);\r
+    }\r
+  }\r
+\r
+  *Value = 0;\r
+  CopyMem (\r
+    Value,\r
+    Buf,\r
+    (((Length + 1) / 2) < sizeof (UINTN)) ? ((Length + 1) / 2) : sizeof (UINTN)\r
+    );\r
+\r
+  FreePool (Buf);\r
+  Status = EFI_SUCCESS;\r
+\r
+Exit:\r
+  if (Str != NULL) {\r
+    FreePool (Str);\r
+  }\r
+\r
+  return Status;\r
+}\r
+\r
 /**\r
   Get the attempt config data from global structure by the ConfigIndex.\r
 \r
@@ -350,6 +441,64 @@ IScsiConfigGetAttemptByNic (
   return NULL;\r
 }\r
 \r
+/**\r
+  Extract the Index of the attempt list.\r
+\r
+  @param[in]   AttemptNameList     The Name list of the Attempts.\r
+  @param[out]  AttemptIndexList    The Index list of the Attempts.\r
+  @param[in]   IsAddAttempts       If TRUE, Indicates add one or more attempts.\r
+                                   If FALSE, Indicates delete attempts or change attempt order.\r
+\r
+  @retval EFI_SUCCESS              The Attempt list is valid.\r
+  @retval EFI_INVALID_PARAMETERS   The Attempt List is invalid.\r
+\r
+**/\r
+EFI_STATUS\r
+IScsiGetAttemptIndexList (\r
+  IN      CHAR16                    *AttemptNameList,\r
+     OUT  UINT8                     *AttemptIndexList,\r
+  IN      BOOLEAN                   IsAddAttempts\r
+)\r
+{\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA   *AttemptConfigData;\r
+  CHAR16                        *AttemptStr;\r
+  UINT8                         AttemptIndex;\r
+  UINTN                         Len;\r
+  UINTN                         Index;\r
+\r
+  Index = 0;\r
+\r
+  if ((AttemptNameList == NULL) || (*AttemptNameList == L'\0')) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  AttemptStr = AttemptNameList;\r
+  Len = StrLen (L"attempt:");\r
+\r
+  while (*AttemptStr != L'\0') {\r
+    AttemptStr = StrStr (AttemptStr, L"attempt:");\r
+    if (AttemptStr == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    AttemptStr += Len;\r
+    AttemptIndex = (UINT8)(*AttemptStr - L'0');\r
+    AttemptConfigData  = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (IsAddAttempts) {\r
+      if ((AttemptConfigData != NULL) || ((AttemptIndex) > PcdGet8 (PcdMaxIScsiAttemptNumber))) {\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+    } else {\r
+      if (AttemptConfigData == NULL) {\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+    }\r
+\r
+    AttemptIndexList[Index] = AttemptIndex;\r
+    Index ++;\r
+    AttemptStr += 2;\r
+  }\r
+  return EFI_SUCCESS;\r
+}\r
 \r
 /**\r
   Convert the iSCSI configuration data into the IFR data.\r
@@ -367,6 +516,7 @@ IScsiConvertAttemptConfigDataToIfrNvData (
   ISCSI_SESSION_CONFIG_NVDATA   *SessionConfigData;\r
   ISCSI_CHAP_AUTH_CONFIG_NVDATA *AuthConfigData;\r
   EFI_IP_ADDRESS                Ip;\r
+  BOOLEAN                       DnsMode;\r
 \r
   //\r
   // Normal session configuration parameters.\r
@@ -374,6 +524,7 @@ IScsiConvertAttemptConfigDataToIfrNvData (
   SessionConfigData                 = &Attempt->SessionConfigData;\r
   IfrNvData->Enabled                = SessionConfigData->Enabled;\r
   IfrNvData->IpMode                 = SessionConfigData->IpMode;\r
+  DnsMode                           = SessionConfigData->DnsMode;\r
 \r
   IfrNvData->InitiatorInfoFromDhcp  = SessionConfigData->InitiatorInfoFromDhcp;\r
   IfrNvData->TargetInfoFromDhcp     = SessionConfigData->TargetInfoFromDhcp;\r
@@ -386,15 +537,33 @@ IScsiConvertAttemptConfigDataToIfrNvData (
     IScsiIpToStr (&Ip, FALSE, IfrNvData->SubnetMask);\r
     CopyMem (&Ip.v4, &SessionConfigData->Gateway, sizeof (EFI_IPv4_ADDRESS));\r
     IScsiIpToStr (&Ip, FALSE, IfrNvData->Gateway);\r
-    CopyMem (&Ip.v4, &SessionConfigData->TargetIp, sizeof (EFI_IPv4_ADDRESS));\r
-    IScsiIpToStr (&Ip, FALSE, IfrNvData->TargetIp);\r
+    if (SessionConfigData->TargetIp.v4.Addr[0] != '\0') {\r
+      CopyMem (&Ip.v4, &SessionConfigData->TargetIp, sizeof (EFI_IPv4_ADDRESS));\r
+      IScsiIpToStr (&Ip, FALSE, IfrNvData->TargetIp);\r
+    }\r
+\r
   } else if (IfrNvData->IpMode == IP_MODE_IP6) {\r
     ZeroMem (IfrNvData->TargetIp, sizeof (IfrNvData->TargetIp));\r
-    IP6_COPY_ADDRESS (&Ip.v6, &SessionConfigData->TargetIp);\r
-    IScsiIpToStr (&Ip, TRUE, IfrNvData->TargetIp);\r
+    if (SessionConfigData->TargetIp.v6.Addr[0] != '\0') {\r
+      IP6_COPY_ADDRESS (&Ip.v6, &SessionConfigData->TargetIp);\r
+      IScsiIpToStr (&Ip, TRUE, IfrNvData->TargetIp);\r
+    }\r
+  }\r
+\r
+  AsciiStrToUnicodeStrS (\r
+    SessionConfigData->TargetName,\r
+    IfrNvData->TargetName,\r
+    sizeof (IfrNvData->TargetName) / sizeof (IfrNvData->TargetName[0])\r
+    );\r
+\r
+  if (DnsMode) {\r
+    AsciiStrToUnicodeStrS (\r
+      SessionConfigData->TargetUrl,\r
+      IfrNvData->TargetIp,\r
+      sizeof (IfrNvData->TargetIp) / sizeof (IfrNvData->TargetIp[0])\r
+      );\r
   }\r
 \r
-  AsciiStrToUnicodeStr (SessionConfigData->TargetName, IfrNvData->TargetName);\r
   IScsiLunToUnicodeStr (SessionConfigData->BootLun, IfrNvData->BootLun);\r
   IScsiConvertIsIdToString (IfrNvData->IsId, SessionConfigData->IsId);\r
 \r
@@ -409,27 +578,176 @@ IScsiConvertAttemptConfigDataToIfrNvData (
   if (IfrNvData->AuthenticationType == ISCSI_AUTH_TYPE_CHAP) {\r
     AuthConfigData      = &Attempt->AuthConfigData.CHAP;\r
     IfrNvData->CHAPType = AuthConfigData->CHAPType;\r
-    AsciiStrToUnicodeStr (AuthConfigData->CHAPName, IfrNvData->CHAPName);\r
-    AsciiStrToUnicodeStr (AuthConfigData->CHAPSecret, IfrNvData->CHAPSecret);\r
-    AsciiStrToUnicodeStr (AuthConfigData->ReverseCHAPName, IfrNvData->ReverseCHAPName);\r
-    AsciiStrToUnicodeStr (AuthConfigData->ReverseCHAPSecret, IfrNvData->ReverseCHAPSecret);\r
+    AsciiStrToUnicodeStrS (\r
+      AuthConfigData->CHAPName,\r
+      IfrNvData->CHAPName,\r
+      sizeof (IfrNvData->CHAPName) / sizeof (IfrNvData->CHAPName[0])\r
+      );\r
+    AsciiStrToUnicodeStrS (\r
+      AuthConfigData->CHAPSecret,\r
+      IfrNvData->CHAPSecret,\r
+      sizeof (IfrNvData->CHAPSecret) / sizeof (IfrNvData->CHAPSecret[0])\r
+      );\r
+    AsciiStrToUnicodeStrS (\r
+      AuthConfigData->ReverseCHAPName,\r
+      IfrNvData->ReverseCHAPName,\r
+      sizeof (IfrNvData->ReverseCHAPName) / sizeof (IfrNvData->ReverseCHAPName[0])\r
+      );\r
+    AsciiStrToUnicodeStrS (\r
+      AuthConfigData->ReverseCHAPSecret,\r
+      IfrNvData->ReverseCHAPSecret,\r
+      sizeof (IfrNvData->ReverseCHAPSecret) / sizeof (IfrNvData->ReverseCHAPSecret[0])\r
+      );\r
   }\r
 \r
   //\r
   // Other parameters.\r
   //\r
-  AsciiStrToUnicodeStr (Attempt->AttemptName, IfrNvData->AttemptName);\r
+  AsciiStrToUnicodeStrS (\r
+    Attempt->AttemptName,\r
+    IfrNvData->AttemptName,\r
+    sizeof (IfrNvData->AttemptName) / sizeof (IfrNvData->AttemptName[0])\r
+    );\r
 }\r
 \r
+/**\r
+  Convert the iSCSI configuration data into the IFR data Which will be used\r
+  to extract the iSCSI Keyword configuration in <ConfigAltResp> format.\r
+\r
+  @param[in, out]  IfrNvData              The IFR nv data.\r
+\r
+**/\r
+VOID\r
+EFIAPI\r
+IScsiConvertAttemptConfigDataToIfrNvDataByKeyword (\r
+  IN OUT ISCSI_CONFIG_IFR_NVDATA  *IfrNvData\r
+  )\r
+{\r
+  LIST_ENTRY                    *Entry;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA   *Attempt;\r
+  ISCSI_SESSION_CONFIG_NVDATA   *SessionConfigData;\r
+  ISCSI_CHAP_AUTH_CONFIG_NVDATA *AuthConfigData;\r
+  CHAR16                        AttemptNameList[ATTEMPT_NAME_LIST_SIZE];\r
+  EFI_IP_ADDRESS                Ip;\r
+  UINTN                         Index;\r
+\r
+  ZeroMem (AttemptNameList, sizeof (AttemptNameList));\r
+\r
+  NET_LIST_FOR_EACH (Entry, &mPrivate->AttemptConfigs) {\r
+    Attempt = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
+    //\r
+    // Normal session configuration parameters.\r
+    //\r
+    SessionConfigData                 = &Attempt->SessionConfigData;\r
+\r
+    Index   = Attempt->AttemptConfigIndex - 1;\r
+\r
+    //\r
+    // Save the attempt to AttemptNameList as Attempt:1 Attempt:2\r
+    //\r
+    AsciiStrToUnicodeStrS (\r
+      Attempt->AttemptName,\r
+      AttemptNameList + StrLen (AttemptNameList),\r
+      ATTEMPT_NAME_LIST_SIZE\r
+    );\r
+    *(AttemptNameList + StrLen (AttemptNameList) - 2) = L':';\r
+    *(AttemptNameList + StrLen (AttemptNameList))     = L' ';\r
+\r
+    AsciiStrToUnicodeStrS (\r
+      Attempt->AttemptName,\r
+      IfrNvData->ISCSIAttemptName  + ATTEMPT_NAME_SIZE * Index,\r
+      ATTEMPT_NAME_SIZE\r
+    );\r
+\r
+    IfrNvData->ISCSIBootEnableList[Index]          = SessionConfigData->Enabled;\r
+    IfrNvData->ISCSIIpAddressTypeList[Index]       = SessionConfigData->IpMode;\r
+\r
+    IfrNvData->ISCSIInitiatorInfoViaDHCP[Index]    = SessionConfigData->InitiatorInfoFromDhcp;\r
+    IfrNvData->ISCSITargetInfoViaDHCP[Index]       = SessionConfigData->TargetInfoFromDhcp;\r
+    IfrNvData->ISCSIConnectRetry[Index]            = SessionConfigData->ConnectRetryCount;\r
+    IfrNvData->ISCSIConnectTimeout[Index]          = SessionConfigData->ConnectTimeout;\r
+    IfrNvData->ISCSITargetTcpPort[Index]           = SessionConfigData->TargetPort;\r
+\r
+    if (SessionConfigData->IpMode == IP_MODE_IP4) {\r
+      CopyMem (&Ip.v4, &SessionConfigData->LocalIp, sizeof (EFI_IPv4_ADDRESS));\r
+      IScsiIpToStr (&Ip, FALSE, IfrNvData->Keyword[Index].ISCSIInitiatorIpAddress);\r
+      CopyMem (&Ip.v4, &SessionConfigData->SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
+      IScsiIpToStr (&Ip, FALSE, IfrNvData->Keyword[Index].ISCSIInitiatorNetmask);\r
+      CopyMem (&Ip.v4, &SessionConfigData->Gateway, sizeof (EFI_IPv4_ADDRESS));\r
+      IScsiIpToStr (&Ip, FALSE, IfrNvData->Keyword[Index].ISCSIInitiatorGateway);\r
+      if (SessionConfigData->TargetIp.v4.Addr[0] != '\0') {\r
+        CopyMem (&Ip.v4, &SessionConfigData->TargetIp, sizeof (EFI_IPv4_ADDRESS));\r
+        IScsiIpToStr (&Ip, FALSE, IfrNvData->Keyword[Index].ISCSITargetIpAddress);\r
+      }\r
+    } else if (SessionConfigData->IpMode == IP_MODE_IP6) {\r
+      ZeroMem (IfrNvData->Keyword[Index].ISCSITargetIpAddress, sizeof (IfrNvData->TargetIp));\r
+      if (SessionConfigData->TargetIp.v6.Addr[0] != '\0') {\r
+        IP6_COPY_ADDRESS (&Ip.v6, &SessionConfigData->TargetIp);\r
+        IScsiIpToStr (&Ip, TRUE, IfrNvData->Keyword[Index].ISCSITargetIpAddress);\r
+      }\r
+    }\r
+\r
+    AsciiStrToUnicodeStrS (\r
+      SessionConfigData->TargetName,\r
+      IfrNvData->Keyword[Index].ISCSITargetName,\r
+      ISCSI_NAME_MAX_SIZE\r
+      );\r
+\r
+    if (SessionConfigData->DnsMode) {\r
+      AsciiStrToUnicodeStrS (\r
+        SessionConfigData->TargetUrl,\r
+        IfrNvData->TargetIp,\r
+        sizeof (IfrNvData->TargetIp) / sizeof (IfrNvData->TargetIp[0])\r
+        );\r
+    }\r
+\r
+    IScsiLunToUnicodeStr (SessionConfigData->BootLun, IfrNvData->Keyword[Index].ISCSILun);\r
+    IScsiConvertIsIdToString (IfrNvData->Keyword[Index].ISCSIIsId, SessionConfigData->IsId);\r
+\r
+    IfrNvData->ISCSIAuthenticationMethod[Index]    = Attempt->AuthenticationType;\r
+\r
+    if (Attempt->AuthenticationType == ISCSI_AUTH_TYPE_CHAP) {\r
+      AuthConfigData      = &Attempt->AuthConfigData.CHAP;\r
+      IfrNvData->ISCSIChapType[Index] = AuthConfigData->CHAPType;\r
+      AsciiStrToUnicodeStrS (\r
+        AuthConfigData->CHAPName,\r
+        IfrNvData->Keyword[Index].ISCSIChapUsername,\r
+        ISCSI_CHAP_NAME_STORAGE\r
+        );\r
+\r
+      AsciiStrToUnicodeStrS (\r
+        AuthConfigData->CHAPSecret,\r
+        IfrNvData->Keyword[Index].ISCSIChapSecret,\r
+        ISCSI_CHAP_SECRET_STORAGE\r
+        );\r
+\r
+      AsciiStrToUnicodeStrS (\r
+        AuthConfigData->ReverseCHAPName,\r
+        IfrNvData->Keyword[Index].ISCSIReverseChapUsername,\r
+        ISCSI_CHAP_NAME_STORAGE\r
+        );\r
+\r
+      AsciiStrToUnicodeStrS (\r
+        AuthConfigData->ReverseCHAPSecret,\r
+        IfrNvData->Keyword[Index].ISCSIReverseChapSecret,\r
+        ISCSI_CHAP_SECRET_STORAGE\r
+        );\r
+    }\r
+  }\r
+\r
+  CopyMem(IfrNvData->ISCSIDisplayAttemptList, AttemptNameList, ATTEMPT_NAME_LIST_SIZE);\r
+}\r
 \r
 /**\r
   Convert the IFR data to iSCSI configuration data.\r
 \r
-  @param[in]       IfrNvData              The IFR nv data.\r
+  @param[in]       IfrNvData              Point to ISCSI_CONFIG_IFR_NVDATA.\r
   @param[in, out]  Attempt                The iSCSI attempt config data.\r
 \r
   @retval EFI_INVALID_PARAMETER  Any input or configured parameter is invalid.\r
   @retval EFI_NOT_FOUND          Cannot find the corresponding variable.\r
+  @retval EFI_OUT_OF_RESOURCES   The operation is failed due to lack of resources.\r
+  @retval EFI_ABORTED            The operation is aborted.\r
   @retval EFI_SUCCESS            The operation is completed successfully.\r
 \r
 **/\r
@@ -451,6 +769,11 @@ IScsiConvertIfrNvDataToAttemptConfigData (
   CHAR16                      IpMode[64];\r
   ISCSI_NIC_INFO              *NicInfo;\r
   EFI_INPUT_KEY               Key;\r
+  UINT8                       *AttemptConfigOrder;\r
+  UINTN                       AttemptConfigOrderSize;\r
+  UINT8                       *AttemptOrderTmp;\r
+  UINTN                       TotalNumber;\r
+  EFI_STATUS                  Status;\r
 \r
   if (IfrNvData == NULL || Attempt == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -491,7 +814,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
         L"Connection Establishing Timeout is less than minimum value 100ms.",\r
         NULL\r
         );\r
-      \r
+\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
@@ -512,7 +835,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
             L"Gateway address is set but subnet mask is zero.",\r
             NULL\r
             );\r
-          \r
+\r
           return EFI_INVALID_PARAMETER;\r
         } else if (!IP4_NET_EQUAL (HostIp.Addr[0], Gateway.Addr[0], SubnetMask.Addr[0])) {\r
           CreatePopUp (\r
@@ -521,7 +844,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
             L"Local IP and Gateway are not in the same subnet.",\r
             NULL\r
             );\r
-          \r
+\r
           return EFI_INVALID_PARAMETER;\r
         }\r
       }\r
@@ -530,16 +853,44 @@ IScsiConvertIfrNvDataToAttemptConfigData (
     // Validate target configuration if DHCP isn't deployed.\r
     //\r
     if (!Attempt->SessionConfigData.TargetInfoFromDhcp && Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) {\r
-      if (!IpIsUnicast (&Attempt->SessionConfigData.TargetIp, IfrNvData->IpMode)) {\r
+      if (!Attempt->SessionConfigData.DnsMode) {\r
+        if (!IpIsUnicast (&Attempt->SessionConfigData.TargetIp, IfrNvData->IpMode)) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Target IP is invalid!",\r
+            NULL\r
+            );\r
+          return EFI_INVALID_PARAMETER;\r
+        }\r
+      } else {\r
+        if (Attempt->SessionConfigData.TargetUrl[0] == '\0') {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"iSCSI target Url should not be NULL!",\r
+            NULL\r
+            );\r
+          return EFI_INVALID_PARAMETER;\r
+        }\r
+      }\r
+\r
+      //\r
+      // Validate iSCSI target name configuration again:\r
+      // The format of iSCSI target name is already verified in IScsiFormCallback() when\r
+      // user input the name; here we only check the case user does not input the name.\r
+      //\r
+      if (Attempt->SessionConfigData.TargetName[0] == '\0') {\r
         CreatePopUp (\r
           EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
           &Key,\r
-          L"Target IP is invalid!",\r
+          L"iSCSI target name is NULL!",\r
           NULL\r
           );\r
         return EFI_INVALID_PARAMETER;\r
       }\r
     }\r
+\r
     //\r
     // Validate the authentication info.\r
     //\r
@@ -573,26 +924,19 @@ IScsiConvertIfrNvDataToAttemptConfigData (
     //\r
     SameNicAttempt = IScsiConfigGetAttemptByNic (Attempt, IfrNvData->Enabled);\r
     if (SameNicAttempt != NULL) {\r
-      AttemptName1 = (CHAR16 *) AllocateZeroPool (ATTEMPT_NAME_MAX_SIZE * sizeof (CHAR16));\r
+      AttemptName1 = (CHAR16 *) AllocateZeroPool (ATTEMPT_NAME_SIZE * sizeof (CHAR16));\r
       if (AttemptName1 == NULL) {\r
         return EFI_OUT_OF_RESOURCES;\r
       }\r
 \r
-      AttemptName2 = (CHAR16 *) AllocateZeroPool (ATTEMPT_NAME_MAX_SIZE * sizeof (CHAR16));\r
+      AttemptName2 = (CHAR16 *) AllocateZeroPool (ATTEMPT_NAME_SIZE * sizeof (CHAR16));\r
       if (AttemptName2 == NULL) {\r
         FreePool (AttemptName1);\r
         return EFI_OUT_OF_RESOURCES;\r
       }      \r
-      \r
-      AsciiStrToUnicodeStr (Attempt->AttemptName, AttemptName1);\r
-      if (StrLen (AttemptName1) > ATTEMPT_NAME_SIZE) {\r
-        CopyMem (&AttemptName1[ATTEMPT_NAME_SIZE], L"...", 4 * sizeof (CHAR16));\r
-      }\r
 \r
-      AsciiStrToUnicodeStr (SameNicAttempt->AttemptName, AttemptName2);\r
-      if (StrLen (AttemptName2) > ATTEMPT_NAME_SIZE) {\r
-        CopyMem (&AttemptName2[ATTEMPT_NAME_SIZE], L"...", 4 * sizeof (CHAR16));\r
-      }\r
+      AsciiStrToUnicodeStrS (Attempt->AttemptName, AttemptName1, ATTEMPT_NAME_SIZE);\r
+      AsciiStrToUnicodeStrS (SameNicAttempt->AttemptName, AttemptName2, ATTEMPT_NAME_SIZE);\r
 \r
       UnicodeSPrint (\r
         mPrivate->PortString,\r
@@ -614,6 +958,60 @@ IScsiConvertIfrNvDataToAttemptConfigData (
     }\r
   }\r
 \r
+  //\r
+  // Update the iSCSI Mode data and record it in attempt help info.\r
+  //\r
+  if (IfrNvData->Enabled == ISCSI_DISABLED) {\r
+    UnicodeSPrint (IScsiMode, 64, L"Disabled");\r
+  } else if (IfrNvData->Enabled == ISCSI_ENABLED) {\r
+    UnicodeSPrint (IScsiMode, 64, L"Enabled");\r
+  } else if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
+    UnicodeSPrint (IScsiMode, 64, L"Enabled for MPIO");\r
+  }\r
+\r
+  if (IfrNvData->IpMode == IP_MODE_IP4) {\r
+    UnicodeSPrint (IpMode, 64, L"IP4");\r
+  } else if (IfrNvData->IpMode == IP_MODE_IP6) {\r
+    UnicodeSPrint (IpMode, 64, L"IP6");\r
+  } else if (IfrNvData->IpMode == IP_MODE_AUTOCONFIG) {\r
+    UnicodeSPrint (IpMode, 64, L"Autoconfigure");\r
+  }\r
+\r
+  NicInfo = IScsiGetNicInfoByIndex (Attempt->NicIndex);\r
+  if (NicInfo == NULL) {\r
+    return EFI_NOT_FOUND;\r
+  }\r
+\r
+  MacString = (CHAR16 *) AllocateZeroPool (ISCSI_MAX_MAC_STRING_LEN * sizeof (CHAR16));\r
+  if (MacString == NULL) {\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+\r
+  AsciiStrToUnicodeStrS (Attempt->MacString, MacString, ISCSI_MAX_MAC_STRING_LEN);\r
+\r
+  UnicodeSPrint (\r
+    mPrivate->PortString,\r
+    (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+    L"MAC: %s, PFA: Bus %d | Dev %d | Func %d, iSCSI mode: %s, IP version: %s",\r
+    MacString,\r
+    NicInfo->BusNumber,\r
+    NicInfo->DeviceNumber,\r
+    NicInfo->FunctionNumber,\r
+    IScsiMode,\r
+    IpMode\r
+    );\r
+\r
+  Attempt->AttemptTitleHelpToken = HiiSetString (\r
+                                     mCallbackInfo->RegisteredHandle,\r
+                                     Attempt->AttemptTitleHelpToken,\r
+                                     mPrivate->PortString,\r
+                                     NULL\r
+                                     );\r
+  if (Attempt->AttemptTitleHelpToken == 0) {\r
+    FreePool (MacString);\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+\r
   //\r
   // Check whether this attempt is an existing one.\r
   //\r
@@ -683,86 +1081,655 @@ IScsiConvertIfrNvDataToAttemptConfigData (
       }\r
     }\r
 \r
-  } else if (ExistAttempt == NULL && IfrNvData->Enabled != ISCSI_DISABLED) {\r
-    if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
-      //\r
-      // This new Attempt is enabled for MPIO; enable the multipath mode.\r
-      //\r
-      mPrivate->EnableMpio = TRUE;\r
-      mPrivate->MpioCount++;\r
-    } else if (IfrNvData->Enabled == ISCSI_ENABLED) {\r
-      mPrivate->SinglePathCount++;\r
-    }\r
-  }\r
+  } else if (ExistAttempt == NULL) {\r
+    //\r
+    // When a new attempt is created, pointer of the attempt is saved to\r
+    // mCallbackInfo->Current in IScsiConfigProcessDefault. If input Attempt\r
+    // does not match any existing attempt, it should be a new created attempt.\r
+    // Save it to system now.\r
+    //\r
 \r
-  //\r
-  // Update the iSCSI Mode data and record it in attempt help info.\r
-  //\r
-  Attempt->SessionConfigData.Enabled = IfrNvData->Enabled;\r
-  if (IfrNvData->Enabled == ISCSI_DISABLED) {\r
-    UnicodeSPrint (IScsiMode, 64, L"Disabled");\r
-  } else if (IfrNvData->Enabled == ISCSI_ENABLED) {\r
-    UnicodeSPrint (IScsiMode, 64, L"Enabled");\r
-  } else if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
-    UnicodeSPrint (IScsiMode, 64, L"Enabled for MPIO");\r
-  }\r
+    //\r
+    // Save current order number for this attempt.\r
+    //\r
+    AttemptConfigOrder = IScsiGetVariableAndSize (\r
+                           L"AttemptOrder",\r
+                           &gIScsiConfigGuid,\r
+                           &AttemptConfigOrderSize\r
+                           );\r
 \r
-  if (IfrNvData->IpMode == IP_MODE_IP4) {\r
-    UnicodeSPrint (IpMode, 64, L"IP4");\r
-  } else if (IfrNvData->IpMode == IP_MODE_IP6) {\r
-    UnicodeSPrint (IpMode, 64, L"IP6");\r
-  } else if (IfrNvData->IpMode == IP_MODE_AUTOCONFIG) {\r
-    UnicodeSPrint (IpMode, 64, L"Autoconfigure");\r
-  }\r
+    TotalNumber = AttemptConfigOrderSize / sizeof (UINT8);\r
+    TotalNumber++;\r
 \r
-  NicInfo = IScsiGetNicInfoByIndex (Attempt->NicIndex);\r
-  if (NicInfo == NULL) {\r
-    return EFI_NOT_FOUND;\r
+    //\r
+    // Append the new created attempt order to the end.\r
+    //\r
+    AttemptOrderTmp = AllocateZeroPool (TotalNumber * sizeof (UINT8));\r
+    if (AttemptOrderTmp == NULL) {\r
+      if (AttemptConfigOrder != NULL) {\r
+        FreePool (AttemptConfigOrder);\r
+      }\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
+\r
+    if (AttemptConfigOrder != NULL) {\r
+      CopyMem (AttemptOrderTmp, AttemptConfigOrder, AttemptConfigOrderSize);\r
+      FreePool (AttemptConfigOrder);\r
+    }\r
+\r
+    AttemptOrderTmp[TotalNumber - 1] = Attempt->AttemptConfigIndex;\r
+    AttemptConfigOrder               = AttemptOrderTmp;\r
+    AttemptConfigOrderSize           = TotalNumber * sizeof (UINT8);\r
+\r
+    Status = gRT->SetVariable (\r
+                    L"AttemptOrder",\r
+                    &gIScsiConfigGuid,\r
+                    EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE,\r
+                    AttemptConfigOrderSize,\r
+                    AttemptConfigOrder\r
+                    );\r
+    FreePool (AttemptConfigOrder);\r
+    if (EFI_ERROR (Status)) {\r
+      return Status;\r
+    }\r
+\r
+    //\r
+    // Insert new created attempt to array.\r
+    //\r
+    InsertTailList (&mPrivate->AttemptConfigs, &Attempt->Link);\r
+    mPrivate->AttemptCount++;\r
+\r
+    if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
+      //\r
+      // This new Attempt is enabled for MPIO; enable the multipath mode.\r
+      //\r
+      mPrivate->EnableMpio = TRUE;\r
+      mPrivate->MpioCount++;\r
+    } else if (IfrNvData->Enabled == ISCSI_ENABLED) {\r
+      mPrivate->SinglePathCount++;\r
+    }\r
+\r
+    IScsiConfigUpdateAttempt ();\r
   }\r
+  Attempt->SessionConfigData.Enabled = IfrNvData->Enabled;\r
 \r
-  MacString = (CHAR16 *) AllocateZeroPool (ISCSI_MAX_MAC_STRING_LEN * sizeof (CHAR16));\r
-  if (MacString == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
+  //\r
+  // Record the user configuration information in NVR.\r
+  //\r
+  UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"Attempt %d", Attempt->AttemptConfigIndex);\r
+\r
+  FreePool (MacString);\r
+\r
+  return gRT->SetVariable (\r
+                mPrivate->PortString,\r
+                &gEfiIScsiInitiatorNameProtocolGuid,\r
+                ISCSI_CONFIG_VAR_ATTR,\r
+                sizeof (ISCSI_ATTEMPT_CONFIG_NVDATA),\r
+                Attempt\r
+                );\r
+}\r
+\r
+/**\r
+  Convert the IFR data configured by keyword to iSCSI configuration data.\r
+\r
+  @param[in]  IfrNvData      Point to ISCSI_CONFIG_IFR_NVDATA.\r
+  @param[in]  OffSet         The offset of the variable to the configuration structure.\r
+\r
+  @retval EFI_INVALID_PARAMETER  Any input or configured parameter is invalid.\r
+  @retval EFI_SUCCESS            The operation is completed successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+IScsiConvertlfrNvDataToAttemptConfigDataByKeyword (\r
+  IN ISCSI_CONFIG_IFR_NVDATA          *IfrNvData,\r
+  IN UINTN                             OffSet\r
+  )\r
+{\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA      *Attempt;\r
+  UINT8                            AttemptIndex;\r
+  UINT8                            Index;\r
+  CHAR16                           *AttemptName1;\r
+  CHAR16                           *AttemptName2;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA      *SameNicAttempt;\r
+  CHAR8                            LunString[ISCSI_LUN_STR_MAX_LEN];\r
+  CHAR8                            IScsiName[ISCSI_NAME_MAX_SIZE];\r
+  CHAR8                            IpString[IP_STR_MAX_SIZE];\r
+  EFI_IP_ADDRESS                   HostIp;\r
+  EFI_IP_ADDRESS                   SubnetMask;\r
+  EFI_IP_ADDRESS                   Gateway;\r
+  EFI_INPUT_KEY                    Key;\r
+  UINT64                           Lun;\r
+  EFI_STATUS                       Status;\r
+\r
+  ZeroMem (IScsiName, sizeof (IScsiName));\r
+\r
+  if (OffSet < ATTEMPT_BOOTENABLE_VAR_OFFSET) {\r
+    return EFI_SUCCESS;\r
+\r
+  } else if ((OffSet >= ATTEMPT_BOOTENABLE_VAR_OFFSET) && (OffSet < ATTEMPT_ADDRESS_TYPE_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_BOOTENABLE_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    IfrNvData->Enabled = IfrNvData->ISCSIBootEnableList[AttemptIndex - 1];\r
+    //\r
+    // Validate the configuration of attempt.\r
+    //\r
+    if (IfrNvData->Enabled != ISCSI_DISABLED) {\r
+      //\r
+      // Check whether this attempt uses NIC which is already used by existing attempt.\r
+      //\r
+      SameNicAttempt = IScsiConfigGetAttemptByNic (Attempt, IfrNvData->Enabled);\r
+      if (SameNicAttempt != NULL) {\r
+        AttemptName1 = (CHAR16 *) AllocateZeroPool (ATTEMPT_NAME_SIZE * sizeof (CHAR16));\r
+        if (AttemptName1 == NULL) {\r
+          return EFI_OUT_OF_RESOURCES;\r
+        }\r
+\r
+        AttemptName2 = (CHAR16 *) AllocateZeroPool (ATTEMPT_NAME_SIZE * sizeof (CHAR16));\r
+        if (AttemptName2 == NULL) {\r
+          FreePool (AttemptName1);\r
+          return EFI_OUT_OF_RESOURCES;\r
+        }\r
+\r
+        AsciiStrToUnicodeStrS (Attempt->AttemptName, AttemptName1, ATTEMPT_NAME_SIZE);\r
+        AsciiStrToUnicodeStrS (SameNicAttempt->AttemptName, AttemptName2, ATTEMPT_NAME_SIZE);\r
+\r
+        UnicodeSPrint (\r
+          mPrivate->PortString,\r
+          (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+          L"Warning! \"%s\" uses same NIC as Attempt \"%s\".",\r
+          AttemptName1,\r
+          AttemptName2\r
+          );\r
+\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          mPrivate->PortString,\r
+          NULL\r
+          );\r
+\r
+        FreePool (AttemptName1);\r
+        FreePool (AttemptName2);\r
+      }\r
+    }\r
+\r
+    if (IfrNvData->Enabled == ISCSI_DISABLED &&\r
+        Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+\r
+      //\r
+      // User updates the Attempt from "Enabled"/"Enabled for MPIO" to "Disabled".\r
+      //\r
+      if (Attempt->SessionConfigData.Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
+        if (mPrivate->MpioCount < 1) {\r
+          return EFI_ABORTED;\r
+        }\r
+\r
+        if (--mPrivate->MpioCount == 0) {\r
+          mPrivate->EnableMpio = FALSE;\r
+        }\r
+      } else if (Attempt->SessionConfigData.Enabled == ISCSI_ENABLED) {\r
+        if (mPrivate->SinglePathCount < 1) {\r
+          return EFI_ABORTED;\r
+        }\r
+        mPrivate->SinglePathCount--;\r
+      }\r
+\r
+    } else if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO &&\r
+               Attempt->SessionConfigData.Enabled == ISCSI_ENABLED) {\r
+      //\r
+      // User updates the Attempt from "Enabled" to "Enabled for MPIO".\r
+      //\r
+      if (mPrivate->SinglePathCount < 1) {\r
+        return EFI_ABORTED;\r
+      }\r
+\r
+      mPrivate->EnableMpio = TRUE;\r
+      mPrivate->MpioCount++;\r
+      mPrivate->SinglePathCount--;\r
+\r
+    } else if (IfrNvData->Enabled == ISCSI_ENABLED &&\r
+               Attempt->SessionConfigData.Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
+      //\r
+      // User updates the Attempt from "Enabled for MPIO" to "Enabled".\r
+      //\r
+      if (mPrivate->MpioCount < 1) {\r
+        return EFI_ABORTED;\r
+      }\r
+\r
+      if (--mPrivate->MpioCount == 0) {\r
+        mPrivate->EnableMpio = FALSE;\r
+      }\r
+      mPrivate->SinglePathCount++;\r
+\r
+    } else if (IfrNvData->Enabled != ISCSI_DISABLED &&\r
+               Attempt->SessionConfigData.Enabled == ISCSI_DISABLED) {\r
+      //\r
+      // User updates the Attempt from "Disabled" to "Enabled"/"Enabled for MPIO".\r
+      //\r
+      if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
+        mPrivate->EnableMpio = TRUE;\r
+        mPrivate->MpioCount++;\r
+\r
+      } else if (IfrNvData->Enabled == ISCSI_ENABLED) {\r
+        mPrivate->SinglePathCount++;\r
+      }\r
+    }\r
+    Attempt->SessionConfigData.Enabled = IfrNvData->Enabled;\r
+\r
+  } else if ((OffSet >= ATTEMPT_ADDRESS_TYPE_VAR_OFFSET) && (OffSet < ATTEMPT_CONNECT_RETRY_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_ADDRESS_TYPE_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    Attempt->SessionConfigData.IpMode = IfrNvData->ISCSIIpAddressTypeList[AttemptIndex - 1];\r
+    if (Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) {\r
+      Attempt->AutoConfigureMode = 0;\r
+    }\r
+\r
+  } else if ((OffSet >= ATTEMPT_CONNECT_RETRY_VAR_OFFSET) && (OffSet < ATTEMPT_CONNECT_TIMEOUT_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_CONNECT_RETRY_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    Attempt->SessionConfigData.ConnectRetryCount = IfrNvData->ISCSIConnectRetry[AttemptIndex - 1];\r
+    if (Attempt->SessionConfigData.ConnectRetryCount == 0) {\r
+      Attempt->SessionConfigData.ConnectRetryCount = CONNECT_MIN_RETRY;\r
+    }\r
+\r
+  } else if ((OffSet >= ATTEMPT_CONNECT_TIMEOUT_VAR_OFFSET) && (OffSet < ATTEMPT_INITIATOR_VIA_DHCP_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_CONNECT_TIMEOUT_VAR_OFFSET) / 2 + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    Attempt->SessionConfigData.ConnectTimeout = IfrNvData->ISCSIConnectTimeout[AttemptIndex - 1];\r
+    if (Attempt->SessionConfigData.ConnectTimeout == 0) {\r
+      Attempt->SessionConfigData.ConnectTimeout = CONNECT_DEFAULT_TIMEOUT;\r
+    }\r
+\r
+    if (Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+      if (Attempt->SessionConfigData.ConnectTimeout < CONNECT_MIN_TIMEOUT) {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Connection Establishing Timeout is less than minimum value 100ms.",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+    }\r
+\r
+  } else if ((OffSet >= ATTEMPT_INITIATOR_VIA_DHCP_VAR_OFFSET) && (OffSet < ATTEMPT_TARGET_VIA_DHCP_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_INITIATOR_VIA_DHCP_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    Attempt->SessionConfigData.InitiatorInfoFromDhcp = IfrNvData->ISCSIInitiatorInfoViaDHCP[AttemptIndex - 1];\r
+\r
+  } else if ((OffSet >= ATTEMPT_TARGET_VIA_DHCP_VAR_OFFSET) && (OffSet < ATTEMPT_TARGET_TCP_PORT_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_TARGET_VIA_DHCP_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+    if ((Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) && (Attempt->SessionConfigData.InitiatorInfoFromDhcp)) {\r
+      Attempt->SessionConfigData.TargetInfoFromDhcp = IfrNvData->ISCSITargetInfoViaDHCP[AttemptIndex - 1];\r
+    } else {\r
+      CreatePopUp (\r
+        EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+        &Key,\r
+        L"Invalid Configuration, Check value of IpMode or Enable DHCP!",\r
+        NULL\r
+        );\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+  } else if ((OffSet >= ATTEMPT_TARGET_TCP_PORT_VAR_OFFSET) && (OffSet < ATTEMPT_AUTHENTICATION_METHOD_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_TARGET_TCP_PORT_VAR_OFFSET) / 2 + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    if ((Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) && (!Attempt->SessionConfigData.TargetInfoFromDhcp)) {\r
+      Attempt->SessionConfigData.TargetPort = IfrNvData->ISCSITargetTcpPort[AttemptIndex - 1];\r
+      if (Attempt->SessionConfigData.TargetPort == 0) {\r
+        Attempt->SessionConfigData.TargetPort = ISCSI_WELL_KNOWN_PORT;\r
+      }\r
+    } else {\r
+      CreatePopUp (\r
+        EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+        &Key,\r
+        L"Invalid Configuration, Check value of IpMode or Target Via DHCP!",\r
+        NULL\r
+        );\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+  } else if ((OffSet >= ATTEMPT_AUTHENTICATION_METHOD_VAR_OFFSET) && (OffSet < ATTEMPT_CHARTYPE_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_AUTHENTICATION_METHOD_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+    Attempt->AuthenticationType = IfrNvData->ISCSIAuthenticationMethod[AttemptIndex - 1];\r
+\r
+  } else if ((OffSet >= ATTEMPT_CHARTYPE_VAR_OFFSET) && (OffSet < ATTEMPT_ISID_VAR_OFFSET)) {\r
+    AttemptIndex = (UINT8) ((OffSet - ATTEMPT_CHARTYPE_VAR_OFFSET) + 1);\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    if (Attempt->AuthenticationType == ISCSI_AUTH_TYPE_CHAP) {\r
+      Attempt->AuthConfigData.CHAP.CHAPType = IfrNvData->ISCSIChapType[AttemptIndex - 1];\r
+    }\r
+\r
+  } else if (OffSet >= ATTEMPT_ISID_VAR_OFFSET) {\r
+    Index = (UINT8) ((OffSet - ATTEMPT_ISID_VAR_OFFSET) / sizeof (KEYWORD_STR));\r
+    AttemptIndex = Index + 1;\r
+    Attempt = IScsiConfigGetAttemptByConfigIndex (AttemptIndex);\r
+    if (Attempt == NULL) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+    OffSet = OffSet - Index * sizeof (KEYWORD_STR);\r
+\r
+    if ((OffSet >= ATTEMPT_ISID_VAR_OFFSET) && (OffSet < ATTEMPT_INITIATOR_IP_ADDRESS_VAR_OFFSET)) {\r
+      IScsiParseIsIdFromString (IfrNvData->Keyword[Index].ISCSIIsId, Attempt->SessionConfigData.IsId);\r
+\r
+    }  else if ((OffSet >= ATTEMPT_INITIATOR_IP_ADDRESS_VAR_OFFSET) && (OffSet < ATTEMPT_INITIATOR_NET_MASK_VAR_OFFSET)) {\r
+      if ((Attempt->SessionConfigData.IpMode == IP_MODE_IP4) && (!Attempt->SessionConfigData.InitiatorInfoFromDhcp)) {\r
+        //\r
+        // Config Local ip\r
+        //\r
+        Status = NetLibStrToIp4 (IfrNvData->Keyword[Index].ISCSIInitiatorIpAddress, &HostIp.v4);\r
+        if (EFI_ERROR (Status) || ((Attempt->SessionConfigData.SubnetMask.Addr[0] != 0) &&\r
+             !NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), NTOHL(*(UINT32*)Attempt->SessionConfigData.SubnetMask.Addr)))) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Invalid IP address!",\r
+            NULL\r
+            );\r
+          return EFI_INVALID_PARAMETER;\r
+        } else {\r
+          CopyMem (&Attempt->SessionConfigData.LocalIp, &HostIp.v4, sizeof (HostIp.v4));\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of IpMode or Enable DHCP!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_INITIATOR_NET_MASK_VAR_OFFSET) && (OffSet < ATTEMPT_INITIATOR_GATE_WAY_VAR_OFFSET)) {\r
+      if ((Attempt->SessionConfigData.IpMode == IP_MODE_IP4) && (!Attempt->SessionConfigData.InitiatorInfoFromDhcp)) {\r
+        Status = NetLibStrToIp4 (IfrNvData->Keyword[Index].ISCSIInitiatorNetmask, &SubnetMask.v4);\r
+        if (EFI_ERROR (Status) || ((SubnetMask.Addr[0] != 0) && (IScsiGetSubnetMaskPrefixLength (&SubnetMask.v4) == 0))) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Invalid Subnet Mask!",\r
+            NULL\r
+            );\r
+          return EFI_INVALID_PARAMETER;\r
+        } else {\r
+          CopyMem (&Attempt->SessionConfigData.SubnetMask, &SubnetMask.v4, sizeof (SubnetMask.v4));\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of IpMode or Enable DHCP!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_INITIATOR_GATE_WAY_VAR_OFFSET) && (OffSet < ATTEMPT_TARGET_NAME_VAR_OFFSET)) {\r
+      if ((Attempt->SessionConfigData.IpMode == IP_MODE_IP4) && (!Attempt->SessionConfigData.InitiatorInfoFromDhcp)) {\r
+        Status = NetLibStrToIp4 (IfrNvData->Keyword[Index].ISCSIInitiatorGateway, &Gateway.v4);\r
+        if (EFI_ERROR (Status) ||\r
+          ((Gateway.Addr[0] != 0) && (Attempt->SessionConfigData.SubnetMask.Addr[0] != 0) &&\r
+             !NetIp4IsUnicast (NTOHL (Gateway.Addr[0]), NTOHL(*(UINT32*)Attempt->SessionConfigData.SubnetMask.Addr)))) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Invalid Gateway!",\r
+            NULL\r
+            );\r
+          return EFI_INVALID_PARAMETER;\r
+        } else {\r
+          CopyMem (&Attempt->SessionConfigData.Gateway, &Gateway.v4, sizeof (Gateway.v4));\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of IpMode or Enable DHCP!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_TARGET_NAME_VAR_OFFSET) && (OffSet < ATTEMPT_TARGET_IP_ADDRESS_VAR_OFFSET)) {\r
+      if ((Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) && (!Attempt->SessionConfigData.TargetInfoFromDhcp)) {\r
+        UnicodeStrToAsciiStrS (IfrNvData->Keyword[Index].ISCSITargetName, IScsiName, ISCSI_NAME_MAX_SIZE);\r
+        Status = IScsiNormalizeName (IScsiName, AsciiStrLen (IScsiName));\r
+        if (EFI_ERROR (Status)) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Invalid iSCSI Name!",\r
+            NULL\r
+            );\r
+        } else {\r
+          AsciiStrCpyS (Attempt->SessionConfigData.TargetName, ISCSI_NAME_MAX_SIZE, IScsiName);\r
+        }\r
+        if (Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+          if (Attempt->SessionConfigData.TargetName[0] == L'\0') {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"iSCSI target name is NULL!",\r
+              NULL\r
+              );\r
+            return EFI_INVALID_PARAMETER;\r
+          }\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of IpMode or Target Via DHCP!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_TARGET_IP_ADDRESS_VAR_OFFSET) && (OffSet < ATTEMPT_LUN_VAR_OFFSET)) {\r
+      if ((Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) && (!Attempt->SessionConfigData.TargetInfoFromDhcp)) {\r
+        UnicodeStrToAsciiStrS (IfrNvData->Keyword[Index].ISCSITargetIpAddress, IpString, sizeof (IpString));\r
+        Status = IScsiAsciiStrToIp (IpString, Attempt->SessionConfigData.IpMode, &HostIp);\r
+        if (EFI_ERROR (Status) || !IpIsUnicast (&HostIp, Attempt->SessionConfigData.IpMode)) {\r
+          Attempt->SessionConfigData.DnsMode = TRUE;\r
+          ZeroMem (&Attempt->SessionConfigData.TargetIp, sizeof (Attempt->SessionConfigData.TargetIp));\r
+          UnicodeStrToAsciiStrS (IfrNvData->Keyword[Index].ISCSITargetIpAddress, Attempt->SessionConfigData.TargetUrl, ISCSI_NAME_MAX_SIZE);\r
+        } else {\r
+          Attempt->SessionConfigData.DnsMode = FALSE;\r
+          CopyMem (&Attempt->SessionConfigData.TargetIp, &HostIp, sizeof (HostIp));\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of IpMode or Target Via DHCP!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_LUN_VAR_OFFSET) && (OffSet < ATTEMPT_CHAR_USER_NAME_VAR_OFFSET)) {\r
+      if ((Attempt->SessionConfigData.IpMode < IP_MODE_AUTOCONFIG) && (Attempt->SessionConfigData.TargetInfoFromDhcp == 0)) {\r
+        //\r
+        // Config LUN.\r
+        //\r
+        UnicodeStrToAsciiStrS (IfrNvData->Keyword[Index].ISCSILun, LunString, ISCSI_LUN_STR_MAX_LEN);\r
+        Status = IScsiAsciiStrToLun (LunString, (UINT8 *) &Lun);\r
+        if (EFI_ERROR (Status)) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Invalid LUN string, Examples are: 4752-3A4F-6b7e-2F99, 6734-9-156f-127, 4186-9!",\r
+            NULL\r
+            );\r
+        } else {\r
+          CopyMem (&Attempt->SessionConfigData.BootLun, &Lun, sizeof (Lun));\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of IpMode or Target Via DHCP!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_CHAR_USER_NAME_VAR_OFFSET) && (OffSet < ATTEMPT_CHAR_SECRET_VAR_OFFSET)) {\r
+      if (Attempt->AuthenticationType == ISCSI_AUTH_TYPE_CHAP) {\r
+        UnicodeStrToAsciiStrS (\r
+          IfrNvData->Keyword[Index].ISCSIChapUsername,\r
+          Attempt->AuthConfigData.CHAP.CHAPName,\r
+          ISCSI_CHAP_NAME_STORAGE\r
+          );\r
+\r
+        if (Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+          if (IfrNvData->CHAPName[0] == L'\0') {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"CHAP Name is invalid!",\r
+              NULL\r
+              );\r
+            return EFI_INVALID_PARAMETER;\r
+          }\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of AuthenticationType!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_CHAR_SECRET_VAR_OFFSET) && (OffSet < ATTEMPT_CHAR_REVERSE_USER_NAME_VAR_OFFSET)) {\r
+      if (Attempt->AuthenticationType == ISCSI_AUTH_TYPE_CHAP) {\r
+        UnicodeStrToAsciiStrS (\r
+          IfrNvData->Keyword[Index].ISCSIChapSecret,\r
+          Attempt->AuthConfigData.CHAP.CHAPSecret,\r
+          ISCSI_CHAP_SECRET_STORAGE\r
+          );\r
+\r
+        if (Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+          if (IfrNvData->CHAPSecret[0] == L'\0') {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"CHAP Secret is invalid!",\r
+              NULL\r
+              );\r
+            return EFI_INVALID_PARAMETER;\r
+          }\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of AuthenticationType!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if ((OffSet >= ATTEMPT_CHAR_REVERSE_USER_NAME_VAR_OFFSET) && (OffSet < ATTEMPT_CHAR_REVERSE_SECRET_VAR_OFFSET)) {\r
+      if (Attempt->AuthConfigData.CHAP.CHAPType == ISCSI_CHAP_MUTUAL) {\r
+        UnicodeStrToAsciiStrS (\r
+          IfrNvData->Keyword[Index].ISCSIReverseChapUsername,\r
+          Attempt->AuthConfigData.CHAP.ReverseCHAPName,\r
+          ISCSI_CHAP_NAME_STORAGE\r
+          );\r
+        if (Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+          if (IfrNvData->ReverseCHAPName[0] == L'\0') {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"Reverse CHAP Name is invalid!",\r
+              NULL\r
+              );\r
+            return EFI_INVALID_PARAMETER;\r
+          }\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of AuthenticationType or Chap Type!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+\r
+    } else if (OffSet >= ATTEMPT_CHAR_REVERSE_SECRET_VAR_OFFSET) {\r
+      if (Attempt->AuthConfigData.CHAP.CHAPType == ISCSI_CHAP_MUTUAL) {\r
+        UnicodeStrToAsciiStrS (\r
+          IfrNvData->Keyword[Index].ISCSIReverseChapSecret,\r
+          Attempt->AuthConfigData.CHAP.ReverseCHAPSecret,\r
+          ISCSI_CHAP_SECRET_STORAGE\r
+          );\r
+\r
+        if (Attempt->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+          if (IfrNvData->ReverseCHAPSecret[0] == L'\0') {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"Reverse CHAP Secret is invalid!",\r
+              NULL\r
+              );\r
+            return EFI_INVALID_PARAMETER;\r
+          }\r
+        }\r
+      } else {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid Configuration, Check value of AuthenticationType or Chap Type!",\r
+          NULL\r
+          );\r
+        return EFI_INVALID_PARAMETER;\r
+      }\r
+    }\r
   }\r
 \r
-  AsciiStrToUnicodeStr (Attempt->MacString, MacString);\r
-\r
-  UnicodeSPrint (\r
-    mPrivate->PortString,\r
-    (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
-    L"MAC: %s, PFA: Bus %d | Dev %d | Func %d, iSCSI mode: %s, IP version: %s",\r
-    MacString,\r
-    NicInfo->BusNumber,\r
-    NicInfo->DeviceNumber,\r
-    NicInfo->FunctionNumber,\r
-    IScsiMode,\r
-    IpMode\r
-    );\r
 \r
-  Attempt->AttemptTitleHelpToken = HiiSetString (\r
-                                     mCallbackInfo->RegisteredHandle,\r
-                                     Attempt->AttemptTitleHelpToken,\r
-                                     mPrivate->PortString,\r
-                                     NULL\r
-                                     );\r
-  if (Attempt->AttemptTitleHelpToken == 0) {\r
-    FreePool (MacString);\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
 \r
   //\r
   // Record the user configuration information in NVR.\r
   //\r
-  UnicodeSPrint (\r
-    mPrivate->PortString,\r
-    (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
-    L"%s%d",\r
-    MacString,\r
-    (UINTN) Attempt->AttemptConfigIndex\r
-    );\r
-\r
-  FreePool (MacString);\r
 \r
+  UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"Attempt %d", Attempt->AttemptConfigIndex);\r
   return gRT->SetVariable (\r
                 mPrivate->PortString,\r
                 &gEfiIScsiInitiatorNameProtocolGuid,\r
@@ -770,10 +1737,11 @@ IScsiConvertIfrNvDataToAttemptConfigData (
                 sizeof (ISCSI_ATTEMPT_CONFIG_NVDATA),\r
                 Attempt\r
                 );\r
+\r
 }\r
 \r
 /**\r
-  Create Hii Extend Label OpCode as the start opcode and end opcode. It is \r
+  Create Hii Extend Label OpCode as the start opcode and end opcode. It is\r
   a help function.\r
 \r
   @param[in]  StartLabelNumber   The number of start label.\r
@@ -784,7 +1752,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
 \r
   @retval EFI_OUT_OF_RESOURCES   Do not have sufficient resource to finish this\r
                                  operation.\r
-  @retval EFI_INVALID_PARAMETER  Any input parameter is invalid.                                 \r
+  @retval EFI_INVALID_PARAMETER  Any input parameter is invalid.\r
   @retval EFI_SUCCESS            The operation is completed successfully.\r
 \r
 **/\r
@@ -834,7 +1802,7 @@ IScsiCreateOpCode (
   if (InternalStartLabel == NULL) {\r
     goto Exit;\r
   }\r
-  \r
+\r
   InternalStartLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
   InternalStartLabel->Number       = StartLabelNumber;\r
 \r
@@ -868,10 +1836,77 @@ Exit:
   if (*EndOpCodeHandle != NULL) {\r
     HiiFreeOpCodeHandle (*EndOpCodeHandle);\r
   }\r
-  \r
   return Status;\r
 }\r
 \r
+/**\r
+  Update the MAIN form to display the configured attempts.\r
+\r
+**/\r
+VOID\r
+IScsiConfigUpdateAttempt (\r
+  VOID\r
+  )\r
+{\r
+  LIST_ENTRY                    *Entry;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA   *AttemptConfigData;\r
+  VOID                          *StartOpCodeHandle;\r
+  EFI_IFR_GUID_LABEL            *StartLabel;\r
+  VOID                          *EndOpCodeHandle;\r
+  EFI_IFR_GUID_LABEL            *EndLabel;\r
+  EFI_STATUS                    Status;\r
+\r
+  Status = IScsiCreateOpCode (\r
+             ATTEMPT_ENTRY_LABEL,\r
+             &StartOpCodeHandle,\r
+             &StartLabel,\r
+             &EndOpCodeHandle,\r
+             &EndLabel\r
+             );\r
+  if (EFI_ERROR (Status)) {\r
+    return ;\r
+  }\r
+\r
+  NET_LIST_FOR_EACH (Entry, &mPrivate->AttemptConfigs) {\r
+    AttemptConfigData = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
+    if (AttemptConfigData->Actived == ISCSI_ACTIVE_ENABLED) {\r
+      //\r
+      // Update Attempt Help Info.\r
+      //\r
+      UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"Attempt %d", (UINTN) AttemptConfigData->AttemptConfigIndex);\r
+      AttemptConfigData->AttemptTitleToken = HiiSetString (\r
+                                               mCallbackInfo->RegisteredHandle,\r
+                                               0,\r
+                                               mPrivate->PortString,\r
+                                               NULL\r
+                                               );\r
+      if (AttemptConfigData->AttemptTitleToken == 0) {\r
+        return ;\r
+      }\r
+\r
+      HiiCreateGotoOpCode (\r
+        StartOpCodeHandle,                         // Container for dynamic created opcodes\r
+        FORMID_ATTEMPT_FORM,                       // Form ID\r
+        AttemptConfigData->AttemptTitleToken,      // Prompt text\r
+        AttemptConfigData->AttemptTitleHelpToken,  // Help text\r
+        EFI_IFR_FLAG_CALLBACK,                     // Question flag\r
+        (UINT16) (KEY_ATTEMPT_ENTRY_BASE + AttemptConfigData->AttemptConfigIndex)   // Question ID\r
+        );\r
+    }\r
+  }\r
+\r
+  HiiUpdateForm (\r
+    mCallbackInfo->RegisteredHandle, // HII handle\r
+    &gIScsiConfigGuid,               // Formset GUID\r
+    FORMID_MAIN_FORM,                // Form ID\r
+    StartOpCodeHandle,               // Label for where to insert opcodes\r
+    EndOpCodeHandle                  // Replace data\r
+  );\r
+\r
+  HiiFreeOpCodeHandle (StartOpCodeHandle);\r
+  HiiFreeOpCodeHandle (EndOpCodeHandle);\r
+}\r
+\r
 /**\r
   Callback function when user presses "Add an Attempt".\r
 \r
@@ -919,7 +1954,7 @@ IScsiConfigAddAttempt (
       MacString\r
       );\r
 \r
-    UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"Port %s", MacString);\r
+    UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"MAC %s", MacString);\r
     PortTitleToken = HiiSetString (\r
                        mCallbackInfo->RegisteredHandle,\r
                        0,\r
@@ -957,7 +1992,7 @@ IScsiConfigAddAttempt (
 \r
   Status = HiiUpdateForm (\r
              mCallbackInfo->RegisteredHandle, // HII handle\r
-             &mVendorGuid,                    // Formset GUID\r
+             &gIScsiConfigGuid,               // Formset GUID\r
              FORMID_MAC_FORM,                 // Form ID\r
              StartOpCodeHandle,               // Label for where to insert opcodes\r
              EndOpCodeHandle                  // Replace data\r
@@ -970,76 +2005,184 @@ Exit:
   return Status;\r
 }\r
 \r
-\r
 /**\r
-  Update the MAIN form to display the configured attempts.\r
+  Add the attempts by keyword 'iSCSIAddAttempts', you can use this keyword with\r
+  value 'attempt:1 attempt:2' etc to add one or more attempts once. This is different\r
+  with IScsiConfigAddAttempt function which is used to add attempt by UI configuration. \r
+\r
+  @param[in]  AttemptList        The new attempt List will be added.\r
+\r
+  @retval EFI_SUCCESS            The operation to add attempt list successfully.\r
+  @retval EFI_INVALID_PARAMETER  Any parameter is invalid.\r
+  @retval EFI_NOT_FOUND          Cannot find the corresponding variable.\r
+  @retval EFI_OUT_OF_RESOURCES   Fail to finish the operation due to lack of\r
+                                 resources.\r
 \r
 **/\r
-VOID\r
-IScsiConfigUpdateAttempt (\r
-  VOID\r
+EFI_STATUS\r
+IScsiConfigAddAttemptsByKeywords (\r
+  IN UINT8                   *AttemptList\r
   )\r
 {\r
-  CHAR16                        AttemptName[ATTEMPT_NAME_MAX_SIZE];\r
-  LIST_ENTRY                    *Entry;\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA   *AttemptConfigData;\r
-  VOID                          *StartOpCodeHandle;\r
-  EFI_IFR_GUID_LABEL            *StartLabel;\r
-  VOID                          *EndOpCodeHandle;\r
-  EFI_IFR_GUID_LABEL            *EndLabel;\r
-  EFI_STATUS                    Status;\r
+  UINT8                       Index;\r
+  UINT8                       Number;\r
+  UINTN                       TotalNumber;\r
+  UINT8                       Nic;\r
+  UINT8                       *AttemptConfigOrder;\r
+  UINTN                       AttemptConfigOrderSize;\r
+  UINT8                       *AttemptConfigOrderTmp;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptConfigData;\r
+  ISCSI_NIC_INFO              *NicInfo;\r
+  CHAR16                      MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR16                      IScsiMode[64];\r
+  CHAR16                      IpMode[64];\r
+  EFI_STATUS                  Status;\r
 \r
-  Status = IScsiCreateOpCode (\r
-             ATTEMPT_ENTRY_LABEL,\r
-             &StartOpCodeHandle,\r
-             &StartLabel,\r
-             &EndOpCodeHandle,\r
-             &EndLabel\r
-             );\r
-  if (EFI_ERROR (Status)) {\r
-    return ;\r
+  Nic = mPrivate->CurrentNic;\r
+  NicInfo = IScsiGetNicInfoByIndex (Nic);\r
+  if (NicInfo == NULL) {\r
+    return EFI_NOT_FOUND;\r
   }\r
 \r
-  NET_LIST_FOR_EACH (Entry, &mPrivate->AttemptConfigs) {\r
-    AttemptConfigData = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
+  //\r
+  // The MAC info will be recorded in Config Data.\r
+  //\r
+  IScsiMacAddrToStr (\r
+    &NicInfo->PermanentAddress,\r
+    NicInfo->HwAddressSize,\r
+    NicInfo->VlanId,\r
+    MacString\r
+    );\r
 \r
-    AsciiStrToUnicodeStr (AttemptConfigData->AttemptName, AttemptName);\r
-    UnicodeSPrint (mPrivate->PortString, (UINTN) 128, L"Attempt %s", AttemptName);\r
-    AttemptConfigData->AttemptTitleToken = HiiSetString (\r
-                                             mCallbackInfo->RegisteredHandle,\r
-                                             0,\r
-                                             mPrivate->PortString,\r
-                                             NULL\r
-                                             );\r
-    if (AttemptConfigData->AttemptTitleToken == 0) {\r
-      return ;\r
+  for (Index = 0; Index < PcdGet8 (PcdMaxIScsiAttemptNumber); Index++) {\r
+    if (AttemptList[Index] == 0) {\r
+      continue;\r
     }\r
 \r
-    HiiCreateGotoOpCode (\r
-      StartOpCodeHandle,                         // Container for dynamic created opcodes\r
-      FORMID_ATTEMPT_FORM,                       // Form ID\r
-      AttemptConfigData->AttemptTitleToken,      // Prompt text\r
-      AttemptConfigData->AttemptTitleHelpToken,  // Help text\r
-      EFI_IFR_FLAG_CALLBACK,                     // Question flag\r
-      (UINT16) (KEY_ATTEMPT_ENTRY_BASE + AttemptConfigData->AttemptConfigIndex)   // Question ID\r
+    //\r
+    // Add the attempt.\r
+    //\r
+    Number = AttemptList[Index];\r
+\r
+    UnicodeSPrint (\r
+      mPrivate->PortString,\r
+      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      L"Attempt %d",\r
+      Number\r
       );\r
-  }\r
 \r
-  HiiUpdateForm (\r
-    mCallbackInfo->RegisteredHandle, // HII handle\r
-    &mVendorGuid,                    // Formset GUID\r
-    FORMID_MAIN_FORM,                // Form ID\r
-    StartOpCodeHandle,               // Label for where to insert opcodes\r
-    EndOpCodeHandle                  // Replace data\r
-  );    \r
+    GetVariable2 (\r
+           mPrivate->PortString,\r
+           &gEfiIScsiInitiatorNameProtocolGuid,\r
+           (VOID**)&AttemptConfigData,\r
+           NULL\r
+           );\r
+    if (AttemptConfigData == NULL || AttemptConfigData->Actived == ISCSI_ACTIVE_ENABLED) {\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
 \r
-  HiiFreeOpCodeHandle (StartOpCodeHandle);\r
-  HiiFreeOpCodeHandle (EndOpCodeHandle);\r
-}\r
+    AttemptConfigData->Actived = ISCSI_ACTIVE_ENABLED;\r
+    AttemptConfigData->NicIndex = NicInfo->NicIndex;\r
+    UnicodeStrToAsciiStrS (MacString, AttemptConfigData->MacString, ISCSI_MAX_MAC_STRING_LEN);\r
+\r
+    //\r
+    // Generate OUI-format ISID based on MAC address.\r
+    //\r
+    CopyMem (AttemptConfigData->SessionConfigData.IsId, &NicInfo->PermanentAddress, 6);\r
+    AttemptConfigData->SessionConfigData.IsId[0] =\r
+      (UINT8) (AttemptConfigData->SessionConfigData.IsId[0] & 0x3F);\r
+\r
+    //\r
+    // Configure the iSCSI Mode and IpMode to default.\r
+    // Add Attempt Help Info.\r
+    //\r
+    UnicodeSPrint (IScsiMode, 64, L"Disabled");\r
+    UnicodeSPrint (IpMode, 64, L"IP4");\r
+    UnicodeSPrint (\r
+      mPrivate->PortString,\r
+      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      L"MAC: %s, PFA: Bus %d | Dev %d | Func %d, iSCSI mode: %s, IP version: %s",\r
+      MacString,\r
+      NicInfo->BusNumber,\r
+      NicInfo->DeviceNumber,\r
+      NicInfo->FunctionNumber,\r
+      IScsiMode,\r
+      IpMode\r
+      );\r
+\r
+    AttemptConfigData->AttemptTitleHelpToken = HiiSetString (\r
+                                                 mCallbackInfo->RegisteredHandle,\r
+                                                 0,\r
+                                                 mPrivate->PortString,\r
+                                                 NULL\r
+                                                 );\r
+    if (AttemptConfigData->AttemptTitleHelpToken == 0) {\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
+\r
+    //\r
+    // Get current Attempt order and number.\r
+    //\r
+    AttemptConfigOrder = IScsiGetVariableAndSize (\r
+                           L"AttemptOrder",\r
+                           &gIScsiConfigGuid,\r
+                           &AttemptConfigOrderSize\r
+                           );\r
+    TotalNumber = AttemptConfigOrderSize / sizeof (UINT8);\r
+    TotalNumber++;\r
+\r
+    //\r
+    // Append the new created attempt order to the end.\r
+    //\r
+    AttemptConfigOrderTmp = AllocateZeroPool (TotalNumber * sizeof (UINT8));\r
+    if (AttemptConfigOrderTmp == NULL) {\r
+      if (AttemptConfigOrder != NULL) {\r
+        FreePool (AttemptConfigOrder);\r
+      }\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
+    if (AttemptConfigOrder != NULL) {\r
+      CopyMem (AttemptConfigOrderTmp, AttemptConfigOrder, AttemptConfigOrderSize);\r
+      FreePool (AttemptConfigOrder);\r
+    }\r
+\r
+    AttemptConfigOrderTmp[TotalNumber - 1] = Number;\r
+    AttemptConfigOrder               = AttemptConfigOrderTmp;\r
+    AttemptConfigOrderSize           = TotalNumber * sizeof (UINT8);\r
+\r
+    Status = gRT->SetVariable (\r
+                    L"AttemptOrder",\r
+                    &gIScsiConfigGuid,\r
+                    EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE,\r
+                    AttemptConfigOrderSize,\r
+                    AttemptConfigOrder\r
+                    );\r
+    FreePool (AttemptConfigOrder);\r
+    if (EFI_ERROR (Status)) {\r
+      return Status;\r
+    }\r
+\r
+    //\r
+    // Record the attempt in global link list.\r
+    //\r
+    InsertTailList (&mPrivate->AttemptConfigs, &AttemptConfigData->Link);\r
+    mPrivate->AttemptCount++;\r
+    UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"Attempt %d", AttemptConfigData->AttemptConfigIndex);\r
+    gRT->SetVariable (\r
+           mPrivate->PortString,\r
+           &gEfiIScsiInitiatorNameProtocolGuid,\r
+           ISCSI_CONFIG_VAR_ATTR,\r
+           sizeof (ISCSI_ATTEMPT_CONFIG_NVDATA),\r
+           AttemptConfigData\r
+           );\r
+\r
+  }\r
 \r
+  return EFI_SUCCESS;\r
+}\r
 \r
 /**\r
-  Callback function when user presses "Commit Changes and Exit" in Delete Attempts.\r
+  Callback function when user presses "Commit Changes and Exit" in Delete Attempts or Delete Attempts by Keyword.\r
 \r
   @param[in]  IfrNvData          The IFR NV data.\r
 \r
@@ -1056,23 +2199,26 @@ IScsiConfigDeleteAttempts (
   IN ISCSI_CONFIG_IFR_NVDATA  *IfrNvData\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  UINTN                       Index;\r
-  UINTN                       NewIndex;\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptConfigData;\r
-  UINT8                       *AttemptConfigOrder;\r
-  UINTN                       AttemptConfigOrderSize;\r
-  UINT8                       *AttemptNewOrder;\r
-  UINT32                      Attribute;\r
-  UINTN                       Total;\r
-  UINTN                       NewTotal;\r
-  LIST_ENTRY                  *Entry;\r
-  LIST_ENTRY                  *NextEntry;\r
-  CHAR16                      MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  EFI_STATUS                    Status;\r
+  UINTN                         Index;\r
+  UINTN                         NewIndex;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA   *AttemptConfigData;\r
+  UINT8                         *AttemptConfigOrder;\r
+  UINTN                         AttemptConfigOrderSize;\r
+  UINT8                         *AttemptNewOrder;\r
+  UINT8                         AttemptConfigIndex;\r
+  UINT32                        Attribute;\r
+  UINTN                         Total;\r
+  UINTN                         NewTotal;\r
+  LIST_ENTRY                    *Entry;\r
+  LIST_ENTRY                    *NextEntry;\r
+  ISCSI_SESSION_CONFIG_NVDATA   *ConfigData;\r
+\r
+  Index     = 0;\r
 \r
   AttemptConfigOrder = IScsiGetVariableAndSize (\r
                          L"AttemptOrder",\r
-                         &mVendorGuid,\r
+                         &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
   if ((AttemptConfigOrder == NULL) || (AttemptConfigOrderSize == 0)) {\r
@@ -1081,12 +2227,12 @@ IScsiConfigDeleteAttempts (
 \r
   AttemptNewOrder = AllocateZeroPool (AttemptConfigOrderSize);\r
   if (AttemptNewOrder == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
+    Status = EFI_OUT_OF_RESOURCES;\r
+    goto Error;\r
   }\r
 \r
   Total    = AttemptConfigOrderSize / sizeof (UINT8);\r
   NewTotal = Total;\r
-  Index    = 0;\r
 \r
   NET_LIST_FOR_EACH_SAFE (Entry, NextEntry, &mPrivate->AttemptConfigs) {\r
     if (IfrNvData->DeleteAttemptList[Index] == 0) {\r
@@ -1131,22 +2277,44 @@ IScsiConfigDeleteAttempts (
       mPrivate->SinglePathCount--;\r
     }\r
 \r
-    AsciiStrToUnicodeStr (AttemptConfigData->MacString, MacString);\r
+    AttemptConfigIndex = AttemptConfigData->AttemptConfigIndex;\r
+    FreePool (AttemptConfigData);\r
+\r
+    //\r
+    // Create a new Attempt\r
+    //\r
+    AttemptConfigData = AllocateZeroPool (sizeof (ISCSI_ATTEMPT_CONFIG_NVDATA));\r
+    if (AttemptConfigData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
+    ConfigData                    = &AttemptConfigData->SessionConfigData;\r
+    ConfigData->TargetPort        = ISCSI_WELL_KNOWN_PORT;\r
+    ConfigData->ConnectTimeout    = CONNECT_DEFAULT_TIMEOUT;\r
+    ConfigData->ConnectRetryCount = CONNECT_MIN_RETRY;\r
+\r
+    AttemptConfigData->AuthenticationType           = ISCSI_AUTH_TYPE_CHAP;\r
+    AttemptConfigData->AuthConfigData.CHAP.CHAPType = ISCSI_CHAP_UNI;\r
+    //\r
+    // Configure the Attempt index and set variable.\r
+    //\r
+    AttemptConfigData->AttemptConfigIndex = AttemptConfigIndex;\r
 \r
+    //\r
+    // Set the attempt name to default.\r
+    //\r
     UnicodeSPrint (\r
       mPrivate->PortString,\r
-      (UINTN) 128,\r
-      L"%s%d",\r
-      MacString,\r
+      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      L"Attempt %d",\r
       (UINTN) AttemptConfigData->AttemptConfigIndex\r
       );\r
-\r
+    UnicodeStrToAsciiStrS (mPrivate->PortString, AttemptConfigData->AttemptName, ATTEMPT_NAME_SIZE);\r
     gRT->SetVariable (\r
            mPrivate->PortString,\r
            &gEfiIScsiInitiatorNameProtocolGuid,\r
-           0,\r
-           0,\r
-           NULL\r
+           ISCSI_CONFIG_VAR_ATTR,\r
+           sizeof (ISCSI_ATTEMPT_CONFIG_NVDATA),\r
+           AttemptConfigData\r
            );\r
 \r
     //\r
@@ -1160,6 +2328,9 @@ IScsiConfigDeleteAttempts (
     }\r
 \r
     NewTotal--;\r
+    if (mCallbackInfo->Current == AttemptConfigData) {\r
+      mCallbackInfo->Current = NULL;\r
+    }\r
     FreePool (AttemptConfigData);\r
 \r
     //\r
@@ -1178,24 +2349,28 @@ IScsiConfigDeleteAttempts (
     }\r
   }\r
 \r
-  Attribute = EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS\r
-              | EFI_VARIABLE_NON_VOLATILE;\r
+  Attribute = EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE;\r
 \r
   //\r
   // Update AttemptOrder in NVR.\r
   //\r
   Status = gRT->SetVariable (\r
                   L"AttemptOrder",\r
-                  &mVendorGuid,\r
+                  &gIScsiConfigGuid,\r
                   Attribute,\r
                   NewTotal * sizeof (UINT8),\r
                   AttemptNewOrder\r
                   );\r
 \r
 Error:\r
-  FreePool (AttemptConfigOrder);\r
-  FreePool (AttemptNewOrder);\r
-  \r
+  if (AttemptConfigOrder != NULL) {\r
+    FreePool (AttemptConfigOrder);\r
+  }\r
+\r
+  if (AttemptNewOrder != NULL) {\r
+    FreePool (AttemptNewOrder);\r
+  }\r
+\r
   return Status;\r
 }\r
 \r
@@ -1240,7 +2415,7 @@ IScsiConfigDisplayDeleteAttempts (
 \r
   AttemptConfigOrder = IScsiGetVariableAndSize (\r
                          L"AttemptOrder",\r
-                         &mVendorGuid,\r
+                         &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
   if (AttemptConfigOrder != NULL) {\r
@@ -1277,7 +2452,7 @@ IScsiConfigDisplayDeleteAttempts (
 \r
   Status = HiiUpdateForm (\r
              mCallbackInfo->RegisteredHandle, // HII handle\r
-             &mVendorGuid,                    // Formset GUID\r
+             &gIScsiConfigGuid,               // Formset GUID\r
              FORMID_DELETE_FORM,              // Form ID\r
              StartOpCodeHandle,               // Label for where to insert opcodes\r
              EndOpCodeHandle                  // Replace data\r
@@ -1324,6 +2499,7 @@ IScsiConfigDisplayOrderAttempts (
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
+  ASSERT (StartOpCodeHandle != NULL);\r
 \r
   OptionsOpCodeHandle = NULL;\r
 \r
@@ -1377,7 +2553,7 @@ IScsiConfigDisplayOrderAttempts (
 Exit:\r
   Status = HiiUpdateForm (\r
              mCallbackInfo->RegisteredHandle, // HII handle\r
-             &mVendorGuid,                    // Formset GUID\r
+             &gIScsiConfigGuid,               // Formset GUID\r
              FORMID_ORDER_FORM,               // Form ID\r
              StartOpCodeHandle,               // Label for where to insert opcodes\r
              EndOpCodeHandle                  // Replace data\r
@@ -1393,9 +2569,8 @@ Error:
   return Status;\r
 }\r
 \r
-\r
 /**\r
-  Callback function when user presses "Commit Changes and Exit" in Change Attempt Order.\r
+  Callback function when user presses "Commit Changes and Exit" in Change Attempt Order or Change Attempt Order by Keyword.\r
 \r
   @param[in]  IfrNvData          The IFR nv data.\r
 \r
@@ -1421,7 +2596,7 @@ IScsiConfigOrderAttempts (
 \r
   AttemptConfigOrder = IScsiGetVariableAndSize (\r
                          L"AttemptOrder",\r
-                         &mVendorGuid,\r
+                         &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
   if (AttemptConfigOrder == NULL) {\r
@@ -1483,8 +2658,8 @@ IScsiConfigOrderAttempts (
 \r
   Status = gRT->SetVariable (\r
                   L"AttemptOrder",\r
-                  &mVendorGuid,\r
-                  EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_NON_VOLATILE,\r
+                  &gIScsiConfigGuid,\r
+                  EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE,\r
                   AttemptConfigOrderSize,\r
                   AttemptConfigOrderTmp\r
                   );\r
@@ -1511,6 +2686,7 @@ Exit:
   @retval EFI_OUT_OF_RESOURCES   Does not have sufficient resources to finish this\r
                                  operation.\r
   @retval EFI_NOT_FOUND          Cannot find the corresponding variable.\r
+  @retval EFI_UNSUPPORTED        Can not create more attempts.\r
   @retval EFI_SUCCESS            The operation is completed successfully.\r
 \r
 **/\r
@@ -1522,18 +2698,18 @@ IScsiConfigProcessDefault (
 {\r
   BOOLEAN                     NewAttempt;\r
   ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptConfigData;\r
-  ISCSI_SESSION_CONFIG_NVDATA *ConfigData;\r
   UINT8                       CurrentAttemptConfigIndex;\r
   ISCSI_NIC_INFO              *NicInfo;\r
   UINT8                       NicIndex;\r
   CHAR16                      MacString[ISCSI_MAX_MAC_STRING_LEN];\r
   UINT8                       *AttemptConfigOrder;\r
   UINTN                       AttemptConfigOrderSize;\r
-  UINTN                       TotalNumber;\r
-  UINT8                       *AttemptOrderTmp;\r
   UINTN                       Index;\r
-  EFI_STATUS                  Status;\r
+  EFI_INPUT_KEY               Key;\r
 \r
+  //\r
+  // Is User creating a new attempt?\r
+  //\r
   NewAttempt = FALSE;\r
 \r
   if ((KeyValue >= KEY_MAC_ENTRY_BASE) &&\r
@@ -1565,94 +2741,53 @@ IScsiConfigProcessDefault (
     if (NicInfo == NULL) {\r
       return EFI_NOT_FOUND;\r
     }\r
-    \r
-    //\r
-    // Create the new attempt and save to NVR.\r
-    //\r
-\r
-    AttemptConfigData = AllocateZeroPool (sizeof (ISCSI_ATTEMPT_CONFIG_NVDATA));\r
-    if (AttemptConfigData == NULL) {\r
-      return EFI_OUT_OF_RESOURCES;\r
-    }\r
-\r
-    ConfigData                    = &AttemptConfigData->SessionConfigData;\r
-    ConfigData->TargetPort        = ISCSI_WELL_KNOWN_PORT;\r
-    ConfigData->ConnectTimeout    = CONNECT_DEFAULT_TIMEOUT;\r
-    ConfigData->ConnectRetryCount = CONNECT_MIN_RETRY;\r
-\r
-    AttemptConfigData->AuthenticationType           = ISCSI_AUTH_TYPE_CHAP;\r
-    AttemptConfigData->AuthConfigData.CHAP.CHAPType = ISCSI_CHAP_UNI;\r
 \r
     //\r
-    // Get current order number for this attempt.\r
+    // Create an attempt following the initialized attempt order.\r
     //\r
     AttemptConfigOrder = IScsiGetVariableAndSize (\r
-                           L"AttemptOrder",\r
-                           &mVendorGuid,\r
+                           L"InitialAttemptOrder",\r
+                           &gIScsiConfigGuid,\r
                            &AttemptConfigOrderSize\r
                            );\r
 \r
-    TotalNumber = AttemptConfigOrderSize / sizeof (UINT8);\r
+    if (AttemptConfigOrder != NULL) {\r
 \r
-    if (AttemptConfigOrder == NULL) {\r
-      CurrentAttemptConfigIndex = 1;\r
-    } else {\r
-      //\r
-      // Get the max attempt config index.\r
-      //\r
-      CurrentAttemptConfigIndex = AttemptConfigOrder[0];\r
-      for (Index = 1; Index < TotalNumber; Index++) {\r
-        if (CurrentAttemptConfigIndex < AttemptConfigOrder[Index]) {\r
-          CurrentAttemptConfigIndex = AttemptConfigOrder[Index];\r
+      for (Index = 0; Index < AttemptConfigOrderSize / sizeof (UINT8); Index++) {\r
+        UnicodeSPrint (\r
+          mPrivate->PortString,\r
+          (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+          L"Attempt %d",\r
+          (UINTN) AttemptConfigOrder[Index]\r
+          );\r
+        GetVariable2 (\r
+                     mPrivate->PortString,\r
+                     &gEfiIScsiInitiatorNameProtocolGuid,\r
+                     (VOID**)&AttemptConfigData,\r
+                     NULL\r
+                     );\r
+        if (AttemptConfigData == NULL || AttemptConfigData->Actived == ISCSI_ACTIVE_ENABLED) {\r
+          continue;\r
         }\r
-      }\r
-\r
-      CurrentAttemptConfigIndex++;\r
-    }\r
 \r
-    TotalNumber++;\r
+        break;\r
+      }\r
 \r
-    //\r
-    // Append the new created attempt order to the end.\r
-    //\r
-    AttemptOrderTmp = AllocateZeroPool (TotalNumber * sizeof (UINT8));\r
-    if (AttemptOrderTmp == NULL) {\r
-      FreePool (AttemptConfigData);\r
-      if (AttemptConfigOrder != NULL) {\r
-        FreePool (AttemptConfigOrder);\r
+      if (Index > PcdGet8 (PcdMaxIScsiAttemptNumber)) {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Can not create more attempts, Please configure the PcdMaxIScsiAttemptNumber if needed!",\r
+          NULL\r
+          );\r
+        return EFI_UNSUPPORTED;\r
       }\r
-      return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
     if (AttemptConfigOrder != NULL) {\r
-      CopyMem (AttemptOrderTmp, AttemptConfigOrder, AttemptConfigOrderSize);\r
-      FreePool (AttemptConfigOrder);      \r
-    }\r
-\r
-    AttemptOrderTmp[TotalNumber - 1] = CurrentAttemptConfigIndex;    \r
-    AttemptConfigOrder               = AttemptOrderTmp;\r
-    AttemptConfigOrderSize           = TotalNumber * sizeof (UINT8);\r
-\r
-    Status = gRT->SetVariable (\r
-                    L"AttemptOrder",\r
-                    &mVendorGuid,\r
-                    EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS | EFI_VARIABLE_NON_VOLATILE,\r
-                    AttemptConfigOrderSize,\r
-                    AttemptConfigOrder\r
-                    );\r
-    FreePool (AttemptConfigOrder);\r
-    if (EFI_ERROR (Status)) {\r
-      FreePool (AttemptConfigData);\r
-      return Status;\r
+      FreePool (AttemptConfigOrder);\r
     }\r
 \r
-    //\r
-    // Record the mapping between attempt order and attempt's configdata.\r
-    //\r
-    AttemptConfigData->AttemptConfigIndex  = CurrentAttemptConfigIndex;\r
-    InsertTailList (&mPrivate->AttemptConfigs, &AttemptConfigData->Link);\r
-    mPrivate->AttemptCount++;\r
-\r
     //\r
     // Record the MAC info in Config Data.\r
     //\r
@@ -1663,8 +2798,9 @@ IScsiConfigProcessDefault (
       MacString\r
       );\r
 \r
-    UnicodeStrToAsciiStr (MacString, AttemptConfigData->MacString);\r
+    UnicodeStrToAsciiStrS (MacString, AttemptConfigData->MacString, sizeof (AttemptConfigData->MacString));\r
     AttemptConfigData->NicIndex = NicIndex;\r
+    AttemptConfigData->Actived = ISCSI_ACTIVE_ENABLED;\r
 \r
     //\r
     // Generate OUI-format ISID based on MAC address.\r
@@ -1694,20 +2830,9 @@ IScsiConfigProcessDefault (
                                                   );\r
     if (AttemptConfigData->AttemptTitleHelpToken == 0) {\r
       FreePool (AttemptConfigData);\r
-      return EFI_INVALID_PARAMETER;\r
+      return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    //\r
-    // Set the attempt name to default.\r
-    //\r
-    UnicodeSPrint (\r
-      mPrivate->PortString,\r
-      (UINTN) 128,\r
-      L"%d",\r
-      (UINTN) AttemptConfigData->AttemptConfigIndex\r
-      );\r
-    UnicodeStrToAsciiStr (mPrivate->PortString, AttemptConfigData->AttemptName);\r
-\r
   } else {\r
     //\r
     // Determine which Attempt user has selected to configure.\r
@@ -1734,10 +2859,11 @@ IScsiConfigProcessDefault (
   \r
   IScsiConvertAttemptConfigDataToIfrNvData (AttemptConfigData, IfrNvData);\r
 \r
+  //\r
+  // Update current attempt to be a new created attempt or an existing attempt.\r
+  //\r
   mCallbackInfo->Current = AttemptConfigData;\r
 \r
-  IScsiConfigUpdateAttempt ();\r
-\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -1837,7 +2963,7 @@ IScsiFormExtractConfig (
   }\r
 \r
   *Progress = Request;\r
-  if ((Request != NULL) && !HiiIsConfigHdrMatch (Request, &mVendorGuid, mVendorStorageName)) {\r
+  if ((Request != NULL) && !HiiIsConfigHdrMatch (Request, &gIScsiConfigGuid, mVendorStorageName)) {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
@@ -1851,11 +2977,17 @@ IScsiFormExtractConfig (
   if (IfrNvData == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
-  \r
-  if (Private->Current != NULL) {\r
+\r
+\r
+  if (Private->Current!= NULL) {\r
     IScsiConvertAttemptConfigDataToIfrNvData (Private->Current, IfrNvData);\r
   }\r
 \r
+  //\r
+  // Extract all AttemptConfigData to Keyword stroage of IfrNvData.\r
+  //\r
+  IScsiConvertAttemptConfigDataToIfrNvDataByKeyword (IfrNvData);\r
+\r
   BufferSize    = ISCSI_NAME_MAX_SIZE;\r
   InitiatorName = (CHAR8 *) AllocateZeroPool (BufferSize);\r
   if (InitiatorName == NULL) {\r
@@ -1867,7 +2999,11 @@ IScsiFormExtractConfig (
   if (EFI_ERROR (Status)) {\r
     IfrNvData->InitiatorName[0] = L'\0';\r
   } else {\r
-    AsciiStrToUnicodeStr (InitiatorName, IfrNvData->InitiatorName);\r
+    AsciiStrToUnicodeStrS (\r
+      InitiatorName,\r
+      IfrNvData->InitiatorName,\r
+      sizeof (IfrNvData->InitiatorName) / sizeof (IfrNvData->InitiatorName[0])\r
+      );\r
   }\r
 \r
   //\r
@@ -1881,10 +3017,14 @@ IScsiFormExtractConfig (
     // Allocate and fill a buffer large enough to hold the <ConfigHdr> template\r
     // followed by "&OFFSET=0&WIDTH=WWWWWWWWWWWWWWWW" followed by a Null-terminator\r
     //\r
-    ConfigRequestHdr = HiiConstructConfigHdr (&mVendorGuid, mVendorStorageName, Private->DriverHandle);\r
+    ConfigRequestHdr = HiiConstructConfigHdr (&gIScsiConfigGuid, mVendorStorageName, Private->DriverHandle);\r
     Size = (StrLen (ConfigRequestHdr) + 32 + 1) * sizeof (CHAR16);\r
     ConfigRequest = AllocateZeroPool (Size);\r
-    ASSERT (ConfigRequest != NULL);\r
+    if (ConfigRequest == NULL) {\r
+      FreePool (IfrNvData);\r
+      FreePool (InitiatorName);\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
     AllocatedRequest = TRUE;\r
     UnicodeSPrint (ConfigRequest, Size, L"%s&OFFSET=0&WIDTH=%016LX", ConfigRequestHdr, (UINT64)BufferSize);\r
     FreePool (ConfigRequestHdr);\r
@@ -1901,92 +3041,282 @@ IScsiFormExtractConfig (
   FreePool (IfrNvData);\r
   FreePool (InitiatorName);\r
 \r
-  //\r
-  // Free the allocated config request string.\r
-  //\r
-  if (AllocatedRequest) {\r
-    FreePool (ConfigRequest);\r
-    ConfigRequest = NULL;\r
-  }\r
-  //\r
-  // Set Progress string to the original request string.\r
-  //\r
-  if (Request == NULL) {\r
-    *Progress = NULL;\r
-  } else if (StrStr (Request, L"OFFSET") == NULL) {\r
-    *Progress = Request + StrLen (Request);\r
-  }\r
+  //\r
+  // Free the allocated config request string.\r
+  //\r
+  if (AllocatedRequest) {\r
+    FreePool (ConfigRequest);\r
+    ConfigRequest = NULL;\r
+  }\r
+  //\r
+  // Set Progress string to the original request string.\r
+  //\r
+  if (Request == NULL) {\r
+    *Progress = NULL;\r
+  } else if (StrStr (Request, L"OFFSET") == NULL) {\r
+    *Progress = Request + StrLen (Request);\r
+  }\r
+\r
+  return Status;\r
+}\r
+\r
+\r
+/**\r
+   \r
+  This function applies changes in a driver's configuration.\r
+  Input is a Configuration, which has the routing data for this\r
+  driver followed by name / value configuration pairs. The driver\r
+  must apply those pairs to its configurable storage. If the\r
+  driver's configuration is stored in a linear block of data\r
+  and the driver's name / value pairs are in <BlockConfig>\r
+  format, it may use the ConfigToBlock helper function (above) to\r
+  simplify the job.\r
+\r
+  @param[in]  This           Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
+\r
+  @param[in]  Configuration  A null-terminated Unicode string in\r
+                             <ConfigString> format. \r
+  \r
+  @param[out] Progress       A pointer to a string filled in with the\r
+                             offset of the most recent '&' before the\r
+                             first failing name / value pair (or the\r
+                             beginning of the string if the failure\r
+                             is in the first name / value pair) or\r
+                             the terminating NULL if all was\r
+                             successful.\r
+\r
+  @retval EFI_SUCCESS             The results have been distributed or are\r
+                                  awaiting distribution.\r
+  \r
+  @retval EFI_OUT_OF_RESOURCES    Not enough memory to store the\r
+                                  parts of the results that must be\r
+                                  stored awaiting possible future\r
+                                  protocols.\r
+  \r
+  @retval EFI_INVALID_PARAMETERS  Passing in a NULL for the\r
+                                  Results parameter would result\r
+                                  in this type of error.\r
+  \r
+  @retval EFI_NOT_FOUND           Target for the specified routing data\r
+                                  was not found.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiFormRouteConfig (\r
+  IN  CONST EFI_HII_CONFIG_ACCESS_PROTOCOL   *This,\r
+  IN  CONST EFI_STRING                       Configuration,\r
+  OUT EFI_STRING                             *Progress\r
+  )\r
+{\r
+  EFI_STATUS                       Status;\r
+  ISCSI_CONFIG_IFR_NVDATA          *IfrNvData;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA      *AttemptConfigData;\r
+  LIST_ENTRY                       *Entry;\r
+  LIST_ENTRY                       *NextEntry;\r
+  ISCSI_NIC_INFO                   *NicInfo;\r
+  EFI_INPUT_KEY                    Key;\r
+  CHAR16                           MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR8                            *InitiatorName;\r
+  UINT8                            *AttemptList;\r
+  UINTN                            BufferSize;\r
+  UINTN                            OffSet;\r
+  UINTN                            Index;\r
+  UINTN                            Index2;\r
+\r
+  Index   = 0;\r
+  Index2  = 0;\r
+  NicInfo = NULL;\r
+  AttemptList = NULL;\r
+\r
+  if (This == NULL || Configuration == NULL || Progress == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  //\r
+  // Check routing data in <ConfigHdr>.\r
+  // Note: if only one Storage is used, then this checking could be skipped.\r
+  //\r
+  if (!HiiIsConfigHdrMatch (Configuration, &gIScsiConfigGuid, mVendorStorageName)) {\r
+    *Progress = Configuration;\r
+    return EFI_NOT_FOUND;\r
+  }\r
+\r
+  IfrNvData = AllocateZeroPool (sizeof (ISCSI_CONFIG_IFR_NVDATA));\r
+  if (IfrNvData == NULL) {\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+\r
+  BufferSize    = ISCSI_NAME_MAX_SIZE;\r
+  InitiatorName = (CHAR8 *) AllocateZeroPool (BufferSize);\r
+  if (InitiatorName == NULL) {\r
+    Status = EFI_OUT_OF_RESOURCES;\r
+    goto Exit;\r
+  }\r
+\r
+  //\r
+  // Convert <ConfigResp> to buffer data by helper function ConfigToBlock().\r
+  //\r
+  BufferSize = sizeof (ISCSI_CONFIG_IFR_NVDATA);\r
+  Status = gHiiConfigRouting->ConfigToBlock (\r
+                             gHiiConfigRouting,\r
+                             Configuration,\r
+                             (UINT8 *) IfrNvData,\r
+                             &BufferSize,\r
+                             Progress\r
+                             );\r
+  if (EFI_ERROR (Status)) {\r
+    goto Exit;\r
+  }\r
+\r
+  if (IfrNvData->InitiatorName[0] != L'\0') {\r
+    UnicodeStrToAsciiStrS (IfrNvData->InitiatorName, InitiatorName, ISCSI_NAME_MAX_SIZE);\r
+    BufferSize  = AsciiStrSize (InitiatorName);\r
+\r
+    Status      = gIScsiInitiatorName.Set (&gIScsiInitiatorName, &BufferSize, InitiatorName);\r
+    if (EFI_ERROR (Status)) {\r
+      CreatePopUp (\r
+        EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+        &Key,\r
+        L"Invalid iSCSI Name!",\r
+        NULL\r
+        );\r
+      goto Exit;\r
+    }\r
+  } else {\r
+    Status = gIScsiInitiatorName.Get (&gIScsiInitiatorName, &BufferSize, InitiatorName);\r
+    if (EFI_ERROR (Status)) {\r
+      CreatePopUp (\r
+        EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+        &Key,\r
+        L"Error: please configure iSCSI initiator name first!",\r
+        NULL\r
+        );\r
+      goto Exit;\r
+    }\r
+\r
+    if (IfrNvData->ISCSIAddAttemptList[0] != L'\0') {\r
+      Status =IScsiGetAttemptIndexList (IfrNvData->ISCSIAddAttemptList, IfrNvData->AddAttemptList, TRUE);\r
+      if (EFI_ERROR (Status)) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Error: The add attempt list is invalid",\r
+            NULL\r
+            );\r
+        goto Exit;\r
+      }\r
+\r
+      Status = IScsiConfigAddAttemptsByKeywords (IfrNvData->AddAttemptList);\r
+      if (EFI_ERROR (Status)) {\r
+        goto Exit;\r
+      }\r
+\r
+    } else if (IfrNvData->ISCSIDeleteAttemptList[0] != L'\0') {\r
+      AttemptList =(UINT8 *) AllocateZeroPool ((ISCSI_MAX_ATTEMPTS_NUM + 1) * sizeof (UINT8));\r
+      if (AttemptList == NULL) {\r
+        Status = EFI_OUT_OF_RESOURCES;\r
+        goto Exit;\r
+      }\r
+      Status = IScsiGetAttemptIndexList (IfrNvData->ISCSIDeleteAttemptList, AttemptList, FALSE);\r
+      if (EFI_ERROR (Status)) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Error: The delete attempt list is invalid",\r
+            NULL\r
+            );\r
+        goto Exit;\r
+      }\r
+\r
+      //\r
+      // Mark the attempt which will be delete in the global list.\r
+      //\r
+      NET_LIST_FOR_EACH_SAFE (Entry, NextEntry, &mPrivate->AttemptConfigs) {\r
+        AttemptConfigData = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
+        while (AttemptList[Index] != 0) {\r
+          if (AttemptConfigData->AttemptConfigIndex == AttemptList[Index]) {\r
+            IfrNvData->DeleteAttemptList[Index2] = 1;\r
+            break;\r
+          }\r
+          Index ++;\r
+        }\r
+        Index2 ++;\r
+        Index = 0;\r
+      }\r
+\r
+      Status = IScsiConfigDeleteAttempts (IfrNvData);\r
+      if (EFI_ERROR (Status)) {\r
+        goto Exit;\r
+      }\r
+\r
+      FreePool (AttemptList);\r
 \r
-  return Status;\r
-}\r
+    } else if (IfrNvData->ISCSIAttemptOrder[0] != L'\0') {\r
+      Status = IScsiGetAttemptIndexList (IfrNvData->ISCSIAttemptOrder, IfrNvData->DynamicOrderedList, FALSE);\r
+      if (EFI_ERROR (Status)) {\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Error: The new attempt order list is invalid",\r
+            NULL\r
+            );\r
+        goto Exit;\r
+      }\r
 \r
+      Status = IScsiConfigOrderAttempts (IfrNvData);\r
+      if (EFI_ERROR (Status)) {\r
+        goto Exit;\r
+      }\r
 \r
-/**\r
-   \r
-  This function applies changes in a driver's configuration.\r
-  Input is a Configuration, which has the routing data for this\r
-  driver followed by name / value configuration pairs. The driver\r
-  must apply those pairs to its configurable storage. If the\r
-  driver's configuration is stored in a linear block of data\r
-  and the driver's name / value pairs are in <BlockConfig>\r
-  format, it may use the ConfigToBlock helper function (above) to\r
-  simplify the job.\r
+    } else if (IfrNvData->ISCSIMacAddr[0] != L'\0') {\r
+      NET_LIST_FOR_EACH (Entry, &mPrivate->NicInfoList) {\r
+        NicInfo = NET_LIST_USER_STRUCT (Entry, ISCSI_NIC_INFO, Link);\r
+        IScsiMacAddrToStr (\r
+        &NicInfo->PermanentAddress,\r
+        NicInfo->HwAddressSize,\r
+        NicInfo->VlanId,\r
+        MacString\r
+        );\r
+        if (!StrCmp(MacString, IfrNvData->ISCSIMacAddr)) {\r
+          mPrivate->CurrentNic = NicInfo->NicIndex;\r
+          break;\r
+        }\r
+      }\r
 \r
-  @param[in]  This           Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
+      if ((NicInfo == NULL) || (NicInfo->NicIndex == 0)) {\r
+        Status = EFI_NOT_FOUND;\r
+        goto Exit;\r
+      }\r
 \r
-  @param[in]  Configuration  A null-terminated Unicode string in\r
-                             <ConfigString> format. \r
-  \r
-  @param[out] Progress       A pointer to a string filled in with the\r
-                             offset of the most recent '&' before the\r
-                             first failing name / value pair (or the\r
-                             beginning of the string if the failure\r
-                             is in the first name / value pair) or\r
-                             the terminating NULL if all was\r
-                             successful.\r
+    } else {\r
+      Status = IScsiGetValue (Configuration, L"&OFFSET=", &OffSet);\r
+      if (EFI_ERROR (Status)) {\r
+        goto Exit;\r
+      }\r
+      Status = IScsiConvertlfrNvDataToAttemptConfigDataByKeyword (IfrNvData, OffSet);\r
+      if (EFI_ERROR (Status)) {\r
+        goto Exit;\r
+      }\r
+    }\r
+  }\r
 \r
-  @retval EFI_SUCCESS             The results have been distributed or are\r
-                                  awaiting distribution.\r
-  \r
-  @retval EFI_OUT_OF_RESOURCES    Not enough memory to store the\r
-                                  parts of the results that must be\r
-                                  stored awaiting possible future\r
-                                  protocols.\r
-  \r
-  @retval EFI_INVALID_PARAMETERS  Passing in a NULL for the\r
-                                  Results parameter would result\r
-                                  in this type of error.\r
-  \r
-  @retval EFI_NOT_FOUND           Target for the specified routing data\r
-                                  was not found.\r
+  IScsiConfigUpdateAttempt ();\r
 \r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-IScsiFormRouteConfig (\r
-  IN  CONST EFI_HII_CONFIG_ACCESS_PROTOCOL   *This,\r
-  IN  CONST EFI_STRING                       Configuration,\r
-  OUT EFI_STRING                             *Progress\r
-  )\r
-{\r
-  if (This == NULL || Configuration == NULL || Progress == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
+  Status = EFI_SUCCESS;\r
+\r
+Exit:\r
+  if (InitiatorName != NULL) {\r
+    FreePool (InitiatorName);\r
   }\r
 \r
-  //\r
-  // Check routing data in <ConfigHdr>.\r
-  // Note: if only one Storage is used, then this checking could be skipped.\r
-  //\r
-  if (!HiiIsConfigHdrMatch (Configuration, &mVendorGuid, mVendorStorageName)) {\r
-    *Progress = Configuration;\r
-    return EFI_NOT_FOUND;\r
+  if (IfrNvData != NULL) {\r
+    FreePool (IfrNvData);\r
   }\r
 \r
-  *Progress = Configuration + StrLen (Configuration);\r
-  return EFI_SUCCESS;\r
+  return Status;\r
 }\r
 \r
-\r
 /**\r
    \r
   This function is called to provide results data to the driver.\r
@@ -2026,7 +3356,7 @@ IScsiFormCallback (
   ISCSI_FORM_CALLBACK_INFO    *Private;\r
   UINTN                       BufferSize;\r
   CHAR8                       *IScsiName;\r
-  CHAR8                       IpString[IP_STR_MAX_SIZE];\r
+  CHAR8                       IpString[ISCSI_NAME_MAX_SIZE];\r
   CHAR8                       LunString[ISCSI_LUN_STR_MAX_LEN];\r
   UINT64                      Lun;\r
   EFI_IP_ADDRESS              HostIp;\r
@@ -2035,7 +3365,6 @@ IScsiFormCallback (
   ISCSI_CONFIG_IFR_NVDATA     *IfrNvData;\r
   ISCSI_CONFIG_IFR_NVDATA     OldIfrNvData;\r
   EFI_STATUS                  Status;\r
-  CHAR16                      AttemptName[ATTEMPT_NAME_SIZE + 4];\r
   EFI_INPUT_KEY               Key;\r
 \r
   if ((Action == EFI_BROWSER_ACTION_FORM_OPEN) || (Action == EFI_BROWSER_ACTION_FORM_CLOSE)) {\r
@@ -2045,56 +3374,65 @@ IScsiFormCallback (
     return EFI_SUCCESS;\r
   }\r
 \r
-  if (Action == EFI_BROWSER_ACTION_CHANGING) {\r
-    if (This == NULL || Value == NULL || ActionRequest == NULL) {\r
-      return EFI_INVALID_PARAMETER;\r
-    }\r
-\r
-    Private = ISCSI_FORM_CALLBACK_INFO_FROM_FORM_CALLBACK (This);\r
-\r
+  if ((Action != EFI_BROWSER_ACTION_CHANGING) && (Action != EFI_BROWSER_ACTION_CHANGED)) {\r
     //\r
-    // Retrieve uncommitted data from Browser\r
+    // All other type return unsupported.\r
     //\r
+    return EFI_UNSUPPORTED;\r
+  }\r
 \r
-    BufferSize = sizeof (ISCSI_CONFIG_IFR_NVDATA);\r
-    IfrNvData = AllocateZeroPool (BufferSize);\r
-    if (IfrNvData == NULL) {\r
-      return EFI_OUT_OF_RESOURCES;\r
-    }\r
-\r
-    IScsiName = (CHAR8 *) AllocateZeroPool (ISCSI_NAME_MAX_SIZE);\r
-    if (IScsiName == NULL) {\r
-      FreePool (IfrNvData);\r
-      return EFI_OUT_OF_RESOURCES;\r
-    }\r
-\r
-    Status = EFI_SUCCESS;\r
-\r
-    ZeroMem (&OldIfrNvData, BufferSize);\r
-\r
-    HiiGetBrowserData (NULL, NULL, BufferSize, (UINT8 *) IfrNvData);\r
+  if ((Value == NULL) || (ActionRequest == NULL)) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
 \r
-    CopyMem (&OldIfrNvData, IfrNvData, BufferSize);\r
+  Private = ISCSI_FORM_CALLBACK_INFO_FROM_FORM_CALLBACK (This);\r
+  \r
+  //\r
+  // Retrieve uncommitted data from Browser\r
+  //\r
+  \r
+  BufferSize = sizeof (ISCSI_CONFIG_IFR_NVDATA);\r
+  IfrNvData = AllocateZeroPool (BufferSize);\r
+  if (IfrNvData == NULL) {\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+  \r
+  IScsiName = (CHAR8 *) AllocateZeroPool (ISCSI_NAME_MAX_SIZE);\r
+  if (IScsiName == NULL) {\r
+    FreePool (IfrNvData);\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+  \r
+  Status = EFI_SUCCESS;\r
+  \r
+  ZeroMem (&OldIfrNvData, BufferSize);\r
+  \r
+  HiiGetBrowserData (NULL, NULL, BufferSize, (UINT8 *) IfrNvData);\r
+  \r
+  CopyMem (&OldIfrNvData, IfrNvData, BufferSize);\r
 \r
+  if (Action == EFI_BROWSER_ACTION_CHANGING) {\r
     switch (QuestionId) {\r
-    case KEY_INITIATOR_NAME:\r
-      UnicodeStrToAsciiStr (IfrNvData->InitiatorName, IScsiName);\r
-      BufferSize  = AsciiStrSize (IScsiName);\r
-\r
-      Status      = gIScsiInitiatorName.Set (&gIScsiInitiatorName, &BufferSize, IScsiName);\r
+    case KEY_ADD_ATTEMPT:\r
+      //\r
+      // Check whether iSCSI initiator name is configured already.\r
+      //\r
+      mPrivate->InitiatorNameLength = ISCSI_NAME_MAX_SIZE;\r
+      Status = gIScsiInitiatorName.Get (\r
+                                     &gIScsiInitiatorName,\r
+                                     &mPrivate->InitiatorNameLength,\r
+                                     mPrivate->InitiatorName\r
+                                     );\r
       if (EFI_ERROR (Status)) {\r
         CreatePopUp (\r
           EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
           &Key,\r
-          L"Invalid iSCSI Name!",\r
+          L"Error: please configure iSCSI initiator name first!",\r
           NULL\r
-          );      \r
+          );        \r
+        break;\r
       }\r
-\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
-      break;\r
-\r
-    case KEY_ADD_ATTEMPT:\r
+      \r
       Status = IScsiConfigAddAttempt ();\r
       break;\r
 \r
@@ -2107,28 +3445,6 @@ IScsiFormCallback (
       Status = IScsiConfigDisplayDeleteAttempts (IfrNvData);\r
       break;\r
 \r
-    case KEY_SAVE_DELETE_ATTEMPT:\r
-      //\r
-      // Delete the Attempt Order from NVR\r
-      //\r
-      Status = IScsiConfigDeleteAttempts (IfrNvData);\r
-      if (EFI_ERROR (Status)) {\r
-        break;\r
-      }\r
-\r
-      IScsiConfigUpdateAttempt ();\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
-      break;\r
-\r
-    case KEY_IGNORE_DELETE_ATTEMPT:\r
-      CopyMem (\r
-        IfrNvData->DeleteAttemptList,\r
-        OldIfrNvData.DeleteAttemptList,\r
-        sizeof (IfrNvData->DeleteAttemptList)\r
-        );\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
-      break;\r
-\r
     case KEY_ORDER_ATTEMPT_CONFIG:\r
       //\r
       // Order the attempt according to user input.\r
@@ -2140,6 +3456,38 @@ IScsiFormCallback (
         );\r
       IScsiConfigDisplayOrderAttempts ();\r
       break;\r
+    \r
+    default:\r
+      Status = IScsiConfigProcessDefault (QuestionId, IfrNvData);\r
+      break;\r
+    }\r
+  } else if (Action == EFI_BROWSER_ACTION_CHANGED) {  \r
+    switch (QuestionId) {\r
+    case KEY_INITIATOR_NAME:\r
+      UnicodeStrToAsciiStrS (IfrNvData->InitiatorName, IScsiName, ISCSI_NAME_MAX_SIZE);\r
+      BufferSize  = AsciiStrSize (IScsiName);\r
+\r
+      Status      = gIScsiInitiatorName.Set (&gIScsiInitiatorName, &BufferSize, IScsiName);\r
+      if (EFI_ERROR (Status)) {\r
+        CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"Invalid iSCSI Name!",\r
+          NULL\r
+          );      \r
+      }\r
+\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_APPLY;\r
+      break;\r
+      \r
+    case KEY_SAVE_ATTEMPT_CONFIG:\r
+      Status = IScsiConvertIfrNvDataToAttemptConfigData (IfrNvData, Private->Current);\r
+      if (EFI_ERROR (Status)) {\r
+        break;\r
+      }\r
+\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_SUBMIT_EXIT;\r
+      break;\r
 \r
     case KEY_SAVE_ORDER_CHANGES:\r
       //\r
@@ -2151,7 +3499,7 @@ IScsiFormCallback (
       }\r
 \r
       IScsiConfigUpdateAttempt ();\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_SUBMIT_EXIT;\r
       break;\r
 \r
     case KEY_IGNORE_ORDER_CHANGES:\r
@@ -2160,39 +3508,36 @@ IScsiFormCallback (
         OldIfrNvData.DynamicOrderedList,\r
         sizeof (IfrNvData->DynamicOrderedList)\r
         );\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_DISCARD_EXIT;\r
       break;\r
 \r
-    case KEY_ATTEMPT_NAME:\r
-      if (StrLen (IfrNvData->AttemptName) > ATTEMPT_NAME_SIZE) {\r
-        CopyMem (AttemptName, IfrNvData->AttemptName, ATTEMPT_NAME_SIZE * sizeof (CHAR16));\r
-        CopyMem (&AttemptName[ATTEMPT_NAME_SIZE], L"...", 4 * sizeof (CHAR16));\r
-      } else {\r
-        CopyMem (\r
-          AttemptName,\r
-          IfrNvData->AttemptName,\r
-          (StrLen (IfrNvData->AttemptName) + 1) * sizeof (CHAR16)\r
-          );\r
+    case KEY_SAVE_DELETE_ATTEMPT:\r
+      //\r
+      // Delete the Attempt Order from NVR\r
+      //\r
+      Status = IScsiConfigDeleteAttempts (IfrNvData);\r
+      if (EFI_ERROR (Status)) {\r
+        break;\r
       }\r
 \r
-      UnicodeStrToAsciiStr (IfrNvData->AttemptName, Private->Current->AttemptName);\r
-\r
       IScsiConfigUpdateAttempt ();\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_SUBMIT_EXIT;\r
+      break;\r
 \r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+    case KEY_IGNORE_DELETE_ATTEMPT:\r
+      CopyMem (\r
+        IfrNvData->DeleteAttemptList,\r
+        OldIfrNvData.DeleteAttemptList,\r
+        sizeof (IfrNvData->DeleteAttemptList)\r
+        );\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_DISCARD_EXIT;\r
       break;\r
 \r
     case KEY_IP_MODE:\r
       switch (Value->u8) {\r
       case IP_MODE_IP6:\r
-        ZeroMem (IfrNvData->TargetIp, sizeof (IfrNvData->TargetIp));\r
-        IScsiIpToStr (&Private->Current->SessionConfigData.TargetIp, TRUE, IfrNvData->TargetIp);\r
-        Private->Current->AutoConfigureMode = 0;\r
-        break;\r
-\r
       case IP_MODE_IP4:\r
         ZeroMem (IfrNvData->TargetIp, sizeof (IfrNvData->TargetIp));\r
-        IScsiIpToStr (&Private->Current->SessionConfigData.TargetIp, FALSE, IfrNvData->TargetIp);\r
         Private->Current->AutoConfigureMode = 0;\r
 \r
         break;\r
@@ -2202,7 +3547,9 @@ IScsiFormCallback (
 \r
     case KEY_LOCAL_IP:\r
       Status = NetLibStrToIp4 (IfrNvData->LocalIp, &HostIp.v4);\r
-      if (EFI_ERROR (Status) || !NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
+      if (EFI_ERROR (Status) || \r
+          ((Private->Current->SessionConfigData.SubnetMask.Addr[0] != 0) && \r
+           !NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), NTOHL(*(UINT32*)Private->Current->SessionConfigData.SubnetMask.Addr)))) {\r
         CreatePopUp (\r
           EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
           &Key,\r
@@ -2236,7 +3583,10 @@ IScsiFormCallback (
 \r
     case KEY_GATE_WAY:\r
       Status = NetLibStrToIp4 (IfrNvData->Gateway, &Gateway.v4);\r
-      if (EFI_ERROR (Status) || ((Gateway.Addr[0] != 0) && !NetIp4IsUnicast (NTOHL (Gateway.Addr[0]), 0))) {\r
+      if (EFI_ERROR (Status) || \r
+          ((Gateway.Addr[0] != 0) && \r
+           (Private->Current->SessionConfigData.SubnetMask.Addr[0] != 0) && \r
+           !NetIp4IsUnicast (NTOHL (Gateway.Addr[0]), NTOHL(*(UINT32*)Private->Current->SessionConfigData.SubnetMask.Addr)))) {\r
         CreatePopUp (\r
           EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
           &Key,\r
@@ -2251,24 +3601,24 @@ IScsiFormCallback (
       break;\r
 \r
     case KEY_TARGET_IP:\r
-      UnicodeStrToAsciiStr (IfrNvData->TargetIp, IpString);\r
+      UnicodeStrToAsciiStrS (IfrNvData->TargetIp, IpString, sizeof (IpString));\r
       Status = IScsiAsciiStrToIp (IpString, IfrNvData->IpMode, &HostIp);\r
       if (EFI_ERROR (Status) || !IpIsUnicast (&HostIp, IfrNvData->IpMode)) {\r
-        CreatePopUp (\r
-          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
-          &Key,\r
-          L"Invalid IP address!",\r
-          NULL\r
-          );       \r
-        Status = EFI_INVALID_PARAMETER;\r
+      //\r
+      // The target is expressed in URL format or an invalid Ip address, just save.\r
+      //\r
+      Private->Current->SessionConfigData.DnsMode = TRUE;\r
+      ZeroMem (&Private->Current->SessionConfigData.TargetIp, sizeof (Private->Current->SessionConfigData.TargetIp));\r
+      UnicodeStrToAsciiStrS (IfrNvData->TargetIp, Private->Current->SessionConfigData.TargetUrl, ISCSI_NAME_MAX_SIZE);\r
       } else {\r
+        Private->Current->SessionConfigData.DnsMode = FALSE;\r
         CopyMem (&Private->Current->SessionConfigData.TargetIp, &HostIp, sizeof (HostIp));\r
       }\r
 \r
       break;\r
 \r
     case KEY_TARGET_NAME:\r
-      UnicodeStrToAsciiStr (IfrNvData->TargetName, IScsiName);\r
+      UnicodeStrToAsciiStrS (IfrNvData->TargetName, IScsiName, ISCSI_NAME_MAX_SIZE);\r
       Status = IScsiNormalizeName (IScsiName, AsciiStrLen (IScsiName));\r
       if (EFI_ERROR (Status)) {\r
         CreatePopUp (\r
@@ -2276,9 +3626,9 @@ IScsiFormCallback (
           &Key,\r
           L"Invalid iSCSI Name!",\r
           NULL\r
-          );       \r
+          );\r
       } else {\r
-        AsciiStrCpy (Private->Current->SessionConfigData.TargetName, IScsiName);\r
+        AsciiStrCpyS (Private->Current->SessionConfigData.TargetName, ISCSI_NAME_MAX_SIZE, IScsiName);\r
       }\r
 \r
       break;\r
@@ -2291,7 +3641,7 @@ IScsiFormCallback (
       break;\r
 \r
     case KEY_BOOT_LUN:\r
-      UnicodeStrToAsciiStr (IfrNvData->BootLun, LunString);\r
+      UnicodeStrToAsciiStrS (IfrNvData->BootLun, LunString, sizeof (LunString));\r
       Status = IScsiAsciiStrToLun (LunString, (UINT8 *) &Lun);\r
       if (EFI_ERROR (Status)) {\r
         CreatePopUp (\r
@@ -2318,30 +3668,34 @@ IScsiFormCallback (
       break;\r
 \r
     case KEY_CHAP_NAME:\r
-      UnicodeStrToAsciiStr (\r
+      UnicodeStrToAsciiStrS (\r
         IfrNvData->CHAPName,\r
-        Private->Current->AuthConfigData.CHAP.CHAPName\r
+        Private->Current->AuthConfigData.CHAP.CHAPName,\r
+        sizeof (Private->Current->AuthConfigData.CHAP.CHAPName)\r
         );\r
       break;\r
 \r
     case KEY_CHAP_SECRET:\r
-      UnicodeStrToAsciiStr (\r
+      UnicodeStrToAsciiStrS (\r
         IfrNvData->CHAPSecret,\r
-        Private->Current->AuthConfigData.CHAP.CHAPSecret\r
+        Private->Current->AuthConfigData.CHAP.CHAPSecret,\r
+        sizeof (Private->Current->AuthConfigData.CHAP.CHAPSecret)\r
         );\r
       break;\r
 \r
     case KEY_REVERSE_CHAP_NAME:\r
-      UnicodeStrToAsciiStr (\r
+      UnicodeStrToAsciiStrS (\r
         IfrNvData->ReverseCHAPName,\r
-        Private->Current->AuthConfigData.CHAP.ReverseCHAPName\r
+        Private->Current->AuthConfigData.CHAP.ReverseCHAPName,\r
+        sizeof (Private->Current->AuthConfigData.CHAP.ReverseCHAPName)\r
         );\r
       break;\r
 \r
     case KEY_REVERSE_CHAP_SECRET:\r
-      UnicodeStrToAsciiStr (\r
+      UnicodeStrToAsciiStrS (\r
         IfrNvData->ReverseCHAPSecret,\r
-        Private->Current->AuthConfigData.CHAP.ReverseCHAPSecret\r
+        Private->Current->AuthConfigData.CHAP.ReverseCHAPSecret,\r
+        sizeof (Private->Current->AuthConfigData.CHAP.ReverseCHAPSecret)\r
         );\r
       break;\r
 \r
@@ -2351,38 +3705,23 @@ IScsiFormCallback (
 \r
       break;\r
 \r
-    case KEY_SAVE_ATTEMPT_CONFIG:\r
-      Status = IScsiConvertIfrNvDataToAttemptConfigData (IfrNvData, Private->Current);\r
-      if (EFI_ERROR (Status)) {\r
-        break;\r
-      }\r
-\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
-      break;\r
-\r
     default:\r
-      Status = IScsiConfigProcessDefault (QuestionId, IfrNvData);\r
       break;\r
     }\r
+  }\r
 \r
-    if (!EFI_ERROR (Status)) {\r
-      //\r
-      // Pass changed uncommitted data back to Form Browser.\r
-      //\r
-      BufferSize = sizeof (ISCSI_CONFIG_IFR_NVDATA);\r
-      HiiSetBrowserData (NULL, NULL, BufferSize, (UINT8 *) IfrNvData, NULL);\r
-    }\r
-\r
-    FreePool (IfrNvData);\r
-    FreePool (IScsiName);\r
-\r
-    return Status;\r
+  if (!EFI_ERROR (Status)) {\r
+    //\r
+    // Pass changed uncommitted data back to Form Browser.\r
+    //\r
+    BufferSize = sizeof (ISCSI_CONFIG_IFR_NVDATA);\r
+    HiiSetBrowserData (NULL, NULL, BufferSize, (UINT8 *) IfrNvData, NULL);\r
   }\r
 \r
-  //\r
-  // All other action return unsupported.\r
-  //\r
-  return EFI_UNSUPPORTED;\r
+  FreePool (IfrNvData);\r
+  FreePool (IScsiName);\r
+\r
+  return Status;\r
 }\r
 \r
 \r
@@ -2432,7 +3771,7 @@ IScsiConfigFormInit (
   // Publish our HII data.\r
   //\r
   CallbackInfo->RegisteredHandle = HiiAddPackages (\r
-                                     &mVendorGuid,\r
+                                     &gIScsiConfigGuid,\r
                                      CallbackInfo->DriverHandle,\r
                                      IScsiDxeStrings,\r
                                      IScsiConfigVfrBin,\r