Messages in this thread | | | From | Andrey Mirkin <> | Subject | Re: [PATCH 1/2] Add flags for user-space and in-kernel process creation | Date | Wed, 26 Nov 2008 08:55:45 +0400 |
| |
On Monday 24 November 2008 19:02 Louis Rilling wrote: > On 24/11/08 18:39 +0300, Andrey Mirkin wrote: > > Introduce 2 flags for user-space and in-kernel process creation during > > restart procedure. > > Also a stub function for in-kernel process restart is introduced. > > [...] > > > diff --git a/checkpoint/sys.c b/checkpoint/sys.c > > index 7745500..e4a9287 100644 > > --- a/checkpoint/sys.c > > +++ b/checkpoint/sys.c > > @@ -264,8 +264,8 @@ asmlinkage long sys_restart(int crid, int fd, > > unsigned long flags) pid_t pid; > > int ret; > > > > - /* no flags for now */ > > - if (flags) > > + if ((flags & (CR_CTX_RSTR_IN_USERSPACE | CR_CTX_RSTR_IN_USERSPACE)) == > > + (CR_CTX_RSTR_IN_USERSPACE | CR_CTX_RSTR_IN_USERSPACE)) > > I guess that the intent was: > + if ((flags & (CR_CTX_RSTR_IN_USERSPACE | CR_CTX_RSTR_IN_KERNEL)) == > + (CR_CTX_RSTR_IN_USERSPACE | CR_CTX_RSTR_IN_KERNEL)) > > ?
Oh, of course it should be like this. I've sent wrong patch. Will resend correct one.
Andrey
| |