From 9be29006683dd5fa6c02059691735715d28070c7 Mon Sep 17 00:00:00 2001 From: vanjeff Date: Wed, 16 Sep 2009 03:05:46 +0000 Subject: [PATCH] 1. updated "the Bus Driver that creates all of its child handles on the first call to Start()" not to create any child handle if RemainingDeviepath is the End of Device Path Node, per UEFI 2.3. The others changes include: a. Check RemainingDevicePath at beginning of Supported(), make sure it has been verified before Start() is called. b. Check IO protocol firstly rather than EfiDevicePathProtocolGuid, reduce the times entering into Start() function because EfiDevicePathProtocolGuid existed on most of handle. 2. roll back serial drivers not to create child device, if the device speicifed by remainingdevicepath cannot find in the created devices list. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@9267 6f19259b-4bc3-4df7-8a09-765794883524 --- .../Bus/Isa/IsaSerialDxe/Serial.c | 13 +-- .../Bus/Pci/PciBusDxe/PciBus.c | 93 +++++++++++++------ .../Universal/Disk/PartitionDxe/Partition.c | 91 +++++++++++------- Nt32Pkg/WinNtSerialIoDxe/WinNtSerialIo.c | 13 +-- UnixPkg/UnixSerialIoDxe/UnixSerialIo.c | 11 +-- 5 files changed, 127 insertions(+), 94 deletions(-) diff --git a/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/Serial.c b/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/Serial.c index a5ee00049e..d5b5cfbbc7 100644 --- a/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/Serial.c +++ b/IntelFrameworkModulePkg/Bus/Isa/IsaSerialDxe/Serial.c @@ -385,18 +385,9 @@ SerialControllerDriverStart ( break; } } - FreePool (OpenInfoBuffer); - if (Index < EntryCount) { - // - // If gEfiSerialIoProtocolGuid is opened by one child device, return - // - return Status; - } - // - // If gEfiSerialIoProtocolGuid is not opened by any child device, - // go further to create child device handle based on RemainingDevicePath - // + FreePool (OpenInfoBuffer); + return Status; } if (RemainingDevicePath != NULL) { diff --git a/IntelFrameworkModulePkg/Bus/Pci/PciBusDxe/PciBus.c b/IntelFrameworkModulePkg/Bus/Pci/PciBusDxe/PciBus.c index 7af1b10f7e..1a4adf54b9 100644 --- a/IntelFrameworkModulePkg/Bus/Pci/PciBusDxe/PciBus.c +++ b/IntelFrameworkModulePkg/Bus/Pci/PciBusDxe/PciBus.c @@ -132,39 +132,27 @@ PciBusDriverBindingSupported ( EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL *PciRootBridgeIo; EFI_DEV_PATH_PTR Node; + // + // Check RemainingDevicePath validation + // if (RemainingDevicePath != NULL) { - Node.DevPath = RemainingDevicePath; - if (Node.DevPath->Type != HARDWARE_DEVICE_PATH || - Node.DevPath->SubType != HW_PCI_DP || - DevicePathNodeLength(Node.DevPath) != sizeof(PCI_DEVICE_PATH)) { - return EFI_UNSUPPORTED; + // + // Check if RemainingDevicePath is the End of Device Path Node, + // if yes, go on checking other conditions + // + if (!IsDevicePathEnd (RemainingDevicePath)) { + // + // If RemainingDevicePath isn't the End of Device Path Node, + // check its validation + // + Node.DevPath = RemainingDevicePath; + if (Node.DevPath->Type != HARDWARE_DEVICE_PATH || + Node.DevPath->SubType != HW_PCI_DP || + DevicePathNodeLength(Node.DevPath) != sizeof(PCI_DEVICE_PATH)) { + return EFI_UNSUPPORTED; + } } } - // - // Open the IO Abstraction(s) needed to perform the supported test - // - Status = gBS->OpenProtocol ( - Controller, - &gEfiDevicePathProtocolGuid, - (VOID **) &ParentDevicePath, - This->DriverBindingHandle, - Controller, - EFI_OPEN_PROTOCOL_BY_DRIVER - ); - if (Status == EFI_ALREADY_STARTED) { - return EFI_SUCCESS; - } - - if (EFI_ERROR (Status)) { - return Status; - } - - gBS->CloseProtocol ( - Controller, - &gEfiDevicePathProtocolGuid, - This->DriverBindingHandle, - Controller - ); // // Check if Pci Root Bridge IO protocol is installed by platform @@ -185,6 +173,9 @@ PciBusDriverBindingSupported ( return Status; } + // + // Close the I/O Abstraction(s) used to perform the supported test + // gBS->CloseProtocol ( Controller, &gEfiPciRootBridgeIoProtocolGuid, @@ -192,6 +183,35 @@ PciBusDriverBindingSupported ( Controller ); + // + // Open the EFI Device Path protocol needed to perform the supported test + // + Status = gBS->OpenProtocol ( + Controller, + &gEfiDevicePathProtocolGuid, + (VOID **) &ParentDevicePath, + This->DriverBindingHandle, + Controller, + EFI_OPEN_PROTOCOL_BY_DRIVER + ); + if (Status == EFI_ALREADY_STARTED) { + return EFI_SUCCESS; + } + + if (EFI_ERROR (Status)) { + return Status; + } + + // + // Close protocol, don't use device path protocol in the Support() function + // + gBS->CloseProtocol ( + Controller, + &gEfiDevicePathProtocolGuid, + This->DriverBindingHandle, + Controller + ); + return EFI_SUCCESS; } @@ -219,6 +239,19 @@ PciBusDriverBindingStart ( { EFI_STATUS Status; + // + // Check RemainingDevicePath validation + // + if (RemainingDevicePath != NULL) { + // + // Check if RemainingDevicePath is the End of Device Path Node, + // if yes, return EFI_SUCCESS + // + if (IsDevicePathEnd (RemainingDevicePath)) { + return EFI_SUCCESS; + } + } + Status = gBS->LocateProtocol ( &gEfiIncompatiblePciDeviceSupportProtocolGuid, NULL, diff --git a/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c b/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c index 375f07aa18..fa9f3e3cc4 100644 --- a/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c +++ b/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c @@ -69,42 +69,27 @@ PartitionDriverBindingSupported ( EFI_DISK_IO_PROTOCOL *DiskIo; EFI_DEV_PATH *Node; + // + // Check RemainingDevicePath validation + // if (RemainingDevicePath != NULL) { - Node = (EFI_DEV_PATH *) RemainingDevicePath; - if (Node->DevPath.Type != MEDIA_DEVICE_PATH || + // + // Check if RemainingDevicePath is the End of Device Path Node, + // if yes, go on checking other conditions + // + if (!IsDevicePathEnd (RemainingDevicePath)) { + // + // If RemainingDevicePath isn't the End of Device Path Node, + // check its validation + // + Node = (EFI_DEV_PATH *) RemainingDevicePath; + if (Node->DevPath.Type != MEDIA_DEVICE_PATH || Node->DevPath.SubType != MEDIA_HARDDRIVE_DP || - DevicePathNodeLength (&Node->DevPath) != sizeof (HARDDRIVE_DEVICE_PATH) - ) { + DevicePathNodeLength (&Node->DevPath) != sizeof (HARDDRIVE_DEVICE_PATH)) { return EFI_UNSUPPORTED; + } } } - // - // Open the IO Abstraction(s) needed to perform the supported test - // - Status = gBS->OpenProtocol ( - ControllerHandle, - &gEfiDevicePathProtocolGuid, - (VOID **) &ParentDevicePath, - This->DriverBindingHandle, - ControllerHandle, - EFI_OPEN_PROTOCOL_BY_DRIVER - ); - if (Status == EFI_ALREADY_STARTED) { - return EFI_SUCCESS; - } - - if (EFI_ERROR (Status)) { - return Status; - } - // - // Close the I/O Abstraction(s) used to perform the supported test - // - gBS->CloseProtocol ( - ControllerHandle, - &gEfiDevicePathProtocolGuid, - This->DriverBindingHandle, - ControllerHandle - ); // // Open the IO Abstraction(s) needed to perform the supported test @@ -127,9 +112,38 @@ PartitionDriverBindingSupported ( // // Close the I/O Abstraction(s) used to perform the supported test // + gBS->CloseProtocol ( + ControllerHandle, + &gEfiDiskIoProtocolGuid, + This->DriverBindingHandle, + ControllerHandle + ); + + // + // Open the EFI Device Path protocol needed to perform the supported test + // + Status = gBS->OpenProtocol ( + ControllerHandle, + &gEfiDevicePathProtocolGuid, + (VOID **) &ParentDevicePath, + This->DriverBindingHandle, + ControllerHandle, + EFI_OPEN_PROTOCOL_BY_DRIVER + ); + if (Status == EFI_ALREADY_STARTED) { + return EFI_SUCCESS; + } + + if (EFI_ERROR (Status)) { + return Status; + } + + // + // Close protocol, don't use device path protocol in the Support() function + // gBS->CloseProtocol ( ControllerHandle, - &gEfiDiskIoProtocolGuid, + &gEfiDevicePathProtocolGuid, This->DriverBindingHandle, ControllerHandle ); @@ -181,6 +195,19 @@ PartitionDriverBindingStart ( PARTITION_DETECT_ROUTINE *Routine; BOOLEAN MediaPresent; + // + // Check RemainingDevicePath validation + // + if (RemainingDevicePath != NULL) { + // + // Check if RemainingDevicePath is the End of Device Path Node, + // if yes, return EFI_SUCCESS + // + if (IsDevicePathEnd (RemainingDevicePath)) { + return EFI_SUCCESS; + } + } + Status = gBS->OpenProtocol ( ControllerHandle, &gEfiBlockIoProtocolGuid, diff --git a/Nt32Pkg/WinNtSerialIoDxe/WinNtSerialIo.c b/Nt32Pkg/WinNtSerialIoDxe/WinNtSerialIo.c index 52d7ee8b06..17e4fdacf6 100644 --- a/Nt32Pkg/WinNtSerialIoDxe/WinNtSerialIo.c +++ b/Nt32Pkg/WinNtSerialIoDxe/WinNtSerialIo.c @@ -372,18 +372,9 @@ Returns: break; } } + FreePool (OpenInfoBuffer); - - if (Index < EntryCount) { - // - // If gEfiWinNtIoProtocolGuid is opened by one child device, return - // - return Status; - } - // - // If gEfiWinNtIoProtocolGuid is not opened by any child device, - // go further to create child device handle based on RemainingDevicePath - // + return Status; } if (RemainingDevicePath == NULL) { diff --git a/UnixPkg/UnixSerialIoDxe/UnixSerialIo.c b/UnixPkg/UnixSerialIoDxe/UnixSerialIo.c index ff9bced4e9..e41e74964f 100644 --- a/UnixPkg/UnixSerialIoDxe/UnixSerialIo.c +++ b/UnixPkg/UnixSerialIoDxe/UnixSerialIo.c @@ -466,16 +466,7 @@ Returns: } FreePool (OpenInfoBuffer); - if (Index < EntryCount) { - // - // If gEfiUnixIoProtocolGuid is opened by one child device, return - // - return Status; - } - // - // If gEfiUnixIoProtocolGuid is not opened by any child device, - // go further to create child device handle based on RemainingDevicePath - // + return Status; } if (RemainingDevicePath == NULL) {