[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] Current HVM acpi support is broken


  • To: "Wang, Winston L" <winston.l.wang@xxxxxxxxx>
  • From: "Christian Limpach" <christian.limpach@xxxxxxxxx>
  • Date: Tue, 8 Aug 2006 18:54:41 +0100
  • Cc: Ian Pratt <m+Ian.Pratt@xxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, Tang Liang <tangliang@xxxxxxxxxx>
  • Delivery-date: Tue, 08 Aug 2006 10:55:05 -0700
  • Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=tNhO79n3W6cxnVIgCY9qmNWhYEmcegrFJtHEJvj8bSLdYmV2C1u/kQWouFV3EqtSKJyM5rkD/B0PiIC5uOUMD6OPishIetB/yuRCkzlFQB5BeYtHN/CYTl1owCcv3bt06Ll2t7cV8XrzdmfRpOgY0A7YnkHyIFdBB/TWcDT14ZU=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

On 8/8/06, Wang, Winston L <winston.l.wang@xxxxxxxxx> wrote:
Hi Christian, Ian and Kair,

I think current hvm acpi support is broken.
We noticed qemu to 0.82 with acpi support is introduced on changset
10957, it is conflict with xen hvm acpi code, in ../ioemu/hw/cp.c start
from ln 860:

Yes, I tried to disable the acpi which comes with qemu 0.8.2 but I
missed the 2nd call-site (acpi_bios_init).

To move forward:
- does it work if you disable the call to acpi_bios_init?
- can you compare the qemu acpi support to the acpi support you've
provided and maybe merge some of it?

I don't feel strongly about using one or the other, I guess it would
be good if we could leverage some of the qemu acpi work but maybe it's
still quite immature?

   christian


    cmos_init(ram_size, boot_device, bs_table, timeoffset);

    /* using PIIX4 acpi model */
    if (pci_enabled && acpi_enabled)
        pci_piix4_acpi_init(pci_bus, piix3_devfn + 3);

    if (pci_enabled && usb_enabled) {
        usb_uhci_init(pci_bus, piix3_devfn + 2);
    }

    if (pci_enabled && acpi_enabled && 0) {
        piix4_pm_init(pci_bus, piix3_devfn + 3);
    }
...
    /* XXX: should be done in the Bochs BIOS */
    if (pci_enabled) {
        pci_bios_init();
        if (acpi_enabled)
            acpi_bios_init();
    }
}

The above code generate two acpi tables and acpi pci piix func3 device
when acpi=1 in hvm guest platform.

Two sets of acpi can not co-exist:) What acpi set do we need to keep?
What is the advatage for that?

We are cureent have to use changset 10955 puls acpi_init_fix.patch for
acpi related development.

Regards,

Winston,

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.