Re: [Xen-devel] [PATCH] hvmloader: Load proper ACPI tables with OVMF

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Mon, 2015-12-14 at 16:08 +0000, Anthony PERARD wrote:
> This patch loads the ACPI tables associated with QEMU instead of the one
> for qemu-traditional.

I'dÂadd "... we only support OVMF with qemu-xen" or something, just to make
it clear why this is correct.

> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

> ---
> Âtools/firmware/hvmloader/ovmf.c | 8 ++++----
> Â1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tools/firmware/hvmloader/ovmf.c
> b/tools/firmware/hvmloader/ovmf.c
> index bb3da93..db9fa7a 100644
> --- a/tools/firmware/hvmloader/ovmf.c
> +++ b/tools/firmware/hvmloader/ovmf.c
> @@ -47,8 +47,8 @@
> Â#define LOWCHUNK_ENDÂÂÂÂÂÂÂÂÂÂÂÂ(OVMF_BEGIN + OVMF_SIZE)
> Â#define OVMF_INFO_PHYSICAL_ADDRESS 0x00001000
> Â
> -extern unsigned char dsdt_anycpu[];
> -extern int dsdt_anycpu_len;
> +extern unsigned char dsdt_anycpu_qemu_xen[];
> +extern int dsdt_anycpu_qemu_xen_len;
> Â
> Â#define OVMF_INFO_MAX_TABLES 4
> Âstruct ovmf_info {
> @@ -119,8 +119,8 @@ static void ovmf_load(const struct bios_config
> *config)
> Âstatic void ovmf_acpi_build_tables(void)
> Â{
> ÂÂÂÂÂstruct acpi_config config = {
> -ÂÂÂÂÂÂÂÂ.dsdt_anycpu = dsdt_anycpu,
> -ÂÂÂÂÂÂÂÂ.dsdt_anycpu_len = dsdt_anycpu_len,
> +ÂÂÂÂÂÂÂÂ.dsdt_anycpu = dsdt_anycpu_qemu_xen,
> +ÂÂÂÂÂÂÂÂ.dsdt_anycpu_len = dsdt_anycpu_qemu_xen_len,
> ÂÂÂÂÂÂÂÂÂ.dsdt_15cpu = NULL, 
> ÂÂÂÂÂÂÂÂÂ.dsdt_15cpu_len = 0
> ÂÂÂÂÂ};

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel