On Mon, Dec 1, 2008 at 7:18 PM, Li Zefan <lizf@xxxxxxxxxxxxxx> wrote: > > How about: > > /** > * link_css_set - a helper function to link a css_set to a cgroup > * @ tmp_cg_links: cg_cgroup_link objects allocated by allocate_cg_links() > * @ cg: the css_set to be linked > * @ cgrp: the destination cgroup > */ Sounds good. > > Because we are linking all the css_sets to the root cgroup, so I think root_cgrp > helps readability. > But we're creating the hierarchy at this point, so there can clearly only be one cgroup any (which is the root cgroup). I don't think it's any more or less readable, it just seems an unnecessary change. Paul _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers