On Thu, Sep 24, 2015 at 12:45:46PM +0300, Kirill Tkhai wrote: ... > > @@ -438,6 +445,10 @@ int ve_start_container(struct ve_struct *ve) > > ve_grab_context(ve); > > ve_list_add(ve); > > > > + err = ve_alloc_id(ve); > > + if (err) > > + goto err_id; > > I'm confused in a fact we add a ve to &ve_list_head, before we know > the ve's id is unique. for_each_ve() users are able to pick it at the moment.
Right, will fix. > > Furthermore, it looks confusing, we add a ve to the list, before it's > completely > initialized. I don't want to change the original behavior in this patch. May be I'll do it separately. Thanks. _______________________________________________ Devel mailing list [email protected] https://lists.openvz.org/mailman/listinfo/devel
