Re: [PATCH 0/9] LXC: Use virDomainObjGetDefs more

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

 



On Wed, Jun 08, 2016 at 08:45:21AM -0400, John Ferlan wrote:
> 
> 
> On 06/06/2016 04:08 AM, Ján Tomko wrote:
> > In a few API implementations, we use virDomainLiveConfigHelperMethod
> > to check the VIR_DOMAIN_AFFECT_* flags (and change the flags variable
> > to either AFFECT_LIVE or AFFECT_CONFIG if AFFECT_CURRENT was requested)
> > and optionally give us a pointer to the persistent definition.
> > 
> > This function can also create the persistent definition in vm->newDef
> > for a live domain if it does not exist yet. This cannot be the case
> > in LXC driver because we create it unconditionally on domain startup.
> > 
> > 
> > By switching to virDomainObjGetDefs we do not need to pass virCaps
> > and virDomainXMLOption (because the defintion does not need to be copied).
> > 
> > Also, instead of altering the flags, it fills the pointers to live and persistent
> > definitions depending on which one was requested to be modified,
> > making the code easier to read for static analyzers.
> > 
> > 
> > Ján Tomko (9):
> >   lxc: rename vmdef to persistentDef
> >   Use virDomainObjGetDefs in lxcDomainSetMemoryFlags
> >   Use virDomainObjGetDefs in lxcDomainSetSchedulerParametersFlags
> >   Use virDomainObjGetDefs in lxcDomainGetSchedulerParametersFlags
> >   Use virDomainObjGetDefs in lxcDomainGetMemoryParameters
> >   Use virDomainObjGetDefs in lxcDomainSetBlkioParameters
> >   Use virDomainObjGetDefs in lxcDomainGetBlkioParameters
> >   Export virDomainGetBlkioParametersAssignFromDef
> >   lxc: simplify lxcDomainGetBlkioParameters
> > 
> >  src/conf/domain_conf.c   |  51 +++++
> >  src/conf/domain_conf.h   |   6 +
> >  src/libvirt_private.syms |   1 +
> >  src/lxc/lxc_driver.c     | 511 +++++++++--------------------------------------
> >  src/qemu/qemu_driver.c   |  59 +-----
> >  5 files changed, 154 insertions(+), 474 deletions(-)
> > 
> 
> In patch 9 you could reference commit id '9f50f6e28' ... although it's
> also possible to figure it out with a bit of work...
> 
> Should lxcDomainSetMemoryParameters get the same virDomainObjGetDefs usage?
> 

Yes, I  must've missed it for some reason.

> ACK series - just adjust patch 5 to remove the caps.

Thanks, I have added the commit reference, fixed the nits and pushed the
series.

Jan

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]