All of lore.kernel.org
 help / color / mirror / Atom feed
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.

  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.