[lxc-devel] [PATCH 2/2] split cgroup handling into discrete backends

Serge Hallyn serge.hallyn at ubuntu.com
Thu Feb 6 05:34:16 UTC 2014


Quoting Serge Hallyn (serge.hallyn at ubuntu.com):
> Quoting Dwight Engen (dwight.engen at oracle.com):
> > - refactor cgroup into two backends, the classic cgfs driver and the new
> >   cgmanager. Instead of lxc_handler knowing about the internals of each,
> >   have it just store an opaque pointer to a struct that is private to
> >   each backend.
> > 
> > - rename a couple of cgroup functions for consistency: those that are
> >   considered an API (ie. exported by lxc.h) begin with lxc_ and those that
> >   are not are just cgroup_*
> > 
> > - made as many backend routines static as possible, only cg*_ops_init is
> >   exported
> > 
> > - made a nrtasks op which is needed by the utmp code for monitoring
> >   container shutdown, currently only implemented for the cgfs backend
> 
> Hi Dwight,
> 
> boy this diff got into some unfortunate strides (with near-identical
> code being removed then inserted pages later) making it harder
> to read than necessary :)
> 
> > Signed-off-by: Dwight Engen <dwight.engen at oracle.com>
> 
> Acked-by: Serge E. Hallyn <serge.hallyn at ubuntu.com>
> 
> (I'll apply after I run some tests)

Unfortunately, while containers started fine, lxc-attach and
lxc-cgroup no longer seemed to work.  If it isn't obvious to
you what the problem migth be, I'll look into it tomorrow.

thanks,
-serge


More information about the lxc-devel mailing list