OpenVZ Forum


Home » Mailing lists » Devel » [PATCH] Define CLONE_NEWPID flag
[PATCH] Define CLONE_NEWPID flag [message #17943] Wed, 21 March 2007 19:41 Go to next message
Sukadev Bhattiprolu is currently offline  Sukadev Bhattiprolu
Messages: 413
Registered: August 2006
Senior Member
This was discussed on containers and we thought it would be useful
to reserve this flag.
---

From: Sukadev Bhattiprolu <sukadev@us.ibm.com>
Subject: [PATCH] Define CLONE_NEWPID flag

Define CLONE_NEWPID flag that will be used to clone pid namespaces.

Signed-off-by: Sukadev Bhattiprolu <sukadev@us.ibm.com>
---
 include/linux/sched.h |    1 +
 1 file changed, 1 insertion(+)

Index: lx26-21-rc3-mm2/include/linux/sched.h
===================================================================
--- lx26-21-rc3-mm2.orig/include/linux/sched.h	2007-03-20 20:13:19.000000000 -0700
+++ lx26-21-rc3-mm2/include/linux/sched.h	2007-03-21 11:10:33.000000000 -0700
@@ -26,6 +26,7 @@
 #define CLONE_STOPPED		0x02000000	/* Start in stopped state */
 #define CLONE_NEWUTS		0x04000000	/* New utsname group? */
 #define CLONE_NEWIPC		0x08000000	/* New ipcs */
+#define CLONE_NEWPID		0x10000000      /* New pid namespace */
 
 /*
  * Scheduling policies
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
Re: [PATCH] Define CLONE_NEWPID flag [message #17944 is a reply to message #17943] Wed, 21 March 2007 20:39 Go to previous messageGo to next message
akpm is currently offline  akpm
Messages: 224
Registered: March 2007
Senior Member
On Wed, 21 Mar 2007 12:41:03 -0700
sukadev@us.ibm.com wrote:

> 
> This was discussed on containers and we thought it would be useful
> to reserve this flag.
> ---
> 
> From: Sukadev Bhattiprolu <sukadev@us.ibm.com>
> Subject: [PATCH] Define CLONE_NEWPID flag
> 
> Define CLONE_NEWPID flag that will be used to clone pid namespaces.
> 
> Signed-off-by: Sukadev Bhattiprolu <sukadev@us.ibm.com>
> ---
>  include/linux/sched.h |    1 +
>  1 file changed, 1 insertion(+)
> 
> Index: lx26-21-rc3-mm2/include/linux/sched.h
> ===================================================================
> --- lx26-21-rc3-mm2.orig/include/linux/sched.h	2007-03-20 20:13:19.000000000 -0700
> +++ lx26-21-rc3-mm2/include/linux/sched.h	2007-03-21 11:10:33.000000000 -0700
> @@ -26,6 +26,7 @@
>  #define CLONE_STOPPED		0x02000000	/* Start in stopped state */
>  #define CLONE_NEWUTS		0x04000000	/* New utsname group? */
>  #define CLONE_NEWIPC		0x08000000	/* New ipcs */
> +#define CLONE_NEWPID		0x10000000      /* New pid namespace */
>  

Do we actually have any need to reserve it at this time?  I'd have thought
that we could defer adding this until we have some code in-kernel which
uses it.

_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
Re: [PATCH] Define CLONE_NEWPID flag [message #17948 is a reply to message #17944] Wed, 21 March 2007 20:57 Go to previous messageGo to next message
ebiederm is currently offline  ebiederm
Messages: 1354
Registered: February 2006
Senior Member
Andrew Morton <akpm@linux-foundation.org> writes:

>> Index: lx26-21-rc3-mm2/include/linux/sched.h
>> ===================================================================
>> --- lx26-21-rc3-mm2.orig/include/linux/sched.h 2007-03-20 20:13:19.000000000
> -0700
>> +++ lx26-21-rc3-mm2/include/linux/sched.h 2007-03-21 11:10:33.000000000 -0700
>> @@ -26,6 +26,7 @@
>> #define CLONE_STOPPED 0x02000000 /* Start in stopped state */
>>  #define CLONE_NEWUTS		0x04000000	/* New utsname group? */
>>  #define CLONE_NEWIPC		0x08000000	/* New ipcs */
>> +#define CLONE_NEWPID		0x10000000      /* New pid namespace */
>>  
>
> Do we actually have any need to reserve it at this time?  I'd have thought
> that we could defer adding this until we have some code in-kernel which
> uses it.

In practice this is pretty much reserved already but I understand the sentiment.

Currently the plan is to work on the core pid namespace and get those functions
merged at least to -mm with a big fat CONFIG_EXPERIMENTAL so people
can really understand what we are talking about when we say a
pid_namespace.  Then go through and finish up all converting the rest
of the pid uses that we haven't tackled yet.  There aren't that many
left and most of the remaining conversions only make sense in the
context of a pid namespace.

Currently we are one or two review cycles away from being able to push
out the core pid namespace code.

Further it is actually critical that we have a clone flag for the pid
namespace because unshare is very much harder if it is possible at
all.

Personally if you want to delay it a week or two, until the rest of
the code is ready that is fine.  Mostly getting it out now is about
release early and release often.

Eric
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
Re: [PATCH] Define CLONE_NEWPID flag [message #17999 is a reply to message #17944] Fri, 23 March 2007 01:36 Go to previous message
Herbert Poetzl is currently offline  Herbert Poetzl
Messages: 239
Registered: February 2006
Senior Member
On Wed, Mar 21, 2007 at 01:39:38PM -0700, Andrew Morton wrote:
> On Wed, 21 Mar 2007 12:41:03 -0700
> sukadev@us.ibm.com wrote:
> 
> > This was discussed on containers and we thought it would be useful
> > to reserve this flag.
> > ---
> > 
> > From: Sukadev Bhattiprolu <sukadev@us.ibm.com>
> > Subject: [PATCH] Define CLONE_NEWPID flag
> > 
> > Define CLONE_NEWPID flag that will be used to clone pid namespaces.
> > 
> > Signed-off-by: Sukadev Bhattiprolu <sukadev@us.ibm.com>
> > ---
> >  include/linux/sched.h |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > Index: lx26-21-rc3-mm2/include/linux/sched.h
> > ===================================================================
> > --- lx26-21-rc3-mm2.orig/include/linux/sched.h	2007-03-20 20:13:19.000000000 -0700
> > +++ lx26-21-rc3-mm2/include/linux/sched.h	2007-03-21 11:10:33.000000000 -0700
> > @@ -26,6 +26,7 @@
> >  #define CLONE_STOPPED		0x02000000	/* Start in stopped state */
> >  #define CLONE_NEWUTS		0x04000000	/* New utsname group? */
> >  #define CLONE_NEWIPC		0x08000000	/* New ipcs */
> > +#define CLONE_NEWPID		0x10000000      /* New pid namespace */
> 
> Do we actually have any need to reserve it at this time?  
> I'd have thought that we could defer adding this until we 
> have some code in-kernel which uses it.

FWIW, I'm fine with the reservation, we won't get around
it for the pid space, so we can as well register it now
YMMV

best,
Herbert

> _______________________________________________
> Containers mailing list
> Containers@lists.linux-foundation.org
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
Previous Topic: [RFC][PATCH 13/14] Define CLONE_NEWPID flag
Next Topic: [PATCH] Protect tty drivers list a little
Goto Forum:
  


Current Time: Sun Oct 13 04:33:42 GMT 2024

Total time taken to generate the page: 0.06362 seconds