From: "Cédric Le Goater" <clg@fr.ibm.com>
To: minyard@acm.org, qemu-devel@nongnu.org,
Paolo Bonzini <pbonzini@redhat.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
Igor Mammedov <imammedo@redhat.com>,
"list@suse.de:PowerPC" <qemu-ppc@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH v4 00/17] Add an IPMI device to QEMU
Date: Sat, 14 Nov 2015 18:25:25 +0100 [thread overview]
Message-ID: <56476E85.4070406@fr.ibm.com> (raw)
In-Reply-To: <1447354953-18893-1-git-send-email-minyard@acm.org>
On 11/12/2015 08:02 PM, 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. This is more
> consistent with the way most other things work, anyway. It did
> require adding stubs for systems without IPMI.
>
> The first nine patches are unchanged.
>
> The IPMI firmware configuration storage now holds the firmware information
> in a data structure and lets it be iterated.
>
> 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. This required pulling some code out of smbios.c into an
> include file and making some things global, as IPMI has to be
> configurable.
>
> The BIOS table tests were also modified since the ACPI info is
> now in the existing SSDT.
>
> 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.
>
> Thanks all!
>
> -corey
Hi Corey,
Adding your patchset to Ben's work on the PowerNV platform gives us
a promising OpenPower system simulator !
I did not have to do too much to make it work. A few SENSOR_EVENT
commands are missing which Openpower systems use. I did the merge
and quick addons here :
https://github.com/legoater/qemu/tree/powernv-ipmi
For a first try, that went extremely well. The system boots and
the ipmi device is fully functionnal. I guess we would need to
define a larger set of sdrs. They can be added to the bmc simulator
device probably with time.
OpenPower systems also use some specific OEM event upon reboot
and reset when initiated from the bmc. We need will to find a way
to fill the event buffer with such data. Nothing complex.
I was wondering how much fix up we should be doing in the firmware
(skiboot) when running a qemu guest. I guess we should try to
generate as much we can of the device tree in qemu depending on
the devices available ? or shall we force the qemu platform in
skiboot to be strictly openpower oriented ?
Thanks,
C.
next prev parent reply other threads:[~2015-11-14 17:25 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-12 19:02 [Qemu-devel] [PATCH v4 00/17] Add an IPMI device to QEMU minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 02/17] Add a base IPMI interface minyard
2015-11-18 18:41 ` Corey Minyard
2015-11-18 20:42 ` Michael S. Tsirkin
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 03/17] ipmi: Add a local BMC simulation minyard
2015-11-24 13:31 ` Cédric Le Goater
2015-11-24 19:46 ` Corey Minyard
2015-11-26 18:07 ` Cédric Le Goater
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 04/17] ipmi: Add an external connection simulation interface minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 05/17] ipmi: Add an ISA KCS low-level interface minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 06/17] ipmi: Add a BT " minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 07/17] ipmi: Add tests minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 08/17] ipmi: Add documentation minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 09/17] ipmi: Add migration capability to the IPMI devices minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 10/17] ipmi: Add a firmware configuration repository minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 11/17] ipmi: Add firmware registration to the ISA interface minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 12/17] smbios: Move table build tools into an include file minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 13/17] pc: Postpone SMBIOS table installation to post machine init minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 14/17] ipmi: Add SMBIOS table entry minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 15/17] acpi: Add IPMI table entries minyard
2015-11-12 19:02 ` [Qemu-devel] [PATCH v4 17/17] ipmi: Add a force off function minyard
2015-11-13 9:15 ` Daniel P. Berrange
2015-11-13 13:22 ` Corey Minyard
2015-11-13 13:23 ` Paolo Bonzini
2015-11-13 13:34 ` Corey Minyard
2015-11-13 14:04 ` Daniel P. Berrange
2015-11-14 17:25 ` Cédric Le Goater [this message]
2015-11-16 3:22 ` [Qemu-devel] [PATCH v4 00/17] Add an IPMI device to QEMU Benjamin Herrenschmidt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56476E85.4070406@fr.ibm.com \
--to=clg@fr.ibm.com \
--cc=imammedo@redhat.com \
--cc=minyard@acm.org \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.