]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/IScsiDxe/IScsiDriver.c
NetworkPkg: Apply uncrustify changes
[mirror_edk2.git] / NetworkPkg / IScsiDxe / IScsiDriver.c
index 2fd4c95a08ec3a7d673356c55787231ed0273498..ba4abd4ef585e86c7a681ec5601fa775031798b6 100644 (file)
@@ -1,41 +1,47 @@
 /** @file\r
   The entry point of IScsi driver.\r
 \r
-Copyright (c) 2004 - 2012, 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
+Copyright (c) 2019, NVIDIA Corporation. All rights reserved.\r
+Copyright (c) 2004 - 2018, Intel Corporation. All rights reserved.<BR>\r
+(C) Copyright 2017 Hewlett Packard Enterprise Development LP<BR>\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
+SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
 \r
 #include "IScsiImpl.h"\r
 \r
-EFI_DRIVER_BINDING_PROTOCOL gIScsiDriverBinding = {\r
-  IScsiDriverBindingSupported,\r
-  IScsiDriverBindingStart,\r
-  IScsiDriverBindingStop,\r
+EFI_DRIVER_BINDING_PROTOCOL  gIScsiIp4DriverBinding = {\r
+  IScsiIp4DriverBindingSupported,\r
+  IScsiIp4DriverBindingStart,\r
+  IScsiIp4DriverBindingStop,\r
   0xa,\r
   NULL,\r
   NULL\r
 };\r
 \r
-EFI_GUID                    gIScsiV4PrivateGuid = ISCSI_V4_PRIVATE_GUID;\r
-EFI_GUID                    gIScsiV6PrivateGuid = ISCSI_V6_PRIVATE_GUID;\r
-ISCSI_PRIVATE_DATA          *mPrivate           = NULL;\r
+EFI_DRIVER_BINDING_PROTOCOL  gIScsiIp6DriverBinding = {\r
+  IScsiIp6DriverBindingSupported,\r
+  IScsiIp6DriverBindingStart,\r
+  IScsiIp6DriverBindingStop,\r
+  0xa,\r
+  NULL,\r
+  NULL\r
+};\r
+\r
+EFI_GUID            gIScsiV4PrivateGuid = ISCSI_V4_PRIVATE_GUID;\r
+EFI_GUID            gIScsiV6PrivateGuid = ISCSI_V6_PRIVATE_GUID;\r
+ISCSI_PRIVATE_DATA  *mPrivate           = NULL;\r
 \r
 /**\r
   Tests to see if this driver supports the RemainingDevicePath.\r
 \r
-  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This \r
-                                   parameter is ignored by device drivers, and is optional for bus \r
-                                   drivers. For bus drivers, if this parameter is not NULL, then \r
-                                   the bus driver must determine if the bus controller specified \r
-                                   by ControllerHandle and the child controller specified \r
-                                   by RemainingDevicePath are both supported by this \r
+  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This\r
+                                   parameter is ignored by device drivers, and is optional for bus\r
+                                   drivers. For bus drivers, if this parameter is not NULL, then\r
+                                   the bus driver must determine if the bus controller specified\r
+                                   by ControllerHandle and the child controller specified\r
+                                   by RemainingDevicePath are both supported by this\r
                                    bus driver.\r
 \r
   @retval EFI_SUCCESS              The RemainingDevicePath is supported or NULL.\r
@@ -44,7 +50,7 @@ ISCSI_PRIVATE_DATA          *mPrivate           = NULL;
 **/\r
 EFI_STATUS\r
 IScsiIsDevicePathSupported (\r
-  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
+  IN EFI_DEVICE_PATH_PROTOCOL  *RemainingDevicePath OPTIONAL\r
   )\r
 {\r
   EFI_DEVICE_PATH_PROTOCOL  *CurrentDevicePath;\r
@@ -65,166 +71,312 @@ IScsiIsDevicePathSupported (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Check whether an iSCSI HBA adapter already installs an AIP instance with\r
+  network boot policy matching the value specified in PcdIScsiAIPNetworkBootPolicy.\r
+  If yes, return EFI_SUCCESS.\r
+\r
+  @retval EFI_SUCCESS              Found an AIP with matching network boot policy.\r
+  @retval EFI_NOT_FOUND            AIP is unavailable or the network boot policy\r
+                                   not matched.\r
+**/\r
+EFI_STATUS\r
+IScsiCheckAip (\r
+  VOID\r
+  )\r
+{\r
+  UINTN                             AipHandleCount;\r
+  EFI_HANDLE                        *AipHandleBuffer;\r
+  UINTN                             AipIndex;\r
+  EFI_ADAPTER_INFORMATION_PROTOCOL  *Aip;\r
+  EFI_EXT_SCSI_PASS_THRU_PROTOCOL   *ExtScsiPassThru;\r
+  EFI_GUID                          *InfoTypesBuffer;\r
+  UINTN                             InfoTypeBufferCount;\r
+  UINTN                             TypeIndex;\r
+  VOID                              *InfoBlock;\r
+  UINTN                             InfoBlockSize;\r
+  BOOLEAN                           Supported;\r
+  EFI_ADAPTER_INFO_NETWORK_BOOT     *NetworkBoot;\r
+  EFI_STATUS                        Status;\r
+  UINT8                             NetworkBootPolicy;\r
+\r
+  //\r
+  // Check any AIP instances exist in system.\r
+  //\r
+  AipHandleCount  = 0;\r
+  AipHandleBuffer = NULL;\r
+  Status          = gBS->LocateHandleBuffer (\r
+                           ByProtocol,\r
+                           &gEfiAdapterInformationProtocolGuid,\r
+                           NULL,\r
+                           &AipHandleCount,\r
+                           &AipHandleBuffer\r
+                           );\r
+  if (EFI_ERROR (Status) || (AipHandleCount == 0)) {\r
+    return EFI_NOT_FOUND;\r
+  }\r
+\r
+  ASSERT (AipHandleBuffer != NULL);\r
+\r
+  InfoBlock = NULL;\r
+\r
+  for (AipIndex = 0; AipIndex < AipHandleCount; AipIndex++) {\r
+    Status = gBS->HandleProtocol (\r
+                    AipHandleBuffer[AipIndex],\r
+                    &gEfiAdapterInformationProtocolGuid,\r
+                    (VOID *)&Aip\r
+                    );\r
+    ASSERT_EFI_ERROR (Status);\r
+    ASSERT (Aip != NULL);\r
+\r
+    Status = gBS->HandleProtocol (\r
+                    AipHandleBuffer[AipIndex],\r
+                    &gEfiExtScsiPassThruProtocolGuid,\r
+                    (VOID *)&ExtScsiPassThru\r
+                    );\r
+    if (EFI_ERROR (Status) || (ExtScsiPassThru == NULL)) {\r
+      continue;\r
+    }\r
+\r
+    InfoTypesBuffer     = NULL;\r
+    InfoTypeBufferCount = 0;\r
+    Status              = Aip->GetSupportedTypes (Aip, &InfoTypesBuffer, &InfoTypeBufferCount);\r
+    if (EFI_ERROR (Status) || (InfoTypesBuffer == NULL)) {\r
+      continue;\r
+    }\r
+\r
+    //\r
+    // Check whether the AIP instance has Network boot information block.\r
+    //\r
+    Supported = FALSE;\r
+    for (TypeIndex = 0; TypeIndex < InfoTypeBufferCount; TypeIndex++) {\r
+      if (CompareGuid (&InfoTypesBuffer[TypeIndex], &gEfiAdapterInfoNetworkBootGuid)) {\r
+        Supported = TRUE;\r
+        break;\r
+      }\r
+    }\r
+\r
+    FreePool (InfoTypesBuffer);\r
+    if (!Supported) {\r
+      continue;\r
+    }\r
+\r
+    //\r
+    // We now have network boot information block.\r
+    //\r
+    InfoBlock     = NULL;\r
+    InfoBlockSize = 0;\r
+    Status        = Aip->GetInformation (Aip, &gEfiAdapterInfoNetworkBootGuid, &InfoBlock, &InfoBlockSize);\r
+    if (EFI_ERROR (Status) || (InfoBlock == NULL)) {\r
+      continue;\r
+    }\r
+\r
+    //\r
+    // Check whether the network boot policy matches.\r
+    //\r
+    NetworkBoot       = (EFI_ADAPTER_INFO_NETWORK_BOOT *)InfoBlock;\r
+    NetworkBootPolicy = PcdGet8 (PcdIScsiAIPNetworkBootPolicy);\r
+\r
+    if (NetworkBootPolicy == STOP_UEFI_ISCSI_IF_HBA_INSTALL_AIP) {\r
+      Status = EFI_SUCCESS;\r
+      goto Exit;\r
+    }\r
+\r
+    if ((((NetworkBootPolicy & STOP_UEFI_ISCSI_IF_AIP_SUPPORT_IP4) != 0) &&\r
+         !NetworkBoot->iScsiIpv4BootCapablity) ||\r
+        (((NetworkBootPolicy & STOP_UEFI_ISCSI_IF_AIP_SUPPORT_IP6) != 0) &&\r
+         !NetworkBoot->iScsiIpv6BootCapablity) ||\r
+        (((NetworkBootPolicy & STOP_UEFI_ISCSI_IF_AIP_SUPPORT_OFFLOAD) != 0) &&\r
+         !NetworkBoot->OffloadCapability) ||\r
+        (((NetworkBootPolicy & STOP_UEFI_ISCSI_IF_AIP_SUPPORT_MPIO) != 0) &&\r
+         !NetworkBoot->iScsiMpioCapability) ||\r
+        (((NetworkBootPolicy & STOP_UEFI_ISCSI_IF_AIP_CONFIGURED_IP4) != 0) &&\r
+         !NetworkBoot->iScsiIpv4Boot) ||\r
+        (((NetworkBootPolicy & STOP_UEFI_ISCSI_IF_AIP_CONFIGURED_IP6) != 0) &&\r
+         !NetworkBoot->iScsiIpv6Boot))\r
+    {\r
+      FreePool (InfoBlock);\r
+      continue;\r
+    }\r
+\r
+    Status = EFI_SUCCESS;\r
+    goto Exit;\r
+  }\r
+\r
+  Status = EFI_NOT_FOUND;\r
+\r
+Exit:\r
+  if (InfoBlock != NULL) {\r
+    FreePool (InfoBlock);\r
+  }\r
+\r
+  if (AipHandleBuffer != NULL) {\r
+    FreePool (AipHandleBuffer);\r
+  }\r
+\r
+  return Status;\r
+}\r
 \r
 /**\r
-  Tests to see if this driver supports a given controller. If a child device is provided, \r
-  it further tests to see if this driver supports creating a handle for the specified child device.\r
-\r
-  This function checks to see if the driver specified by This supports the device specified by \r
-  ControllerHandle. Drivers typically use the device path attached to \r
-  ControllerHandle and/or the services from the bus I/O abstraction attached to \r
-  ControllerHandle to determine if the driver supports ControllerHandle. This function \r
-  may be called many times during platform initialization. In order to reduce boot times, the tests \r
-  performed by this function must be very small and take as little time as possible to execute. This \r
-  function must not change the state of any hardware devices, and this function must be aware that the \r
-  device specified by ControllerHandle may already be managed by the same driver or a \r
-  different driver. This function must match its calls to AllocatePages() with FreePages(), \r
-  AllocatePool() with FreePool(), and OpenProtocol() with CloseProtocol().  \r
-  Since ControllerHandle may have been previously started by the same driver, if a protocol is \r
-  already in the opened state, then it must not be closed with CloseProtocol(). This is required \r
-  to guarantee the state of ControllerHandle is not modified by this function.\r
+  Tests to see if this driver supports a given controller. This is the worker function for\r
+  IScsiIp4(6)DriverBindingSupported.\r
 \r
   @param[in]  This                 A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
-  @param[in]  ControllerHandle     The handle of the controller to test. This handle \r
-                                   must support a protocol interface that supplies \r
+  @param[in]  ControllerHandle     The handle of the controller to test. This handle\r
+                                   must support a protocol interface that supplies\r
                                    an I/O abstraction to the driver.\r
-  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This \r
-                                   parameter is ignored by device drivers, and is optional for bus \r
-                                   drivers. For bus drivers, if this parameter is not NULL, then \r
-                                   the bus driver must determine if the bus controller specified \r
-                                   by ControllerHandle and the child controller specified \r
-                                   by RemainingDevicePath are both supported by this \r
+  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This\r
+                                   parameter is ignored by device drivers, and is optional for bus\r
+                                   drivers. For bus drivers, if this parameter is not NULL, then\r
+                                   the bus driver must determine if the bus controller specified\r
+                                   by ControllerHandle and the child controller specified\r
+                                   by RemainingDevicePath are both supported by this\r
                                    bus driver.\r
+  @param[in]  IpVersion            IP_VERSION_4 or IP_VERSION_6.\r
 \r
   @retval EFI_SUCCESS              The device specified by ControllerHandle and\r
                                    RemainingDevicePath is supported by the driver specified by This.\r
   @retval EFI_ALREADY_STARTED      The device specified by ControllerHandle and\r
                                    RemainingDevicePath is already being managed by the driver\r
                                    specified by This.\r
-  @retval EFI_ACCESS_DENIED        The device specified by ControllerHandle and\r
-                                   RemainingDevicePath is already being managed by a different\r
-                                   driver or an application that requires exclusive access.\r
-                                   Currently not implemented.\r
   @retval EFI_UNSUPPORTED          The device specified by ControllerHandle and\r
                                    RemainingDevicePath is not supported by the driver specified by This.\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
-IScsiDriverBindingSupported (\r
+IScsiSupported (\r
   IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
   IN EFI_HANDLE                   ControllerHandle,\r
-  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL,\r
+  IN UINT8                        IpVersion\r
   )\r
 {\r
-  EFI_STATUS                Status;\r
-  BOOLEAN                   IsIscsi4Started;\r
+  EFI_STATUS  Status;\r
+  EFI_GUID    *IScsiServiceBindingGuid;\r
+  EFI_GUID    *TcpServiceBindingGuid;\r
+  EFI_GUID    *DhcpServiceBindingGuid;\r
+  EFI_GUID    *DnsServiceBindingGuid;\r
+\r
+  if (IpVersion == IP_VERSION_4) {\r
+    IScsiServiceBindingGuid = &gIScsiV4PrivateGuid;\r
+    TcpServiceBindingGuid   = &gEfiTcp4ServiceBindingProtocolGuid;\r
+    DhcpServiceBindingGuid  = &gEfiDhcp4ServiceBindingProtocolGuid;\r
+    DnsServiceBindingGuid   = &gEfiDns4ServiceBindingProtocolGuid;\r
+  } else {\r
+    IScsiServiceBindingGuid = &gIScsiV6PrivateGuid;\r
+    TcpServiceBindingGuid   = &gEfiTcp6ServiceBindingProtocolGuid;\r
+    DhcpServiceBindingGuid  = &gEfiDhcp6ServiceBindingProtocolGuid;\r
+    DnsServiceBindingGuid   = &gEfiDns6ServiceBindingProtocolGuid;\r
+  }\r
 \r
   Status = gBS->OpenProtocol (\r
                   ControllerHandle,\r
-                  &gIScsiV4PrivateGuid,\r
+                  IScsiServiceBindingGuid,\r
                   NULL,\r
                   This->DriverBindingHandle,\r
                   ControllerHandle,\r
                   EFI_OPEN_PROTOCOL_TEST_PROTOCOL\r
                   );\r
   if (!EFI_ERROR (Status)) {\r
-    IsIscsi4Started = TRUE;\r
-  } else {\r
-    Status = gBS->OpenProtocol (\r
-                    ControllerHandle,\r
-                    &gEfiTcp4ServiceBindingProtocolGuid,\r
-                    NULL,\r
-                    This->DriverBindingHandle,\r
-                    ControllerHandle,\r
-                    EFI_OPEN_PROTOCOL_TEST_PROTOCOL\r
-                    );\r
-    if (!EFI_ERROR (Status)) {\r
-      Status = IScsiIsDevicePathSupported (RemainingDevicePath);\r
-      if (!EFI_ERROR (Status)) {\r
-        return EFI_SUCCESS;\r
-      }\r
-    }\r
-\r
-    IsIscsi4Started = FALSE;\r
+    return EFI_ALREADY_STARTED;\r
   }\r
 \r
   Status = gBS->OpenProtocol (\r
                   ControllerHandle,\r
-                  &gIScsiV6PrivateGuid,\r
+                  TcpServiceBindingGuid,\r
                   NULL,\r
                   This->DriverBindingHandle,\r
                   ControllerHandle,\r
                   EFI_OPEN_PROTOCOL_TEST_PROTOCOL\r
                   );\r
-  if (!EFI_ERROR (Status)) {\r
-    if (IsIscsi4Started) {\r
-      return EFI_ALREADY_STARTED;\r
+  if (EFI_ERROR (Status)) {\r
+    return EFI_UNSUPPORTED;\r
+  }\r
+\r
+  Status = IScsiIsDevicePathSupported (RemainingDevicePath);\r
+  if (EFI_ERROR (Status)) {\r
+    return EFI_UNSUPPORTED;\r
+  }\r
+\r
+  if (IScsiDhcpIsConfigured (ControllerHandle, IpVersion)) {\r
+    Status = gBS->OpenProtocol (\r
+                    ControllerHandle,\r
+                    DhcpServiceBindingGuid,\r
+                    NULL,\r
+                    This->DriverBindingHandle,\r
+                    ControllerHandle,\r
+                    EFI_OPEN_PROTOCOL_TEST_PROTOCOL\r
+                    );\r
+    if (EFI_ERROR (Status)) {\r
+      return EFI_UNSUPPORTED;\r
     }\r
-  } else {\r
+  }\r
+\r
+  if (IScsiDnsIsConfigured (ControllerHandle)) {\r
     Status = gBS->OpenProtocol (\r
                     ControllerHandle,\r
-                    &gEfiTcp6ServiceBindingProtocolGuid,\r
+                    DnsServiceBindingGuid,\r
                     NULL,\r
                     This->DriverBindingHandle,\r
                     ControllerHandle,\r
                     EFI_OPEN_PROTOCOL_TEST_PROTOCOL\r
                     );\r
-    if (!EFI_ERROR (Status)) {\r
-      Status = IScsiIsDevicePathSupported (RemainingDevicePath);\r
-      if (!EFI_ERROR (Status)) {\r
-        return EFI_SUCCESS;\r
-      }\r
+    if (EFI_ERROR (Status)) {\r
+      return EFI_UNSUPPORTED;\r
     }\r
   }\r
 \r
-  return EFI_UNSUPPORTED;\r
+  return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Start to manage the controller. This is the worker function for\r
-  IScsiDriverBindingStart.\r
+  IScsiIp4(6)DriverBindingStart.\r
 \r
   @param[in]  Image                Handle of the image.\r
   @param[in]  ControllerHandle     Handle of the controller.\r
-  @param[in]  IpVersion            Ip4 or Ip6\r
+  @param[in]  IpVersion            IP_VERSION_4 or IP_VERSION_6.\r
 \r
-  @retval EFI_SUCCES            This driver supports this device.\r
+  @retval EFI_SUCCESS           This driver was started.\r
   @retval EFI_ALREADY_STARTED   This driver is already running on this device.\r
   @retval EFI_INVALID_PARAMETER Any input parameter is invalid.\r
   @retval EFI_NOT_FOUND         There is no sufficient information to establish\r
                                 the iScsi session.\r
-  @retval EFI_DEVICE_ERROR      Failed to get TCP connection device path.                              \r
+  @retval EFI_OUT_OF_RESOURCES  Failed to allocate memory.\r
+  @retval EFI_DEVICE_ERROR      Failed to get TCP connection device path.\r
+  @retval EFI_ACCESS_DENIED     The protocol could not be removed from the Handle\r
+                                because its interfaces are being used.\r
 \r
 **/\r
 EFI_STATUS\r
 IScsiStart (\r
-  IN EFI_HANDLE                   Image,\r
-  IN EFI_HANDLE                   ControllerHandle,\r
-  IN UINT8                        IpVersion\r
+  IN EFI_HANDLE  Image,\r
+  IN EFI_HANDLE  ControllerHandle,\r
+  IN UINT8       IpVersion\r
   )\r
 {\r
-  EFI_STATUS                      Status;\r
-  ISCSI_DRIVER_DATA               *Private;\r
-  LIST_ENTRY                      *Entry;\r
-  LIST_ENTRY                      *NextEntry;\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA     *AttemptConfigData;\r
-  ISCSI_SESSION                   *Session;\r
-  UINT8                           Index;\r
-  EFI_EXT_SCSI_PASS_THRU_PROTOCOL *ExistIScsiExtScsiPassThru;\r
-  ISCSI_DRIVER_DATA               *ExistPrivate;\r
-  UINT8                           *AttemptConfigOrder;\r
-  UINTN                           AttemptConfigOrderSize;\r
-  UINT8                           BootSelected;\r
-  EFI_HANDLE                      *HandleBuffer;\r
-  UINTN                           NumberOfHandles;\r
-  EFI_DEVICE_PATH_PROTOCOL        *DevicePath;\r
-  EFI_GUID                        *IScsiPrivateGuid;\r
-  EFI_GUID                        *TcpServiceBindingGuid;\r
-  CHAR16                          MacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  BOOLEAN                         NeedUpdate;\r
-  VOID                            *Interface;\r
-  EFI_GUID                        *ProtocolGuid;\r
+  EFI_STATUS                       Status;\r
+  ISCSI_DRIVER_DATA                *Private;\r
+  LIST_ENTRY                       *Entry;\r
+  LIST_ENTRY                       *NextEntry;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA      *AttemptConfigData;\r
+  ISCSI_SESSION                    *Session;\r
+  UINT8                            Index;\r
+  EFI_EXT_SCSI_PASS_THRU_PROTOCOL  *ExistIScsiExtScsiPassThru;\r
+  ISCSI_DRIVER_DATA                *ExistPrivate;\r
+  UINT8                            *AttemptConfigOrder;\r
+  UINTN                            AttemptConfigOrderSize;\r
+  UINT8                            BootSelected;\r
+  EFI_HANDLE                       *HandleBuffer;\r
+  UINTN                            NumberOfHandles;\r
+  EFI_DEVICE_PATH_PROTOCOL         *DevicePath;\r
+  EFI_GUID                         *IScsiPrivateGuid;\r
+  EFI_GUID                         *TcpServiceBindingGuid;\r
+  BOOLEAN                          NeedUpdate;\r
+  VOID                             *Interface;\r
+  EFI_GUID                         *ProtocolGuid;\r
+  UINT8                            NetworkBootPolicy;\r
+  ISCSI_SESSION_CONFIG_NVDATA      *NvData;\r
 \r
   //\r
   // Test to see if iSCSI driver supports the given controller.\r
@@ -266,10 +418,28 @@ IScsiStart (
     return EFI_UNSUPPORTED;\r
   }\r
 \r
+  NetworkBootPolicy = PcdGet8 (PcdIScsiAIPNetworkBootPolicy);\r
+  if (NetworkBootPolicy == ALWAYS_USE_ISCSI_HBA_AND_IGNORE_UEFI_ISCSI) {\r
+    return EFI_ABORTED;\r
+  }\r
+\r
+  if (NetworkBootPolicy != ALWAYS_USE_UEFI_ISCSI_AND_IGNORE_ISCSI_HBA) {\r
+    //\r
+    // Check existing iSCSI AIP.\r
+    //\r
+    Status = IScsiCheckAip ();\r
+    if (!EFI_ERROR (Status)) {\r
+      //\r
+      // Find iSCSI AIP with specified network boot policy. return EFI_ABORTED.\r
+      //\r
+      return EFI_ABORTED;\r
+    }\r
+  }\r
+\r
   //\r
   // Record the incoming NIC info.\r
   //\r
-  Status = IScsiAddNic (ControllerHandle);\r
+  Status = IScsiAddNic (ControllerHandle, Image);\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
@@ -300,20 +470,20 @@ IScsiStart (
   }\r
 \r
   Status = gBS->OpenProtocol (\r
-                  Private->ChildHandle,\r
+                  Private->ChildHandle, /// Default Tcp child\r
                   ProtocolGuid,\r
                   &Interface,\r
                   Image,\r
                   ControllerHandle,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
                   );\r
-                  \r
+\r
   if (EFI_ERROR (Status)) {\r
     goto ON_ERROR;\r
   }\r
 \r
   //\r
-  // Always install private protocol no matter what happens later. We need to \r
+  // Always install private protocol no matter what happens later. We need to\r
   // keep the relationship between ControllerHandle and ChildHandle.\r
   //\r
   Status = gBS->InstallProtocolInterface (\r
@@ -325,7 +495,7 @@ IScsiStart (
   if (EFI_ERROR (Status)) {\r
     goto ON_ERROR;\r
   }\r
-  \r
+\r
   if (IpVersion == IP_VERSION_4) {\r
     mPrivate->Ipv6Flag = FALSE;\r
   } else {\r
@@ -351,7 +521,7 @@ IScsiStart (
   if (mPrivate->OneSessionEstablished && mPrivate->EnableMpio) {\r
     AttemptConfigData = NULL;\r
     NET_LIST_FOR_EACH (Entry, &mPrivate->AttemptConfigs) {\r
-     AttemptConfigData = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
+      AttemptConfigData = NET_LIST_USER_STRUCT (Entry, ISCSI_ATTEMPT_CONFIG_NVDATA, Link);\r
       if (AttemptConfigData->SessionConfigData.Enabled == ISCSI_ENABLED_FOR_MPIO) {\r
         break;\r
       }\r
@@ -391,7 +561,7 @@ IScsiStart (
       Status = gBS->HandleProtocol (\r
                       HandleBuffer[Index],\r
                       &gEfiDevicePathProtocolGuid,\r
-                      (VOID **) &DevicePath\r
+                      (VOID **)&DevicePath\r
                       );\r
       if (EFI_ERROR (Status)) {\r
         continue;\r
@@ -405,7 +575,7 @@ IScsiStart (
           Status = gBS->HandleProtocol (\r
                           HandleBuffer[Index],\r
                           &gEfiExtScsiPassThruProtocolGuid,\r
-                          (VOID **) &ExistIScsiExtScsiPassThru\r
+                          (VOID **)&ExistIScsiExtScsiPassThru\r
                           );\r
           ASSERT_EFI_ERROR (Status);\r
           break;\r
@@ -455,9 +625,10 @@ IScsiStart (
     // Don't process the attempt that does not associate with the current NIC or\r
     // this attempt is disabled or established.\r
     //\r
-    if (AttemptConfigData->NicIndex != mPrivate->CurrentNic ||\r
-        AttemptConfigData->SessionConfigData.Enabled == ISCSI_DISABLED ||\r
-        AttemptConfigData->ValidPath) {\r
+    if ((AttemptConfigData->NicIndex != mPrivate->CurrentNic) ||\r
+        (AttemptConfigData->SessionConfigData.Enabled == ISCSI_DISABLED) ||\r
+        AttemptConfigData->ValidPath)\r
+    {\r
       continue;\r
     }\r
 \r
@@ -466,9 +637,10 @@ IScsiStart (
     // In default single path mode, don't process attempts configured for multipath.\r
     //\r
     if ((mPrivate->EnableMpio &&\r
-         AttemptConfigData->SessionConfigData.Enabled != ISCSI_ENABLED_FOR_MPIO) ||\r
+         (AttemptConfigData->SessionConfigData.Enabled != ISCSI_ENABLED_FOR_MPIO)) ||\r
         (!mPrivate->EnableMpio &&\r
-         AttemptConfigData->SessionConfigData.Enabled != ISCSI_ENABLED)) {\r
+         (AttemptConfigData->SessionConfigData.Enabled != ISCSI_ENABLED)))\r
+    {\r
       continue;\r
     }\r
 \r
@@ -476,18 +648,21 @@ IScsiStart (
     // Don't process the attempt that fails to get the init/target information from DHCP.\r
     //\r
     if (AttemptConfigData->SessionConfigData.InitiatorInfoFromDhcp &&\r
-        !AttemptConfigData->DhcpSuccess) {\r
-      if (!mPrivate->EnableMpio && mPrivate->ValidSinglePathCount > 0) {\r
+        !AttemptConfigData->DhcpSuccess)\r
+    {\r
+      if (!mPrivate->EnableMpio && (mPrivate->ValidSinglePathCount > 0)) {\r
         mPrivate->ValidSinglePathCount--;\r
       }\r
+\r
       continue;\r
     }\r
 \r
     //\r
     // Don't process the autoconfigure path if it is already established.\r
     //\r
-    if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG &&\r
-        AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_SUCCESS) {\r
+    if ((AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) &&\r
+        AttemptConfigData->AutoConfigureSuccess)\r
+    {\r
       continue;\r
     }\r
 \r
@@ -498,16 +673,20 @@ IScsiStart (
       if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_IP6) {\r
         continue;\r
       }\r
-      if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG &&\r
-          AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP6) {\r
+\r
+      if ((AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) &&\r
+          (AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP6))\r
+      {\r
         continue;\r
       }\r
     } else {\r
       if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_IP4) {\r
         continue;\r
       }\r
-      if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG &&\r
-          AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP4) {\r
+\r
+      if ((AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) &&\r
+          (AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP4))\r
+      {\r
         continue;\r
       }\r
     }\r
@@ -515,7 +694,7 @@ IScsiStart (
     //\r
     // Fill in the Session and init it.\r
     //\r
-    Session = (ISCSI_SESSION *) AllocateZeroPool (sizeof (ISCSI_SESSION));\r
+    Session = (ISCSI_SESSION *)AllocateZeroPool (sizeof (ISCSI_SESSION));\r
     if (Session == NULL) {\r
       Status = EFI_OUT_OF_RESOURCES;\r
       goto ON_ERROR;\r
@@ -525,13 +704,11 @@ IScsiStart (
     Session->ConfigData = AttemptConfigData;\r
     Session->AuthType   = AttemptConfigData->AuthenticationType;\r
 \r
-    AsciiStrToUnicodeStr (AttemptConfigData->MacString, MacString);\r
     UnicodeSPrint (\r
       mPrivate->PortString,\r
-      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
-      L"%s%d",\r
-      MacString,\r
-      (UINTN) AttemptConfigData->AttemptConfigIndex\r
+      (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
+      L"Attempt %d",\r
+      (UINTN)AttemptConfigData->AttemptConfigIndex\r
       );\r
 \r
     if (Session->AuthType == ISCSI_AUTH_TYPE_CHAP) {\r
@@ -554,17 +731,34 @@ IScsiStart (
       Status = IScsiSessionReLogin (Session);\r
     }\r
 \r
+    //\r
+    // Restore the original user setting which specifies the proxy/virtual iSCSI target to NV region.\r
+    //\r
+    NvData = &AttemptConfigData->SessionConfigData;\r
+    if (NvData->RedirectFlag) {\r
+      NvData->TargetPort = NvData->OriginalTargetPort;\r
+      CopyMem (&NvData->TargetIp, &NvData->OriginalTargetIp, sizeof (EFI_IP_ADDRESS));\r
+      NvData->RedirectFlag = FALSE;\r
+\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
     if (EFI_ERROR (Status)) {\r
       //\r
       // In Single path mode, only the successful attempt will be recorded in iBFT;\r
       // in multi-path mode, all the attempt entries in MPIO will be recorded in iBFT.\r
       //\r
-      if (!mPrivate->EnableMpio && mPrivate->ValidSinglePathCount > 0) {\r
+      if (!mPrivate->EnableMpio && (mPrivate->ValidSinglePathCount > 0)) {\r
         mPrivate->ValidSinglePathCount--;\r
       }\r
 \r
       FreePool (Session);\r
-\r
     } else {\r
       AttemptConfigData->ValidPath = TRUE;\r
 \r
@@ -573,7 +767,7 @@ IScsiStart (
       // TODO: record KRB5 attempt information in the iSCSI device path.\r
       //\r
       if (Session->AuthType == ISCSI_AUTH_TYPE_KRB) {\r
-        if (!mPrivate->EnableMpio && mPrivate->ValidSinglePathCount > 0) {\r
+        if (!mPrivate->EnableMpio && (mPrivate->ValidSinglePathCount > 0)) {\r
           mPrivate->ValidSinglePathCount--;\r
         }\r
 \r
@@ -586,7 +780,7 @@ IScsiStart (
       // IScsi session success. Update the attempt state to NVR.\r
       //\r
       if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) {\r
-        AttemptConfigData->AutoConfigureMode = IP_MODE_AUTOCONFIG_SUCCESS;\r
+        AttemptConfigData->AutoConfigureSuccess = TRUE;\r
       }\r
 \r
       gRT->SetVariable (\r
@@ -631,7 +825,7 @@ IScsiStart (
     //\r
     // Reinstall the original ExtScsiPassThru back.\r
     //\r
-    if (mPrivate->OneSessionEstablished && ExistPrivate != NULL) {\r
+    if (mPrivate->OneSessionEstablished && (ExistPrivate != NULL)) {\r
       Status = gBS->InstallProtocolInterface (\r
                       &ExistPrivate->ExtScsiPassThruHandle,\r
                       &gEfiExtScsiPassThruProtocolGuid,\r
@@ -654,17 +848,20 @@ IScsiStart (
   //\r
   // More than one attempt successes.\r
   //\r
-  if (Private->Session != NULL && mPrivate->OneSessionEstablished) {\r
-\r
+  if ((Private->Session != NULL) && mPrivate->OneSessionEstablished) {\r
     AttemptConfigOrder = IScsiGetVariableAndSize (\r
                            L"AttemptOrder",\r
                            &gIScsiConfigGuid,\r
                            &AttemptConfigOrderSize\r
                            );\r
-    ASSERT (AttemptConfigOrder != NULL);\r
+    if (AttemptConfigOrder == NULL) {\r
+      goto ON_ERROR;\r
+    }\r
+\r
     for (Index = 0; Index < AttemptConfigOrderSize / sizeof (UINT8); Index++) {\r
-      if (AttemptConfigOrder[Index] == mPrivate->BootSelectedIndex ||\r
-          AttemptConfigOrder[Index] == BootSelected) {\r
+      if ((AttemptConfigOrder[Index] == mPrivate->BootSelectedIndex) ||\r
+          (AttemptConfigOrder[Index] == BootSelected))\r
+      {\r
         break;\r
       }\r
     }\r
@@ -699,7 +896,10 @@ IScsiStart (
 \r
         goto ON_EXIT;\r
       } else {\r
-        ASSERT (AttemptConfigOrder[Index] == BootSelected);\r
+        if (AttemptConfigOrder[Index] != BootSelected) {\r
+          goto ON_ERROR;\r
+        }\r
+\r
         mPrivate->BootSelectedIndex = BootSelected;\r
         //\r
         // Clear the resource in ExistPrivate.\r
@@ -708,14 +908,28 @@ IScsiStart (
                ExistPrivate->Controller,\r
                IScsiPrivateGuid,\r
                &ExistPrivate->IScsiIdentifier\r
-               ); \r
-        \r
+               );\r
+\r
         IScsiRemoveNic (ExistPrivate->Controller);\r
         if (ExistPrivate->Session != NULL) {\r
           IScsiSessionAbort (ExistPrivate->Session);\r
         }\r
 \r
-        IScsiCleanDriverData (ExistPrivate);\r
+        if (ExistPrivate->DevicePath != NULL) {\r
+          Status = gBS->UninstallProtocolInterface (\r
+                          ExistPrivate->ExtScsiPassThruHandle,\r
+                          &gEfiDevicePathProtocolGuid,\r
+                          ExistPrivate->DevicePath\r
+                          );\r
+          if (EFI_ERROR (Status)) {\r
+            goto ON_ERROR;\r
+          }\r
+\r
+          FreePool (ExistPrivate->DevicePath);\r
+        }\r
+\r
+        gBS->CloseEvent (ExistPrivate->ExitBootServiceEvent);\r
+        FreePool (ExistPrivate);\r
       }\r
     } else {\r
       //\r
@@ -725,7 +939,6 @@ IScsiStart (
         NeedUpdate = FALSE;\r
       }\r
     }\r
-\r
   }\r
 \r
   if (NeedUpdate) {\r
@@ -743,6 +956,7 @@ IScsiStart (
     Status = EFI_DEVICE_ERROR;\r
     goto ON_ERROR;\r
   }\r
+\r
   //\r
   // Install the updated device path onto the ExtScsiPassThruHandle.\r
   //\r
@@ -756,6 +970,30 @@ IScsiStart (
     goto ON_ERROR;\r
   }\r
 \r
+  //\r
+  // ISCSI children should share the default Tcp child, just open the default Tcp child via BY_CHILD_CONTROLLER.\r
+  //\r
+  Status = gBS->OpenProtocol (\r
+                  Private->ChildHandle, /// Default Tcp child\r
+                  ProtocolGuid,\r
+                  &Interface,\r
+                  Image,\r
+                  Private->ExtScsiPassThruHandle,\r
+                  EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
+                  );\r
+  if (EFI_ERROR (Status)) {\r
+    gBS->UninstallMultipleProtocolInterfaces (\r
+           Private->ExtScsiPassThruHandle,\r
+           &gEfiExtScsiPassThruProtocolGuid,\r
+           &Private->IScsiExtScsiPassThru,\r
+           &gEfiDevicePathProtocolGuid,\r
+           Private->DevicePath,\r
+           NULL\r
+           );\r
+\r
+    goto ON_ERROR;\r
+  }\r
+\r
 ON_EXIT:\r
 \r
   //\r
@@ -777,115 +1015,44 @@ ON_ERROR:
 }\r
 \r
 /**\r
-  Starts a device controller or a bus controller.\r
-\r
-  The Start() function is designed to be invoked from the EFI boot service ConnectController().\r
-  As a result, much of the error checking on the parameters to Start() has been moved into this \r
-  common boot service. It is legal to call Start() from other locations, \r
-  but the following calling restrictions must be followed or the system behavior will not be deterministic.\r
-  1. ControllerHandle must be a valid EFI_HANDLE.\r
-  2. If RemainingDevicePath is not NULL, then it must be a pointer to a naturally aligned\r
-     EFI_DEVICE_PATH_PROTOCOL.\r
-  3. Prior to calling Start(), the Supported() function for the driver specified by This must\r
-     have been called with the same calling parameters, and Supported() must have returned EFI_SUCCESS.  \r
-\r
-  @param[in]  This                 A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
-  @param[in]  ControllerHandle     The handle of the controller to start. This handle \r
-                                   must support a protocol interface that supplies \r
-                                   an I/O abstraction to the driver.\r
-  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This \r
-                                   parameter is ignored by device drivers, and is optional for bus \r
-                                   drivers. For a bus driver, if this parameter is NULL, then handles \r
-                                   for all the children of Controller are created by this driver.  \r
-                                   If this parameter is not NULL and the first Device Path Node is \r
-                                   not the End of Device Path Node, then only the handle for the \r
-                                   child device specified by the first Device Path Node of \r
-                                   RemainingDevicePath is created by this driver.\r
-                                   If the first Device Path Node of RemainingDevicePath is \r
-                                   the End of Device Path Node, no child handle is created by this\r
-                                   driver.\r
-\r
-  @retval EFI_SUCCESS              The device was started.\r
-  @retval EFI_DEVICE_ERROR         The device could not be started due to a device error.Currently not implemented.\r
-  @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
-  @retval Others                   The driver failed to start the device.\r
+  Stops a device controller or a bus controller. This is the worker function for\r
+  IScsiIp4(6)DriverBindingStop.\r
 \r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-IScsiDriverBindingStart (\r
-  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
-  IN EFI_HANDLE                   ControllerHandle,\r
-  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
-  )\r
-{\r
-  EFI_STATUS        V4Status;\r
-  EFI_STATUS        V6Status;\r
-\r
-  V4Status = IScsiStart (This->DriverBindingHandle, ControllerHandle, IP_VERSION_4);\r
-  if (V4Status == EFI_ALREADY_STARTED) {\r
-    V4Status = EFI_SUCCESS;\r
-  }\r
-\r
-  V6Status = IScsiStart (This->DriverBindingHandle, ControllerHandle, IP_VERSION_6);\r
-  if (V6Status == EFI_ALREADY_STARTED) {\r
-    V6Status = EFI_SUCCESS;\r
-  }\r
-\r
-  if (!EFI_ERROR (V4Status) || !EFI_ERROR (V6Status)) {\r
-    return EFI_SUCCESS;\r
-  } else if (EFI_ERROR (V4Status)) {\r
-    return V4Status;\r
-  } else {\r
-    return V6Status;\r
-  }\r
-}\r
-\r
-/**\r
-  Stops a device controller or a bus controller.\r
-  \r
-  The Stop() function is designed to be invoked from the EFI boot service DisconnectController(). \r
-  As a result, much of the error checking on the parameters to Stop() has been moved \r
-  into this common boot service. It is legal to call Stop() from other locations, \r
-  but the following calling restrictions must be followed or the system behavior will not be deterministic.\r
-  1. ControllerHandle must be a valid EFI_HANDLE that was used on a previous call to this\r
-     same driver's Start() function.\r
-  2. The first NumberOfChildren handles of ChildHandleBuffer must all be a valid\r
-     EFI_HANDLE. In addition, all of these handles must have been created in this driver's\r
-     Start() function, and the Start() function must have called OpenProtocol() on\r
-     ControllerHandle with an Attribute of EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER.\r
-  \r
   @param[in]  This              A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
-  @param[in]  ControllerHandle  A handle to the device being stopped. The handle must \r
-                                support a bus specific I/O protocol for the driver \r
+  @param[in]  ControllerHandle  A handle to the device being stopped. The handle must\r
+                                support a bus specific I/O protocol for the driver\r
                                 to use to stop the device.\r
   @param[in]  NumberOfChildren  The number of child device handles in ChildHandleBuffer.\r
-  @param[in]  ChildHandleBuffer An array of child handles to be freed. May be NULL \r
+  @param[in]  ChildHandleBuffer An array of child handles to be freed. May be NULL\r
                                 if NumberOfChildren is 0.\r
+  @param[in]  IpVersion         IP_VERSION_4 or IP_VERSION_6.\r
 \r
   @retval EFI_SUCCESS           The device was stopped.\r
   @retval EFI_DEVICE_ERROR      The device could not be stopped due to a device error.\r
+  @retval EFI_INVALID_PARAMETER Child handle is NULL.\r
+  @retval EFI_ACCESS_DENIED     The protocol could not be removed from the Handle\r
+                                because its interfaces are being used.\r
 \r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
-IScsiDriverBindingStop (\r
+IScsiStop (\r
   IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
   IN EFI_HANDLE                   ControllerHandle,\r
   IN UINTN                        NumberOfChildren,\r
-  IN EFI_HANDLE                   *ChildHandleBuffer OPTIONAL\r
+  IN EFI_HANDLE                   *ChildHandleBuffer OPTIONAL,\r
+  IN UINT8                        IpVersion\r
   )\r
 {\r
-  EFI_HANDLE                      IScsiController;\r
-  EFI_STATUS                      Status;\r
-  ISCSI_PRIVATE_PROTOCOL          *IScsiIdentifier;\r
-  ISCSI_DRIVER_DATA               *Private;\r
-  EFI_EXT_SCSI_PASS_THRU_PROTOCOL *PassThru;\r
-  ISCSI_CONNECTION                *Conn;\r
-  EFI_GUID                        *ProtocolGuid;\r
-  EFI_GUID                        *TcpServiceBindingGuid;\r
-  EFI_GUID                        *TcpProtocolGuid;\r
-\r
+  EFI_HANDLE                       IScsiController;\r
+  EFI_STATUS                       Status;\r
+  ISCSI_PRIVATE_PROTOCOL           *IScsiIdentifier;\r
+  ISCSI_DRIVER_DATA                *Private;\r
+  EFI_EXT_SCSI_PASS_THRU_PROTOCOL  *PassThru;\r
+  ISCSI_CONNECTION                 *Conn;\r
+  EFI_GUID                         *ProtocolGuid;\r
+  EFI_GUID                         *TcpServiceBindingGuid;\r
+  EFI_GUID                         *TcpProtocolGuid;\r
 \r
   if (NumberOfChildren != 0) {\r
     //\r
@@ -894,7 +1061,7 @@ IScsiDriverBindingStop (
     Status = gBS->OpenProtocol (\r
                     ChildHandleBuffer[0],\r
                     &gEfiExtScsiPassThruProtocolGuid,\r
-                    (VOID **) &PassThru,\r
+                    (VOID **)&PassThru,\r
                     This->DriverBindingHandle,\r
                     ControllerHandle,\r
                     EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -911,12 +1078,19 @@ IScsiDriverBindingStop (
     // the protocol here, but do not uninstall the device path protocol and\r
     // EXT SCSI PASS THRU protocol installed on ExtScsiPassThruHandle.\r
     //\r
-    if (!Conn->Ipv6Flag) {\r
+    if (IpVersion == IP_VERSION_4) {\r
       ProtocolGuid = &gEfiTcp4ProtocolGuid;\r
     } else {\r
       ProtocolGuid = &gEfiTcp6ProtocolGuid;\r
     }\r
 \r
+    gBS->CloseProtocol (\r
+           Private->ChildHandle,\r
+           ProtocolGuid,\r
+           Private->Image,\r
+           Private->ExtScsiPassThruHandle\r
+           );\r
+\r
     gBS->CloseProtocol (\r
            Conn->TcpIo.Handle,\r
            ProtocolGuid,\r
@@ -926,26 +1100,29 @@ IScsiDriverBindingStop (
 \r
     return EFI_SUCCESS;\r
   }\r
+\r
   //\r
-  // Get the handle of the controller we are controling.\r
+  // Get the handle of the controller we are controlling.\r
   //\r
-  IScsiController = NetLibGetNicHandle (ControllerHandle, &gEfiTcp4ProtocolGuid);\r
-  if (IScsiController != NULL) {\r
-    ProtocolGuid            = &gIScsiV4PrivateGuid;\r
-    TcpProtocolGuid         = &gEfiTcp4ProtocolGuid;\r
-    TcpServiceBindingGuid   = &gEfiTcp4ServiceBindingProtocolGuid;\r
+  if (IpVersion == IP_VERSION_4) {\r
+    ProtocolGuid          = &gIScsiV4PrivateGuid;\r
+    TcpProtocolGuid       = &gEfiTcp4ProtocolGuid;\r
+    TcpServiceBindingGuid = &gEfiTcp4ServiceBindingProtocolGuid;\r
   } else {\r
-    IScsiController = NetLibGetNicHandle (ControllerHandle, &gEfiTcp6ProtocolGuid);\r
-    ASSERT (IScsiController != NULL);\r
-    ProtocolGuid            = &gIScsiV6PrivateGuid;\r
-    TcpProtocolGuid         = &gEfiTcp6ProtocolGuid;\r
-    TcpServiceBindingGuid   = &gEfiTcp6ServiceBindingProtocolGuid;\r
+    ProtocolGuid          = &gIScsiV6PrivateGuid;\r
+    TcpProtocolGuid       = &gEfiTcp6ProtocolGuid;\r
+    TcpServiceBindingGuid = &gEfiTcp6ServiceBindingProtocolGuid;\r
+  }\r
+\r
+  IScsiController = NetLibGetNicHandle (ControllerHandle, TcpProtocolGuid);\r
+  if (IScsiController == NULL) {\r
+    return EFI_SUCCESS;\r
   }\r
 \r
   Status = gBS->OpenProtocol (\r
                   IScsiController,\r
                   ProtocolGuid,\r
-                  (VOID **) &IScsiIdentifier,\r
+                  (VOID **)&IScsiIdentifier,\r
                   This->DriverBindingHandle,\r
                   ControllerHandle,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -964,7 +1141,7 @@ IScsiDriverBindingStop (
                     This->DriverBindingHandle,\r
                     IScsiController\r
                     );\r
-                    \r
+\r
     ASSERT (!EFI_ERROR (Status));\r
 \r
     Status = NetLibDestroyServiceChild (\r
@@ -981,7 +1158,7 @@ IScsiDriverBindingStop (
          IScsiController,\r
          ProtocolGuid,\r
          &Private->IScsiIdentifier\r
-         ); \r
+         );\r
 \r
   //\r
   // Remove this NIC.\r
@@ -989,7 +1166,7 @@ IScsiDriverBindingStop (
   IScsiRemoveNic (IScsiController);\r
 \r
   //\r
-  // Update the iSCSI Boot Firware Table.\r
+  // Update the iSCSI Boot Firmware Table.\r
   //\r
   IScsiPublishIbft ();\r
 \r
@@ -997,34 +1174,349 @@ IScsiDriverBindingStop (
     IScsiSessionAbort (Private->Session);\r
   }\r
 \r
-  IScsiCleanDriverData (Private);\r
+  Status = IScsiCleanDriverData (Private);\r
+\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
 \r
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
-  Unload the iSCSI driver.\r
-\r
-  @param[in]  ImageHandle          The handle of the driver image.\r
+  Tests to see if this driver supports a given controller. If a child device is provided,\r
+  it tests to see if this driver supports creating a handle for the specified child device.\r
+\r
+  This function checks to see if the driver specified by This supports the device specified by\r
+  ControllerHandle. Drivers typically use the device path attached to\r
+  ControllerHandle and/or the services from the bus I/O abstraction attached to\r
+  ControllerHandle to determine if the driver supports ControllerHandle. This function\r
+  may be called many times during platform initialization. In order to reduce boot times, the tests\r
+  performed by this function must be very small and take as little time as possible to execute. This\r
+  function must not change the state of any hardware devices, and this function must be aware that the\r
+  device specified by ControllerHandle may already be managed by the same driver or a\r
+  different driver. This function must match its calls to AllocatePages() with FreePages(),\r
+  AllocatePool() with FreePool(), and OpenProtocol() with CloseProtocol().\r
+  Since ControllerHandle may have been previously started by the same driver, if a protocol is\r
+  already in the opened state, then it must not be closed with CloseProtocol(). This is required\r
+  to guarantee the state of ControllerHandle is not modified by this function.\r
 \r
-  @retval     EFI_SUCCESS          The driver is unloaded.\r
-  @retval     EFI_DEVICE_ERROR     An unexpected error occurred.\r
+  @param[in]  This                 A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
+  @param[in]  ControllerHandle     The handle of the controller to test. This handle\r
+                                   must support a protocol interface that supplies\r
+                                   an I/O abstraction to the driver.\r
+  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This\r
+                                   parameter is ignored by device drivers, and is optional for bus\r
+                                   drivers. For bus drivers, if this parameter is not NULL, then\r
+                                   the bus driver must determine if the bus controller specified\r
+                                   by ControllerHandle and the child controller specified\r
+                                   by RemainingDevicePath are both supported by this\r
+                                   bus driver.\r
 \r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
+  @retval EFI_SUCCESS              The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is supported by the driver specified by This.\r
+  @retval EFI_ALREADY_STARTED      The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is already managed by the driver\r
+                                   specified by This.\r
+  @retval EFI_ACCESS_DENIED        The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is already managed by a different\r
+                                   driver or an application that requires exclusive access.\r
+                                   Currently not implemented.\r
+  @retval EFI_UNSUPPORTED          The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is not supported by the driver specified by This.\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiIp4DriverBindingSupported (\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   ControllerHandle,\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
+  )\r
+{\r
+  return IScsiSupported (\r
+           This,\r
+           ControllerHandle,\r
+           RemainingDevicePath,\r
+           IP_VERSION_4\r
+           );\r
+}\r
+\r
+/**\r
+  Starts a device controller or a bus controller.\r
+\r
+  The Start() function is designed to be invoked from the EFI boot service ConnectController().\r
+  As a result, much of the error checking on the parameters to Start() has been moved into this\r
+  common boot service. It is legal to call Start() from other locations,\r
+  but the following calling restrictions must be followed or the system behavior will not be deterministic.\r
+  1. ControllerHandle must be a valid EFI_HANDLE.\r
+  2. If RemainingDevicePath is not NULL, then it must be a pointer to a naturally aligned\r
+     EFI_DEVICE_PATH_PROTOCOL.\r
+  3. Prior to calling Start(), the Supported() function for the driver specified by This must\r
+     have been called with the same calling parameters, and Supported() must have returned EFI_SUCCESS.\r
+\r
+  @param[in]  This                 A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
+  @param[in]  ControllerHandle     The handle of the controller to start. This handle\r
+                                   must support a protocol interface that supplies\r
+                                   an I/O abstraction to the driver.\r
+  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This\r
+                                   parameter is ignored by device drivers, and is optional for bus\r
+                                   drivers. For a bus driver, if this parameter is NULL, then handles\r
+                                   for all the children of Controller are created by this driver.\r
+                                   If this parameter is not NULL and the first Device Path Node is\r
+                                   not the End of Device Path Node, then only the handle for the\r
+                                   child device specified by the first Device Path Node of\r
+                                   RemainingDevicePath is created by this driver.\r
+                                   If the first Device Path Node of RemainingDevicePath is\r
+                                   the End of Device Path Node, no child handle is created by this\r
+                                   driver.\r
+\r
+  @retval EFI_SUCCESS              The device was started.\r
+  @retval EFI_DEVICE_ERROR         The device could not be started due to a device error. Currently not implemented.\r
+  @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
+  @retval Others                   The driver failed to start the device.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiIp4DriverBindingStart (\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   ControllerHandle,\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
+  )\r
+{\r
+  EFI_STATUS  Status;\r
+\r
+  Status = IScsiStart (This->DriverBindingHandle, ControllerHandle, IP_VERSION_4);\r
+  if (Status == EFI_ALREADY_STARTED) {\r
+    Status = EFI_SUCCESS;\r
+  }\r
+\r
+  return Status;\r
+}\r
+\r
+/**\r
+  Stops a device controller or a bus controller.\r
+\r
+  The Stop() function is designed to be invoked from the EFI boot service DisconnectController().\r
+  As a result, much of the error checking on the parameters to Stop() has been moved\r
+  into this common boot service. It is legal to call Stop() from other locations,\r
+  but the following calling restrictions must be followed or the system behavior will not be deterministic.\r
+  1. ControllerHandle must be a valid EFI_HANDLE that was used on a previous call to this\r
+     same driver's Start() function.\r
+  2. The first NumberOfChildren handles of ChildHandleBuffer must all be a valid\r
+     EFI_HANDLE. In addition, all of these handles must have been created in this driver's\r
+     Start() function, and the Start() function must have called OpenProtocol() on\r
+     ControllerHandle with an Attribute of EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER.\r
+\r
+  @param[in]  This              A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
+  @param[in]  ControllerHandle  A handle to the device being stopped. The handle must\r
+                                support a bus specific I/O protocol for the driver\r
+                                to use to stop the device.\r
+  @param[in]  NumberOfChildren  The number of child device handles in ChildHandleBuffer.\r
+  @param[in]  ChildHandleBuffer An array of child handles to be freed. May be NULL\r
+                                if NumberOfChildren is 0.\r
+\r
+  @retval EFI_SUCCESS           The device was stopped.\r
+  @retval EFI_DEVICE_ERROR      The device could not be stopped due to a device error.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiIp4DriverBindingStop (\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   ControllerHandle,\r
+  IN UINTN                        NumberOfChildren,\r
+  IN EFI_HANDLE                   *ChildHandleBuffer OPTIONAL\r
+  )\r
+{\r
+  return IScsiStop (\r
+           This,\r
+           ControllerHandle,\r
+           NumberOfChildren,\r
+           ChildHandleBuffer,\r
+           IP_VERSION_4\r
+           );\r
+}\r
+\r
+/**\r
+  Tests to see if this driver supports a given controller. If a child device is provided,\r
+  it tests to see if this driver supports creating a handle for the specified child device.\r
+\r
+  This function checks to see if the driver specified by This supports the device specified by\r
+  ControllerHandle. Drivers typically use the device path attached to\r
+  ControllerHandle and/or the services from the bus I/O abstraction attached to\r
+  ControllerHandle to determine if the driver supports ControllerHandle. This function\r
+  may be called many times during platform initialization. In order to reduce boot times, the tests\r
+  performed by this function must be very small and take as little time as possible to execute. This\r
+  function must not change the state of any hardware devices, and this function must be aware that the\r
+  device specified by ControllerHandle may already be managed by the same driver or a\r
+  different driver. This function must match its calls to AllocatePages() with FreePages(),\r
+  AllocatePool() with FreePool(), and OpenProtocol() with CloseProtocol().\r
+  Since ControllerHandle may have been previously started by the same driver, if a protocol is\r
+  already in the opened state, then it must not be closed with CloseProtocol(). This is required\r
+  to guarantee the state of ControllerHandle is not modified by this function.\r
+\r
+  @param[in]  This                 A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
+  @param[in]  ControllerHandle     The handle of the controller to test. This handle\r
+                                   must support a protocol interface that supplies\r
+                                   an I/O abstraction to the driver.\r
+  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This\r
+                                   parameter is ignored by device drivers, and is optional for bus\r
+                                   drivers. For bus drivers, if this parameter is not NULL, then\r
+                                   the bus driver must determine if the bus controller specified\r
+                                   by ControllerHandle and the child controller specified\r
+                                   by RemainingDevicePath are both supported by this\r
+                                   bus driver.\r
+\r
+  @retval EFI_SUCCESS              The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is supported by the driver specified by This.\r
+  @retval EFI_ALREADY_STARTED      The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is already managed by the driver\r
+                                   specified by This.\r
+  @retval EFI_ACCESS_DENIED        The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is already managed by a different\r
+                                   driver or an application that requires exclusive access.\r
+                                   Currently not implemented.\r
+  @retval EFI_UNSUPPORTED          The device specified by ControllerHandle and\r
+                                   RemainingDevicePath is not supported by the driver specified by This.\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiIp6DriverBindingSupported (\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   ControllerHandle,\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
+  )\r
+{\r
+  return IScsiSupported (\r
+           This,\r
+           ControllerHandle,\r
+           RemainingDevicePath,\r
+           IP_VERSION_6\r
+           );\r
+}\r
+\r
+/**\r
+  Starts a device controller or a bus controller.\r
+\r
+  The Start() function is designed to be invoked from the EFI boot service ConnectController().\r
+  As a result, much of the error checking on the parameters to Start() has been moved into this\r
+  common boot service. It is legal to call Start() from other locations,\r
+  but the following calling restrictions must be followed or the system behavior will not be deterministic.\r
+  1. ControllerHandle must be a valid EFI_HANDLE.\r
+  2. If RemainingDevicePath is not NULL, then it must be a pointer to a naturally aligned\r
+     EFI_DEVICE_PATH_PROTOCOL.\r
+  3. Prior to calling Start(), the Supported() function for the driver specified by This must\r
+     have been called with the same calling parameters, and Supported() must have returned EFI_SUCCESS.\r
+\r
+  @param[in]  This                 A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
+  @param[in]  ControllerHandle     The handle of the controller to start. This handle\r
+                                   must support a protocol interface that supplies\r
+                                   an I/O abstraction to the driver.\r
+  @param[in]  RemainingDevicePath  A pointer to the remaining portion of a device path.  This\r
+                                   parameter is ignored by device drivers, and is optional for bus\r
+                                   drivers. For a bus driver, if this parameter is NULL, then handles\r
+                                   for all the children of Controller are created by this driver.\r
+                                   If this parameter is not NULL and the first Device Path Node is\r
+                                   not the End of Device Path Node, then only the handle for the\r
+                                   child device specified by the first Device Path Node of\r
+                                   RemainingDevicePath is created by this driver.\r
+                                   If the first Device Path Node of RemainingDevicePath is\r
+                                   the End of Device Path Node, no child handle is created by this\r
+                                   driver.\r
+\r
+  @retval EFI_SUCCESS              The device was started.\r
+  @retval EFI_DEVICE_ERROR         The device could not be started due to a device error. Currently not implemented.\r
+  @retval EFI_OUT_OF_RESOURCES     The request could not be completed due to a lack of resources.\r
+  @retval Others                   The driver failed to start the device.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiIp6DriverBindingStart (\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   ControllerHandle,\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath OPTIONAL\r
+  )\r
+{\r
+  EFI_STATUS  Status;\r
+\r
+  Status = IScsiStart (This->DriverBindingHandle, ControllerHandle, IP_VERSION_6);\r
+  if (Status == EFI_ALREADY_STARTED) {\r
+    Status = EFI_SUCCESS;\r
+  }\r
+\r
+  return Status;\r
+}\r
+\r
+/**\r
+  Stops a device controller or a bus controller.\r
+\r
+  The Stop() function is designed to be invoked from the EFI boot service DisconnectController().\r
+  As a result, much of the error checking on the parameters to Stop() has been moved\r
+  into this common boot service. It is legal to call Stop() from other locations,\r
+  but the following calling restrictions must be followed or the system behavior will not be deterministic.\r
+  1. ControllerHandle must be a valid EFI_HANDLE that was used on a previous call to this\r
+     same driver's Start() function.\r
+  2. The first NumberOfChildren handles of ChildHandleBuffer must all be a valid\r
+     EFI_HANDLE. In addition, all of these handles must have been created in this driver's\r
+     Start() function, and the Start() function must have called OpenProtocol() on\r
+     ControllerHandle with an Attribute of EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER.\r
+\r
+  @param[in]  This              A pointer to the EFI_DRIVER_BINDING_PROTOCOL instance.\r
+  @param[in]  ControllerHandle  A handle to the device being stopped. The handle must\r
+                                support a bus specific I/O protocol for the driver\r
+                                to use to stop the device.\r
+  @param[in]  NumberOfChildren  The number of child device handles in ChildHandleBuffer.\r
+  @param[in]  ChildHandleBuffer An array of child handles to be freed. May be NULL\r
+                                if NumberOfChildren is 0.\r
+\r
+  @retval EFI_SUCCESS           The device was stopped.\r
+  @retval EFI_DEVICE_ERROR      The device could not be stopped due to a device error.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
+IScsiIp6DriverBindingStop (\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   ControllerHandle,\r
+  IN UINTN                        NumberOfChildren,\r
+  IN EFI_HANDLE                   *ChildHandleBuffer OPTIONAL\r
+  )\r
+{\r
+  return IScsiStop (\r
+           This,\r
+           ControllerHandle,\r
+           NumberOfChildren,\r
+           ChildHandleBuffer,\r
+           IP_VERSION_6\r
+           );\r
+}\r
+\r
+/**\r
+  Unload the iSCSI driver.\r
+\r
+  @param[in]  ImageHandle          The handle of the driver image.\r
+\r
+  @retval     EFI_SUCCESS          The driver is unloaded.\r
+  @retval     EFI_DEVICE_ERROR     An unexpected error occurred.\r
+\r
+**/\r
+EFI_STATUS\r
+EFIAPI\r
 IScsiUnload (\r
   IN EFI_HANDLE  ImageHandle\r
   )\r
 {\r
-  EFI_STATUS  Status;\r
-  UINTN       DeviceHandleCount;\r
-  EFI_HANDLE  *DeviceHandleBuffer;\r
-  UINTN       Index;\r
+  EFI_STATUS                    Status;\r
+  UINTN                         DeviceHandleCount;\r
+  EFI_HANDLE                    *DeviceHandleBuffer;\r
+  UINTN                         Index;\r
+  EFI_COMPONENT_NAME_PROTOCOL   *ComponentName;\r
+  EFI_COMPONENT_NAME2_PROTOCOL  *ComponentName2;\r
 \r
   //\r
-  // Try to disonnect the driver from the devices it's controlling.\r
+  // Try to disconnect the driver from the devices it's controlling.\r
   //\r
   Status = gBS->LocateHandleBuffer (\r
                   AllHandles,\r
@@ -1033,62 +1525,203 @@ IScsiUnload (
                   &DeviceHandleCount,\r
                   &DeviceHandleBuffer\r
                   );\r
-  if (!EFI_ERROR (Status)) {\r
-    for (Index = 0; Index < DeviceHandleCount; Index++) {\r
-      Status = gBS->DisconnectController (\r
-                      DeviceHandleBuffer[Index],\r
-                      ImageHandle,\r
-                      NULL\r
-                      );\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
+\r
+  //\r
+  // Disconnect the iSCSI4 driver from the controlled device.\r
+  //\r
+  for (Index = 0; Index < DeviceHandleCount; Index++) {\r
+    Status = IScsiTestManagedDevice (\r
+               DeviceHandleBuffer[Index],\r
+               gIScsiIp4DriverBinding.DriverBindingHandle,\r
+               &gEfiTcp4ProtocolGuid\r
+               )\r
+    ;\r
+    if (EFI_ERROR (Status)) {\r
+      continue;\r
     }\r
 \r
-    if (DeviceHandleBuffer != NULL) {\r
-      FreePool (DeviceHandleBuffer);\r
+    Status = gBS->DisconnectController (\r
+                    DeviceHandleBuffer[Index],\r
+                    gIScsiIp4DriverBinding.DriverBindingHandle,\r
+                    NULL\r
+                    );\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
     }\r
   }\r
+\r
+  //\r
+  // Disconnect the iSCSI6 driver from the controlled device.\r
+  //\r
+  for (Index = 0; Index < DeviceHandleCount; Index++) {\r
+    Status = IScsiTestManagedDevice (\r
+               DeviceHandleBuffer[Index],\r
+               gIScsiIp6DriverBinding.DriverBindingHandle,\r
+               &gEfiTcp6ProtocolGuid\r
+               );\r
+    if (EFI_ERROR (Status)) {\r
+      continue;\r
+    }\r
+\r
+    Status = gBS->DisconnectController (\r
+                    DeviceHandleBuffer[Index],\r
+                    gIScsiIp6DriverBinding.DriverBindingHandle,\r
+                    NULL\r
+                    );\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
+    }\r
+  }\r
+\r
   //\r
   // Unload the iSCSI configuration form.\r
   //\r
-  IScsiConfigFormUnload (gIScsiDriverBinding.DriverBindingHandle);\r
+  Status = IScsiConfigFormUnload (gIScsiIp4DriverBinding.DriverBindingHandle);\r
+  if (EFI_ERROR (Status)) {\r
+    goto ON_EXIT;\r
+  }\r
 \r
   //\r
   // Uninstall the protocols installed by iSCSI driver.\r
   //\r
-  gBS->UninstallMultipleProtocolInterfaces (\r
-         ImageHandle,\r
-         &gEfiAuthenticationInfoProtocolGuid,\r
-         &gIScsiAuthenticationInfo,\r
-         NULL\r
-         );\r
+  Status = gBS->UninstallMultipleProtocolInterfaces (\r
+                  ImageHandle,\r
+                  &gEfiAuthenticationInfoProtocolGuid,\r
+                  &gIScsiAuthenticationInfo,\r
+                  NULL\r
+                  );\r
+  if (EFI_ERROR (Status)) {\r
+    goto ON_EXIT;\r
+  }\r
+\r
+  if (gIScsiControllerNameTable != NULL) {\r
+    Status = FreeUnicodeStringTable (gIScsiControllerNameTable);\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
+    }\r
 \r
-  if (gIScsiControllerNameTable!= NULL) {\r
-    FreeUnicodeStringTable (gIScsiControllerNameTable);\r
     gIScsiControllerNameTable = NULL;\r
   }\r
-  \r
-  return gBS->UninstallMultipleProtocolInterfaces (\r
-                ImageHandle,\r
-                &gEfiDriverBindingProtocolGuid,\r
-                &gIScsiDriverBinding,\r
-                &gEfiComponentName2ProtocolGuid,\r
-                &gIScsiComponentName2,\r
-                &gEfiComponentNameProtocolGuid,\r
-                &gIScsiComponentName,\r
-                &gEfiIScsiInitiatorNameProtocolGuid,\r
-                &gIScsiInitiatorName,\r
-                NULL\r
-                );\r
+\r
+  //\r
+  // Uninstall the ComponentName and ComponentName2 protocol from iSCSI4 driver binding handle\r
+  // if it has been installed.\r
+  //\r
+  Status = gBS->HandleProtocol (\r
+                  gIScsiIp4DriverBinding.DriverBindingHandle,\r
+                  &gEfiComponentNameProtocolGuid,\r
+                  (VOID **)&ComponentName\r
+                  );\r
+  if (!EFI_ERROR (Status)) {\r
+    Status = gBS->UninstallMultipleProtocolInterfaces (\r
+                    gIScsiIp4DriverBinding.DriverBindingHandle,\r
+                    &gEfiComponentNameProtocolGuid,\r
+                    ComponentName,\r
+                    NULL\r
+                    );\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
+    }\r
+  }\r
+\r
+  Status = gBS->HandleProtocol (\r
+                  gIScsiIp4DriverBinding.DriverBindingHandle,\r
+                  &gEfiComponentName2ProtocolGuid,\r
+                  (VOID **)&ComponentName2\r
+                  );\r
+  if (!EFI_ERROR (Status)) {\r
+    gBS->UninstallMultipleProtocolInterfaces (\r
+           gIScsiIp4DriverBinding.DriverBindingHandle,\r
+           &gEfiComponentName2ProtocolGuid,\r
+           ComponentName2,\r
+           NULL\r
+           );\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
+    }\r
+  }\r
+\r
+  //\r
+  // Uninstall the ComponentName and ComponentName2 protocol from iSCSI6 driver binding handle\r
+  // if it has been installed.\r
+  //\r
+  Status = gBS->HandleProtocol (\r
+                  gIScsiIp6DriverBinding.DriverBindingHandle,\r
+                  &gEfiComponentNameProtocolGuid,\r
+                  (VOID **)&ComponentName\r
+                  );\r
+  if (!EFI_ERROR (Status)) {\r
+    Status = gBS->UninstallMultipleProtocolInterfaces (\r
+                    gIScsiIp6DriverBinding.DriverBindingHandle,\r
+                    &gEfiComponentNameProtocolGuid,\r
+                    ComponentName,\r
+                    NULL\r
+                    );\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
+    }\r
+  }\r
+\r
+  Status = gBS->HandleProtocol (\r
+                  gIScsiIp6DriverBinding.DriverBindingHandle,\r
+                  &gEfiComponentName2ProtocolGuid,\r
+                  (VOID **)&ComponentName2\r
+                  );\r
+  if (!EFI_ERROR (Status)) {\r
+    gBS->UninstallMultipleProtocolInterfaces (\r
+           gIScsiIp6DriverBinding.DriverBindingHandle,\r
+           &gEfiComponentName2ProtocolGuid,\r
+           ComponentName2,\r
+           NULL\r
+           );\r
+    if (EFI_ERROR (Status)) {\r
+      goto ON_EXIT;\r
+    }\r
+  }\r
+\r
+  //\r
+  // Uninstall the IScsiInitiatorNameProtocol and all the driver binding protocols.\r
+  //\r
+  Status = gBS->UninstallMultipleProtocolInterfaces (\r
+                  gIScsiIp4DriverBinding.DriverBindingHandle,\r
+                  &gEfiDriverBindingProtocolGuid,\r
+                  &gIScsiIp4DriverBinding,\r
+                  &gEfiIScsiInitiatorNameProtocolGuid,\r
+                  &gIScsiInitiatorName,\r
+                  NULL\r
+                  );\r
+  if (EFI_ERROR (Status)) {\r
+    goto ON_EXIT;\r
+  }\r
+\r
+  Status = gBS->UninstallMultipleProtocolInterfaces (\r
+                  gIScsiIp6DriverBinding.DriverBindingHandle,\r
+                  &gEfiDriverBindingProtocolGuid,\r
+                  &gIScsiIp6DriverBinding,\r
+                  NULL\r
+                  );\r
+\r
+ON_EXIT:\r
+\r
+  if (DeviceHandleBuffer != NULL) {\r
+    FreePool (DeviceHandleBuffer);\r
+  }\r
+\r
+  return Status;\r
 }\r
 \r
 /**\r
   This is the declaration of an EFI image entry point. This entry point is\r
   the same for UEFI Applications, UEFI OS Loaders, and UEFI Drivers including\r
   both device drivers and bus drivers.\r
-  \r
+\r
   The entry point for iSCSI driver which initializes the global variables and\r
   installs the driver binding, component name protocol, iSCSI initiator name\r
   protocol and Authentication Info protocol on its image.\r
-  \r
+\r
   @param[in]  ImageHandle       The firmware allocated handle for the UEFI image.\r
   @param[in]  SystemTable       A pointer to the EFI System Table.\r
 \r
@@ -1099,8 +1732,8 @@ IScsiUnload (
 EFI_STATUS\r
 EFIAPI\r
 IScsiDriverEntryPoint (\r
-  IN EFI_HANDLE         ImageHandle,\r
-  IN EFI_SYSTEM_TABLE   *SystemTable\r
+  IN EFI_HANDLE        ImageHandle,\r
+  IN EFI_SYSTEM_TABLE  *SystemTable\r
   )\r
 {\r
   EFI_STATUS                         Status;\r
@@ -1113,7 +1746,7 @@ IScsiDriverEntryPoint (
   Status = gBS->LocateProtocol (\r
                   &gEfiIScsiInitiatorNameProtocolGuid,\r
                   NULL,\r
-                  (VOID **) &IScsiInitiatorName\r
+                  (VOID **)&IScsiInitiatorName\r
                   );\r
   if (!EFI_ERROR (Status)) {\r
     return EFI_ACCESS_DENIED;\r
@@ -1125,7 +1758,7 @@ IScsiDriverEntryPoint (
   Status = EfiLibInstallDriverBindingComponentName2 (\r
              ImageHandle,\r
              SystemTable,\r
-             &gIScsiDriverBinding,\r
+             &gIScsiIp4DriverBinding,\r
              ImageHandle,\r
              &gIScsiComponentName,\r
              &gIScsiComponentName2\r
@@ -1134,6 +1767,18 @@ IScsiDriverEntryPoint (
     return Status;\r
   }\r
 \r
+  Status = EfiLibInstallDriverBindingComponentName2 (\r
+             ImageHandle,\r
+             SystemTable,\r
+             &gIScsiIp6DriverBinding,\r
+             NULL,\r
+             &gIScsiComponentName,\r
+             &gIScsiComponentName2\r
+             );\r
+  if (EFI_ERROR (Status)) {\r
+    goto Error1;\r
+  }\r
+\r
   //\r
   // Install the iSCSI Initiator Name Protocol.\r
   //\r
@@ -1144,16 +1789,18 @@ IScsiDriverEntryPoint (
                   &gIScsiInitiatorName\r
                   );\r
   if (EFI_ERROR (Status)) {\r
-    goto Error1;\r
-  } \r
+    goto Error2;\r
+  }\r
 \r
   //\r
   // Create the private data structures.\r
   //\r
+  IScsiCHAPInitHashList ();\r
+\r
   mPrivate = AllocateZeroPool (sizeof (ISCSI_PRIVATE_DATA));\r
   if (mPrivate == NULL) {\r
     Status = EFI_OUT_OF_RESOURCES;\r
-    goto Error2;\r
+    goto Error3;\r
   }\r
 \r
   InitializeListHead (&mPrivate->NicInfoList);\r
@@ -1162,9 +1809,25 @@ IScsiDriverEntryPoint (
   //\r
   // Initialize the configuration form of iSCSI.\r
   //\r
-  Status = IScsiConfigFormInit (gIScsiDriverBinding.DriverBindingHandle);\r
+  Status = IScsiConfigFormInit (gIScsiIp4DriverBinding.DriverBindingHandle);\r
   if (EFI_ERROR (Status)) {\r
-    goto Error3;\r
+    goto Error4;\r
+  }\r
+\r
+  //\r
+  // Create the Maximum Attempts.\r
+  //\r
+  Status = IScsiCreateAttempts (PcdGet8 (PcdMaxIScsiAttemptNumber));\r
+  if (EFI_ERROR (Status)) {\r
+    goto Error5;\r
+  }\r
+\r
+  //\r
+  // Create Keywords for all the Attempts.\r
+  //\r
+  Status = IScsiCreateKeywords (PcdGet8 (PcdMaxIScsiAttemptNumber));\r
+  if (EFI_ERROR (Status)) {\r
+    goto Error6;\r
   }\r
 \r
   //\r
@@ -1174,7 +1837,7 @@ IScsiDriverEntryPoint (
   Status = gBS->LocateProtocol (\r
                   &gEfiAuthenticationInfoProtocolGuid,\r
                   NULL,\r
-                  (VOID **) &AuthenticationInfo\r
+                  (VOID **)&AuthenticationInfo\r
                   );\r
   if (Status == EFI_NOT_FOUND) {\r
     Status = gBS->InstallProtocolInterface (\r
@@ -1184,19 +1847,25 @@ IScsiDriverEntryPoint (
                     &gIScsiAuthenticationInfo\r
                     );\r
     if (EFI_ERROR (Status)) {\r
-      goto Error4;\r
-    }    \r
+      goto Error6;\r
+    }\r
   }\r
 \r
   return EFI_SUCCESS;\r
 \r
+Error6:\r
+  IScsiCleanAttemptVariable ();\r
+\r
+Error5:\r
+  IScsiConfigFormUnload (gIScsiIp4DriverBinding.DriverBindingHandle);\r
+\r
 Error4:\r
-  IScsiConfigFormUnload (gIScsiDriverBinding.DriverBindingHandle);\r
+  if (mPrivate != NULL) {\r
+    FreePool (mPrivate);\r
+    mPrivate = NULL;\r
+  }\r
 \r
 Error3:\r
-  FreePool (mPrivate);\r
-\r
-Error2:\r
   gBS->UninstallMultipleProtocolInterfaces (\r
          ImageHandle,\r
          &gEfiIScsiInitiatorNameProtocolGuid,\r
@@ -1204,18 +1873,19 @@ Error2:
          NULL\r
          );\r
 \r
+Error2:\r
+  EfiLibUninstallDriverBindingComponentName2 (\r
+    &gIScsiIp6DriverBinding,\r
+    &gIScsiComponentName,\r
+    &gIScsiComponentName2\r
+    );\r
+\r
 Error1:\r
-  gBS->UninstallMultipleProtocolInterfaces (\r
-         ImageHandle,\r
-         &gEfiDriverBindingProtocolGuid,\r
-         &gIScsiDriverBinding,\r
-         &gEfiComponentName2ProtocolGuid,\r
-         &gIScsiComponentName2,\r
-         &gEfiComponentNameProtocolGuid,\r
-         &gIScsiComponentName,\r
-         NULL\r
-         );\r
+  EfiLibUninstallDriverBindingComponentName2 (\r
+    &gIScsiIp4DriverBinding,\r
+    &gIScsiComponentName,\r
+    &gIScsiComponentName2\r
+    );\r
 \r
   return Status;\r
 }\r
-\r