]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/IScsiDxe/IScsiDriver.c
Fix a bug about the iSCSI DHCP dependency issue.
[mirror_edk2.git] / NetworkPkg / IScsiDxe / IScsiDriver.c
index 2fd4c95a08ec3a7d673356c55787231ed0273498..6d6f9a13ed8b8f33059872bc1f1dd3b7b27682ca 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   The entry point of IScsi driver.\r
 \r
-Copyright (c) 2004 - 2012, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2013, Intel Corporation. All rights reserved.<BR>\r
 This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -14,10 +14,19 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \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_DRIVER_BINDING_PROTOCOL gIScsiIp6DriverBinding = {\r
+  IScsiIp6DriverBindingSupported,\r
+  IScsiIp6DriverBindingStart,\r
+  IScsiIp6DriverBindingStop,\r
   0xa,\r
   NULL,\r
   NULL\r
@@ -28,7 +37,7 @@ EFI_GUID                    gIScsiV6PrivateGuid = ISCSI_V6_PRIVATE_GUID;
 ISCSI_PRIVATE_DATA          *mPrivate           = NULL;\r
 \r
 /**\r
-  Tests to see if this driver supports the RemainingDevicePath.\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
@@ -67,22 +76,8 @@ IScsiIsDevicePathSupported (
 \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
@@ -95,101 +90,96 @@ IScsiIsDevicePathSupported (
                                    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_GUID                  *IScsiServiceBindingGuid;\r
+  EFI_GUID                  *TcpServiceBindingGuid;\r
+  EFI_GUID                  *DhcpServiceBindingGuid;\r
+\r
+  if (IpVersion == IP_VERSION_4) {\r
+    IScsiServiceBindingGuid  = &gIScsiV4PrivateGuid;\r
+    TcpServiceBindingGuid    = &gEfiTcp4ServiceBindingProtocolGuid;\r
+    DhcpServiceBindingGuid   = &gEfiDhcp4ServiceBindingProtocolGuid;\r
+  } else {\r
+    IScsiServiceBindingGuid  = &gIScsiV6PrivateGuid;\r
+    TcpServiceBindingGuid    = &gEfiTcp6ServiceBindingProtocolGuid;\r
+    DhcpServiceBindingGuid   = &gEfiDhcp6ServiceBindingProtocolGuid;\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
-    }\r
-  } else {\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
-                    &gEfiTcp6ServiceBindingProtocolGuid,\r
+                    DhcpServiceBindingGuid,\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
+  \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_SUCCES            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
@@ -777,83 +767,8 @@ 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
-\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
+  Stops a device controller or a bus controller. This is the worker function for\r
+  IScsiIp4(6)DriverBindingStop.\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
@@ -862,18 +777,20 @@ IScsiDriverBindingStart (
   @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
+  @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
 \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
@@ -911,7 +828,7 @@ 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
@@ -929,18 +846,19 @@ IScsiDriverBindingStop (
   //\r
   // Get the handle of the controller we are controling.\r
   //\r
-  IScsiController = NetLibGetNicHandle (ControllerHandle, &gEfiTcp4ProtocolGuid);\r
-  if (IScsiController != NULL) {\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
   }\r
+  IScsiController = NetLibGetNicHandle (ControllerHandle, TcpProtocolGuid);\r
+  if (IScsiController == NULL) {\r
+    return EFI_SUCCESS;\r
+  }\r
 \r
   Status = gBS->OpenProtocol (\r
                   IScsiController,\r
@@ -1002,6 +920,315 @@ IScsiDriverBindingStop (
   return EFI_SUCCESS;\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
+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
@@ -1033,23 +1260,27 @@ 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
-    }\r
+  if (EFI_ERROR (Status)) {\r
+    return Status;\r
+  }\r
 \r
-    if (DeviceHandleBuffer != NULL) {\r
-      FreePool (DeviceHandleBuffer);\r
-    }\r
+  for (Index = 0; Index < DeviceHandleCount; Index++) {\r
+    gBS->DisconnectController (\r
+           DeviceHandleBuffer[Index],\r
+           gIScsiIp4DriverBinding.DriverBindingHandle,\r
+           NULL\r
+           );\r
+    gBS->DisconnectController (\r
+           DeviceHandleBuffer[Index],\r
+           gIScsiIp6DriverBinding.DriverBindingHandle,\r
+           NULL\r
+           );\r
   }\r
+\r
   //\r
   // Unload the iSCSI configuration form.\r
   //\r
-  IScsiConfigFormUnload (gIScsiDriverBinding.DriverBindingHandle);\r
+  IScsiConfigFormUnload (gIScsiIp4DriverBinding.DriverBindingHandle);\r
 \r
   //\r
   // Uninstall the protocols installed by iSCSI driver.\r
@@ -1066,18 +1297,31 @@ IScsiUnload (
     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
+  gBS->UninstallMultipleProtocolInterfaces (\r
+         gIScsiIp4DriverBinding.DriverBindingHandle,\r
+         &gEfiDriverBindingProtocolGuid,\r
+         &gIScsiIp4DriverBinding,\r
+         &gEfiComponentName2ProtocolGuid,\r
+         &gIScsiComponentName2,\r
+         &gEfiComponentNameProtocolGuid,\r
+         &gIScsiComponentName,\r
+         &gEfiIScsiInitiatorNameProtocolGuid,\r
+         &gIScsiInitiatorName,\r
+         NULL\r
+         );\r
+\r
+  gBS->UninstallMultipleProtocolInterfaces (\r
+         gIScsiIp6DriverBinding.DriverBindingHandle,\r
+         &gEfiDriverBindingProtocolGuid,\r
+         &gIScsiIp6DriverBinding,\r
+         &gEfiComponentName2ProtocolGuid,\r
+         &gIScsiComponentName2,\r
+         &gEfiComponentNameProtocolGuid,\r
+         &gIScsiComponentName,\r
+         NULL\r
+         );\r
+\r
+  return EFI_SUCCESS;\r
 }\r
 \r
 /**\r
@@ -1125,7 +1369,7 @@ IScsiDriverEntryPoint (
   Status = EfiLibInstallDriverBindingComponentName2 (\r
              ImageHandle,\r
              SystemTable,\r
-             &gIScsiDriverBinding,\r
+             &gIScsiIp4DriverBinding,\r
              ImageHandle,\r
              &gIScsiComponentName,\r
              &gIScsiComponentName2\r
@@ -1134,6 +1378,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,7 +1400,7 @@ IScsiDriverEntryPoint (
                   &gIScsiInitiatorName\r
                   );\r
   if (EFI_ERROR (Status)) {\r
-    goto Error1;\r
+    goto Error2;\r
   } \r
 \r
   //\r
@@ -1153,7 +1409,7 @@ IScsiDriverEntryPoint (
   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 +1418,9 @@ 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
@@ -1184,19 +1440,19 @@ IScsiDriverEntryPoint (
                     &gIScsiAuthenticationInfo\r
                     );\r
     if (EFI_ERROR (Status)) {\r
-      goto Error4;\r
+      goto Error5;\r
     }    \r
   }\r
 \r
   return EFI_SUCCESS;\r
 \r
-Error4:\r
-  IScsiConfigFormUnload (gIScsiDriverBinding.DriverBindingHandle);\r
+Error5:\r
+  IScsiConfigFormUnload (gIScsiIp4DriverBinding.DriverBindingHandle);\r
 \r
-Error3:\r
+Error4:\r
   FreePool (mPrivate);\r
 \r
-Error2:\r
+Error3:\r
   gBS->UninstallMultipleProtocolInterfaces (\r
          ImageHandle,\r
          &gEfiIScsiInitiatorNameProtocolGuid,\r
@@ -1204,11 +1460,23 @@ Error2:
          NULL\r
          );\r
 \r
+Error2:\r
+  gBS->UninstallMultipleProtocolInterfaces (\r
+         gIScsiIp6DriverBinding.DriverBindingHandle,\r
+         &gEfiDriverBindingProtocolGuid,\r
+         &gIScsiIp6DriverBinding,\r
+         &gEfiComponentName2ProtocolGuid,\r
+         &gIScsiComponentName2,\r
+         &gEfiComponentNameProtocolGuid,\r
+         &gIScsiComponentName,\r
+         NULL\r
+         );\r
+\r
 Error1:\r
   gBS->UninstallMultipleProtocolInterfaces (\r
          ImageHandle,\r
          &gEfiDriverBindingProtocolGuid,\r
-         &gIScsiDriverBinding,\r
+         &gIScsiIp4DriverBinding,\r
          &gEfiComponentName2ProtocolGuid,\r
          &gIScsiComponentName2,\r
          &gEfiComponentNameProtocolGuid,\r