From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48857) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vdjo2-0005Le-GP for qemu-devel@nongnu.org; Tue, 05 Nov 2013 11:48:47 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vdjnw-0001Zt-SX for qemu-devel@nongnu.org; Tue, 05 Nov 2013 11:48:41 -0500 Received: from mail-oa0-x233.google.com ([2607:f8b0:4003:c02::233]:59699) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vdjnw-0001Zp-NZ for qemu-devel@nongnu.org; Tue, 05 Nov 2013 11:48:36 -0500 Received: by mail-oa0-f51.google.com with SMTP id o6so830092oag.10 for ; Tue, 05 Nov 2013 08:48:36 -0800 (PST) Sender: Corey Minyard Message-ID: <52792162.1060808@acm.org> Date: Tue, 05 Nov 2013 10:48:34 -0600 From: Corey Minyard MIME-Version: 1.0 References: <1369865296-19584-1-git-send-email-minyard@acm.org> <20131105135644.GA3692@redhat.com> <5279183A.2070106@suse.de> In-Reply-To: <5279183A.2070106@suse.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 00/20] Add an IPMI device to QEMU Reply-To: minyard@acm.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?ISO-8859-1?Q?Andreas_F=E4rber?= Cc: openipmi-developer@lists.sourceforge.net, qemu-devel@nongnu.org, "Michael S. Tsirkin" On 11/05/2013 10:09 AM, Andreas Färber wrote: > Hi, > > Am 05.11.2013 14:56, schrieb Michael S. Tsirkin: >> On Wed, May 29, 2013 at 05:07:56PM -0500, minyard@acm.org wrote: >>> I have finally gotten some time to work on this, this series of >>> patches adds an IPMI interface to qemu. The changes are roughly: >>> >>> patches 01-05 - Add the capability to have a chardev reconnect if >>> the connections fails. This way, if using an external BMC, qemu >>> will detect the failure and periodically attempt to reconnect. >>> This also adds ways for the device code to get an event on a >>> disconnect and connect so it can handle it properly. This is >>> probably useful for things besides IPMI. There are also a few >>> small bugfixes in this. >>> >>> patches 06-14 - Add the IPMI device itself, with an ISA interface >>> for now (PCI and others can also be added easily). >>> >>> patches 15-18 - Add a way to dynamically add content to the ACPI >>> tables, and add the capability to add the IPMI information to the >>> table. >>> >>> Patches 19-20 - Add a way to dynamically add content to the SMBIOS >>> tables, and add an IPMI entry to the table. >>> >> I was pointed at these patches as an example of useful >> functionality that's out of qemu merely for lack of review >> resources. I'd like to help. >> >> Now that we have code to generate ACPI tables >> directly in qemu, this series can be rebased on top of >> that, with no need for new FW CFG entries or bios changes. >> >> If you have the time, pls Cc me on patches and I'll try to >> help shepherd them upstream. > Me too, I have IPMI on my radar for 1.8/2.0. > > Our use case would be using IPMI tools to boot/reboot/shutdown a guest > and to access its serial console as done for the physical hosts; we had > discussed in the past where to locate such a TCP server component, > whether in QEMU or libvirt - am I interpreting correctly that patch 11 > is preparing an interface to have that as external process now? Or is it > hidden somewhere in patch 10 or is this unimplemented in this version? Patch 11 does that, it gives an external interface. The OpenIPMI library has a simulator that will plug in to this interface to provide BMC functions and an IPMI LAN interface. You can do the boot/reboot/shutdown via the normal mechanisms with the external simulator. The internal simulator doesn't have an IPMI LAN interface, though, so you would need the external interface. I also have not provided a serial over LAN interface through this interface. If that's what you are looking for, more work would need to be done. Thanks, -corey > > Also, some of the IPMI acronyms in the commit messages (KCS, BT, ...) > could use some explanation in the next version. ;) > > Regards, > Andreas >