Quoting Christian Seiler (christ...@iwakd.de): > Hi Serge, > > > Thanks, Christian. I did need the trivial white-space-damaged patch below, > > but > > with that it built and ran for me, both with %n and default (/lxc/%n) > > patterns. > > This was in a nested container, I haven't tested at host level but have no > > reason to think that would fail if nested worked :) > > > > However I didn't get occasional cgroups left behind - every cgroup was left > > behind in my case. I'll take another look. I am to send acks and push > > tomorrow. > > I think I found the problem with that... cgroup.c line 907, > lxc_cgroup_process_info_free_and_remove should call itself recursively, > but a C&P error on my end made it call lxc_cgroup_process_info_free > instead... So only in the first hierarchy are cgroups removed properly, > in the others they aren't. > > As I said, there are probably still bugs in there. ;-) > > -- Christian
I've pushed patches 1-5 (plus the tweaks) to staging. Patch 6 and the two one-liners so far are in git://github.com/hallyn/lxc #cgroup1. I will look over it some more and make sure mountcgroup works before pushing that. thanks, -serge ------------------------------------------------------------------------------ How ServiceNow helps IT people transform IT departments: 1. Consolidate legacy IT systems to a single system of record for IT 2. Standardize and globalize service processes across IT 3. Implement zero-touch automation to replace manual, redundant tasks http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk _______________________________________________ Lxc-devel mailing list Lxc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/lxc-devel
