[edk2] [PATCH 0/3] IntelFrameworkModulePkg: Fixes for Intel BDS

2016-04-29 Thread Daniil Egranov
This set of patches fixes the following: Crash when BDS loads a device handle based onthe device path but the device driver has not been loaded. BDS boot timeout. Timeout progress message for RS-232 serial console. Thanks, Daniil Daniil Egranov (3): IntelFrameworkModulePkg/GenericBdsLib: Che

[edk2] [PATCH 2/3] IntelFrameworkModulePkg/BdsDxe: Fix for the BDS boot timeout

2016-04-29 Thread Daniil Egranov
The patch loads timeout value from the "Timeout" global variable and passes it to PlatformBdsEnterFrontPage(), which handles delay and key input. The PcdPlatformBootTimeOut is only used at the BDS entry point and updates the "Timeout" value. This will allow the modification of the timeout value th

[edk2] [PATCH 1/3] IntelFrameworkModulePkg/GenericBdsLib: Check for invalid device handle

2016-04-29 Thread Daniil Egranov
Added error control for invalid device handle when LocateDevicePath() fails. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Daniil Egranov --- IntelFrameworkModulePkg/Library/GenericBdsLib/BdsBoot.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git

[edk2] [PATCH 3/3] IntelFrameworkModulePkg/BdsDxe: Show boot timeout message

2016-04-29 Thread Daniil Egranov
The PlatformBdsShowProgress() supports graphics mode only, which is not applicable for RS-232 serial console. Show the progress message as a console text message in case PlatformBdsShowProgress() fails. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Daniil Egranov --- Int

Re: [edk2] [Patch v4 00/23] Use MdeModulePkg/BDS in OVMF platform

2016-04-29 Thread Ni, Ruiyu
https://github.com/niruiyu/edk2/commits/Ovmf_Bds Regards, Ray From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo Ersek Sent: Friday, April 29, 2016 7:21 PM To: Ni, Ruiyu ; edk2-de...@ml01.01.org Subject: Re: [edk2] [Patch v4 00/23] Use MdeModulePkg/BDS in OVMF platform

Re: [edk2] [Patch v4 00/23] Use MdeModulePkg/BDS in OVMF platform

2016-04-29 Thread Ni, Ruiyu
https://github.com/niruiyu/edk2/commits/Ovmf_Bds Regards, Ray From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo Ersek Sent: Friday, April 29, 2016 7:21 PM To: Ni, Ruiyu ; edk2-de...@ml01.01.org Subject: Re: [edk2] [Patch v4 00/23] Use MdeModulePkg/BDS in OVMF platfor

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Carsey, Jaben
I missed that the default was from the RFC. I agree with you. -Jaben > -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Friday, April 29, 2016 11:00 AM > To: Carsey, Jaben > Cc: Marcin Wojtas ; Fu, Siyuan ; > ha...@marvell.com; edk2-devel@lists.01.org;

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Leif Lindholm
Since the default is actually defined in the RFC, I would prefer this to not be PCD configurable. Having it as an option to the shell command makes a lot of sense, though. / Leif On Fri, Apr 29, 2016 at 05:44:50PM +, Carsey, Jaben wrote: > Maybe we should have a PCD control the default va

Re: [edk2] [review]Please review changes to UNI file format document

2016-04-29 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge From: Carsey, Jaben Sent: Friday, April 29, 2016 10:53 AM To: edk2-devel@lists.01.org; Bjorge, Erik C ; Qiu, Shumin Cc: Carsey, Jaben Subject: [review]Please review changes to UNI file format document Updated UNI spec to remove info specific to some consumers, not for

[edk2] [review]Please review changes to UNI file format document

2016-04-29 Thread Carsey, Jaben
Updated UNI spec to remove info specific to some consumers, not format. Updated UNI spec to allow non-ascii characters in strings. Cc: Qiu Shumin mailto:shumin@intel.com>> Cc: Erik Bjorge Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jaben Carsey mailto:jaben.car

Re: [edk2] [PATCH 1/2] ShellPkg/UefiDpLib: Fix the error message "Timer library instance error!"

2016-04-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Thursday, April 28, 2016 7:42 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Zeng, Star > Subject: [edk2] [PATCH 1/2] Sh

Re: [edk2] [PATCH 2/2] PerformancePkg/Dp_App: Fix the error message "Timer library instance error!"

2016-04-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Thursday, April 28, 2016 7:42 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Zeng, Star > Subject: [edk2] [PATCH 2/2] Pe

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Carsey, Jaben
Maybe we should have a PCD control the default value so that people who build it can set it as per their own best value. Adding a parameter seems like a great (but separate) solution. -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >

Re: [edk2] [PATCH] ArmPlatformPkg/PrePi: allow unicore version to be used on MP hardware

2016-04-29 Thread Ard Biesheuvel
On 28 April 2016 at 17:29, Leif Lindholm wrote: > On Thu, Apr 28, 2016 at 05:02:19PM +0200, Ard Biesheuvel wrote: >> On 19 April 2016 at 09:26, Ard Biesheuvel wrote: >> > When combining UEFI firmware built from Tianocore with ARM Trusted >> > Firmware running in EL3, it is the responsibility of A

Re: [edk2] [PATCH 2/2] ArmPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-29 Thread Leif Lindholm
On Fri, Apr 29, 2016 at 05:41:20PM +0200, Ard Biesheuvel wrote: > The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, > which relies on it to back its own I/O and MMIO operations. > > Since ARM has no native I/O port equivalent, such accesses can only > originate from PCI drive

Re: [edk2] [PATCH 1/2] ArmPlatformPkg: move PCI related PCD definitions to ArmPkg

2016-04-29 Thread Leif Lindholm
On Fri, Apr 29, 2016 at 05:41:19PM +0200, Ard Biesheuvel wrote: > The PCI related PCDs are not platform specific, and architectural > protocols such as CpuIo2 are based on PCI provided MMIO to IO > translation, so these PCDs belong in ArmPkg not ArmPlatformPkg. Very happy with this - but, could yo

[edk2] [PATCH 1/2] ArmPlatformPkg: move PCI related PCD definitions to ArmPkg

2016-04-29 Thread Ard Biesheuvel
The PCI related PCDs are not platform specific, and architectural protocols such as CpuIo2 are based on PCI provided MMIO to IO translation, so these PCDs belong in ArmPkg not ArmPlatformPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmPkg/ArmPkg.

[edk2] [PATCH 2/2] ArmPkg: implement CpuIo2 protocol driver specific for PCI

2016-04-29 Thread Ard Biesheuvel
The CpuIo2 protocol is required by the generic PciHostBridgeDxe driver, which relies on it to back its own I/O and MMIO operations. Since ARM has no native I/O port equivalent, such accesses can only originate from PCI drivers, and the PCI I/O space is translated to MMIO in this case. So we can i

Re: [edk2] [PATCH] SecurityPkg: Reduce DEBUG verbosity in Tcg2Dxe

2016-04-29 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Derek Lin > Sent: Friday, April 29, 2016 7:09 PM > To: edk2-devel@lists.01.org > Cc: Derek Lin ; Yao, Jiewen ; > Zhang, Chao B > Subject: [edk2] [PATCH] Securi

Re: [edk2] [Patch v4 00/23] Use MdeModulePkg/BDS in OVMF platform

2016-04-29 Thread Laszlo Ersek
On 04/29/16 09:00, Ruiyu Ni wrote: > The patch serials creates a flag USE_OLD_BDS and by default the value > of the flag is FALSE so that the new MdeModulePkg/BDS is used. > User can define USE_OLD_BDS as TRUE to force to use IntelFrameworkModulePkg > /BDS. > > The v3 adopts comments for v1 and v2

Re: [edk2] [PATCH] SecurityPkg: Reduce DEBUG verbosity in Tcg2Dxe

2016-04-29 Thread Laszlo Ersek
On 04/29/16 13:09, Derek Lin wrote: > Reduce several DEBUG messages verbosity from INFO to VERBOSE, so that will > not see debug message around each driver loading when TPM 2.0 part present. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Derek Lin > --- > SecurityP

[edk2] [PATCH] SecurityPkg: Reduce DEBUG verbosity in Tcg2Dxe

2016-04-29 Thread Derek Lin
Reduce several DEBUG messages verbosity from INFO to VERBOSE, so that will not see debug message around each driver loading when TPM 2.0 part present. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Derek Lin --- SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c | 13 +++-- 1 fil

Re: [edk2] [PATCH v4 2/2] OvmfPkg: Modify FDF/DSC files for RamDiskDxe's adding NFIT report feature

2016-04-29 Thread Laszlo Ersek
On 04/29/16 04:23, Hao Wu wrote: > The RamDiskDxe driver in MdeModulePkg now will use EFI_ACPI_TABLE_PROTOCOL > and EFI_ACPI_SDT_PROTOCOL during reporting RAM disks to NVDIMM Firmware > Interface Table (NFIT). > > A Pcd 'PcdInstallAcpiSdtProtocol' controls whether the > EFI_ACPI_SDT_PROTOCOL will

Re: [edk2] [PATCH 2/2] PerformancePkg/Dp_App: Fix the error message "Timer library instance error!"

2016-04-29 Thread Shia, Cinnamon
Hi Star, Thanks for your review. "duration < StartTimeStamp" is for the case of the timer counting down. Below is the source code for your reference: if (TimerInfo.CountUp) { Duration = Measurement->EndTimeStamp - Measurement->StartTimeStamp; Error = (BOOLEAN)(Duration > Measurement->EndTim

Re: [edk2] [PATCH] MdeModulePkg/DxeCore: set ImageContext Handle and ImageRead() fields

2016-04-29 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Friday, April 29, 2016 3:44 PM To: edk2-devel@lists.01.org; Tian, Feng ; Zeng, Star Cc: Ard Biesheuvel Subject: [edk2] [PATCH] MdeModulePkg/DxeCore: se

Re: [edk2] [PATCH] SecurityPkg: Reduce DEBUG verbosity in Tcg2Dxe

2016-04-29 Thread Lin, Derek (HPS UEFI Dev)
Thanks. I agree, will send a patch soon. Derek -Original Message- From: Yao, Jiewen [mailto:jiewen@intel.com] Sent: Friday, April 29, 2016 6:29 PM To: Lin, Derek (HPS UEFI Dev) ; Zhang, Chao B ; El-Haj-Mahmoud, Samer ; edk2-devel@lists.01.org Subject: RE: [PATCH] SecurityPkg: Reduc

Re: [edk2] [PATCH] SecurityPkg: Reduce TPM debug verbosity

2016-04-29 Thread Yao, Jiewen
Reviewed by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Samer El-Haj-Mahmoud > Sent: Friday, April 29, 2016 5:08 AM > To: edk2-devel@lists.01.org > Cc: Samer El-Haj-Mahmoud ; Zhang, Chao B > > Subject: [edk2] [PATCH

Re: [edk2] [PATCH] SecurityPkg: Reduce DEBUG verbosity in Tcg2Dxe

2016-04-29 Thread Yao, Jiewen
Thanks. That makes sense now. Would you please update commit message? I agree to reduce debug message. I think below message is of much importance to show TPM capability and narrow BitLock failure, per my experience. I suggest we keep those as INFO. For others, I am OK to change to VERBOSE. DE

Re: [edk2] [PATCH] SecurityPkg: Reduce DEBUG verbosity in Tcg2Dxe

2016-04-29 Thread Lin, Derek (HPS UEFI Dev)
Hi Jiewen, It is test with TPM 2.0 part present. There is debug message around each driver loading, which we think should be verbose. The commit description is wrong. Thanks, Derek -Original Message- From: Yao, Jiewen [mailto:jiewen@intel.com] Sent: Friday, April 29, 2016 10:38 AM

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Marcin Wojtas
Hi Siyuan, Thanks a lot for thorough explanation. The default 512B won't be changed then - PCD and extending tftp command seem a good alternative. We will come back later with more proper solution. Best regards, Marcin 2016-04-29 10:40 GMT+02:00 Fu, Siyuan : > Hi, Marcin > > The RFC2348 has a de

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Fu, Siyuan
Hi, Marcin The RFC2348 has a detail discussion about the block size setting in tftp and give the answer: using value of MTU is not the best choice. You may find you still can gain more efficiency by increasing the block size over the path MTU, until the IP fragmentation gives more overhead than

[edk2] [PATCH] MdeModulePkg/DxeCore: set ImageContext Handle and ImageRead() fields

2016-04-29 Thread Ard Biesheuvel
Set the ImageContext.Handle and ImageContext.ImageRead() fields so that PeCoffLoaderRelocateImageExtraAction() can invoke PeCoffLoaderGetImageInfo or PeCoffLoaderGetPeHeader if desired to obtain additional metadata. We will use this to create a PeCoffLoaderRelocateImageExtraAction() implementation

Re: [edk2] [PATCH] MdeModulePkg/PciBusDxe: don't create bogus descriptor if no resources needed

2016-04-29 Thread Ard Biesheuvel
On 29 April 2016 at 09:19, Ni, Ruiyu wrote: > Ard, > At first glance of this patch, I immediately checked the PI spec > SubResources() description, > it says when a root bridge doesn't require any resource, "a zero-length > resource request > must explicitly be submitted." > "zero-length resourc

Re: [edk2] [PATCH] MdeModulePkg/PciBusDxe: don't create bogus descriptor if no resources needed

2016-04-29 Thread Ni, Ruiyu
Ard, At first glance of this patch, I immediately checked the PI spec SubResources() description, it says when a root bridge doesn't require any resource, "a zero-length resource request must explicitly be submitted." "zero-length resource" can be interpreted in two ways: 1. only END descriptor 2

[edk2] [Patch v4 23/23] OvmfPkg/OvmfPkgIa32X64.dsc: Move PcdShellFile to [PcdsFixedAtBuild.X64]

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Laszlo Ersek --- OvmfPkg/OvmfPkgIa32X64.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc index e5a5e8a..6e04dd6 100644 --- a/OvmfPkg/

[edk2] [Patch v4 21/23] OvmfPkg/PlatformBootManagerLib: Remove unused C structures definitions

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- .../Library/PlatformBootManagerLib/BdsPlatform.h | 47 -- 1 file changed, 47 deletions(-) diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.h b/Ovmf

[edk2] [Patch v4 15/23] OvmfPkg/PlatformBootManagerLib: Use EfiBootManagerRefreshAllBootOption()

2016-04-29 Thread Ruiyu Ni
The patch uses EfiBootManagerRefreshAllBootOption() to collect all boot options and uses SetBootOrderFromQemu exposed by QemuNewBootOrderLib to adjust the boot option order. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- OvmfPkg/Lib

[edk2] [Patch v4 22/23] OvmfPkg: Use MdeModulePkg/BDS

2016-04-29 Thread Ruiyu Ni
By default the new MdeModulePkg/BDS is used. If USE_OLD_BDS is defined to TRUE, IntelFrameworkModulePkg/BDS is used. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- OvmfPkg/OvmfPkgIa32.dsc| 41 ++

[edk2] [Patch v4 16/23] OvmfPkg/PlatformBootManagerLib: Remove PlatformBdsGetDriverOption()

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- .../Library/PlatformBootManagerLib/BdsPlatform.c | 30 -- .../Library/PlatformBootManagerLib/BdsPlatform.h | 6 - 2 files changed, 36 deletions(-) diff --gi

[edk2] [Patch v4 20/23] OvmfPkg/PlatformBootManagerLib: Add EnableQuietBoot & DisableQuietBoot

2016-04-29 Thread Ruiyu Ni
EnableQuietBoot and DisableQuietBoot are copied from IntelFrameworkModulePkg/Library/GenericBdsLib/BdsConsole.c. Because these two functions are not in UefiBootManagerLib. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- ...

[edk2] [Patch v4 18/23] OvmfPkg/PlatformBootManagerLib: Remove unnecessary memory test

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- .../Library/PlatformBootManagerLib/BdsPlatform.c | 60 +- .../Library/PlatformBootManagerLib/BdsPlatform.h | 5 -- 2 files changed, 2 insertions(+), 63 deletions

[edk2] [Patch v4 19/23] OvmfPkg/PlatformBootManagerLib: Remove unused vars and func prototypes

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- .../Library/PlatformBootManagerLib/BdsPlatform.h | 42 -- .../Library/PlatformBootManagerLib/PlatformData.c | 12 +-- 2 files changed, 1 insertion(+)

[edk2] [Patch v4 17/23] OvmfPkg/PlatformBootManagerLib: Use GetBootModeHob() in HobLib

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c b/OvmfPkg

[edk2] [Patch v4 10/23] OvmfPkg/PlatformBootManagerLib: Init console vars in *BeforeConsole()

2016-04-29 Thread Ruiyu Ni
The major difference between IntelFrameworkModulePkg/BDS and MdeModulePkg/BDS is the latter connects the consoles in core code while the former connects in platform code. The change initializes the console variables in PlatformBootManagerBeforeConsole() and removes the console connection code. It a

[edk2] [Patch v4 08/23] OvmfPkg/PlatformBootManagerLib: link to UefiBootManagerLib

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.h | 2 +- OvmfPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 2 +- 2 files changed, 2 insertions(+), 2 de

[edk2] [Patch v4 05/23] OvmfPkg: Duplicate PlatformBdsLib to PlatformBootManagerLib

2016-04-29 Thread Ruiyu Ni
It will be changed to build with MdeModulePkg/BDS. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- .../{PlatformBdsLib => PlatformBootManagerLib}/BdsPlatform.c| 0 .../{PlatformBdsLib => PlatformBootManagerLib}/BdsP

[edk2] [Patch v4 12/23] OvmfPkg/PlatformBootManagerLib: Register boot options and hot keys

2016-04-29 Thread Ruiyu Ni
The patch registers "Enter" key as the continue key (hot key to skip the boot timeout wait), maps "F2" key to UI, and registers Shell boot option. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- .../Library/PlatformBootMana

[edk2] [Patch v4 11/23] OvmfPkg/PlatformBootManagerLib: Do not launch Boot Manager Menu

2016-04-29 Thread Ruiyu Ni
MdeModulePkg/BDS doesn't launch UI (Boot Manager Menu) from platform side. The change removes the code which launches the UI but still set the boot timeout. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- OvmfPkg/Library/PlatformBoot

[edk2] [Patch v4 03/23] OvmfPkg: Duplicate QemuBootOrderLib to QemuNewBootOrderLib

2016-04-29 Thread Ruiyu Ni
QemuNewBootOrderLib will be changed to work with MdeModulePkg/BDS. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Cc: Laszlo Ersek --- .../{QemuBootOrderLib => QemuNewBootOrderLib}/ExtraRootBusMap.c | 0 .../{QemuBootOrderLib => QemuNewBoo

[edk2] [Patch v4 04/23] OvmfPkg/QemuNewBootOrderLib: Build with UefiBootManagerLib

2016-04-29 Thread Ruiyu Ni
NOTE: SetBootOrderFromQemu() interface is not changed. But when the old IntelFrameworkModulePkg/BDS is no longer used in OVMF and ArmVirtPkg, additional patch will be submitted to change this interface to remove parameter BootOptionList. Contributed-under: TianoCore Contribution Agreement 1.0 Sign

[edk2] [Patch v4 14/23] OvmfPkg/PlatformBootManagerLib: port PlatformBdsConnectSequence to UefiBootManagerLib

2016-04-29 Thread Ruiyu Ni
The patch changes PlatformBdsConnectSequence() to use library API exposed from UefiBootManagerLib and removes the additional connect ALL action. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- OvmfPkg/Library/PlatformBootManagerLib/B

[edk2] [Patch v4 09/23] OvmfPkg/PlatformBootManagerLib: Use ConvertDevicePathToText()

2016-04-29 Thread Ruiyu Ni
The DevicePathToStr() function (in "IntelFrameworkModulePkg/Library/GenericBdsLib/DevicePath.c") is a simple wrapper around ConvertDevicePathToText(). DevicePathToStr() passes DisplayOnly=TRUE and AllowShortcuts=TRUE to ConvertDevicePathToText(), whereas in this patch, both parameters are flipped

[edk2] [Patch v4 06/23] OvmfPkg/PlatformBootManagerLib: Follow PlatformBootManagerLib interfaces

2016-04-29 Thread Ruiyu Ni
Change the function name to follow new library class PlatformBootManagerLib interfaces. NOTE: There is no progress bar during BDS timeout waiting. In order to show the progress bar, PlatformBootManagerWaitCallback () needs to change to draw it. Contributed-under: TianoCore Contribution Agreement

[edk2] [Patch v4 13/23] OvmfPkg/PlatformBootManagerLib: Remove unused local functions.

2016-04-29 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jordan Justen Reviewed-by: Laszlo Ersek --- .../Library/PlatformBootManagerLib/BdsPlatform.c | 94 -- 1 file changed, 94 deletions(-) diff --git a/OvmfPkg/Library/PlatformBootManagerLib/Bd

[edk2] [Patch v4 07/23] OvmfPkg/PlatformBootManagerLib: use EfiBootManagerUpdateConsoleVariable

2016-04-29 Thread Ruiyu Ni
Call EfiBootManagerUpdateConsoleVariable in UefiBootManagerLib instead of BdsLibUpdateConsoleVariable in GenericBdsLib. Still cannot pass build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- .../Library/PlatformBootManagerLib/BdsP

[edk2] [Patch v4 01/23] MdeModulePkg/UefiBootManagerLib: Expose *GetLoadOptionBuffer() API

2016-04-29 Thread Ruiyu Ni
Expose EfiBootManagerGetLoadOptionBuffer() API. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Reviewed-by: Laszlo Ersek --- MdeModulePkg/Include/Library/UefiBootManagerLib.h | 23 +- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 11 +

[edk2] [Patch v4 02/23] OvmfPkg/PlatformPei: Add memory above 4GB as tested

2016-04-29 Thread Ruiyu Ni
Since PlatformBootManagerLib do not run memory test to convert untested memory to tested. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Tested-by: Laszlo Ersek Reviewed-by: Laszlo Ersek --- OvmfPkg/PlatformPei/MemDetect.c | 4 ++-- OvmfPkg/PlatformPei/Platfor

[edk2] [Patch v4 00/23] Use MdeModulePkg/BDS in OVMF platform

2016-04-29 Thread Ruiyu Ni
The patch serials creates a flag USE_OLD_BDS and by default the value of the flag is FALSE so that the new MdeModulePkg/BDS is used. User can define USE_OLD_BDS as TRUE to force to use IntelFrameworkModulePkg /BDS. The v3 adopts comments for v1 and v2 to split the big changes to small changes and