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 fi

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 > > > > > e40

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_op

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_op

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

2013-09-25 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) { > Acp

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

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 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 > > > > e

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 separat

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 > > > >

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 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 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 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

[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 so