Re: [PATCH 1/5] The config option itself
On Wed, 31 Oct 2007 14:05:01 +0300 Pavel Emelyanov <[EMAIL PROTECTED]> wrote: > The option is called NAMESPACES. It can be selectable only > if EMBEDDED is chosen (this was Eric's requisition). When > the EMBEDDED is off namespaces will be on automatically. I'm thinking that Eric's pidns-place-under-config_experimental.patch is for 2.6.24, so I attempted to rework your patches atop that but gave up on the last one. So if we're agreeeable, can you please redo these against next -mm, thanks. Also, please do put a little thought into the choice of patch subjects. "the config option itself" will look rather dopey if it goes into the permanent kernel record under that name. Probably putting "CONFIG_NAMESPACES: " in front of each of these would suit. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Randy Dunlap wrote: > On Wed, 31 Oct 2007 14:05:01 +0300 Pavel Emelyanov wrote: > >> The option is called NAMESPACES. It can be selectable only >> if EMBEDDED is chosen (this was Eric's requisition). When >> the EMBEDDED is off namespaces will be on automatically. >> Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > > > and when EMBEDDED is on, it looks like NAMESPACES will be off > automatically (but user-changeable)? Is that intended? Yes. The namespaces are likely not needed on cell-phones or similar, but if the user does want them - he may turn them on. This was one of Eric's wishes to give only the embedded people the choice. > >> --- >> >> diff --git a/init/Kconfig b/init/Kconfig >> index c3de3ed..fc76773 100644 >> --- a/init/Kconfig >> +++ b/init/Kconfig >> @@ -400,6 +400,15 @@ config RELAY >> >>If unsure, say N. >> >> +config NAMESPACES >> +bool "Namespaces support" if EMBEDDED >> +default !EMBEDDED >> +help >> + Provides the way to make tasks work with different objects using >> + the same id. For example same IPC id may refer to different objects >> + or same user id or pid may refer to different tasks when used in >> + different namespaces. >> + >> config BLK_DEV_INITRD >> bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" >> depends on BROKEN || !FRV > > > --- > ~Randy > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
On Wed, 31 Oct 2007 14:05:01 +0300 Pavel Emelyanov wrote: > The option is called NAMESPACES. It can be selectable only > if EMBEDDED is chosen (this was Eric's requisition). When > the EMBEDDED is off namespaces will be on automatically. > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> and when EMBEDDED is on, it looks like NAMESPACES will be off automatically (but user-changeable)? Is that intended? > --- > > diff --git a/init/Kconfig b/init/Kconfig > index c3de3ed..fc76773 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -400,6 +400,15 @@ config RELAY > > If unsure, say N. > > +config NAMESPACES > + bool "Namespaces support" if EMBEDDED > + default !EMBEDDED > + help > + Provides the way to make tasks work with different objects using > + the same id. For example same IPC id may refer to different objects > + or same user id or pid may refer to different tasks when used in > + different namespaces. > + > config BLK_DEV_INITRD > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > depends on BROKEN || !FRV --- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
On Mon, 01 Oct 2007 19:33:27 +0400 Pavel Emelyanov wrote: > diff --git a/init/Kconfig b/init/Kconfig > index 6bb603a..67b1e41 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -390,6 +381,23 @@ config RELAY > > If unsure, say N. > > +config NAMESPACES > + bool "Namespaces support" if EMBEDDED > + default !EMBEDDED > + help > + Provides the way to make tasks work with different objects using > + the same id. For example same IPC id may refer to different objects > + or same user id or pid may refer to different tasks when used in > + different namespaces. > + > +config NAMESPACES_EXPERIMENTAL > + bool "Add experimantal namespaces support" experimental > + depends on NAMESPACES && EXPERIMENTAL > + default n > + help > + Also include the support for the namespaces that are not fnished finished > + or well developed yet > + > config BLK_DEV_INITRD > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > depends on BROKEN || !FRV --- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Quoting Pavel Emelyanov ([EMAIL PROTECTED]): > The option is called NAMESPACES. It can be selectable only > if EMBEDDED is chosen (this was Eric's requisition). When > the EMBEDDED is off namespaces will be on automatically. > > One more option (NAMESPACES_EXPERIMENTAL) was added by > Serge's request to move there all the namespaces that are > not finished yet. Currently only the user and the network > namespaces are such. This option is selectable all the > time NAMESPACES are "y" regardless of EMBEDDED one. > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > Acked-by: Cedric Le Goater <[EMAIL PROTECTED]> Looks good. Thanks. Acked-by: Serge Hallyn <[EMAIL PROTECTED]> (for the whole set, just to save the list from a few emails) -serge > > --- > > diff --git a/init/Kconfig b/init/Kconfig > index 6bb603a..67b1e41 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -390,6 +381,23 @@ config RELAY > > If unsure, say N. > > +config NAMESPACES > + bool "Namespaces support" if EMBEDDED > + default !EMBEDDED > + help > + Provides the way to make tasks work with different objects using > + the same id. For example same IPC id may refer to different objects > + or same user id or pid may refer to different tasks when used in > + different namespaces. > + > +config NAMESPACES_EXPERIMENTAL > + bool "Add experimantal namespaces support" > + depends on NAMESPACES && EXPERIMENTAL > + default n > + help > + Also include the support for the namespaces that are not fnished > + or well developed yet > + > config BLK_DEV_INITRD > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > depends on BROKEN || !FRV - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Serge E. Hallyn wrote: > Quoting Pavel Emelyanov ([EMAIL PROTECTED]): >> [EMAIL PROTECTED] wrote: >>> Serge E. Hallyn [EMAIL PROTECTED] wrote: >>> | Quoting Serge E. Hallyn ([EMAIL PROTECTED]): >>> | > Quoting Pavel Emelyanov ([EMAIL PROTECTED]): >>> | > > The option is called NAMESPACES. It can be selectable only >>> | > > if EMBEDDED is chosen (this was Eric's requisition). When >>> | > > the EMBEDDED is off namespaces will be on automatically. >>> | > > >>> | > > One more option (NAMESPACES_EXPERIMENTAL) was added by >>> | > > Serge's request to move there all the namespaces that are >>> | > > not finished yet. Currently only the user and the network >>> | > > namespaces are such. >>> | > > >>> | > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> >>> | > > >>> | > > --- >>> | > > >>> | > > diff --git a/init/Kconfig b/init/Kconfig >>> | > > index 684ccfb..05a71d7 100644 >>> | > > --- a/init/Kconfig >>> | > > +++ b/init/Kconfig >>> | > > @@ -369,6 +360,23 @@ config RELAY >>> | > > >>> | > > If unsure, say N. >>> | > > >>> | > > +config NAMESPACES >>> | > > + bool "The namespaces support" if EMBEDDED >>> | > > + default !EMBEDDED >>> | > > + help >>> | > > + Provides the way to make tasks work with different objects >>> using >>> | > > + the same id. For example same IPC id may refer to different >>> objects >>> | > > + or same user id or pid may refer to different tasks when used >>> in >>> | > > + different namespaces. >>> | > > + >>> | > > +config NAMESPACES_EXPERIMENTAL >>> | > > + bool "Add the experimantal namespaces support" if EMBEDDED >>> | > >>> | > Hi Pavel, >>> | > >>> | > Overall this patchset looks good. >>> | > >>> | > However the NAMESPACES_EXPERIMENTAL option should not have the >>> | > 'if EMBEDDED', right? This is about EXPERIMENTAL, not about >>> | > EMBEDDED, unlike the plain NAMESPACES option. >>> | > >>> | > -serge >>> | >>> | Actually that doesn't seem to work either. Even though >>> | SECURITY_NAMESPACES=y, the config system seems to infer that >>> | since NAMESPACES is not user-selectable, NAMESPACES_EXPERIMENTAL >>> | shouldn't be either. So we end up with NAMESPACES_EXPERIMENTAL >>> | being on and not un-selectable if !EMBEDDED. >>> >>> Yes. Given that NAMESPACES depends on EMBEDDED, NAMESPACES_EXPERIMENTAL >>> can simply depend on NAMESPACES and EXPERIMENTAL with a default of N. >> Nope. The intention is that the NAMESPACES and NAMESPACES_EXPERIMENTAL > > No, Eric wanted NAMESPACES to only be unselecteable if EMBEDDED=y, but > NAMESPACES_EXPERIMENTAL should always be unselectable. If current > config logic can't do that then we should forget about the EMBEDDED > requirement altogether. OK, I see and agree :) I will remake the patches shortly. > The rest of this patchset looks great. Thanks, Pavel. You're welcome. > -serge > >> must not be selectable if EMBEDDED=n, but if we make NS_EXPERIMENTAL >> depend on NAMESPACES only we'll have to make a choice in !EMBEDDED. >> >>> BTW, does the position of 'config NAMESPACES' in init/Kconfig file >>> matter ? If it is dependent on EMBEDDED, should it not come later >>> in the file, after 'config EMBEDDED' ? >>> >>> | >>> | > > + depends on NAMESPACES && EXPERIMENTAL >>> | > > + default !EMBEDDED >>> | > > + help >>> | > > + Also include the support for the namespaces that are not >>> fnished >>> | > > + or well developed yet >>> | > > + >>> | > > config BLK_DEV_INITRD >>> | > > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) >>> support" >>> | > > depends on BROKEN || !FRV >>> | ___ >>> | Containers mailing list >>> | [EMAIL PROTECTED] >>> | https://lists.linux-foundation.org/mailman/listinfo/containers >>> > - > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Quoting Pavel Emelyanov ([EMAIL PROTECTED]): > [EMAIL PROTECTED] wrote: > > Serge E. Hallyn [EMAIL PROTECTED] wrote: > > | Quoting Serge E. Hallyn ([EMAIL PROTECTED]): > > | > Quoting Pavel Emelyanov ([EMAIL PROTECTED]): > > | > > The option is called NAMESPACES. It can be selectable only > > | > > if EMBEDDED is chosen (this was Eric's requisition). When > > | > > the EMBEDDED is off namespaces will be on automatically. > > | > > > > | > > One more option (NAMESPACES_EXPERIMENTAL) was added by > > | > > Serge's request to move there all the namespaces that are > > | > > not finished yet. Currently only the user and the network > > | > > namespaces are such. > > | > > > > | > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > > | > > > > | > > --- > > | > > > > | > > diff --git a/init/Kconfig b/init/Kconfig > > | > > index 684ccfb..05a71d7 100644 > > | > > --- a/init/Kconfig > > | > > +++ b/init/Kconfig > > | > > @@ -369,6 +360,23 @@ config RELAY > > | > > > > | > > If unsure, say N. > > | > > > > | > > +config NAMESPACES > > | > > + bool "The namespaces support" if EMBEDDED > > | > > + default !EMBEDDED > > | > > + help > > | > > + Provides the way to make tasks work with different objects > > using > > | > > + the same id. For example same IPC id may refer to different > > objects > > | > > + or same user id or pid may refer to different tasks when used > > in > > | > > + different namespaces. > > | > > + > > | > > +config NAMESPACES_EXPERIMENTAL > > | > > + bool "Add the experimantal namespaces support" if EMBEDDED > > | > > > | > Hi Pavel, > > | > > > | > Overall this patchset looks good. > > | > > > | > However the NAMESPACES_EXPERIMENTAL option should not have the > > | > 'if EMBEDDED', right? This is about EXPERIMENTAL, not about > > | > EMBEDDED, unlike the plain NAMESPACES option. > > | > > > | > -serge > > | > > | Actually that doesn't seem to work either. Even though > > | SECURITY_NAMESPACES=y, the config system seems to infer that > > | since NAMESPACES is not user-selectable, NAMESPACES_EXPERIMENTAL > > | shouldn't be either. So we end up with NAMESPACES_EXPERIMENTAL > > | being on and not un-selectable if !EMBEDDED. > > > > Yes. Given that NAMESPACES depends on EMBEDDED, NAMESPACES_EXPERIMENTAL > > can simply depend on NAMESPACES and EXPERIMENTAL with a default of N. > > Nope. The intention is that the NAMESPACES and NAMESPACES_EXPERIMENTAL No, Eric wanted NAMESPACES to only be unselecteable if EMBEDDED=y, but NAMESPACES_EXPERIMENTAL should always be unselectable. If current config logic can't do that then we should forget about the EMBEDDED requirement altogether. The rest of this patchset looks great. Thanks, Pavel. -serge > must not be selectable if EMBEDDED=n, but if we make NS_EXPERIMENTAL > depend on NAMESPACES only we'll have to make a choice in !EMBEDDED. > > > BTW, does the position of 'config NAMESPACES' in init/Kconfig file > > matter ? If it is dependent on EMBEDDED, should it not come later > > in the file, after 'config EMBEDDED' ? > > > > | > > | > > + depends on NAMESPACES && EXPERIMENTAL > > | > > + default !EMBEDDED > > | > > + help > > | > > + Also include the support for the namespaces that are not > > fnished > > | > > + or well developed yet > > | > > + > > | > > config BLK_DEV_INITRD > > | > > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) > > support" > > | > > depends on BROKEN || !FRV > > | ___ > > | Containers mailing list > > | [EMAIL PROTECTED] > > | https://lists.linux-foundation.org/mailman/listinfo/containers > > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Pavel Emelyanov wrote: > The option is called NAMESPACES. It can be selectable only > if EMBEDDED is chosen (this was Eric's requisition). When > the EMBEDDED is off namespaces will be on automatically. > > One more option (NAMESPACES_EXPERIMENTAL) was added by > Serge's request to move there all the namespaces that are > not finished yet. Currently only the user and the network > namespaces are such. > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> Acked-by: Cedric Le Goater <[EMAIL PROTECTED]> Thanks ! C. > > --- > > diff --git a/init/Kconfig b/init/Kconfig > index 684ccfb..05a71d7 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -369,6 +360,23 @@ config RELAY > > If unsure, say N. > > +config NAMESPACES > + bool "The namespaces support" if EMBEDDED > + default !EMBEDDED > + help > + Provides the way to make tasks work with different objects using > + the same id. For example same IPC id may refer to different objects > + or same user id or pid may refer to different tasks when used in > + different namespaces. > + > +config NAMESPACES_EXPERIMENTAL > + bool "Add the experimantal namespaces support" if EMBEDDED > + depends on NAMESPACES && EXPERIMENTAL > + default !EMBEDDED > + help > + Also include the support for the namespaces that are not fnished > + or well developed yet > + > config BLK_DEV_INITRD > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > depends on BROKEN || !FRV > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
[EMAIL PROTECTED] wrote: > Serge E. Hallyn [EMAIL PROTECTED] wrote: > | Quoting Serge E. Hallyn ([EMAIL PROTECTED]): > | > Quoting Pavel Emelyanov ([EMAIL PROTECTED]): > | > > The option is called NAMESPACES. It can be selectable only > | > > if EMBEDDED is chosen (this was Eric's requisition). When > | > > the EMBEDDED is off namespaces will be on automatically. > | > > > | > > One more option (NAMESPACES_EXPERIMENTAL) was added by > | > > Serge's request to move there all the namespaces that are > | > > not finished yet. Currently only the user and the network > | > > namespaces are such. > | > > > | > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > | > > > | > > --- > | > > > | > > diff --git a/init/Kconfig b/init/Kconfig > | > > index 684ccfb..05a71d7 100644 > | > > --- a/init/Kconfig > | > > +++ b/init/Kconfig > | > > @@ -369,6 +360,23 @@ config RELAY > | > > > | > > If unsure, say N. > | > > > | > > +config NAMESPACES > | > > + bool "The namespaces support" if EMBEDDED > | > > + default !EMBEDDED > | > > + help > | > > + Provides the way to make tasks work with different objects > using > | > > + the same id. For example same IPC id may refer to different > objects > | > > + or same user id or pid may refer to different tasks when used > in > | > > + different namespaces. > | > > + > | > > +config NAMESPACES_EXPERIMENTAL > | > > + bool "Add the experimantal namespaces support" if EMBEDDED > | > > | > Hi Pavel, > | > > | > Overall this patchset looks good. > | > > | > However the NAMESPACES_EXPERIMENTAL option should not have the > | > 'if EMBEDDED', right? This is about EXPERIMENTAL, not about > | > EMBEDDED, unlike the plain NAMESPACES option. > | > > | > -serge > | > | Actually that doesn't seem to work either. Even though > | SECURITY_NAMESPACES=y, the config system seems to infer that > | since NAMESPACES is not user-selectable, NAMESPACES_EXPERIMENTAL > | shouldn't be either. So we end up with NAMESPACES_EXPERIMENTAL > | being on and not un-selectable if !EMBEDDED. > > Yes. Given that NAMESPACES depends on EMBEDDED, NAMESPACES_EXPERIMENTAL > can simply depend on NAMESPACES and EXPERIMENTAL with a default of N. Nope. The intention is that the NAMESPACES and NAMESPACES_EXPERIMENTAL must not be selectable if EMBEDDED=n, but if we make NS_EXPERIMENTAL depend on NAMESPACES only we'll have to make a choice in !EMBEDDED. > BTW, does the position of 'config NAMESPACES' in init/Kconfig file > matter ? If it is dependent on EMBEDDED, should it not come later > in the file, after 'config EMBEDDED' ? > > | > | > > + depends on NAMESPACES && EXPERIMENTAL > | > > + default !EMBEDDED > | > > + help > | > > + Also include the support for the namespaces that are not > fnished > | > > + or well developed yet > | > > + > | > > config BLK_DEV_INITRD > | > > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) > support" > | > > depends on BROKEN || !FRV > | ___ > | Containers mailing list > | [EMAIL PROTECTED] > | https://lists.linux-foundation.org/mailman/listinfo/containers > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Serge E. Hallyn [EMAIL PROTECTED] wrote: | Quoting Serge E. Hallyn ([EMAIL PROTECTED]): | > Quoting Pavel Emelyanov ([EMAIL PROTECTED]): | > > The option is called NAMESPACES. It can be selectable only | > > if EMBEDDED is chosen (this was Eric's requisition). When | > > the EMBEDDED is off namespaces will be on automatically. | > > | > > One more option (NAMESPACES_EXPERIMENTAL) was added by | > > Serge's request to move there all the namespaces that are | > > not finished yet. Currently only the user and the network | > > namespaces are such. | > > | > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> | > > | > > --- | > > | > > diff --git a/init/Kconfig b/init/Kconfig | > > index 684ccfb..05a71d7 100644 | > > --- a/init/Kconfig | > > +++ b/init/Kconfig | > > @@ -369,6 +360,23 @@ config RELAY | > > | > > If unsure, say N. | > > | > > +config NAMESPACES | > > + bool "The namespaces support" if EMBEDDED | > > + default !EMBEDDED | > > + help | > > + Provides the way to make tasks work with different objects using | > > + the same id. For example same IPC id may refer to different objects | > > + or same user id or pid may refer to different tasks when used in | > > + different namespaces. | > > + | > > +config NAMESPACES_EXPERIMENTAL | > > + bool "Add the experimantal namespaces support" if EMBEDDED | > | > Hi Pavel, | > | > Overall this patchset looks good. | > | > However the NAMESPACES_EXPERIMENTAL option should not have the | > 'if EMBEDDED', right? This is about EXPERIMENTAL, not about | > EMBEDDED, unlike the plain NAMESPACES option. | > | > -serge | | Actually that doesn't seem to work either. Even though | SECURITY_NAMESPACES=y, the config system seems to infer that | since NAMESPACES is not user-selectable, NAMESPACES_EXPERIMENTAL | shouldn't be either. So we end up with NAMESPACES_EXPERIMENTAL | being on and not un-selectable if !EMBEDDED. Yes. Given that NAMESPACES depends on EMBEDDED, NAMESPACES_EXPERIMENTAL can simply depend on NAMESPACES and EXPERIMENTAL with a default of N. BTW, does the position of 'config NAMESPACES' in init/Kconfig file matter ? If it is dependent on EMBEDDED, should it not come later in the file, after 'config EMBEDDED' ? | | > > + depends on NAMESPACES && EXPERIMENTAL | > > + default !EMBEDDED | > > + help | > > + Also include the support for the namespaces that are not fnished | > > + or well developed yet | > > + | > > config BLK_DEV_INITRD | > > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" | > > depends on BROKEN || !FRV | ___ | Containers mailing list | [EMAIL PROTECTED] | https://lists.linux-foundation.org/mailman/listinfo/containers - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Quoting Serge E. Hallyn ([EMAIL PROTECTED]): > Quoting Pavel Emelyanov ([EMAIL PROTECTED]): > > The option is called NAMESPACES. It can be selectable only > > if EMBEDDED is chosen (this was Eric's requisition). When > > the EMBEDDED is off namespaces will be on automatically. > > > > One more option (NAMESPACES_EXPERIMENTAL) was added by > > Serge's request to move there all the namespaces that are > > not finished yet. Currently only the user and the network > > namespaces are such. > > > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > > > > --- > > > > diff --git a/init/Kconfig b/init/Kconfig > > index 684ccfb..05a71d7 100644 > > --- a/init/Kconfig > > +++ b/init/Kconfig > > @@ -369,6 +360,23 @@ config RELAY > > > > If unsure, say N. > > > > +config NAMESPACES > > + bool "The namespaces support" if EMBEDDED > > + default !EMBEDDED > > + help > > + Provides the way to make tasks work with different objects using > > + the same id. For example same IPC id may refer to different objects > > + or same user id or pid may refer to different tasks when used in > > + different namespaces. > > + > > +config NAMESPACES_EXPERIMENTAL > > + bool "Add the experimantal namespaces support" if EMBEDDED > > Hi Pavel, > > Overall this patchset looks good. > > However the NAMESPACES_EXPERIMENTAL option should not have the > 'if EMBEDDED', right? This is about EXPERIMENTAL, not about > EMBEDDED, unlike the plain NAMESPACES option. > > -serge Actually that doesn't seem to work either. Even though SECURITY_NAMESPACES=y, the config system seems to infer that since NAMESPACES is not user-selectable, NAMESPACES_EXPERIMENTAL shouldn't be either. So we end up with NAMESPACES_EXPERIMENTAL being on and not un-selectable if !EMBEDDED. > > + depends on NAMESPACES && EXPERIMENTAL > > + default !EMBEDDED > > + help > > + Also include the support for the namespaces that are not fnished > > + or well developed yet > > + > > config BLK_DEV_INITRD > > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > > depends on BROKEN || !FRV - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
Quoting Pavel Emelyanov ([EMAIL PROTECTED]): > The option is called NAMESPACES. It can be selectable only > if EMBEDDED is chosen (this was Eric's requisition). When > the EMBEDDED is off namespaces will be on automatically. > > One more option (NAMESPACES_EXPERIMENTAL) was added by > Serge's request to move there all the namespaces that are > not finished yet. Currently only the user and the network > namespaces are such. > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > > --- > > diff --git a/init/Kconfig b/init/Kconfig > index 684ccfb..05a71d7 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -369,6 +360,23 @@ config RELAY > > If unsure, say N. > > +config NAMESPACES > + bool "The namespaces support" if EMBEDDED > + default !EMBEDDED > + help > + Provides the way to make tasks work with different objects using > + the same id. For example same IPC id may refer to different objects > + or same user id or pid may refer to different tasks when used in > + different namespaces. > + > +config NAMESPACES_EXPERIMENTAL > + bool "Add the experimantal namespaces support" if EMBEDDED Hi Pavel, Overall this patchset looks good. However the NAMESPACES_EXPERIMENTAL option should not have the 'if EMBEDDED', right? This is about EXPERIMENTAL, not about EMBEDDED, unlike the plain NAMESPACES option. -serge > + depends on NAMESPACES && EXPERIMENTAL > + default !EMBEDDED > + help > + Also include the support for the namespaces that are not fnished > + or well developed yet > + > config BLK_DEV_INITRD > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > depends on BROKEN || !FRV - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] The config option itself
On Wed, 26 Sep 2007 19:43:28 +0400 Pavel Emelyanov wrote: > The option is called NAMESPACES. It can be selectable only > if EMBEDDED is chosen (this was Eric's requisition). When > the EMBEDDED is off namespaces will be on automatically. and when EMBEDDED is on, namespaces will be off automatically, until it is enabled? Is that what you want? (It's OK by me.) Same way for NAMESPACES_EXPERIMENTAL. > One more option (NAMESPACES_EXPERIMENTAL) was added by > Serge's request to move there all the namespaces that are > not finished yet. Currently only the user and the network > namespaces are such. > > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> > > --- > > diff --git a/init/Kconfig b/init/Kconfig > index 684ccfb..05a71d7 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -369,6 +360,23 @@ config RELAY > > If unsure, say N. > > +config NAMESPACES > + bool "The namespaces support" if EMBEDDED bool "Namespaces support" if EMBEDDED > + default !EMBEDDED > + help > + Provides the way to make tasks work with different objects using > + the same id. For example same IPC id may refer to different objects > + or same user id or pid may refer to different tasks when used in > + different namespaces. > + > +config NAMESPACES_EXPERIMENTAL > + bool "Add the experimantal namespaces support" if EMBEDDED experimental > + depends on NAMESPACES && EXPERIMENTAL > + default !EMBEDDED > + help > + Also include the support for the namespaces that are not fnished > + or well developed yet > + > config BLK_DEV_INITRD > bool "Initial RAM filesystem and RAM disk (initramfs/initrd) support" > depends on BROKEN || !FRV > - --- ~Randy Phaedrus says that Quality is about caring. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/