[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Third shot at adding IPMI to qemu
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] Third shot at adding IPMI to qemu |
Date: |
Mon, 30 Jul 2012 16:05:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120713 Thunderbird/14.0 |
Am 30.07.2012 15:34, schrieb Corey Minyard:
> I haven't heard anything on this.
FWIW you did hear from Paolo. :)
> Any comments or movement on this set
> of patches?
For a third shot, you should've used --subject-prefix="PATCH v3" so that
each patch becomes distinguishable on Patchwork, in inbox searches, etc.
http://wiki.qemu.org/Contribute/SubmitAPatch
I've started commenting on some non-functional code issues that you
should review throughout the series. It seems in lack of how-tos you
have oriented yourself on some "old-style" devices, would be nice to
update that for v4 as anti-typo measure and to increase chances of
people copying a modern device. ;)
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH 18/18] IPMI: Add documentation., (continued)
- [Qemu-devel] [PATCH 18/18] IPMI: Add documentation., minyard, 2012/07/19
- [Qemu-devel] [PATCH 12/18] IPMI: Add a PC ISA type structure, minyard, 2012/07/19
- [Qemu-devel] [PATCH 17/18] IPMI: Add tests, minyard, 2012/07/19
- [Qemu-devel] [PATCH 08/18] qemu-char: Close fd at end of file, minyard, 2012/07/19
- [Qemu-devel] [PATCH 16/18] IPMI: Add an external connection simulation interface, minyard, 2012/07/19
- [Qemu-devel] [PATCH 13/18] IPMI: Add a KCS low-level interface, minyard, 2012/07/19
- [Qemu-devel] [PATCH 15/18] IPMI: Add a local BMC simulation, minyard, 2012/07/19
- Re: [Qemu-devel] Third shot at adding IPMI to qemu, Paolo Bonzini, 2012/07/20
- Re: [Qemu-devel] Third shot at adding IPMI to qemu, Corey Minyard, 2012/07/30
- Re: [Qemu-devel] Third shot at adding IPMI to qemu,
Andreas Färber <=