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

Reply via email to