Reviewed-By: Olivier Martin <[email protected]>

> -----Original Message-----
> From: Laszlo Ersek [mailto:[email protected]]
> Sent: 14 February 2015 22:02
> To: [email protected]; [email protected]; Peter Maydell; Ard
> Biesheuvel; Drew Jones; Wei Huang; Donald Dutile; Wei Liu
> Subject: [edk2] [PATCH 22/27] ArmVirtualizationPkg:
> PlatformIntelBdsLib: kernel boot should provide ACPI
> 
> If there is a PCI host, then PCI enumeration (which happens inside
> BdsLibConnectAll()) blocks ACPI table installation (correctly). Make
> sure
> we install ACPI tables before trying to direct-boot a QEMU kernel.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Laszlo Ersek <[email protected]>
> ---
> 
> ArmPlatformPkg/ArmVirtualizationPkg/Library/PlatformIntelBdsLib/IntelBd
> sPlatform.c | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)
> 
> diff --git
> a/ArmPlatformPkg/ArmVirtualizationPkg/Library/PlatformIntelBdsLib/Intel
> BdsPlatform.c
> b/ArmPlatformPkg/ArmVirtualizationPkg/Library/PlatformIntelBdsLib/Intel
> BdsPlatform.c
> index a03e909..ae43f9c 100644
> ---
> a/ArmPlatformPkg/ArmVirtualizationPkg/Library/PlatformIntelBdsLib/Intel
> BdsPlatform.c
> +++
> b/ArmPlatformPkg/ArmVirtualizationPkg/Library/PlatformIntelBdsLib/Intel
> BdsPlatform.c
> @@ -289,14 +289,17 @@ PlatformBdsPolicyBehavior (
>    Status = PlatformBdsConnectConsole ();
>    ASSERT_EFI_ERROR (Status);
> 
> -  //
> -  // Process QEMU's -kernel command line option
> -  //
> -  TryRunningQemuKernel ();
> -
>    BdsLibConnectAll ();
> +
> +  //
> +  // Process QEMU's -kernel command line option. Note that the kernel
> booted
> +  // this way should receive ACPI tables, which is why we connect all
> devices
> +  // first (see above) -- PCI enumeration blocks ACPI table
> installation, if
> +  // there is a PCI host.
> +  //
> +  TryRunningQemuKernel ();
> +
>    BdsLibEnumerateAllBootOption (BootOptionList);
> -
>    SetBootOrderFromQemu (BootOptionList);
>    //
>    // The BootOrder variable may have changed, reload the in-memory
> list with
> --
> 1.8.3.1
> 
> 
> 
> -----------------------------------------------------------------------
> -------
> Dive into the World of Parallel Programming. The Go Parallel Website,
> sponsored by Intel and developed in partnership with Slashdot Media, is
> your
> hub for all things parallel software development, from weekly thought
> leadership blogs to news, videos, case studies, tutorials and more.
> Take a
> look and join the conversation now. http://goparallel.sourceforge.net/
> _______________________________________________
> edk2-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/edk2-devel





------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to