Gleb Natapov wrote:
On Mon, Jun 14, 2010 at 09:54:25AM -0400, Kevin O'Connor wrote:
Could we just have qemu build the hpet tables and pass them through to seabios? Perhaps using the qemu_cfg_acpi_additional_tables() method.
Possible, and I considered that. I personally prefer to pass minimum information required for seabios to discover underlying HW and leave ACPI table creation to seabios. That is how things done for HW that seabios can actually detect. If we will go your way pretty soon we will move creation of ACPI/SMBIOS/MP tables into qemu and IMHO this will be step backworkds.
Why would creation of all the tables in qemu be a bad thing or a step in the wrong direction?
Crude argument in favour of doing it in qemu: They're both C code, so sharing code between qemu and SeaBIOS may not be as traumatic as it would be for an asm BIOS. Doing in SeaBIOS forces the existence of another API, in effect, to pass the qemu-specific hardware information, so doing it in qemu means one less interface API that needs designing and maintaining.
Argument in favour of doing it all in SeaBIOS: I'm not sure, what is it?
Indeed, why not build all the tables outside qemu using a separate tool ("qemu-build-acpi-tables < machine-description.txt"), invoked from qemu when it starts up, making it easier to examine the tables using ACPI tools?
-- Jamie