[CRIU] [PATCH] restore: Wait for userns being prepared before moving into cgroups (v2)
    Cyrill Gorcunov 
    gorcunov at gmail.com
       
    Thu Apr 21 05:42:24 PDT 2016
    
    
  
On Thu, Apr 21, 2016 at 03:28:54PM +0300, Pavel Emelyanov wrote:
> The commit message doesn't match the patch itself.
These stages are used for completion.
> 
> > @@ -213,7 +213,8 @@ static inline unsigned long restorer_stack(struct thread_restore_args *a)
> >  
> >  enum {
> >  	CR_STATE_FAIL		= -1,
> > -	CR_STATE_RESTORE_NS	= 0, /* is used for executing "setup-namespace" scripts */
> > +	CR_STATE_RESTORE_USERNS = 0,	/* restore uid_map and gid_map */
> > +	CR_STATE_RESTORE_NS,		/* is used for executing "setup-namespace" scripts */
> 
> Why not merge it with CR_STATE_RESTORE_NS?
At least because it makes change more visible, and
moreover CR_STATE_RESTORE_NS already sync'ing
join-ns and such.
    
    
More information about the CRIU
mailing list