Re: [PATCH v1 1/2] fork: add clone3

2019-06-01 Thread Yann Droneaud
Hi, Le samedi 01 juin 2019 à 00:08 +0200, Christian Brauner a écrit : > On Wed, May 29, 2019 at 05:42:14PM +0200, Yann Droneaud wrote: > > Le mercredi 29 mai 2019 à 17:22 +0200, Christian Brauner a écrit : > > > This adds the clone3 system call. > > > > > > > > > diff --git a/kernel/fork.c b/ker

Re: [PATCH v1 1/2] fork: add clone3

2019-05-31 Thread Christian Brauner
On Fri, May 31, 2019 at 01:38:29PM -0700, Linus Torvalds wrote: > On Wed, May 29, 2019 at 3:24 PM Andrei Vagin wrote: > > > > Thank you for thinking about time namespaces. I looked at this patch > > quickly and I would suggest to move a termination signal out of flags. I > > think we can add a sep

Re: [PATCH v1 1/2] fork: add clone3

2019-05-31 Thread Christian Brauner
On Wed, May 29, 2019 at 03:24:15PM -0700, Andrei Vagin wrote: > On Wed, May 29, 2019 at 05:22:36PM +0200, Christian Brauner wrote: > > This adds the clone3 system call. > > > > As mentioned several times already (cf. [7], [8]) here's the promised > > patchset for clone3(). > > > > We recently mer

Re: [PATCH v1 1/2] fork: add clone3

2019-05-31 Thread Christian Brauner
On Wed, May 29, 2019 at 05:42:14PM +0200, Yann Droneaud wrote: > Le mercredi 29 mai 2019 à 17:22 +0200, Christian Brauner a écrit : > > This adds the clone3 system call. > > > > > > diff --git a/kernel/fork.c b/kernel/fork.c > > index b4cba953040a..6bc3e3d17150 100644 > > --- a/kernel/fork.c > >

Re: [PATCH v1 1/2] fork: add clone3

2019-05-31 Thread Linus Torvalds
On Wed, May 29, 2019 at 3:24 PM Andrei Vagin wrote: > > Thank you for thinking about time namespaces. I looked at this patch > quickly and I would suggest to move a termination signal out of flags. I > think we can add a separate field (exit_signal) into clone_args. Does it > make sense? For me, e

Re: [PATCH v1 1/2] fork: add clone3

2019-05-31 Thread Arnd Bergmann
On Thu, May 30, 2019 at 3:20 PM Szabolcs Nagy wrote: > * Christian Brauner [2019-05-29 17:22:36 +0200]: > > /* uapi */ > > struct clone_args { > > __aligned_u64 flags; > > __aligned_u64 pidfd; > > __aligned_u64 parent_tidptr; > > __aligned_u64 child_tidptr; > >

Re: [PATCH v1 1/2] fork: add clone3

2019-05-30 Thread Szabolcs Nagy
* Christian Brauner [2019-05-29 17:22:36 +0200]: > This adds the clone3 system call. > > As mentioned several times already (cf. [7], [8]) here's the promised > patchset for clone3(). > > We recently merged the CLONE_PIDFD patchset (cf. [1]). It took the last > free flag from clone(). > > Indep

Re: [PATCH v1 1/2] fork: add clone3

2019-05-29 Thread Andrei Vagin
On Wed, May 29, 2019 at 05:22:36PM +0200, Christian Brauner wrote: > This adds the clone3 system call. > > As mentioned several times already (cf. [7], [8]) here's the promised > patchset for clone3(). > > We recently merged the CLONE_PIDFD patchset (cf. [1]). It took the last > free flag from cl

Re: [PATCH v1 1/2] fork: add clone3

2019-05-29 Thread Yann Droneaud
Le mercredi 29 mai 2019 à 17:22 +0200, Christian Brauner a écrit : > This adds the clone3 system call. > > > diff --git a/kernel/fork.c b/kernel/fork.c > index b4cba953040a..6bc3e3d17150 100644 > --- a/kernel/fork.c > +++ b/kernel/fork.c > @@ -2472,7 +2475,96 @@ SYSCALL_DEFINE5(clone, unsigned lo