From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:59869)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <lersek@redhat.com>) id 1aOUsC-0007jU-AL
	for qemu-devel@nongnu.org; Wed, 27 Jan 2016 13:31:21 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <lersek@redhat.com>) id 1aOUs8-00020X-2t
	for qemu-devel@nongnu.org; Wed, 27 Jan 2016 13:31:20 -0500
Received: from mx1.redhat.com ([209.132.183.28]:33991)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <lersek@redhat.com>) id 1aOUs7-00020Q-Tu
	for qemu-devel@nongnu.org; Wed, 27 Jan 2016 13:31:16 -0500
References: <1453126333-19474-1-git-send-email-lersek@redhat.com>
From: Laszlo Ersek <lersek@redhat.com>
Message-ID: <56A90CEE.4020100@redhat.com>
Date: Wed, 27 Jan 2016 19:31:10 +0100
MIME-Version: 1.0
In-Reply-To: <1453126333-19474-1-git-send-email-lersek@redhat.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v2 0/4] set the OEM fields in the RSDT and
 the FADT from the SLIC
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>
Cc: Aleksei Kovura <alex3kov@zoho.com>, Xiao Guangrong <guangrong.xiao@linux.intel.com>, Steven Newbury <steve@snewbury.org.uk>, Michael Tokarev <mjt@tls.msk.ru>, qemu-devel@nongnu.org, "Richard W.M. Jones" <rjones@redhat.com>, Shannon Zhao <zhaoshenglong@huawei.com>, Paolo Bonzini <pbonzini@redhat.com>

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" <mst@redhat.com>
> Cc: Aleksei Kovura <alex3kov@zoho.com>
> Cc: Igor Mammedov <imammedo@redhat.com>
> Cc: Michael Tokarev <mjt@tls.msk.ru>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: Richard W.M. Jones <rjones@redhat.com>
> Cc: Shannon Zhao <zhaoshenglong@huawei.com>
> Cc: Steven Newbury <steve@snewbury.org.uk>
> Cc: Xiao Guangrong <guangrong.xiao@linux.intel.com>
> 
> 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 <shannon.zhao@linaro.org>
         Reviewed-by: Steven Newbury <steve@snewbury.org.uk>
         Tested-by: Aleksei Kovura <alex3kov@zoho.com>

patch 2: Reviewed-by: Shannon Zhao <shannon.zhao@linaro.org>
         Reviewed-by: Steven Newbury <steve@snewbury.org.uk>
         Tested-by: Aleksei Kovura <alex3kov@zoho.com>

patch 3: Reviewed-by: Steven Newbury <steve@snewbury.org.uk>
         Tested-by: Aleksei Kovura <alex3kov@zoho.com>

patch 4: Reviewed-by: Steven Newbury <steve@snewbury.org.uk>
         Tested-by: Aleksei Kovura <alex3kov@zoho.com>

(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