From a7a523e07b1f804efed99fcf57cf0421904c75a9 Mon Sep 17 00:00:00 2001 From: tye1 Date: Mon, 25 Jan 2010 03:05:41 +0000 Subject: [PATCH] 1.Change the behavior of BdsBoot, to enumerate Boot Options by EfiLoadFileProtocol, other than EfiManagedNetworkServiceBindingProtocol. 2.Add MSG_IPv4_DP and MSG_IPv6_DP into boot type of BDS_EFI_MESSAGE_MAC_BOOT. 3.Change the file header format of inf file and remove the MNP dependency. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@9799 6f19259b-4bc3-4df7-8a09-765794883524 --- .../Library/GenericBdsLib/BdsBoot.c | 114 ++++++------------ .../Library/GenericBdsLib/GenericBdsLib.inf | 7 +- .../Library/GenericBdsLib/InternalBdsLib.h | 3 +- 3 files changed, 39 insertions(+), 85 deletions(-) diff --git a/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsBoot.c b/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsBoot.c index e6b2fe4d17..e9e3435abd 100644 --- a/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsBoot.c +++ b/IntelFrameworkModulePkg/Library/GenericBdsLib/BdsBoot.c @@ -991,8 +991,8 @@ BdsLibEnumerateAllBootOption ( EFI_HANDLE *BlockIoHandles; EFI_BLOCK_IO_PROTOCOL *BlkIo; UINTN Index; - UINTN NumberNetworkHandles; - EFI_HANDLE *NetworkHandles; + UINTN NumOfLoadFileHandles; + EFI_HANDLE *LoadFileHandles; UINTN FvHandleCount; EFI_HANDLE *FvHandleBuffer; EFI_FV_FILETYPE Type; @@ -1171,37 +1171,25 @@ BdsLibEnumerateAllBootOption ( // // Parse Network Boot Device // - NumberNetworkHandles = 0; + NumOfLoadFileHandles = 0; // - // Search MNP Service Binding protocol for UEFI network stack + // Search Load File protocol for PXE boot option. // gBS->LocateHandleBuffer ( ByProtocol, - &gEfiManagedNetworkServiceBindingProtocolGuid, + &gEfiLoadFileProtocolGuid, NULL, - &NumberNetworkHandles, - &NetworkHandles + &NumOfLoadFileHandles, + &LoadFileHandles ); - if (NumberNetworkHandles == 0) { - // - // MNP Service Binding protocol not found, search SNP for EFI network stack - // - gBS->LocateHandleBuffer ( - ByProtocol, - &gEfiSimpleNetworkProtocolGuid, - NULL, - &NumberNetworkHandles, - &NetworkHandles - ); - } - for (Index = 0; Index < NumberNetworkHandles; Index++) { + for (Index = 0; Index < NumOfLoadFileHandles; Index++) { UnicodeSPrint (Buffer, sizeof (Buffer), L"%d", Index); - BdsLibBuildOptionFromHandle (NetworkHandles[Index], BdsBootOptionList, Buffer); + BdsLibBuildOptionFromHandle (LoadFileHandles[Index], BdsBootOptionList, Buffer); } - if (NumberNetworkHandles != 0) { - FreePool (NetworkHandles); + if (NumOfLoadFileHandles != 0) { + FreePool (LoadFileHandles); } // @@ -1556,16 +1544,9 @@ BdsLibNetworkBootWithMediaPresent ( UpdatedDevicePath = DevicePath; // - // Locate MNP Service Binding protocol for UEFI network stack first + // Locate Load File Protocol for PXE boot option first // - Status = gBS->LocateDevicePath (&gEfiManagedNetworkServiceBindingProtocolGuid, &UpdatedDevicePath, &Handle); - if (EFI_ERROR (Status)) { - // - // MNP Service Binding protocol not found, search SNP for EFI network stack - // - UpdatedDevicePath = DevicePath; - Status = gBS->LocateDevicePath (&gEfiSimpleNetworkProtocolGuid, &UpdatedDevicePath, &Handle); - } + Status = gBS->LocateDevicePath (&gEfiLoadFileProtocolGuid, &UpdatedDevicePath, &Handle); if (EFI_ERROR (Status)) { // // Device not present so see if we need to connect it @@ -1575,11 +1556,7 @@ BdsLibNetworkBootWithMediaPresent ( // // This one should work after we did the connect // - Status = gBS->LocateDevicePath (&gEfiManagedNetworkServiceBindingProtocolGuid, &UpdatedDevicePath, &Handle); - if (EFI_ERROR (Status)) { - UpdatedDevicePath = DevicePath; - Status = gBS->LocateDevicePath (&gEfiSimpleNetworkProtocolGuid, &UpdatedDevicePath, &Handle); - } + Status = gBS->LocateDevicePath (&gEfiLoadFileProtocolGuid, &UpdatedDevicePath, &Handle); } } @@ -1727,6 +1704,8 @@ BdsGetBootTypeFromDevicePath ( case MSG_MAC_ADDR_DP: case MSG_VLAN_DP: + case MSG_IPv4_DP: + case MSG_IPv6_DP: BootType = BDS_EFI_MESSAGE_MAC_BOOT; break; @@ -1794,29 +1773,20 @@ BdsLibIsValidEFIBootOptDevicePathExt ( EFI_DEVICE_PATH_PROTOCOL *TempDevicePath; EFI_DEVICE_PATH_PROTOCOL *LastDeviceNode; EFI_BLOCK_IO_PROTOCOL *BlockIo; - EFI_LOAD_FILE_PROTOCOL *LoadFile; TempDevicePath = DevPath; LastDeviceNode = DevPath; // - // Check if it's a valid boot option for network boot device - // Check if there is MNP Service Binding Protocol or SimpleNetworkProtocol - // installed. If yes, that means there is the network card there. + // Check if it's a valid boot option for network boot device. + // Check if there is EfiLoadFileProtocol installed. + // If yes, that means there is a boot option for network. // Status = gBS->LocateDevicePath ( - &gEfiManagedNetworkServiceBindingProtocolGuid, + &gEfiLoadFileProtocolGuid, &TempDevicePath, &Handle ); - if (EFI_ERROR (Status)) { - TempDevicePath = DevPath; - Status = gBS->LocateDevicePath ( - &gEfiSimpleNetworkProtocolGuid, - &TempDevicePath, - &Handle - ); - } if (EFI_ERROR (Status)) { // // Device not present so see if we need to connect it @@ -1824,44 +1794,30 @@ BdsLibIsValidEFIBootOptDevicePathExt ( TempDevicePath = DevPath; BdsLibConnectDevicePath (TempDevicePath); Status = gBS->LocateDevicePath ( - &gEfiManagedNetworkServiceBindingProtocolGuid, + &gEfiLoadFileProtocolGuid, &TempDevicePath, &Handle ); - if (EFI_ERROR (Status)) { - TempDevicePath = DevPath; - Status = gBS->LocateDevicePath ( - &gEfiSimpleNetworkProtocolGuid, - &TempDevicePath, - &Handle - ); - } } if (!EFI_ERROR (Status)) { - // - // Check whether LoadFile protocol is installed - // - Status = gBS->HandleProtocol (Handle, &gEfiLoadFileProtocolGuid, (VOID **)&LoadFile); - if (!EFI_ERROR (Status)) { - if (!IsDevicePathEnd (TempDevicePath)) { - // - // LoadFile protocol is not installed on handle with exactly the same DevPath - // - return FALSE; - } + if (!IsDevicePathEnd (TempDevicePath)) { + // + // LoadFile protocol is not installed on handle with exactly the same DevPath + // + return FALSE; + } - if (CheckMedia) { - // - // Test if it is ready to boot now - // - if (BdsLibNetworkBootWithMediaPresent(DevPath)) { - return TRUE; - } - } else { + if (CheckMedia) { + // + // Test if it is ready to boot now + // + if (BdsLibNetworkBootWithMediaPresent(DevPath)) { return TRUE; } - } + } else { + return TRUE; + } } // diff --git a/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf b/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf index 4aa9f124dd..c5ba83c4a0 100644 --- a/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf +++ b/IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf @@ -1,11 +1,11 @@ -#/** @file +## @file # # General BDS defines and produce general interfaces for platform BDS driver including: # 1) BDS boot policy interface; # 2) BDS boot device connect interface; # 3) BDS Misc interfaces for mainting boot variable, ouput string, etc. # -# Copyright (c) 2007 - 2009, Intel Corporation.
+# Copyright (c) 2007 - 2010, Intel Corporation.
# All rights reserved. This program and the accompanying materials # are licensed and made available under the terms and conditions of the BSD License # which accompanies this distribution. The full text of the license may be found at @@ -14,7 +14,7 @@ # THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, # WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. # -#**/ +## [Defines] INF_VERSION = 0x00010005 @@ -90,7 +90,6 @@ [Protocols] gEfiSimpleFileSystemProtocolGuid # PROTOCOL CONSUMES - gEfiManagedNetworkServiceBindingProtocolGuid # PROTOCOL CONSUMES gEfiLoadFileProtocolGuid # PROTOCOL CONSUMES gEfiSimpleTextOutProtocolGuid # PROTOCOL CONSUMES gEfiPciIoProtocolGuid # PROTOCOL CONSUMES diff --git a/IntelFrameworkModulePkg/Library/GenericBdsLib/InternalBdsLib.h b/IntelFrameworkModulePkg/Library/GenericBdsLib/InternalBdsLib.h index c119523edd..7b4a69494d 100644 --- a/IntelFrameworkModulePkg/Library/GenericBdsLib/InternalBdsLib.h +++ b/IntelFrameworkModulePkg/Library/GenericBdsLib/InternalBdsLib.h @@ -1,7 +1,7 @@ /** @file BDS library definition, include the file and data structure -Copyright (c) 2004 - 2009, Intel Corporation.
+Copyright (c) 2004 - 2010, Intel Corporation.
All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -24,7 +24,6 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. #include #include #include -#include #include #include #include -- 2.39.2