Re: [Qemu-devel] [PATCH v2 0/4] set the OEM fields in the RSDT and the FADT from the SLIC

[Laszlo Ersek]

On 01/18/16 15:12, Laszlo Ersek wrote:
> This is version 2 of
> <http://thread.gmane.org/gmane.comp.emulators.qemu/386728>.
> 
> Changes in v2 (also annotated per patch):
> - introduce acpi_get_slic_oem() [MST],
> - drop machine property that disables the SLIC-based override [MST].
> 
> Cc: "Michael S. Tsirkin" <address@hidden>
> Cc: Aleksei Kovura <address@hidden>
> Cc: Igor Mammedov <address@hidden>
> Cc: Michael Tokarev <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Richard W.M. Jones <address@hidden>
> Cc: Shannon Zhao <address@hidden>
> Cc: Steven Newbury <address@hidden>
> Cc: Xiao Guangrong <address@hidden>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (4):
>   acpi: take oem_id in build_header(), optionally
>   acpi: expose oem_id and oem_table_id in build_rsdt()
>   acpi: add function to extract oem_id and oem_table_id from the user's
>     SLIC
>   pc: set the OEM fields in the RSDT and the FADT from the SLIC
> 
>  include/hw/acpi/acpi.h      |  7 +++++++
>  include/hw/acpi/aml-build.h |  5 +++--
>  hw/acpi/aml-build.c         | 14 ++++++++++----
>  hw/acpi/core.c              | 16 ++++++++++++++++
>  hw/acpi/nvdimm.c            |  4 ++--
>  hw/arm/virt-acpi-build.c    | 14 +++++++-------
>  hw/i386/acpi-build.c        | 31 ++++++++++++++++++-------------
>  qemu-options.hx             |  4 ++++
>  8 files changed, 67 insertions(+), 28 deletions(-)
> 

Ping -- here's the review & testing feedback thus far:

patch 1: Reviewed-by: Shannon Zhao <address@hidden>
         Reviewed-by: Steven Newbury <address@hidden>
         Tested-by: Aleksei Kovura <address@hidden>

patch 2: Reviewed-by: Shannon Zhao <address@hidden>
         Reviewed-by: Steven Newbury <address@hidden>
         Tested-by: Aleksei Kovura <address@hidden>

patch 3: Reviewed-by: Steven Newbury <address@hidden>
         Tested-by: Aleksei Kovura <address@hidden>

patch 4: Reviewed-by: Steven Newbury <address@hidden>
         Tested-by: Aleksei Kovura <address@hidden>

(Tested-by inferred from
<https://bugzilla.redhat.com/show_bug.cgi?id=1248758#c28>.)

Michael, Igor, can one of you guys please pick this up?

Thanks
Laszlo