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, 14 Aug 2012 23:19:45 -0400 From: Vlad Yasevich <vyasevich@...il.com> To: "Eric W. Biederman" <ebiederm@...ssion.com> CC: David Miller <davem@...emloft.net>, linux-sctp@...r.kernel.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, Jan Ariyasu <jan.ariyasu@...com>, Jan Ariyasu <jan.ariyasu@...il.com>, Neil Horman <nhorman@...driver.com>, Thomas Graf <tgraf@...radead.org>, Xi Wang <xi.wang@...il.com> Subject: Re: [PATCH net-next 6/9] sctp: Move the percpu sockets counter out of sctp_proc_init On 08/06/2012 02:44 PM, Eric W. Biederman wrote: > > The percpu sctp socket counter has nothing at all to do with the sctp > proc files, and having it in the wrong initialization is confusing, > and makes network namespace support a pain. > > Signed-off-by: "Eric W. Biederman" <ebiederm@...ssion.com> Acked-by: Vlad Yasevich <vyasevich@...il.com> > --- > net/sctp/protocol.c | 13 +++++++------ > 1 files changed, 7 insertions(+), 6 deletions(-) > > diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c > index 6193d20..976d765 100644 > --- a/net/sctp/protocol.c > +++ b/net/sctp/protocol.c > @@ -93,8 +93,6 @@ int sysctl_sctp_wmem[3]; > /* Set up the proc fs entry for the SCTP protocol. */ > static __init int sctp_proc_init(void) > { > - if (percpu_counter_init(&sctp_sockets_allocated, 0)) > - goto out_nomem; > #ifdef CONFIG_PROC_FS > if (!proc_net_sctp) { > proc_net_sctp = proc_mkdir("sctp", init_net.proc_net); > @@ -125,12 +123,9 @@ out_snmp_proc_init: > remove_proc_entry("sctp", init_net.proc_net); > } > out_free_percpu: > - percpu_counter_destroy(&sctp_sockets_allocated); > #else > return 0; > #endif /* CONFIG_PROC_FS */ > - > -out_nomem: > return -ENOMEM; > } > > @@ -151,7 +146,6 @@ static void sctp_proc_exit(void) > remove_proc_entry("sctp", init_net.proc_net); > } > #endif > - percpu_counter_destroy(&sctp_sockets_allocated); > } > > /* Private helper to extract ipv4 address and stash them in > @@ -1261,6 +1255,10 @@ SCTP_STATIC __init int sctp_init(void) > if (status) > goto err_init_mibs; > > + status = percpu_counter_init(&sctp_sockets_allocated, 0); > + if (status) > + goto err_percpu_counter_init; > + > /* Initialize proc fs directory. */ > status = sctp_proc_init(); > if (status) > @@ -1481,6 +1479,8 @@ err_ahash_alloc: > sctp_dbg_objcnt_exit(); > sctp_proc_exit(); > err_init_proc: > + percpu_counter_destroy(&sctp_sockets_allocated); > +err_percpu_counter_init: > cleanup_sctp_mibs(); > err_init_mibs: > kmem_cache_destroy(sctp_chunk_cachep); > @@ -1521,6 +1521,7 @@ SCTP_STATIC __exit void sctp_exit(void) > sizeof(struct sctp_bind_hashbucket))); > > sctp_dbg_objcnt_exit(); > + percpu_counter_destroy(&sctp_sockets_allocated); > sctp_proc_exit(); > cleanup_sctp_mibs(); > > -- 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