Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On Tue, 2016-05-03 at 12:37 +0200, Laszlo Ersek wrote: > > However, CSM is apparently generally borked at the moment (see > ), and I don't think I can > spend time on analyzing and fixing that. > > I guess this is not good news, but I thought it better

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On 08/04/15 13:07, Laszlo Ersek wrote: > On 08/04/15 12:42, David Woodhouse wrote: >> On Tue, 2015-08-04 at 03:16 +, Ni, Ruiyu wrote: >>> I forgot to emphasize that I only tested the QEMU boot timeout >>> feature after changing the QemuBootOrderLib. I don't know how to test >>> the boot order

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

that boot order override > from QEMU and legacy boot still works, we are safe to just use the > new BDS. > What do you think? I started looking at the patch. I can't yet say how intrusive it is. Perhaps I'll be okay without a build time flag, not sure yet. Thanks Laszlo >

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

M To: Ni, Ruiyu Cc: edk2-de...@ml01.01.org; David Woodhouse Subject: Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg On 08/04/15 12:53, Laszlo Ersek wrote: > No matter how carefully > we review and test the new code, something will inevitably break, This wasn't meant as lack of trus

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On Tue, 2015-08-04 at 13:07 +0200, Laszlo Ersek wrote: > On 08/04/15 12:42, David Woodhouse wrote: > > Then (if the BDS gets it right) you should be able to boot legacy > > targets as well as native UEFI targets. > > Okay, if you don't have your guests any longer, I guess I can add > this to the

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On 08/04/15 12:53, Laszlo Ersek wrote: > No matter how carefully > we review and test the new code, something will inevitably break, This wasn't meant as lack of trust in your code; it's just that there are many cases and corner cases in the related OVMF code, and it's quite hard to test them all

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On 08/04/15 12:42, David Woodhouse wrote: > On Tue, 2015-08-04 at 03:16 +, Ni, Ruiyu wrote: >> I forgot to emphasize that I only tested the QEMU boot timeout >> feature after changing the QemuBootOrderLib. I don't know how to test >> the boot order feature. >> For the SeaBIOS CSM, can you tel

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

will also require this series to be available in a public branch somewhere. Thanks! Laszlo > > Thanks, > Ray > > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of David > Woodhouse > Sent: Monday, August 3, 2015 6:31 PM &

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On Tue, 2015-08-04 at 03:16 +, Ni, Ruiyu wrote: > I forgot to emphasize that I only tested the QEMU boot timeout > feature after changing the QemuBootOrderLib. I don't know how to test > the boot order feature. > For the SeaBIOS CSM, can you tell me how to test or test it for me if > it's ve

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of David Woodhouse Sent: Monday, August 3, 2015 6:31 PM To: Ni, Ruiyu ; edk2-de...@ml01.01.org Subject: Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg On Mon, 2015-08-03 at 13:41 +0800, Ruiyu Ni wrote: > A new

Re: [edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

On Mon, 2015-08-03 at 13:41 +0800, Ruiyu Ni wrote: > A new BDS and UiApp was created in MdeModulePkg and are already > used in Nt32Pkg. > The patch changes the OvmfPkg to use the new BDS and UiApp as well. Hi Ruiyu, were these changes tested with the SeaBIOS CSM? -- David Woodhouse

[edk2] [Patch 0/3] Use new BDS and UiApp for OvmfPkg

A new BDS and UiApp was created in MdeModulePkg and are already used in Nt32Pkg. The patch changes the OvmfPkg to use the new BDS and UiApp as well. Ruiyu Ni (3): ArmVirtPkg: Copy QemuBootOrderLib to ArmVirtPkg OvmfPkg: use new BDS and UiApp in MdeModulePkg OvmfPkg: Remove unused PlatformBds