Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756566Ab1BYSxT (ORCPT ); Fri, 25 Feb 2011 13:53:19 -0500 Received: from mail-ew0-f46.google.com ([209.85.215.46]:41821 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756533Ab1BYSxP (ORCPT ); Fri, 25 Feb 2011 13:53:15 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=Zj9fLRTtzc7VA5lnjiG0nI4XXE29YvnRm+66r1L9XAiLfAdDGxZqjciAbZUdV0PgMh Yrhb0UOymDBh8VoAhfgIcbPYhzozh+rKY1bJHmjLf0Ip6/PWvPgI3UVHfXxmmhO7gUoO fAliUij1vLMgyixajyn4rN8jrY/Lg6IXnOWcs= From: Lucian Adrian Grijincu To: "David S. Miller" , Alexey Dobriyan , "Eric W. Biederman" , Octavian Purdila , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Lucian Adrian Grijincu Subject: [PATCH 4/9] sysctl: ipv4: ipfrag: share ip4_frags_ns_ctl_table between nets Date: Fri, 25 Feb 2011 20:52:36 +0200 Message-Id: <1298659961-23863-5-git-send-email-lucian.grijincu@gmail.com> X-Mailer: git-send-email 1.7.4.rc1.7.g2cf08.dirty In-Reply-To: <1298659961-23863-1-git-send-email-lucian.grijincu@gmail.com> References: <1298659961-23863-1-git-send-email-lucian.grijincu@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3136 Lines: 105 The only reason we were creating a copy of this table was to set ->data to point to data from within the newly created net. The netns_proc_* handlers do this dynamically. Signed-off-by: Lucian Adrian Grijincu --- net/ipv4/ip_fragment.c | 34 ++++++---------------------------- net/sysctl_net.c | 2 +- 2 files changed, 7 insertions(+), 29 deletions(-) diff --git a/net/ipv4/ip_fragment.c b/net/ipv4/ip_fragment.c index a1151b8..ffca3cc 100644 --- a/net/ipv4/ip_fragment.c +++ b/net/ipv4/ip_fragment.c @@ -677,21 +677,21 @@ static struct ctl_table ip4_frags_ns_ctl_table[] = { .data = &init_net.ipv4.frags.high_thresh, .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec + .proc_handler = (proc_handler *) netns_proc_dointvec }, { .procname = "ipfrag_low_thresh", .data = &init_net.ipv4.frags.low_thresh, .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec + .proc_handler = (proc_handler *) netns_proc_dointvec }, { .procname = "ipfrag_time", .data = &init_net.ipv4.frags.timeout, .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec_jiffies, + .proc_handler = (proc_handler *) netns_proc_dointvec_jiffies, }, { } }; @@ -717,41 +717,19 @@ static struct ctl_table ip4_frags_ctl_table[] = { static int __net_init ip4_frags_ns_ctl_register(struct net *net) { - struct ctl_table *table; struct ctl_table_header *hdr; - - table = ip4_frags_ns_ctl_table; - if (!net_eq(net, &init_net)) { - table = kmemdup(table, sizeof(ip4_frags_ns_ctl_table), GFP_KERNEL); - if (table == NULL) - goto err_alloc; - - table[0].data = &net->ipv4.frags.high_thresh; - table[1].data = &net->ipv4.frags.low_thresh; - table[2].data = &net->ipv4.frags.timeout; - } - - hdr = register_net_sysctl_table(net, net_ipv4_ctl_path, table); + hdr = register_net_sysctl_table(net, net_ipv4_ctl_path, + ip4_frags_ns_ctl_table); if (hdr == NULL) - goto err_reg; + return -ENOMEM; net->ipv4.frags_hdr = hdr; return 0; - -err_reg: - if (!net_eq(net, &init_net)) - kfree(table); -err_alloc: - return -ENOMEM; } static void __net_exit ip4_frags_ns_ctl_unregister(struct net *net) { - struct ctl_table *table; - - table = net->ipv4.frags_hdr->ctl_table_arg; unregister_net_sysctl_table(net->ipv4.frags_hdr); - kfree(table); } static void ip4_frags_ctl_register(void) diff --git a/net/sysctl_net.c b/net/sysctl_net.c index 60b36ad..d80e9c4 100644 --- a/net/sysctl_net.c +++ b/net/sysctl_net.c @@ -110,7 +110,7 @@ struct ctl_table_header *register_net_sysctl_table(struct net *net, namespaces = *current->nsproxy; namespaces.net_ns = net; return __register_sysctl_paths(&net_sysctl_root, &namespaces, path, - table, NULL); + table, net); } EXPORT_SYMBOL_GPL(register_net_sysctl_table); -- 1.7.4.rc1.7.g2cf08.dirty -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/