From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:52640)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZyAO1-0003SS-CS
	for qemu-devel@nongnu.org; Sun, 15 Nov 2015 22:23:22 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZyANy-00065N-9X
	for qemu-devel@nongnu.org; Sun, 15 Nov 2015 22:23:21 -0500
Received: from gate.crashing.org ([63.228.1.57]:52632)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <benh@kernel.crashing.org>) id 1ZyANy-00062B-0g
	for qemu-devel@nongnu.org; Sun, 15 Nov 2015 22:23:18 -0500
Message-ID: <1447644179.3081.81.camel@kernel.crashing.org>
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: Mon, 16 Nov 2015 14:22:59 +1100
In-Reply-To: <1447354953-18893-1-git-send-email-minyard@acm.org>
References: <1447354953-18893-1-git-send-email-minyard@acm.org>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH v4 00/17] Add an IPMI device to QEMU
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: minyard@acm.org, qemu-devel@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>

On Thu, 2015-11-12 at 13:02 -0600, minyard@acm.org wrote:
> This is a long delayed patch set, but I think I have things reworked
> to make Igor happy with the way ACPI and SMBIOS work.=C2=A0=C2=A0This i=
s more
> consistent with the way most other things work, anyway.=C2=A0=C2=A0It d=
id
> require adding stubs for systems without IPMI.

Thanks Corey !

This looks like it's going to be very useful for the new OpenPower
model I have been working on.

Cheers,
Ben.

> The first nine patches are unchanged.
>=20
> The IPMI firmware configuration storage now holds the firmware
> information
> in a data structure and lets it be iterated.
>=20
> SMBIOS and ACPI build the tables with their existing building
> functions and call out to the IPMI one (or the stub) when the time
> comes.=C2=A0=C2=A0This required pulling some code out of smbios.c into =
an
> include file and making some things global, as IPMI has to be
> configurable.
>=20
> The BIOS table tests were also modified since the ACPI info is
> now in the existing SSDT.
>=20
> I've also added a force-off function for external BMCs; an external
> BMC needs a way to do a hard power-off of the system if the soft
> power offs fail.
>=20
> Thanks all!
>=20
> -corey
>=20