[libvirt] [PATCH 00/12] A bunch of extensions to libxl driver

This are some additional features to libxl driver. Some of them require change in domain config structures/syntax. Details described with each patch. The last two patches are bugfix for deadlock during daemon startup. PS Please keep me in To/Cc - I'm not subscribed to the list. Marek Marczykowski (12): libxl: allow script for any network interface, not only bridge libxl: PCI passthrough support libxl: nodeDevice* support for PCI devices libxl: populate xenstore memory entries at startup conf: add 'script' attribute to disk specification libxl: use disk 'script' attribute conf: support backend domain name in disk and network devices libxl: support backend domain setting for disk and net devices libxl: fill HVM SDL and VNC settings based on <graphics/> entries RFC: libxl: special 'stubdom-dm' emulator to use qemu in stub domain conf: virDomainObjListRemoveLocked function libxl: fix deadlock in libxlReconnectDomain docs/schemas/domaincommon.rng | 14 ++ src/conf/domain_conf.c | 54 ++++++++ src/conf/domain_conf.h | 5 + src/libvirt_private.syms | 1 + src/libxl/libxl_conf.c | 297 +++++++++++++++++++++++++++++++++--------- src/libxl/libxl_conf.h | 6 +- src/libxl/libxl_driver.c | 253 +++++++++++++++++++++++++++++++---- 7 files changed, 545 insertions(+), 85 deletions(-) -- 1.8.1.4

On 04/09/2013 07:53 PM, Marek Marczykowski wrote:
This are some additional features to libxl driver. Some of them require change in domain config structures/syntax. Details described with each patch. The last two patches are bugfix for deadlock during daemon startup.
PS Please keep me in To/Cc - I'm not subscribed to the list.
It's list policy to reply-to-all; people who don't want an explicit cc already use list manager features to suppress duplicates, and/or Mail-Reply-To headers to tell reply-to-all how to behave. Your patch series did not come through with any threading, which makes it very hard to review. When using 'git send-email', it's good to send a patch series to yourself before the list to make sure it will come through correctly. You want each one of the messages (1/12, 2/12, etc) to be marked In-Reply-To the Message-id of the 0/12 message. It would be helpful if you could resend, so that all your patches are properly threaded. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org

On 10.04.2013 04:34, Eric Blake wrote:
On 04/09/2013 07:53 PM, Marek Marczykowski wrote:
This are some additional features to libxl driver. Some of them require change in domain config structures/syntax. Details described with each patch. The last two patches are bugfix for deadlock during daemon startup.
PS Please keep me in To/Cc - I'm not subscribed to the list.
It's list policy to reply-to-all; people who don't want an explicit cc already use list manager features to suppress duplicates, and/or Mail-Reply-To headers to tell reply-to-all how to behave.
Your patch series did not come through with any threading, which makes it very hard to review. When using 'git send-email', it's good to send a patch series to yourself before the list to make sure it will come through correctly. You want each one of the messages (1/12, 2/12, etc) to be marked In-Reply-To the Message-id of the 0/12 message.
It would be helpful if you could resend, so that all your patches are properly threaded.
Done, sorry for the noise. -- Best Regards / Pozdrawiam, Marek Marczykowski Invisible Things Lab
participants (2)
-
Eric Blake
-
Marek Marczykowski