OpenVZ Forum - RDF feed
type: channel
items: Re: [PATCH -mm] uts namespace : remove CONFIG_UTS_NS

Re: [PATCH -mm] uts namespace : remove CONFIG_UTS_NS
type: item
description: Quoting Cedric Le Goater (clg@fr.ibm.com): > CONFIG_UTS_NS has very little value as it only deactivates the unshare > of the uts namespace and does not improve performance. > > Signed-off-by: Cedric Le Goater <clg@fr.ibm.com> It really is worthless complication, with the sole effect being to complicate testing... Acked-by: Serge Hallyn <serue@us.ibm.com> > --- > include/linux/utsname.h | 19 ------------------- > init/Kconfig | 8 -------- > kernel/Makefile | 3 +-- > kernel/sysctl.c | 3 +-- > 4 files changed, 2 insertions(+), 31 deletions(-) > > Index: 2.6.20-rc4-mm1/include/linux/utsname.h > =================================================================== > --- 2.6.20-rc4-mm1.orig/include/linux/utsname.h > +++ 2.6.20-rc4-mm1/include/linux/utsname.h > @@ -48,7 +48,6 @@ static inline void get_uts_ns(struct uts > kref_get(&ns->kref); > } > > -#ifdef CONFIG_UTS_NS > extern int unshare_utsname(unsigned long unshare_flags, > struct uts_namespace **new_uts); > extern int copy_utsname(int flags, struct task_struct *tsk); > @@ -58,24 +57,6 @@ static inline void put_uts_ns(struct uts > { > kref_put(&ns->kref, free_uts_ns); > } > -#else > -static inline int unshare_utsname(unsigned long unshare_flags, > - struct uts_namespace **new_uts) > -{ > - if (unshare_flags & CLONE_NEWUTS) > - return -EINVAL; > - > - return 0; > -} > - > -static inline int copy_utsname(int flags, struct task_struct *tsk) > -{ > - return 0; > -} > -static inline void put_uts_ns(struct uts_namespace *ns) > -{ > -} > -#endif > > static inline struct new_utsname *utsname(void) > { > Index: 2.6.20-rc4-mm1/init/Kconfig > =================================================================== > --- 2.6.20-rc4-mm1.orig/init/Kconfig > +++ 2.6.20-rc4-mm1/init/Kconfig > @@ -205,14 +205,6 @@ config TASK_DELAY_ACCT > > Say N if unsure. > > -config UTS_NS > - bool "UTS Namespaces" > - default n > - help > - Support uts namespaces. This allows containers, i.e. > - vservers, to use uts namespaces to provide different > - …
creator: serue
date: 2007-01-16T15:27:34-00:00

Re: [PATCH -mm] uts namespace : remove CONFIG_UTS_NS
type: item
description: On Tue, Jan 16, 2007 at 09:27:34AM -0600, Serge E. Hallyn wrote: > Quoting Cedric Le Goater (clg@fr.ibm.com): > > CONFIG_UTS_NS has very little value as it only deactivates the unshare > > of the uts namespace and does not improve performance. > > > > Signed-off-by: Cedric Le Goater <clg@fr.ibm.com> > > It really is worthless complication, with the sole effect being to > complicate testing... we already force that on by default (in Linux-VServer) and I don't think the code can be considered critical enough for special casing that ... so I'm perfectly fine with removing that for UTS best, Herbert > Acked-by: Serge Hallyn <serue@us.ibm.com> > > > --- > > include/linux/utsname.h | 19 ------------------- > > init/Kconfig | 8 -------- > > kernel/Makefile | 3 +-- > > kernel/sysctl.c | 3 +-- > > 4 files changed, 2 insertions(+), 31 deletions(-) > > > > Index: 2.6.20-rc4-mm1/include/linux/utsname.h > > =================================================================== > > --- 2.6.20-rc4-mm1.orig/include/linux/utsname.h > > +++ 2.6.20-rc4-mm1/include/linux/utsname.h > > @@ -48,7 +48,6 @@ static inline void get_uts_ns(struct uts > > kref_get(&ns->kref); > > } > > > > -#ifdef CONFIG_UTS_NS > > extern int unshare_utsname(unsigned long unshare_flags, > > struct uts_namespace **new_uts); > > extern int copy_utsname(int flags, struct task_struct *tsk); > > @@ -58,24 +57,6 @@ static inline void put_uts_ns(struct uts > > { > > kref_put(&ns->kref, free_uts_ns); > > } > > -#else > > -static inline int unshare_utsname(unsigned long unshare_flags, > > - struct uts_namespace **new_uts) > > -{ > > - if (unshare_flags & CLONE_NEWUTS) > > - return -EINVAL; > > - > > - return 0; > > -} > > - > > -static inline int copy_utsname(int flags, struct task_struct *tsk) > > -{ > > - return 0; > > -} > > -static inline void put_uts_ns(struct uts_namespace *ns) > > -{ > > -} > > -#endif > > > > static inline struct new_utsname *utsname(void) > > { > > Index: 2.6.20-rc4-mm1/init/Kconfig > > ==========…
creator: Herbert Poetzl
date: 2007-01-22T22:09:58-00:00