On 09/14/12 09:08, Paolo Bonzini wrote: > Il 13/09/2012 22:12, Jason Baron ha scritto: >> Automatically, locate the required q35 dsdt table on load. Otherwise we error >> out. This could be done in the bios, but its harder to produce a good error >> message.
>> /* PC hardware initialisation */ >> static void pc_q35_init(ram_addr_t ram_size, >> const char *boot_device, >> @@ -356,6 +375,13 @@ static void pc_q35_init(ram_addr_t ram_size, >> qemu_irq *i8259; >> int i; >> >> + /* let's first see if we can find the proper dsdt */ >> + if (find_and_load_dsdt("q35-acpi-dsdt.aml")) { >> + fprintf(stderr, "Couldn't find q35 dsdt table!\n" >> + "Try updating your bios.\n"); >> + exit(1); >> + } >> + >> pc_cpus_init(cpu_model); >> >> kvmclock_create(); >> > > Should we take the PIIX4 DSDT into the QEMU tree, and do the same thing > for PIIX4? We should surely have both at the same place. Dunno what the best place is, we have interactions with both seabios code and acpi tables and qemu + acpi tables. I'd tend to put them into seabios.git, add a config option to select piix/q35, then go build two bios binaries. cheers, Gerd