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
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
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
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
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
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
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;
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
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
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
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
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
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
>
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
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
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
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.
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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/
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
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
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 ++
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
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
---
...
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
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(+)
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
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
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
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
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
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
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
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
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
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
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
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
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
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 +
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
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
58 matches
Mail list logo