On 06.03.2013 16:37, Peter Krempa wrote: > --- > src/Makefile.am | 2 ++ > src/lxc/lxc_conf.c | 8 ++++++++ > src/lxc/lxc_conf.h | 2 ++ > src/lxc/lxc_controller.c | 7 ++++++- > src/lxc/lxc_domain.c | 13 ++++++------- > src/lxc/lxc_domain.h | 2 +- > src/lxc/lxc_driver.c | 43 ++++++++++++++++++++++++------------------- > src/lxc/lxc_process.c | 9 +++++---- > 8 files changed, 54 insertions(+), 32 deletions(-) > > diff --git a/src/Makefile.am b/src/Makefile.am > index c1659a4..b615902 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -493,6 +493,8 @@ LXC_CONTROLLER_SOURCES = \ > lxc/lxc_conf.c lxc/lxc_conf.h \ > lxc/lxc_container.c lxc/lxc_container.h \ > lxc/lxc_cgroup.c lxc/lxc_cgroup.h \ > + lxc/lxc_domain.c lxc/lxc_domain.h \ > + lxc/lxc_hostdev.c lxc/lxc_hostdev.h \ > lxc/lxc_fuse.c lxc/lxc_fuse.h \ > lxc/lxc_controller.c This one looks spurious. At least lxc_hostdev.h > > diff --git a/src/lxc/lxc_conf.c b/src/lxc/lxc_conf.c > index 7b808e7..c723e77 100644 > --- a/src/lxc/lxc_conf.c > +++ b/src/lxc/lxc_conf.c > @@ -27,6 +27,7 @@ > #include <config.h> > > #include "lxc_conf.h" > +#include "lxc_domain.h" > #include "nodeinfo.h" > #include "virerror.h" > #include "virconf.h" > @@ -154,6 +155,13 @@ error: > return NULL; > } > > + > +virDomainXMLConfPtr > +lxcDomainXMLConfInit(void) > +{ > + return virDomainXMLConfNew(&virLXCDriverPrivateDataCallbacks, NULL); > +} > + Is this wrapper function really needed? > int lxcLoadDriverConfig(virLXCDriverPtr driver) > { > char *filename; -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list