Paul Menage wrote: > On 4/5/07, Pavel Emelianov <[EMAIL PROTECTED]> wrote: >> >> Right now I do not. Calling container_init_eary() is OK from >> my POV. What I do not like is that I have to reinitialize >> container->create() callback. > > I don't think you should need to do that. The way cpuset_create() > handles it is for the root cpuset to be statically allocated, so the > code looks like: > > if (!cont->parent) { > /* This is early initialization for the top container */ > set_container_cs(cont, &top_cpuset); > top_cpuset.css.container = cont; > top_cpuset.mems_generation = cpuset_mems_generation++; > return 0; > } > > ... other normal code that allocates a cpuset ... > > Mildly ugly, but it solves the problem.
Well, it's less ugly than resetting pointer but it's worse from performance POV. See, we have an if (xxx) that is true only once during system lifetime. This is not that good... Anyway. When are you going to send your patches with containers and what kernel will it be built upon? I'll try to prepare the RSS patches soon after this. > Paul > ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ ckrm-tech mailing list https://lists.sourceforge.net/lists/listinfo/ckrm-tech