[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH V4 2/3] Introduce file descriptor set for QEMU domains



On Thu, Feb 07, 2013 at 10:35:15PM -0500, Stefan Berger wrote:
> Extend the QEMU private domain structure with virFdset.
> Persist the virFdset using XML and parse its XML.
> Reset the fdset upon domain stop.
> 
> Stefan Berger <stefanb linux vnet ibm com>
> 
> ---
>  src/qemu/qemu_domain.c  |   10 ++++++++++
>  src/qemu/qemu_domain.h  |    3 +++
>  src/qemu/qemu_process.c |    2 ++
>  3 files changed, 15 insertions(+)
> 

> Index: libvirt/src/qemu/qemu_domain.h
> ===================================================================
> --- libvirt.orig/src/qemu/qemu_domain.h
> +++ libvirt/src/qemu/qemu_domain.h
> @@ -32,6 +32,7 @@
>  # include "qemu_conf.h"
>  # include "qemu_capabilities.h"
>  # include "virchrdev.h"
> +# include "virfdset.h"
>  
>  # define QEMU_EXPECTED_VIRT_TYPES      \
>      ((1 << VIR_DOMAIN_VIRT_QEMU) |     \
> @@ -160,6 +161,8 @@ struct _qemuDomainObjPrivate {
>      qemuDomainCleanupCallback *cleanupCallbacks;
>      size_t ncleanupCallbacks;
>      size_t ncleanupCallbacks_max;
> +
> +    virFdset fdset;

s/virFdset/virFdsetPtr/ so we can just use the normal alloc/free
pattern for this.


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]