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: Tue, 11 Dec 2007 10:33:45 +0300 From: "Denis V. Lunev" <den@...ru> To: "Eric W. Biederman" <ebiederm@...ssion.com> CC: "Denis V. Lunev" <den@...nvz.org>, davem@...emloft.net, containers@...ts.osdl.org, netdev@...r.kernel.org, herbert@...dor.apana.org.au Subject: Re: [PATCH 2.6.25] netns: struct net content re-work Eric W. Biederman wrote: > The idea of separate structures make sense, and seems needed and useful. > > "Denis V. Lunev" <den@...nvz.org> writes: > >> diff --git a/include/net/netns/unix.h b/include/net/netns/unix.h >> new file mode 100644 >> index 0000000..27b4e7f >> --- /dev/null >> +++ b/include/net/netns/unix.h > ^^^^^^ > Given that we are making this per protocol adding a separate directory > to hold them seems to be the wrong grouping. Ideally we want everything > for the protocol all together in the same location so it is easy > to find. Possibly with a user/kernel split. > > So perhaps unix_net.h The idea was simple: - I can name 5 files right now - I want them to be shown to gather by ls - so, there are 2 ways, namely: # include/net/netns/unix.h # include/net/netns-unix.h Regards, Den > >> @@ -0,0 +1,13 @@ >> +/* >> + * Unix network namespace >> + */ >> +#ifndef __NETNS_UNIX_H__ >> +#define __NETNS_UNIX_H__ >> + >> +struct ctl_table_header; >> +struct netns_unix { >> + int sysctl_unix_max_dgram_qlen; >> + struct ctl_table_header *unix_ctl; >> +}; > > How about struct unix_net? I think that tracks a little better > with how we have done struct in_device, ip6_dev and their friends. > > Eric > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists