[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