Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-27 Thread Gerd Hoffmann
Hi, ACPI gives a way to do this: supply PBlockLength = 0. I applied the following: though this means bytecode is not 1:1 identical to what we have with seabios, so I will keep it as a separate commit. Sure. Retested -- works fine now with both seabios + coreboot. Glad to see this

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-26 Thread Gerd Hoffmann
Hi, diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 1ba86d0..d1ccdf7 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -961,8 +961,8 @@ static void acpi_build_update(void *build_opaque, uint32_t offset) if (build_state-mcfg_base) {

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-26 Thread Michael S. Tsirkin
On Thu, Sep 26, 2013 at 08:26:51AM +0200, Gerd Hoffmann wrote: Hi, diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 1ba86d0..d1ccdf7 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -961,8 +961,8 @@ static void acpi_build_update(void *build_opaque,

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-26 Thread Michael S. Tsirkin
On Thu, Sep 26, 2013 at 08:26:51AM +0200, Gerd Hoffmann wrote: Hi, diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 1ba86d0..d1ccdf7 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -961,8 +961,8 @@ static void acpi_build_update(void *build_opaque,

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-26 Thread Michael S. Tsirkin
On Wed, Sep 25, 2013 at 03:26:12PM +0200, Gerd Hoffmann wrote: [root@fedora ~]# cat /proc/ioports [ ... ] afe0-afe3 : ACPI GPE0_BLK b010-b015 : ACPI CPU throttle b100-b10f : :00:01.3 b100-b107 : piix4_smbus e400-e43f : :00:01.3

[Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Michael S. Tsirkin
This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi While this patch still uses info not available in QOM, I think it's reasonable to merge it and then refactor as QOM properties cover more ground. In particular, merging this patchset blocks other projects

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Gerd Hoffmann
On Mi, 2013-09-25 at 14:22 +0300, Michael S. Tsirkin wrote: This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi While this patch still uses info not available in QOM, I think it's reasonable to merge it and then refactor as QOM properties cover more

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Michael S. Tsirkin
On Wed, Sep 25, 2013 at 02:48:52PM +0200, Gerd Hoffmann wrote: On Mi, 2013-09-25 at 14:22 +0300, Michael S. Tsirkin wrote: This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi While this patch still uses info not available in QOM, I think it's

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Michael S. Tsirkin
On Wed, Sep 25, 2013 at 03:02:35PM +0200, Gerd Hoffmann wrote: On Mi, 2013-09-25 at 15:59 +0300, Michael S. Tsirkin wrote: On Wed, Sep 25, 2013 at 02:48:52PM +0200, Gerd Hoffmann wrote: On Mi, 2013-09-25 at 14:22 +0300, Michael S. Tsirkin wrote: This code can also be found here:

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Michael S. Tsirkin
On Wed, Sep 25, 2013 at 03:59:48PM +0300, Michael S. Tsirkin wrote: On Wed, Sep 25, 2013 at 02:48:52PM +0200, Gerd Hoffmann wrote: On Mi, 2013-09-25 at 14:22 +0300, Michael S. Tsirkin wrote: This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Gerd Hoffmann
On Mi, 2013-09-25 at 15:59 +0300, Michael S. Tsirkin wrote: On Wed, Sep 25, 2013 at 02:48:52PM +0200, Gerd Hoffmann wrote: On Mi, 2013-09-25 at 14:22 +0300, Michael S. Tsirkin wrote: This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi While

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Gerd Hoffmann
Hi, This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi Crashes on coreboot with -M q35: qemu-system-x86_64: /home/kraxel/projects/qemu/hw/i386/acpi-build.c:965: acpi_build_update: Assertion `mcfg_size' failed. I'll go send a coreboot rom with separate

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Gerd Hoffmann
[root@fedora ~]# cat /proc/ioports [ ... ] afe0-afe3 : ACPI GPE0_BLK b010-b015 : ACPI CPU throttle b100-b10f : :00:01.3 b100-b107 : piix4_smbus e400-e43f : :00:01.3 e400-e403 : ACPI PM1a_EVT_BLK e404-e405 : ACPI PM1a_CNT_BLK

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Gerd Hoffmann
On Mi, 2013-09-25 at 15:12 +0200, Gerd Hoffmann wrote: Hi, This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi Crashes on coreboot with -M q35: qemu-system-x86_64: /home/kraxel/projects/qemu/hw/i386/acpi-build.c:965: acpi_build_update:

Re: [Qemu-devel] [PATCH v5 00/23] qemu: generate acpi tables for the guest

2013-09-25 Thread Michael S. Tsirkin
On Wed, Sep 25, 2013 at 04:15:03PM +0200, Gerd Hoffmann wrote: On Mi, 2013-09-25 at 15:12 +0200, Gerd Hoffmann wrote: Hi, This code can also be found here: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git acpi Crashes on coreboot with -M q35: qemu-system-x86_64: