[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Xen-devel] [PATCH v2] xenfv: set has_acpi_build to fal
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [Xen-devel] [PATCH v2] xenfv: set has_acpi_build to false |
Date: |
Thu, 27 Oct 2016 11:58:29 -0700 (PDT) |
User-agent: |
Alpine 2.10 (DEB 1266 2009-07-14) |
On Thu, 27 Oct 2016, Sander Eikelenboom wrote:
> Thursday, October 27, 2016, 3:51:09 PM, you wrote:
>
> > Xen's toolstack is in charge of building ACPI tables. Skip ACPI table
> > building and loading in QEMU by setting has_acpi_build to false for
> > xenfv machine.
>
> > This issue is discovered due to direct kernel boot on Xen doesn't boot
> > anymore, because the new ACPI tables cause the guest to exceed its
> > memory allocation limit.
>
> > Reported-by: Sander Eikelenboom <address@hidden>
> > Signed-off-by: Wei Liu <address@hidden>
>
> Just given this patch a spin and you may add a:
> Tested-by: Sander Eikelenboom <address@hidden>
The problem with this patch is that it only covers the xenfv machine
case, which is default, but QEMU can also be invoked with -M
pc,accel=xen. That case wouldn't be fixed by this patch. Wei, you can
test it by adding "xen_platform_pci=0" to the VM config file.
That's why we probably need a new option, similar to has_acpi_build, but
that can be changed at accelerator init time.
> > ---
> > Cc: Igor Mammedov <address@hidden>
> > Cc: Anthony PERARD <address@hidden>
> > Cc: Stefano Stabellini <address@hidden>
> > Cc: Sander Eikelenboom <address@hidden>
> > ---
> > hw/i386/pc_piix.c | 4 ++++
> > 1 file changed, 4 insertions(+)
>
> > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
> > index a54a468..08596a1 100644
> > --- a/hw/i386/pc_piix.c
> > +++ b/hw/i386/pc_piix.c
> > @@ -1094,10 +1094,14 @@ DEFINE_PC_MACHINE(isapc, "isapc", pc_init_isa,
> > #ifdef CONFIG_XEN
> > static void xenfv_machine_options(MachineClass *m)
> > {
> > + PCMachineClass *pcmc = PC_MACHINE_CLASS(m);
> > +
> > m->desc = "Xen Fully-virtualized PC";
> > m->max_cpus = HVM_MAX_VCPUS;
> > m->default_machine_opts = "accel=xen";
> > m->hot_add_cpu = pc_hot_add_cpu;
> > +
> + pcmc->>has_acpi_build = false;
> > }
> >
> > DEFINE_PC_MACHINE(xenfv, "xenfv", pc_xen_hvm_init,
>
>
> _______________________________________________
> Xen-devel mailing list
> address@hidden
> https://lists.xen.org/xen-devel
>