* [Qemu-devel] [PATCH] pc: drop external DSDT loading
@ 2013-08-08 16:38 Anthony Liguori
2013-08-09 6:48 ` Gerd Hoffmann
0 siblings, 1 reply; 6+ messages in thread
From: Anthony Liguori @ 2013-08-08 16:38 UTC (permalink / raw)
To: qemu-devel; +Cc: Anthony Liguori
This breaks migration and is unneeded with modern SeaBIOS.
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
---
hw/i386/pc_piix.c | 1 -
hw/i386/pc_q35.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
index ab25458..daaff8a 100644
--- a/hw/i386/pc_piix.c
+++ b/hw/i386/pc_piix.c
@@ -103,7 +103,6 @@ static void pc_init1(MemoryRegion *system_memory,
OBJECT(icc_bridge), NULL);
pc_cpus_init(cpu_model, icc_bridge);
- pc_acpi_init("acpi-dsdt.aml");
if (kvm_enabled() && kvmclock_enabled) {
kvmclock_create();
diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c
index 2f35d12..8c6c7bc 100644
--- a/hw/i386/pc_q35.c
+++ b/hw/i386/pc_q35.c
@@ -86,7 +86,6 @@ static void pc_q35_init(QEMUMachineInitArgs *args)
OBJECT(icc_bridge), NULL);
pc_cpus_init(cpu_model, icc_bridge);
- pc_acpi_init("q35-acpi-dsdt.aml");
kvmclock_create();
--
1.8.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [Qemu-devel] [PATCH] pc: drop external DSDT loading
2013-08-08 16:38 [Qemu-devel] [PATCH] pc: drop external DSDT loading Anthony Liguori
@ 2013-08-09 6:48 ` Gerd Hoffmann
2013-08-09 12:38 ` Anthony Liguori
0 siblings, 1 reply; 6+ messages in thread
From: Gerd Hoffmann @ 2013-08-09 6:48 UTC (permalink / raw)
To: Anthony Liguori; +Cc: qemu-devel
On 08/08/13 18:38, Anthony Liguori wrote:
> This breaks migration and is unneeded with modern SeaBIOS.
No. Dropping for piix is fine. It will break q35 though.
Given that q35 can't be migrated anyway due to ahci being tagged as
unmigratable keeping it for q35 (until the new acpi table loading is
sorted) shouldn't hurt though.
cheers,
Gerd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] pc: drop external DSDT loading
2013-08-09 6:48 ` Gerd Hoffmann
@ 2013-08-09 12:38 ` Anthony Liguori
2013-08-09 13:26 ` Gerd Hoffmann
2013-08-09 15:50 ` Michael S. Tsirkin
0 siblings, 2 replies; 6+ messages in thread
From: Anthony Liguori @ 2013-08-09 12:38 UTC (permalink / raw)
To: Gerd Hoffmann; +Cc: qemu-devel, Michael Tsirkin
Gerd Hoffmann <kraxel@redhat.com> writes:
> On 08/08/13 18:38, Anthony Liguori wrote:
>> This breaks migration and is unneeded with modern SeaBIOS.
>
> No. Dropping for piix is fine. It will break q35 though.
Can you elaborate? When Michael and I discussed this I was under the
impression that latest SeaBIOS had full support for q35.
Regards,
Anthony Liguori
>
> Given that q35 can't be migrated anyway due to ahci being tagged as
> unmigratable keeping it for q35 (until the new acpi table loading is
> sorted) shouldn't hurt though.
>
> cheers,
> Gerd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] pc: drop external DSDT loading
2013-08-09 12:38 ` Anthony Liguori
@ 2013-08-09 13:26 ` Gerd Hoffmann
2013-08-09 15:51 ` Michael S. Tsirkin
2013-08-09 15:50 ` Michael S. Tsirkin
1 sibling, 1 reply; 6+ messages in thread
From: Gerd Hoffmann @ 2013-08-09 13:26 UTC (permalink / raw)
To: Anthony Liguori; +Cc: qemu-devel, Michael Tsirkin
On 08/09/13 14:38, Anthony Liguori wrote:
> Gerd Hoffmann <kraxel@redhat.com> writes:
>
>> On 08/08/13 18:38, Anthony Liguori wrote:
>>> This breaks migration and is unneeded with modern SeaBIOS.
>>
>> No. Dropping for piix is fine. It will break q35 though.
>
> Can you elaborate? When Michael and I discussed this I was under the
> impression that latest SeaBIOS had full support for q35.
SeaBIOS has the piix acpi tables compiled in, for the snake of backward
compatibility with old qemu versions which don't provide the acpi tables.
With the q35 merge seabios started to provide the apci tables via
fw_cfg, for both q35 and piix, with the long-term goal to drop the
internal tables some day even for piix.
Later on we figured table loading has live migration issues (due to bios
binary being migrated but acpi tables are not, so you can end up with
mismatches).
The idea to deal with that was to simply turn off acpi table loading for
piix4, then wait for mst's acpi table patches which fix this for real
(including migration). Continue loading the tables on q35, ignoring the
live migration issue as ahci renders q35 unmigratable anyway.
I assumed that happend already, but looks like it slipped though,
otherwise you would not have posted that patch ...
cheers,
Gerd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] pc: drop external DSDT loading
2013-08-09 13:26 ` Gerd Hoffmann
@ 2013-08-09 15:51 ` Michael S. Tsirkin
0 siblings, 0 replies; 6+ messages in thread
From: Michael S. Tsirkin @ 2013-08-09 15:51 UTC (permalink / raw)
To: Gerd Hoffmann; +Cc: Anthony Liguori, qemu-devel
On Fri, Aug 09, 2013 at 03:26:22PM +0200, Gerd Hoffmann wrote:
> On 08/09/13 14:38, Anthony Liguori wrote:
> > Gerd Hoffmann <kraxel@redhat.com> writes:
> >
> >> On 08/08/13 18:38, Anthony Liguori wrote:
> >>> This breaks migration and is unneeded with modern SeaBIOS.
> >>
> >> No. Dropping for piix is fine. It will break q35 though.
> >
> > Can you elaborate? When Michael and I discussed this I was under the
> > impression that latest SeaBIOS had full support for q35.
>
> SeaBIOS has the piix acpi tables compiled in, for the snake of backward
> compatibility with old qemu versions which don't provide the acpi tables.
>
> With the q35 merge seabios started to provide the apci tables via
> fw_cfg, for both q35 and piix, with the long-term goal to drop the
> internal tables some day even for piix.
>
> Later on we figured table loading has live migration issues (due to bios
> binary being migrated but acpi tables are not, so you can end up with
> mismatches).
>
> The idea to deal with that was to simply turn off acpi table loading for
> piix4, then wait for mst's acpi table patches which fix this for real
> (including migration). Continue loading the tables on q35, ignoring the
> live migration issue as ahci renders q35 unmigratable anyway.
Sounds like a plan for 1.6.
> I assumed that happend already, but looks like it slipped though,
> otherwise you would not have posted that patch ...
>
> cheers,
> Gerd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] pc: drop external DSDT loading
2013-08-09 12:38 ` Anthony Liguori
2013-08-09 13:26 ` Gerd Hoffmann
@ 2013-08-09 15:50 ` Michael S. Tsirkin
1 sibling, 0 replies; 6+ messages in thread
From: Michael S. Tsirkin @ 2013-08-09 15:50 UTC (permalink / raw)
To: Anthony Liguori; +Cc: Gerd Hoffmann, qemu-devel
On Fri, Aug 09, 2013 at 07:38:36AM -0500, Anthony Liguori wrote:
> Gerd Hoffmann <kraxel@redhat.com> writes:
>
> > On 08/08/13 18:38, Anthony Liguori wrote:
> >> This breaks migration and is unneeded with modern SeaBIOS.
> >
> > No. Dropping for piix is fine. It will break q35 though.
>
> Can you elaborate? When Michael and I discussed this I was under the
> impression that latest SeaBIOS had full support for q35.
>
> Regards,
>
> Anthony Liguori
I think I made a mistake. SeaBIOS does have the dsdt for Q35
but it doesn not load it.
> >
> > Given that q35 can't be migrated anyway due to ahci being tagged as
> > unmigratable keeping it for q35 (until the new acpi table loading is
> > sorted) shouldn't hurt though.
> >
> > cheers,
> > Gerd
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-08-09 15:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-08 16:38 [Qemu-devel] [PATCH] pc: drop external DSDT loading Anthony Liguori
2013-08-09 6:48 ` Gerd Hoffmann
2013-08-09 12:38 ` Anthony Liguori
2013-08-09 13:26 ` Gerd Hoffmann
2013-08-09 15:51 ` Michael S. Tsirkin
2013-08-09 15:50 ` Michael S. Tsirkin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).