[libvirt] [PATCH] Remove macvtap dependancy on domain configuration
Daniel P. Berrange
berrange at redhat.com
Wed Jun 22 16:36:38 UTC 2011
On Wed, Jun 22, 2011 at 04:45:13PM +0100, Daniel P. Berrange wrote:
> Files under src/util must not depend on src/conf
> Solve the macvtap problem by moving the definition
> of macvtap modes from domain_conf.h into macvtap.h
>
> * src/util/macvtap.c, src/util/macvtap.h: Add enum
> for macvtap modes
> * src/conf/domain_conf.c, src/conf/domain_conf.h: Remove
> enum for macvtap modes
> ---
> src/conf/domain_conf.c | 17 ++++++-----------
> src/conf/domain_conf.h | 15 +--------------
> src/util/macvtap.c | 40 +++++++++++++++++-----------------------
> src/util/macvtap.h | 14 +++++++++++++-
> 4 files changed, 37 insertions(+), 49 deletions(-)
>
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> @@ -1156,7 +1150,7 @@ static virDomainObjPtr virDomainObjNew(virCapsPtr caps)
> }
>
> if (caps->privateDataAllocFunc &&
> - !(domain->privateData = (caps->privateDataAllocFunc)())) {
> + !(domain->privateData = (caps->privateDataAllocFunc)(caps->privateDataOpaque))) {
> virReportOOMError();
> VIR_FREE(domain);
> return NULL;
This chunk is also a rebase mistake, from the qemu monitor timeouts patch
so will be removed from here
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 :|
More information about the libvir-list
mailing list