[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/11] build ACPI MADT for fw_cfg clients
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 00/11] build ACPI MADT for fw_cfg clients |
Date: |
Wed, 03 Apr 2013 15:05:24 -0500 |
User-agent: |
Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Laszlo Ersek <address@hidden> writes:
> On 03/21/13 00:23, Laszlo Ersek wrote:
>> This series reworks the internals of the -acpitable command line option,
>> and if that option is not specified, produces the APIC (MADT) table
>> inside qemu, to be consumed over fw_cfg (alongside the DSDT).
>
>> Laszlo Ersek (11):
>> strip some whitespace
>> change element type from "char" to "unsigned char" in ACPI table data
>> acpi_table_add(): report fatal errors through an internal Error
>> object
>> qapi schema: add AcpiTableOptions
>> acpi_table_add(): accept QemuOpts and parse it with OptsVisitor
>> acpi_table_add(): extract and reimplement internals
>> like acpi_table_install(), acpi_table_add() should propagate Errors
>> extract/unify the constant 0xfee00000 as APIC_DEFAULT_ADDRESS
>> Introduce IO_APIC_DEFAULT_ADDRESS for 0xfec00000
>> pc_acpi_init(): don't bail as soon as failing to find default DSDT
>> i386/pc: build ACPI MADT for fw_cfg clients
>
> Any chance patches 01 to 09 could be considered? Esp. 06 which removes
> an out-of-bounds access (an innocent-looking one, admittedly).
>
> I'm OK too if the series is dropped (patch 11 was the main motivation,
> but the interface that it extends was deemed unsuitable going forward on
> the seabios list). I'd just like to hear the maintainer with
> jurisdiction say the NAK. ("Too expensive even to review for too little
> gain" is a good reason.)
The whole thing looks pretty nice to me. I'll merge the full series in
a day or so unless anyone objects.
This is an under maintained area of QEMU so it's very nice to see
improvements!
Reviewed-by: Anthony Liguori <address@hidden>
Regards,
Anthony Liguori
>
> Thanks
> Laszlo