lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 5 May 2010 17:01:50 -0500 From: "Serge E. Hallyn" <serue@...ibm.com> To: "Eric W. Biederman" <ebiederm@...ssion.com> Cc: Greg Kroah-Hartman <gregkh@...e.de>, Kay Sievers <kay.sievers@...y.org>, linux-kernel@...r.kernel.org, Tejun Heo <tj@...nel.org>, Cornelia Huck <cornelia.huck@...ibm.com>, Eric Dumazet <eric.dumazet@...il.com>, Benjamin LaHaise <bcrl@...et.ca>, netdev@...r.kernel.org, David Miller <davem@...emloft.net> Subject: Re: [PATCH 2/6] netns: Teach network device kobjects which namespace they are in. Quoting Eric W. Biederman (ebiederm@...ssion.com): > "Serge E. Hallyn" <serue@...ibm.com> writes: > > > Quoting Eric W. Biederman (ebiederm@...ssion.com): > >> diff --git a/net/Kconfig b/net/Kconfig > >> index 041c35e..265e33b 100644 > >> --- a/net/Kconfig > >> +++ b/net/Kconfig > >> @@ -45,6 +45,14 @@ config COMPAT_NETLINK_MESSAGES > >> > >> menu "Networking options" > >> > >> +config NET_NS > >> + bool "Network namespace support" > >> + default n > >> + depends on EXPERIMENTAL && NAMESPACES > >> + help > >> + Allow user space to create what appear to be multiple instances > >> + of the network stack. > >> + > > > > Hi Eric, > > > > I'm confused - NET_NS is defined in init/Kconfig right now. Is the tree > > you're working from very different from mine, or is this the unfortunate > > rekult of the patches sitting so long? > > Old patches, nothing that complains when you make a mistake like this, > and apparently I have a blind spot in my personal code review. haha, we all know about that. > At one point it was not possible to enable the network namespace until > the sysfs stuff was enabled, but things have been going on long enough > that we worked around that restriction. Yeah, I remember that, and leaving this wouldn't break anything. > >> int netdev_kobject_init(void) > >> { > >> + kobj_ns_type_register(&net_ns_type_operations); > >> +#ifdef CONFIG_SYSFS > >> + register_pernet_subsys(&sysfs_net_ops); > >> +#endif > >> return class_register(&net_class); > > > > I think the kobj_ns_type_register() needs to be under > > ifdef CONFIG_SYSFS as well, bc net_ns_type_operations is defined > > under ifdef CONFIG_SYSFS. > > kobj_ns_type_register should not be under CONFIG_SYSFS. Which means > that kobj_ns_type_operations needs not to be under CONFIG_SYSFS as > well. That you for spotting that bug. np - outside of that, Acked-by: Serge E. Hallyn <serue@...ibm.com> I saw no problems with the other patches, just don't feel qualified to give an ack. thanks, -serge -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists