]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/IScsiDxe/IScsiConfig.c
MdeModulePkg/DxeCore: invoke the emulator protocol for foreign images
[mirror_edk2.git] / NetworkPkg / IScsiDxe / IScsiConfig.c
index 69a400389dc5d57372b665ac1f0a2e3442dacdb6..b876da7f5ccd3d24732a4221e29b31c00155bb9b 100644 (file)
@@ -1,22 +1,14 @@
 /** @file\r
   Helper functions for configuring or getting the parameters relating to iSCSI.\r
 \r
-Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>\r
-This program and the accompanying materials\r
-are licensed and made available under the terms and conditions of the BSD License\r
-which accompanies this distribution.  The full text of the license may be found at\r
-http://opensource.org/licenses/bsd-license.php\r
-\r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+Copyright (c) 2004 - 2019, Intel Corporation. All rights reserved.<BR>\r
+SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
 \r
 #include "IScsiImpl.h"\r
 \r
 CHAR16          mVendorStorageName[]     = L"ISCSI_CONFIG_IFR_NVDATA";\r
-BOOLEAN         mIScsiDeviceListUpdated  = FALSE;\r
-UINTN           mNumberOfIScsiDevices    = 0;\r
 ISCSI_FORM_CALLBACK_INFO  *mCallbackInfo = NULL;\r
 \r
 HII_VENDOR_DEVICE_PATH  mIScsiHiiVendorDevicePath = {\r
@@ -34,7 +26,7 @@ HII_VENDOR_DEVICE_PATH  mIScsiHiiVendorDevicePath = {
   {\r
     END_DEVICE_PATH_TYPE,\r
     END_ENTIRE_DEVICE_PATH_SUBTYPE,\r
-    { \r
+    {\r
       (UINT8) (END_DEVICE_PATH_LENGTH),\r
       (UINT8) ((END_DEVICE_PATH_LENGTH) >> 8)\r
     }\r
@@ -164,7 +156,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
@@ -202,11 +197,11 @@ IScsiParseIsIdFromString (
 \r
   IsIdStr = (CHAR16 *) String;\r
 \r
-  if (StrLen (IsIdStr) != 6) {\r
+  if (StrLen (IsIdStr) != 6 && StrLen (IsIdStr) != 12) {\r
     UnicodeSPrint (\r
       PortString,\r
       (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
-      L"Error! Input is incorrect, please input 6 hex numbers!\n"\r
+      L"Error! Only last 3 bytes are configurable, please input 6 hex numbers for last 3 bytes only or 12 hex numbers for full SSID!\n"\r
       );\r
 \r
     CreatePopUp (\r
@@ -219,6 +214,10 @@ IScsiParseIsIdFromString (
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
+  if (StrLen (IsIdStr) == 12) {\r
+    IsIdStr += 6;\r
+  }\r
+\r
   for (Index = 3; Index < 6; Index++) {\r
     CopyMem (TempStr, IsIdStr, sizeof (TempStr));\r
     TempStr[2] = L'\0';\r
@@ -286,6 +285,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
@@ -346,6 +437,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
@@ -363,6 +512,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
@@ -370,6 +520,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
@@ -382,15 +533,34 @@ 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
+    ZeroMem (IfrNvData->TargetIp, sizeof (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
@@ -405,16 +575,196 @@ 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
+  ISCSI_NIC_INFO                *NicInfo;\r
+  CHAR16                        MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  EFI_IP_ADDRESS                Ip;\r
+  UINTN                         Index;\r
+  UINTN                         StringLen;\r
+\r
+  NicInfo = NULL;\r
+  ZeroMem (AttemptNameList, sizeof (AttemptNameList));\r
+\r
+  if ((mPrivate != NULL) && (mPrivate->AttemptCount != 0)) {\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
+      ASSERT ((Attempt->AttemptConfigIndex > 0) && (Attempt->AttemptConfigIndex <= FixedPcdGet8 (PcdMaxIScsiAttemptNumber)));\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 - StrLen (AttemptNameList)\r
+      );\r
+\r
+      StringLen = StrLen (AttemptNameList);\r
+      ASSERT (StringLen > 2);\r
+      *(AttemptNameList + StringLen - 2) = L':';\r
+      *(AttemptNameList + StringLen)     = L' ';\r
+\r
+      AsciiStrToUnicodeStrS (\r
+        Attempt->AttemptName,\r
+        IfrNvData->ISCSIAttemptName  + ATTEMPT_NAME_SIZE * Index,\r
+        ATTEMPT_NAME_LIST_SIZE - ATTEMPT_NAME_SIZE * Index\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->Keyword[Index].ISCSITargetIpAddress,\r
+          sizeof (IfrNvData->Keyword[Index].ISCSITargetIpAddress) / sizeof (IfrNvData->Keyword[Index].ISCSITargetIpAddress[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
+    CopyMem(IfrNvData->ISCSIDisplayAttemptList, AttemptNameList, ATTEMPT_NAME_LIST_SIZE);\r
+\r
+    ZeroMem (IfrNvData->ISCSIMacAddr, sizeof (IfrNvData->ISCSIMacAddr));\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
+      CopyMem (\r
+        IfrNvData->ISCSIMacAddr + StrLen (IfrNvData->ISCSIMacAddr),\r
+        MacString,\r
+        StrLen (MacString) * sizeof (CHAR16)\r
+        );\r
+\r
+      *(IfrNvData->ISCSIMacAddr + StrLen (IfrNvData->ISCSIMacAddr)) = L'/';\r
+    }\r
+\r
+    StringLen = StrLen (IfrNvData->ISCSIMacAddr);\r
+    if (StringLen > 0) {\r
+      *(IfrNvData->ISCSIMacAddr + StringLen - 1) = L'\0';\r
+    }\r
+  }\r
 }\r
 \r
 /**\r
@@ -493,7 +843,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
@@ -514,7 +864,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
@@ -523,7 +873,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
@@ -532,14 +882,26 @@ 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
-        CreatePopUp (\r
-          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
-          &Key,\r
-          L"Target IP is invalid!",\r
-          NULL\r
-          );\r
-        return EFI_INVALID_PARAMETER;\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
@@ -558,7 +920,6 @@ IScsiConvertIfrNvDataToAttemptConfigData (
       }\r
     }\r
 \r
-\r
     //\r
     // Validate the authentication info.\r
     //\r
@@ -570,7 +931,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
           L"CHAP Name or CHAP Secret is invalid!",\r
           NULL\r
           );\r
-        \r
+\r
         return EFI_INVALID_PARAMETER;\r
       }\r
 \r
@@ -582,7 +943,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
           &Key,\r
           L"Reverse CHAP Name or Reverse CHAP Secret is invalid!",\r
           NULL\r
-          );        \r
+          );\r
         return EFI_INVALID_PARAMETER;\r
       }\r
     }\r
@@ -592,26 +953,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
@@ -626,7 +980,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
         &Key,\r
         mPrivate->PortString,\r
         NULL\r
-        );       \r
+        );\r
 \r
       FreePool (AttemptName1);\r
       FreePool (AttemptName2);\r
@@ -636,7 +990,6 @@ IScsiConvertIfrNvDataToAttemptConfigData (
   //\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
@@ -663,7 +1016,7 @@ IScsiConvertIfrNvDataToAttemptConfigData (
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
-  AsciiStrToUnicodeStr (Attempt->MacString, MacString);\r
+  AsciiStrToUnicodeStrS (Attempt->MacString, MacString, ISCSI_MAX_MAC_STRING_LEN);\r
 \r
   UnicodeSPrint (\r
     mPrivate->PortString,\r
@@ -760,11 +1113,10 @@ IScsiConvertIfrNvDataToAttemptConfigData (
   } else if (ExistAttempt == NULL) {\r
     //\r
     // When a new attempt is created, pointer of the attempt is saved to\r
-    // mPrivate->NewAttempt, and also saved to mCallbackInfo->Current in\r
-    // IScsiConfigProcessDefault. If input Attempt does not match any existing\r
-    // attempt, it should be a new created attempt. Save it to system now.\r
-    //    \r
-    ASSERT (Attempt == mPrivate->NewAttempt);\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
     // Save current order number for this attempt.\r
@@ -815,11 +1167,6 @@ IScsiConvertIfrNvDataToAttemptConfigData (
     //\r
     InsertTailList (&mPrivate->AttemptConfigs, &Attempt->Link);\r
     mPrivate->AttemptCount++;\r
-    //\r
-    // Reset mPrivate->NewAttempt to NULL, which indicates none attempt is created\r
-    // but not saved now.\r
-    //\r
-    mPrivate->NewAttempt = NULL;\r
 \r
     if (IfrNvData->Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
       //\r
@@ -833,17 +1180,12 @@ IScsiConvertIfrNvDataToAttemptConfigData (
 \r
     IScsiConfigUpdateAttempt ();\r
   }\r
+  Attempt->SessionConfigData.Enabled = IfrNvData->Enabled;\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
+  UnicodeSPrint (mPrivate->PortString, (UINTN) ISCSI_NAME_IFR_MAX_SIZE, L"Attempt %d", Attempt->AttemptConfigIndex);\r
 \r
   FreePool (MacString);\r
 \r
@@ -857,83 +1199,666 @@ IScsiConvertIfrNvDataToAttemptConfigData (
 }\r
 \r
 /**\r
-  Create Hii Extend Label OpCode as the start opcode and end opcode. It is \r
-  a help function.\r
+  Convert the IFR data configured by keyword to iSCSI configuration data.\r
 \r
-  @param[in]  StartLabelNumber   The number of start label.\r
-  @param[out] StartOpCodeHandle  Points to the start opcode handle.\r
-  @param[out] StartLabel         Points to the created start opcode.\r
-  @param[out] EndOpCodeHandle    Points to the end opcode handle.\r
-  @param[out] EndLabel           Points to the created end opcode.\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_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 or configured parameter is invalid.\r
   @retval EFI_SUCCESS            The operation is completed successfully.\r
 \r
 **/\r
 EFI_STATUS\r
-IScsiCreateOpCode (\r
-  IN  UINT16                        StartLabelNumber,\r
-  OUT VOID                          **StartOpCodeHandle,\r
-  OUT EFI_IFR_GUID_LABEL            **StartLabel,\r
-  OUT VOID                          **EndOpCodeHandle,\r
-  OUT EFI_IFR_GUID_LABEL            **EndLabel\r
+IScsiConvertlfrNvDataToAttemptConfigDataByKeyword (\r
+  IN ISCSI_CONFIG_IFR_NVDATA          *IfrNvData,\r
+  IN UINTN                             OffSet\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  EFI_IFR_GUID_LABEL                *InternalStartLabel;\r
-  EFI_IFR_GUID_LABEL                *InternalEndLabel;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA      *Attempt;\r
+  UINT8                            AttemptIndex;\r
+  UINT8                            Index;\r
+  UINT8                            ChapSecretLen;\r
+  UINT8                            ReverseChapSecretLen;\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
-  if (StartOpCodeHandle == NULL || StartLabel == NULL || EndOpCodeHandle == NULL || EndLabel == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
+  Attempt = NULL;\r
+  ZeroMem (IScsiName, sizeof (IScsiName));\r
 \r
-  *StartOpCodeHandle = NULL;\r
-  *EndOpCodeHandle   = NULL;\r
-  Status             = EFI_OUT_OF_RESOURCES;\r
+  if (OffSet < ATTEMPT_BOOTENABLE_VAR_OFFSET) {\r
+    return EFI_SUCCESS;\r
 \r
-  //\r
-  // Initialize the container for dynamic opcodes.\r
-  //\r
-  *StartOpCodeHandle = HiiAllocateOpCodeHandle ();\r
-  if (*StartOpCodeHandle == NULL) {\r
-    return Status;\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
-  *EndOpCodeHandle = HiiAllocateOpCodeHandle ();\r
-  if (*EndOpCodeHandle == NULL) {\r
-    goto Exit;\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
-  //\r
-  // Create Hii Extend Label OpCode as the start opcode.\r
-  //\r
-  InternalStartLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (\r
-                                                *StartOpCodeHandle,\r
-                                                &gEfiIfrTianoGuid,\r
-                                                NULL,\r
-                                                sizeof (EFI_IFR_GUID_LABEL)\r
-                                                );\r
-  if (InternalStartLabel == NULL) {\r
-    goto Exit;\r
-  }\r
-  \r
-  InternalStartLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
-  InternalStartLabel->Number       = StartLabelNumber;\r
+        AsciiStrToUnicodeStrS (Attempt->AttemptName, AttemptName1, ATTEMPT_NAME_SIZE);\r
+        AsciiStrToUnicodeStrS (SameNicAttempt->AttemptName, AttemptName2, ATTEMPT_NAME_SIZE);\r
 \r
-  //\r
-  // Create Hii Extend Label OpCode as the end opcode.\r
-  //\r
-  InternalEndLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (\r
-                                              *EndOpCodeHandle,\r
-                                              &gEfiIfrTianoGuid,\r
-                                              NULL,\r
-                                              sizeof (EFI_IFR_GUID_LABEL)\r
-                                              );\r
-  if (InternalEndLabel == NULL) {\r
-    goto Exit;\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
+\r
+    if (IfrNvData->ISCSIConnectRetry[AttemptIndex - 1] > CONNECT_MAX_RETRY) {\r
+      CreatePopUp (\r
+          EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+          &Key,\r
+          L"The minimum value is 0 and the maximum is 16. 0 means no retry.",\r
+          NULL\r
+          );\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    Attempt->SessionConfigData.ConnectRetryCount = IfrNvData->ISCSIConnectRetry[AttemptIndex - 1];\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
+\r
+    if ((IfrNvData->ISCSIConnectTimeout[AttemptIndex - 1] < CONNECT_MIN_TIMEOUT) ||\r
+        (IfrNvData->ISCSIConnectTimeout[AttemptIndex - 1] > CONNECT_MAX_TIMEOUT)) {\r
+      CreatePopUp (\r
+        EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+        &Key,\r
+        L"The minimum value is 100 milliseconds and the maximum is 20 seconds.",\r
+        NULL\r
+        );\r
+      return EFI_INVALID_PARAMETER;\r
+    }\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
+  } 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->Keyword[Index].ISCSIChapUsername[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
+        ChapSecretLen = (UINT8)StrLen (IfrNvData->Keyword[Index].ISCSIChapSecret);\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 ((ChapSecretLen < ISCSI_CHAP_SECRET_MIN_LEN) || (ChapSecretLen > ISCSI_CHAP_SECRET_MAX_LEN)) {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"The Chap Secret minimum length is 12 bytes and the maximum length is 16 bytes.",\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->Keyword[Index].ISCSIReverseChapUsername[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
+        ReverseChapSecretLen = (UINT8)StrLen (IfrNvData->Keyword[Index].ISCSIReverseChapSecret);\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 ((ReverseChapSecretLen < ISCSI_CHAP_SECRET_MIN_LEN) || (ReverseChapSecretLen > ISCSI_CHAP_SECRET_MAX_LEN)) {\r
+            CreatePopUp (\r
+              EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+              &Key,\r
+              L"The Reverse CHAP Secret minimum length is 12 bytes and the maximum length is 16 bytes.",\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
+\r
+\r
+  //\r
+  // Record the user configuration information in NVR.\r
+  //\r
+  ASSERT (Attempt != NULL);\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
+                ISCSI_CONFIG_VAR_ATTR,\r
+                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
+  a help function.\r
+\r
+  @param[in]  StartLabelNumber   The number of start label.\r
+  @param[out] StartOpCodeHandle  Points to the start opcode handle.\r
+  @param[out] StartLabel         Points to the created start opcode.\r
+  @param[out] EndOpCodeHandle    Points to the end opcode handle.\r
+  @param[out] EndLabel           Points to the created end opcode.\r
+\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_SUCCESS            The operation is completed successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+IScsiCreateOpCode (\r
+  IN  UINT16                        StartLabelNumber,\r
+  OUT VOID                          **StartOpCodeHandle,\r
+  OUT EFI_IFR_GUID_LABEL            **StartLabel,\r
+  OUT VOID                          **EndOpCodeHandle,\r
+  OUT EFI_IFR_GUID_LABEL            **EndLabel\r
+  )\r
+{\r
+  EFI_STATUS                        Status;\r
+  EFI_IFR_GUID_LABEL                *InternalStartLabel;\r
+  EFI_IFR_GUID_LABEL                *InternalEndLabel;\r
+\r
+  if (StartOpCodeHandle == NULL || StartLabel == NULL || EndOpCodeHandle == NULL || EndLabel == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  *StartOpCodeHandle = NULL;\r
+  *EndOpCodeHandle   = NULL;\r
+  Status             = EFI_OUT_OF_RESOURCES;\r
+\r
+  //\r
+  // Initialize the container for dynamic opcodes.\r
+  //\r
+  *StartOpCodeHandle = HiiAllocateOpCodeHandle ();\r
+  if (*StartOpCodeHandle == NULL) {\r
+    return Status;\r
+  }\r
+\r
+  *EndOpCodeHandle = HiiAllocateOpCodeHandle ();\r
+  if (*EndOpCodeHandle == NULL) {\r
+    goto Exit;\r
+  }\r
+\r
+  //\r
+  // Create Hii Extend Label OpCode as the start opcode.\r
+  //\r
+  InternalStartLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (\r
+                                                *StartOpCodeHandle,\r
+                                                &gEfiIfrTianoGuid,\r
+                                                NULL,\r
+                                                sizeof (EFI_IFR_GUID_LABEL)\r
+                                                );\r
+  if (InternalStartLabel == NULL) {\r
+    goto Exit;\r
+  }\r
+\r
+  InternalStartLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
+  InternalStartLabel->Number       = StartLabelNumber;\r
+\r
+  //\r
+  // Create Hii Extend Label OpCode as the end opcode.\r
+  //\r
+  InternalEndLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (\r
+                                              *EndOpCodeHandle,\r
+                                              &gEfiIfrTianoGuid,\r
+                                              NULL,\r
+                                              sizeof (EFI_IFR_GUID_LABEL)\r
+                                              );\r
+  if (InternalEndLabel == NULL) {\r
+    goto Exit;\r
+  }\r
 \r
   InternalEndLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
   InternalEndLabel->Number       = LABEL_END;\r
@@ -952,10 +1877,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
@@ -1026,7 +2018,7 @@ IScsiConfigAddAttempt (
     PortTitleHelpToken = HiiSetString (mCallbackInfo->RegisteredHandle, 0, mPrivate->PortString, NULL);\r
     if (PortTitleHelpToken == 0) {\r
       Status = EFI_INVALID_PARAMETER;\r
-      goto Exit;      \r
+      goto Exit;\r
     }\r
 \r
     HiiCreateGotoOpCode (\r
@@ -1050,80 +2042,188 @@ IScsiConfigAddAttempt (
 Exit:\r
   HiiFreeOpCodeHandle (StartOpCodeHandle);\r
   HiiFreeOpCodeHandle (EndOpCodeHandle);\r
-  \r
+\r
   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
+  for (Index = 0; Index < PcdGet8 (PcdMaxIScsiAttemptNumber); Index++) {\r
+    if (AttemptList[Index] == 0) {\r
+      continue;\r
+    }\r
+\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
-    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
+    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
-    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
+    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
 \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
+    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
-  HiiFreeOpCodeHandle (StartOpCodeHandle);\r
-  HiiFreeOpCodeHandle (EndOpCodeHandle);\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
@@ -1140,19 +2240,22 @@ 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
@@ -1171,7 +2274,6 @@ IScsiConfigDeleteAttempts (
 \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
@@ -1184,10 +2286,6 @@ IScsiConfigDeleteAttempts (
     //\r
 \r
     AttemptConfigData = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
-    if (AttemptConfigData == NULL) {\r
-      Status = EFI_NOT_FOUND;\r
-      goto Error;\r
-    }\r
 \r
     //\r
     // Remove this attempt from UI configured attempt list.\r
@@ -1216,22 +2314,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
@@ -1245,6 +2365,9 @@ IScsiConfigDeleteAttempts (
     }\r
 \r
     NewTotal--;\r
+    if (mCallbackInfo->Current == AttemptConfigData) {\r
+      mCallbackInfo->Current = NULL;\r
+    }\r
     FreePool (AttemptConfigData);\r
 \r
     //\r
@@ -1284,7 +2407,7 @@ Error:
   if (AttemptNewOrder != NULL) {\r
     FreePool (AttemptNewOrder);\r
   }\r
-  \r
+\r
   return Status;\r
 }\r
 \r
@@ -1370,7 +2493,7 @@ IScsiConfigDisplayDeleteAttempts (
              FORMID_DELETE_FORM,              // Form ID\r
              StartOpCodeHandle,               // Label for where to insert opcodes\r
              EndOpCodeHandle                  // Replace data\r
-             );    \r
+             );\r
 \r
   HiiFreeOpCodeHandle (StartOpCodeHandle);\r
   HiiFreeOpCodeHandle (EndOpCodeHandle);\r
@@ -1401,8 +2524,8 @@ IScsiConfigDisplayOrderAttempts (
   EFI_IFR_GUID_LABEL          *StartLabel;\r
   VOID                        *EndOpCodeHandle;\r
   EFI_IFR_GUID_LABEL          *EndLabel;\r
-  VOID                        *OptionsOpCodeHandle;  \r
-  \r
+  VOID                        *OptionsOpCodeHandle;\r
+\r
   Status = IScsiCreateOpCode (\r
              ORDER_ENTRY_LABEL,\r
              &StartOpCodeHandle,\r
@@ -1454,14 +2577,14 @@ IScsiConfigDisplayOrderAttempts (
     DYNAMIC_ORDERED_LIST_QUESTION_ID,           // Question ID\r
     CONFIGURATION_VARSTORE_ID,                  // VarStore ID\r
     DYNAMIC_ORDERED_LIST_VAR_OFFSET,            // Offset in Buffer Storage\r
-    STRING_TOKEN (STR_ORDER_ATTEMPT_ENTRY),     // Question prompt text        \r
-    STRING_TOKEN (STR_ORDER_ATTEMPT_ENTRY),     // Question help text       \r
+    STRING_TOKEN (STR_ORDER_ATTEMPT_ENTRY),     // Question prompt text\r
+    STRING_TOKEN (STR_ORDER_ATTEMPT_ENTRY),     // Question help text\r
     0,                                          // Question flag\r
     EFI_IFR_UNIQUE_SET,                         // Ordered list flag, e.g. EFI_IFR_UNIQUE_SET\r
     EFI_IFR_NUMERIC_SIZE_1,                     // Data type of Question value\r
     ISCSI_MAX_ATTEMPTS_NUM,                     // Maximum container\r
-    OptionsOpCodeHandle,                        // Option Opcode list                        \r
-    NULL                                        // Default Opcode is NULL                               \r
+    OptionsOpCodeHandle,                        // Option Opcode list\r
+    NULL                                        // Default Opcode is NULL\r
     );\r
 \r
 Exit:\r
@@ -1471,11 +2594,11 @@ Exit:
              FORMID_ORDER_FORM,               // Form ID\r
              StartOpCodeHandle,               // Label for where to insert opcodes\r
              EndOpCodeHandle                  // Replace data\r
-             );             \r
+             );\r
 \r
 Error:\r
   HiiFreeOpCodeHandle (StartOpCodeHandle);\r
-  HiiFreeOpCodeHandle (EndOpCodeHandle);  \r
+  HiiFreeOpCodeHandle (EndOpCodeHandle);\r
   if (OptionsOpCodeHandle != NULL) {\r
     HiiFreeOpCodeHandle (OptionsOpCodeHandle);\r
   }\r
@@ -1483,9 +2606,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
@@ -1601,6 +2723,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
@@ -1612,16 +2735,16 @@ 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
   UINTN                       Index;\r
+  EFI_INPUT_KEY               Key;\r
 \r
+  AttemptConfigData = NULL;\r
   //\r
   // Is User creating a new attempt?\r
   //\r
@@ -1646,14 +2769,6 @@ IScsiConfigProcessDefault (
     //\r
     return EFI_SUCCESS;\r
   }\r
-  \r
-  //\r
-  // Free any attempt that is previously created but not saved to system.\r
-  //\r
-  if (mPrivate->NewAttempt != NULL) {\r
-    FreePool (mPrivate->NewAttempt);\r
-    mPrivate->NewAttempt = NULL;\r
-  }\r
 \r
   if (NewAttempt) {\r
     //\r
@@ -1664,57 +2779,49 @@ IScsiConfigProcessDefault (
     if (NicInfo == NULL) {\r
       return EFI_NOT_FOUND;\r
     }\r
-    \r
-    //\r
-    // Create new attempt.\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
+                           L"InitialAttemptOrder",\r
                            &gIScsiConfigGuid,\r
                            &AttemptConfigOrderSize\r
                            );\r
 \r
-    TotalNumber = AttemptConfigOrderSize / sizeof (UINT8);\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
-        }\r
+      return EFI_NOT_FOUND;\r
+    }\r
+\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
-      CurrentAttemptConfigIndex++;\r
+      break;\r
     }\r
 \r
-    TotalNumber++;\r
-\r
-    //\r
-    // Record the mapping between attempt order and attempt's configdata.\r
-    //\r
-    AttemptConfigData->AttemptConfigIndex  = CurrentAttemptConfigIndex;\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
 \r
     if (AttemptConfigOrder != NULL) {\r
       FreePool (AttemptConfigOrder);\r
@@ -1730,14 +2837,16 @@ IScsiConfigProcessDefault (
       MacString\r
       );\r
 \r
-    UnicodeStrToAsciiStr (MacString, AttemptConfigData->MacString);\r
+    ASSERT (AttemptConfigData != NULL);\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
     //\r
     CopyMem (AttemptConfigData->SessionConfigData.IsId, &NicInfo->PermanentAddress, 6);\r
-    AttemptConfigData->SessionConfigData.IsId[0] = \r
+    AttemptConfigData->SessionConfigData.IsId[0] =\r
       (UINT8) (AttemptConfigData->SessionConfigData.IsId[0] & 0x3F);\r
 \r
     //\r
@@ -1761,27 +2870,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
-    //\r
-    // Save the created Attempt temporarily. If user does not save the attempt\r
-    // by press 'KEY_SAVE_ATTEMPT_CONFIG' later, iSCSI driver would know that\r
-    // and free resources.\r
-    //\r
-    mPrivate->NewAttempt = (VOID *) AttemptConfigData;\r
-\r
   } else {\r
     //\r
     // Determine which Attempt user has selected to configure.\r
@@ -1805,7 +2896,7 @@ IScsiConfigProcessDefault (
     ZeroMem (IfrNvData->ReverseCHAPName, sizeof (IfrNvData->ReverseCHAPName));\r
     ZeroMem (IfrNvData->ReverseCHAPSecret, sizeof (IfrNvData->ReverseCHAPSecret));\r
   }\r
-  \r
+\r
   IScsiConvertAttemptConfigDataToIfrNvData (AttemptConfigData, IfrNvData);\r
 \r
   //\r
@@ -1818,7 +2909,7 @@ IScsiConfigProcessDefault (
 \r
 \r
 /**\r
-   \r
+\r
   This function allows the caller to request the current\r
   configuration for one or more named elements. The resulting\r
   string is in <ConfigAltResp> format. Also, any and all alternative\r
@@ -1847,7 +2938,7 @@ IScsiConfigProcessDefault (
                          to the most recent "&" before the first\r
                          failing name / value pair (or the beginning\r
                          of the string if the failure is in the first\r
-                         name / value pair) if the request was not successful.                        \r
+                         name / value pair) if the request was not successful.\r
 \r
   @param[out] Results    A null-terminated Unicode string in\r
                          <ConfigAltResp> format which has all values\r
@@ -1868,7 +2959,7 @@ IScsiConfigProcessDefault (
                                   would result in this type of\r
                                   error. In this case, the\r
                                   Progress parameter would be\r
-                                  set to NULL. \r
+                                  set to NULL.\r
 \r
   @retval EFI_NOT_FOUND           Routing data doesn't match any\r
                                   known driver. Progress set to the\r
@@ -1926,23 +3017,33 @@ 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
     FreePool (IfrNvData);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
-    \r
+\r
   Status = gIScsiInitiatorName.Get (&gIScsiInitiatorName, &BufferSize, InitiatorName);\r
   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
@@ -1959,7 +3060,11 @@ IScsiFormExtractConfig (
     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
@@ -1997,7 +3102,7 @@ IScsiFormExtractConfig (
 \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
@@ -2010,8 +3115,8 @@ IScsiFormExtractConfig (
   @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
+                             <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
@@ -2022,16 +3127,16 @@ IScsiFormExtractConfig (
 \r
   @retval EFI_SUCCESS             The results have been distributed or are\r
                                   awaiting distribution.\r
-  \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
+\r
   @retval EFI_INVALID_PARAMETERS  Passing in a NULL for the\r
                                   Results parameter would result\r
                                   in this type of error.\r
-  \r
+\r
   @retval EFI_NOT_FOUND           Target for the specified routing data\r
                                   was not found.\r
 \r
@@ -2044,6 +3149,27 @@ IScsiFormRouteConfig (
   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
+  Status = EFI_SUCCESS;\r
+\r
   if (This == NULL || Configuration == NULL || Progress == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -2057,13 +3183,186 @@ IScsiFormRouteConfig (
     return EFI_NOT_FOUND;\r
   }\r
 \r
-  *Progress = Configuration + StrLen (Configuration);\r
-  return EFI_SUCCESS;\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 = IScsiGetValue (Configuration, L"&OFFSET=", &OffSet);\r
+    if (EFI_ERROR (Status)) {\r
+      goto Exit;\r
+    }\r
+\r
+    if (OffSet >= ATTEMPT_MAC_ADDR_VAR_OFFSET) {\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
+    } else {\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
+    } 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
+    } 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
+      if ((NicInfo == NULL) || (NicInfo->NicIndex == 0)) {\r
+        Status = EFI_NOT_FOUND;\r
+        goto Exit;\r
+      }\r
+\r
+    } else {\r
+      Status = IScsiConvertlfrNvDataToAttemptConfigDataByKeyword (IfrNvData, OffSet);\r
+      if (EFI_ERROR (Status)) {\r
+        goto Exit;\r
+      }\r
+    }\r
+  }\r
 \r
+  IScsiConfigUpdateAttempt ();\r
+\r
+Exit:\r
+  if (InitiatorName != NULL) {\r
+    FreePool (InitiatorName);\r
+  }\r
+\r
+  if (IfrNvData != NULL) {\r
+    FreePool (IfrNvData);\r
+  }\r
+\r
+  return Status;\r
+}\r
 \r
 /**\r
-   \r
+\r
   This function is called to provide results data to the driver.\r
   This data consists of a unique key that is used to identify\r
   which data is either being passed back or being asked for.\r
@@ -2072,7 +3371,7 @@ IScsiFormRouteConfig (
   @param[in]       Action        Specifies the type of action taken by the browser.\r
   @param[in]       QuestionId    A unique value which is sent to the original\r
                                  exporting driver so that it can identify the type\r
-                                 of data to expect. The format of the data tends to \r
+                                 of data to expect. The format of the data tends to\r
                                  vary based on the opcode that generated the callback.\r
   @param[in]       Type          The type of value for the question.\r
   @param[in, out]  Value         A pointer to the data being sent to the original\r
@@ -2101,7 +3400,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
@@ -2110,8 +3409,10 @@ 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
+  ISCSI_NIC_INFO              *NicInfo;\r
+\r
+  NicInfo = NULL;\r
 \r
   if ((Action == EFI_BROWSER_ACTION_FORM_OPEN) || (Action == EFI_BROWSER_ACTION_FORM_CLOSE)) {\r
     //\r
@@ -2132,29 +3433,29 @@ IScsiFormCallback (
   }\r
 \r
   Private = ISCSI_FORM_CALLBACK_INFO_FROM_FORM_CALLBACK (This);\r
-  \r
+\r
   //\r
   // Retrieve uncommitted data from Browser\r
   //\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
+\r
   IScsiName = (CHAR8 *) AllocateZeroPool (ISCSI_NAME_MAX_SIZE);\r
   if (IScsiName == NULL) {\r
     FreePool (IfrNvData);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
-  \r
+\r
   Status = EFI_SUCCESS;\r
-  \r
+\r
   ZeroMem (&OldIfrNvData, BufferSize);\r
-  \r
+\r
   HiiGetBrowserData (NULL, NULL, BufferSize, (UINT8 *) IfrNvData);\r
-  \r
+\r
   CopyMem (&OldIfrNvData, IfrNvData, BufferSize);\r
 \r
   if (Action == EFI_BROWSER_ACTION_CHANGING) {\r
@@ -2175,10 +3476,10 @@ IScsiFormCallback (
           &Key,\r
           L"Error: please configure iSCSI initiator name first!",\r
           NULL\r
-          );        \r
+          );\r
         break;\r
       }\r
-      \r
+\r
       Status = IScsiConfigAddAttempt ();\r
       break;\r
 \r
@@ -2202,15 +3503,15 @@ IScsiFormCallback (
         );\r
       IScsiConfigDisplayOrderAttempts ();\r
       break;\r
-    \r
+\r
     default:\r
       Status = IScsiConfigProcessDefault (QuestionId, IfrNvData);\r
       break;\r
     }\r
-  } else if (Action == EFI_BROWSER_ACTION_CHANGED) {  \r
+  } else if (Action == EFI_BROWSER_ACTION_CHANGED) {\r
     switch (QuestionId) {\r
     case KEY_INITIATOR_NAME:\r
-      UnicodeStrToAsciiStr (IfrNvData->InitiatorName, IScsiName);\r
+      UnicodeStrToAsciiStrS (IfrNvData->InitiatorName, IScsiName, ISCSI_NAME_MAX_SIZE);\r
       BufferSize  = AsciiStrSize (IScsiName);\r
 \r
       Status      = gIScsiInitiatorName.Set (&gIScsiInitiatorName, &BufferSize, IScsiName);\r
@@ -2220,37 +3521,19 @@ IScsiFormCallback (
           &Key,\r
           L"Invalid iSCSI Name!",\r
           NULL\r
-          );      \r
-      }\r
-\r
-      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_APPLY;\r
-      break;\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
       }\r
 \r
-      UnicodeStrToAsciiStr (IfrNvData->AttemptName, Private->Current->AttemptName);\r
-\r
-      IScsiConfigUpdateAttempt ();\r
-\r
       *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_APPLY;\r
       break;\r
-      \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_APPLY;\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_FORM_SUBMIT_EXIT;\r
       break;\r
 \r
     case KEY_SAVE_ORDER_CHANGES:\r
@@ -2300,15 +3583,35 @@ IScsiFormCallback (
     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
+        NicInfo = IScsiGetNicInfoByIndex (Private->Current->NicIndex);\r
+        if(NicInfo == NULL) {\r
+          break;\r
+        }\r
+\r
+        if(!NicInfo->Ipv6Available) {\r
+          //\r
+          // Current NIC doesn't Support IPv6, hence use IPv4.\r
+          //\r
+          IfrNvData->IpMode = IP_MODE_IP4;\r
+\r
+          CreatePopUp (\r
+            EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,\r
+            &Key,\r
+            L"Current NIC doesn't Support IPv6!",\r
+            NULL\r
+            );\r
+        }\r
 \r
       case IP_MODE_IP4:\r
+        ZeroMem (IfrNvData->LocalIp, sizeof (IfrNvData->LocalIp));\r
+        ZeroMem (IfrNvData->SubnetMask, sizeof (IfrNvData->SubnetMask));\r
+        ZeroMem (IfrNvData->Gateway, sizeof (IfrNvData->Gateway));\r
         ZeroMem (IfrNvData->TargetIp, sizeof (IfrNvData->TargetIp));\r
-        IScsiIpToStr (&Private->Current->SessionConfigData.TargetIp, FALSE, IfrNvData->TargetIp);\r
         Private->Current->AutoConfigureMode = 0;\r
+        ZeroMem (&Private->Current->SessionConfigData.LocalIp, sizeof (EFI_IP_ADDRESS));\r
+        ZeroMem (&Private->Current->SessionConfigData.SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
+        ZeroMem (&Private->Current->SessionConfigData.Gateway, sizeof (EFI_IP_ADDRESS));\r
+        ZeroMem (&Private->Current->SessionConfigData.TargetIp, sizeof (EFI_IP_ADDRESS));\r
 \r
         break;\r
       }\r
@@ -2317,14 +3620,16 @@ 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
           L"Invalid IP address!",\r
           NULL\r
-          ); \r
-        \r
+          );\r
+\r
         Status = EFI_INVALID_PARAMETER;\r
       } else {\r
         CopyMem (&Private->Current->SessionConfigData.LocalIp, &HostIp.v4, sizeof (HostIp.v4));\r
@@ -2340,8 +3645,8 @@ IScsiFormCallback (
           &Key,\r
           L"Invalid Subnet Mask!",\r
           NULL\r
-          ); \r
-        \r
+          );\r
+\r
         Status = EFI_INVALID_PARAMETER;\r
       } else {\r
         CopyMem (&Private->Current->SessionConfigData.SubnetMask, &SubnetMask.v4, sizeof (SubnetMask.v4));\r
@@ -2351,13 +3656,16 @@ 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
           L"Invalid Gateway!",\r
           NULL\r
-          );       \r
+          );\r
         Status = EFI_INVALID_PARAMETER;\r
       } else {\r
         CopyMem (&Private->Current->SessionConfigData.Gateway, &Gateway.v4, sizeof (Gateway.v4));\r
@@ -2366,24 +3674,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
@@ -2406,7 +3714,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
@@ -2414,7 +3722,7 @@ IScsiFormCallback (
           &Key,\r
           L"Invalid LUN string!",\r
           NULL\r
-          );       \r
+          );\r
       } else {\r
         CopyMem (Private->Current->SessionConfigData.BootLun, &Lun, sizeof (Lun));\r
       }\r
@@ -2433,30 +3741,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
@@ -2527,7 +3839,7 @@ IScsiConfigFormInit (
                   NULL\r
                   );\r
   ASSERT_EFI_ERROR (Status);\r
-  \r
+\r
   //\r
   // Publish our HII data.\r
   //\r
@@ -2596,13 +3908,6 @@ IScsiConfigFormUnload (
 \r
   ASSERT (mPrivate->NicCount == 0);\r
 \r
-  //\r
-  // Free attempt is created but not saved to system.\r
-  //\r
-  if (mPrivate->NewAttempt != NULL) {\r
-    FreePool (mPrivate->NewAttempt);\r
-  }\r
-\r
   FreePool (mPrivate);\r
   mPrivate = NULL;\r
 \r