All of lore.kernel.org
 help / color / mirror / Atom feed
From: Corey Minyard <tcminyard@gmail.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] First shot at adding IPMI to qemu
Date: Tue, 10 Jul 2012 10:43:35 -0500	[thread overview]
Message-ID: <4FFC4DA7.7040203@acm.org> (raw)
In-Reply-To: <4FFBF77E.5030705@redhat.com>

On 07/10/2012 04:35 AM, Paolo Bonzini wrote:
> Il 09/07/2012 21:17, minyard@acm.org ha scritto:
>> I had asked about getting an IPMI device into qemu and received some
>> interest, and it's useful to me, so I've done some work to add it.
>> The following patch set has a set of patches to add an IPMI KCS
>> device, and IPMI BT device, a built-in BMC (IPMI management controller),
>> and a way to attach an external BMC through a chardev.
>>
>> There was some discussion on whether to make the BMC internal or
>> external, but I went ahead and added both.  The internal one is
>> fairly basic and not extensible, at least without adding code.
>> I've modified the OpenIPMI library simulator to work with the
>> external interface to allow it to receive connections from the
>> qemu external simulator with a fairly basic protocol.
>>
>> I've also added the ability for the OpenIPMI library to manage
>> a VM to power it on, power it off, reset it, and handle an IPMI
>> watchdog timer.  So it looks quite like a real system.  Instructions
>> for using it are in the OpenIPMI release candidate I uploaded to
>> https://sourceforge.net/projects/openipmi
>>
>> Since IPMI can advertise its presence via SMBIOS, I added a
>> way for a driver to add an SMBIOS entry.  I also added a way
>> to query a free interrupt from the ISA bus, since the interrupt
>> is in the SMBIOS entry and nobody really cares which one is used.
> I provided some feedback in the individual patches, it shouldn't be a
> lot of work compared to what you have done already!

Yes, looks like all good comments.  I'll work on these over the next few 
days and resubmit.  Thanks.

>
> It would be great if you could add a basic testcase using qtest, even if
> only for the internal interface, to ensure it doesn't bitrot.

Yes, certainly.

Thanks again,

-corey

      reply	other threads:[~2012-07-10 15:43 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-09 19:17 [Qemu-devel] First shot at adding IPMI to qemu minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 1/9] smbios: Add a function to directly add an entry minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 2/9] pc: move SMBIOS setup to after device init minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 3/9] isa: Add a way to query for a free interrupt minyard
2012-07-10  9:16   ` Paolo Bonzini
2012-07-09 19:17 ` [Qemu-devel] [PATCH 4/9] Add a base IPMI interface minyard
2012-07-10  9:17   ` Daniel P. Berrange
2012-07-10 11:12     ` Markus Armbruster
2012-07-10 16:19     ` Corey Minyard
2012-07-10  9:29   ` Paolo Bonzini
2012-07-09 19:17 ` [Qemu-devel] [PATCH 5/9] IPMI: Add a PC ISA type structure minyard
2012-07-10  9:17   ` Paolo Bonzini
2012-07-10  9:33   ` Paolo Bonzini
2012-07-10 20:00     ` Corey Minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 6/9] IPMI: Add a KCS low-level interface minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 7/9] IPMI: Add a BT " minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 8/9] IPMI: Add a local BMC simulation minyard
2012-07-09 19:17 ` [Qemu-devel] [PATCH 9/9] IPMI: Add an external connection simulation interface minyard
2012-07-10  9:35 ` [Qemu-devel] First shot at adding IPMI to qemu Paolo Bonzini
2012-07-10 15:43   ` Corey Minyard [this message]

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=4FFC4DA7.7040203@acm.org \
    --to=tcminyard@gmail.com \
    --cc=minyard@acm.org \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@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.