sysctl: Modify __register_sysctl_paths to take a set instead of a root and an nsproxy

An nsproxy argument here has always been awkard and now the nsproxy argument
is completely unnecessary so remove it, replacing it with the set we want
the registered tables to show up in.

Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>

diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index ec54a57c4690ef98063ad777dde1f9847a2d03ee..e0d3e7e59cbdec3fdcfb7f5d82e928643c401524 100644 (file)
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -1084,8 +1084,7 @@ out:
 
 /**
  * __register_sysctl_table - register a leaf sysctl table
- * @root: List of sysctl headers to register on
- * @namespaces: Data to compute which lists of sysctl entries are visible
+ * @set: Sysctl tree to register on
  * @path: The path to the directory the sysctl table is in.
  * @table: the top-level table structure
  *
@@ -1126,26 +1125,24 @@ out:
  * to the table header on success.
  */
 struct ctl_table_header *__register_sysctl_table(
-       struct ctl_table_root *root,
-       struct nsproxy *namespaces,
+       struct ctl_table_set *set,
        const char *path, struct ctl_table *table)
 {
+       struct ctl_table_root *root = set->dir.header.root;
        struct ctl_table_header *links = NULL;
        struct ctl_table_header *header;
        const char *name, *nextname;
-       struct ctl_table_set *set;
        struct ctl_dir *dir;
 
        header = kzalloc(sizeof(struct ctl_table_header), GFP_KERNEL);
        if (!header)
                return NULL;
 
-       init_header(header, root, NULL, table);
+       init_header(header, root, set, table);
        if (sysctl_check_table(path, table))
                goto fail;
 
        spin_lock(&sysctl_lock);
-       header->set = set = lookup_header_set(root, namespaces);
        dir = &set->dir;
        dir->header.nreg++;
        spin_unlock(&sysctl_lock);
@@ -1223,8 +1220,7 @@ static int count_subheaders(struct ctl_table *table)
 }
 
 static int register_leaf_sysctl_tables(const char *path, char *pos,
-       struct ctl_table_header ***subheader,
-       struct ctl_table_root *root, struct nsproxy *namespaces,
+       struct ctl_table_header ***subheader, struct ctl_table_set *set,
        struct ctl_table *table)
 {
        struct ctl_table *ctl_table_arg = NULL;
@@ -1261,7 +1257,7 @@ static int register_leaf_sysctl_tables(const char *path, char *pos,
        /* Register everything except a directory full of subdirectories */
        if (nr_files || !nr_dirs) {
                struct ctl_table_header *header;
-               header = __register_sysctl_table(root, namespaces, path, files);
+               header = __register_sysctl_table(set, path, files);
                if (!header) {
                        kfree(ctl_table_arg);
                        goto out;
@@ -1286,7 +1282,7 @@ static int register_leaf_sysctl_tables(const char *path, char *pos,
                        goto out;
 
                err = register_leaf_sysctl_tables(path, child_pos, subheader,
-                                                 root, namespaces, entry->child);
+                                                 set, entry->child);
                pos[0] = '\0';
                if (err)
                        goto out;
@@ -1299,8 +1295,7 @@ out:
 
 /**
  * __register_sysctl_paths - register a sysctl table hierarchy
- * @root: List of sysctl headers to register on
- * @namespaces: Data to compute which lists of sysctl entries are visible
+ * @set: Sysctl tree to register on
  * @path: The path to the directory the sysctl table is in.
  * @table: the top-level table structure
  *
@@ -1310,8 +1305,7 @@ out:
  * See __register_sysctl_table for more details.
  */
 struct ctl_table_header *__register_sysctl_paths(
-       struct ctl_table_root *root,
-       struct nsproxy *namespaces,
+       struct ctl_table_set *set,
        const struct ctl_path *path, struct ctl_table *table)
 {
        struct ctl_table *ctl_table_arg = table;
@@ -1337,7 +1331,7 @@ struct ctl_table_header *__register_sysctl_paths(
                table = table->child;
        }
        if (nr_subheaders == 1) {
-               header = __register_sysctl_table(root, namespaces, new_path, table);
+               header = __register_sysctl_table(set, new_path, table);
                if (header)
                        header->ctl_table_arg = ctl_table_arg;
        } else {
@@ -1351,7 +1345,7 @@ struct ctl_table_header *__register_sysctl_paths(
                header->ctl_table_arg = ctl_table_arg;
 
                if (register_leaf_sysctl_tables(new_path, pos, &subheader,
-                                               root, namespaces, table))
+                                               set, table))
                        goto err_register_leaves;
        }
 
@@ -1384,7 +1378,7 @@ err_register_leaves:
 struct ctl_table_header *register_sysctl_paths(const struct ctl_path *path,
                                                struct ctl_table *table)
 {
-       return __register_sysctl_paths(&sysctl_table_root, current->nsproxy,
+       return __register_sysctl_paths(&sysctl_table_root.default_set,
                                        path, table);
 }
 EXPORT_SYMBOL(register_sysctl_paths);

diff --git a/include/linux/sysctl.h b/include/linux/sysctl.h
index 2a1446a96094175b3358fea88073ce947a514350..cec59415b3ce37068bc9cb8cec2bb36abba08863 100644 (file)
--- a/include/linux/sysctl.h
+++ b/include/linux/sysctl.h
@@ -1079,10 +1079,10 @@ extern void retire_sysctl_set(struct ctl_table_set *set);
 
 void register_sysctl_root(struct ctl_table_root *root);
 struct ctl_table_header *__register_sysctl_table(
-       struct ctl_table_root *root, struct nsproxy *namespaces,
+       struct ctl_table_set *set,
        const char *path, struct ctl_table *table);
 struct ctl_table_header *__register_sysctl_paths(
-       struct ctl_table_root *root, struct nsproxy *namespaces,
+       struct ctl_table_set *set,
        const struct ctl_path *path, struct ctl_table *table);
 struct ctl_table_header *register_sysctl_table(struct ctl_table * table);
 struct ctl_table_header *register_sysctl_paths(const struct ctl_path *path,

diff --git a/net/sysctl_net.c b/net/sysctl_net.c
index e998c644804675e7bf2f59f2088b740e8bc329ee..c3e65aebecc08f85b3d5d471d9ce65908f7ea2d8 100644 (file)
--- a/net/sysctl_net.c
+++ b/net/sysctl_net.c
@@ -105,19 +105,15 @@ subsys_initcall(net_sysctl_init);
 struct ctl_table_header *register_net_sysctl_table(struct net *net,
        const struct ctl_path *path, struct ctl_table *table)
 {
-       struct nsproxy namespaces;
-       namespaces = *current->nsproxy;
-       namespaces.net_ns = net;
-       return __register_sysctl_paths(&net_sysctl_root,
-                                       &namespaces, path, table);
+       return __register_sysctl_paths(&net->sysctls, path, table);
 }
 EXPORT_SYMBOL_GPL(register_net_sysctl_table);
 
 struct ctl_table_header *register_net_sysctl_rotable(const
                struct ctl_path *path, struct ctl_table *table)
 {
-       return __register_sysctl_paths(&net_sysctl_ro_root,
-                       &init_nsproxy, path, table);
+       return __register_sysctl_paths(&net_sysctl_ro_root.default_set,
+                                       path, table);
 }
 EXPORT_SYMBOL_GPL(register_net_sysctl_rotable);