bury struct proc_ns in fs/proc
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 1 Nov 2014 07:13:17 +0000 (03:13 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Thu, 4 Dec 2014 19:34:54 +0000 (14:34 -0500)
a) make get_proc_ns() return a pointer to struct ns_common
b) mirror ns_ops in dentry->d_fsdata of ns dentries, so that
is_mnt_ns_file() could get away with fewer dereferences.

That way struct proc_ns becomes invisible outside of fs/proc/*.c

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/namespace.c
fs/proc/internal.h
fs/proc/namespaces.c
include/linux/proc_ns.h
kernel/nsproxy.c
net/core/net_namespace.c

index f815218f92d318d84a4c9ba950846dcd16d8a573..9dfb4cac0c41db8c6cdf833b130ccf70f494040f 100644 (file)
@@ -1570,16 +1570,7 @@ static bool is_mnt_ns_file(struct dentry *dentry)
 {
        /* Is this a proxy for a mount namespace? */
        struct inode *inode = dentry->d_inode;
-       struct proc_ns *ei;
-
-       if (!proc_ns_inode(inode))
-               return false;
-
-       ei = get_proc_ns(inode);
-       if (ei->ns_ops != &mntns_operations)
-               return false;
-
-       return true;
+       return proc_ns_inode(inode) && dentry->d_fsdata == &mntns_operations;
 }
 
 struct mnt_namespace *to_mnt_ns(struct ns_common *ns)
@@ -1596,7 +1587,7 @@ static bool mnt_ns_loop(struct dentry *dentry)
        if (!is_mnt_ns_file(dentry))
                return false;
 
-       mnt_ns = to_mnt_ns(get_proc_ns(dentry->d_inode)->ns);
+       mnt_ns = to_mnt_ns(get_proc_ns(dentry->d_inode));
        return current->nsproxy->mnt_ns->seq >= mnt_ns->seq;
 }
 
index aa7a0ee182e19329636b21f079fac074145aaa84..0fabc48d905f4b0f0900acd4428d7a5994bdc6ae 100644 (file)
@@ -57,6 +57,11 @@ union proc_op {
                struct task_struct *task);
 };
 
+struct proc_ns {
+       struct ns_common *ns;
+       const struct proc_ns_operations *ns_ops;
+};
+
 struct proc_inode {
        struct pid *pid;
        int fd;
index 995e8e98237d40027dbe424e3781befad6243b1e..18fc1cf899de863e0711d82cd7f91e1367ed807c 100644 (file)
@@ -45,7 +45,7 @@ static const struct inode_operations ns_inode_operations = {
 static char *ns_dname(struct dentry *dentry, char *buffer, int buflen)
 {
        struct inode *inode = dentry->d_inode;
-       const struct proc_ns_operations *ns_ops = PROC_I(inode)->ns.ns_ops;
+       const struct proc_ns_operations *ns_ops = dentry->d_fsdata;
 
        return dynamic_dname(dentry, buffer, buflen, "%s:[%lu]",
                ns_ops->name, inode->i_ino);
@@ -75,6 +75,7 @@ static struct dentry *proc_ns_get_dentry(struct super_block *sb,
                ns_ops->put(ns);
                return ERR_PTR(-ENOMEM);
        }
+       dentry->d_fsdata = (void *)ns_ops;
 
        inode = iget_locked(sb, ns->inum);
        if (!inode) {
@@ -286,9 +287,9 @@ out_invalid:
        return ERR_PTR(-EINVAL);
 }
 
-struct proc_ns *get_proc_ns(struct inode *inode)
+struct ns_common *get_proc_ns(struct inode *inode)
 {
-       return &PROC_I(inode)->ns;
+       return PROC_I(inode)->ns.ns;
 }
 
 bool proc_ns_inode(struct inode *inode)
index f5780ee7f8f7a2b8d7f1b5a68e612b3e2ddb6a80..2837ff41cfe3998b98cacf6d8a7784db212ae422 100644 (file)
@@ -16,11 +16,6 @@ struct proc_ns_operations {
        int (*install)(struct nsproxy *nsproxy, struct ns_common *ns);
 };
 
-struct proc_ns {
-       struct ns_common *ns;
-       const struct proc_ns_operations *ns_ops;
-};
-
 extern const struct proc_ns_operations netns_operations;
 extern const struct proc_ns_operations utsns_operations;
 extern const struct proc_ns_operations ipcns_operations;
@@ -44,7 +39,7 @@ enum {
 extern int pid_ns_prepare_proc(struct pid_namespace *ns);
 extern void pid_ns_release_proc(struct pid_namespace *ns);
 extern struct file *proc_ns_fget(int fd);
-extern struct proc_ns *get_proc_ns(struct inode *);
+extern struct ns_common *get_proc_ns(struct inode *);
 extern int proc_alloc_inum(unsigned int *pino);
 extern void proc_free_inum(unsigned int inum);
 extern bool proc_ns_inode(struct inode *inode);
@@ -59,7 +54,7 @@ static inline struct file *proc_ns_fget(int fd)
        return ERR_PTR(-EINVAL);
 }
 
-static inline struct proc_ns *get_proc_ns(struct inode *inode) { return NULL; }
+static inline struct ns_common *get_proc_ns(struct inode *inode) { return NULL; }
 
 static inline int proc_alloc_inum(unsigned int *inum)
 {
index 87c37221cb7f66a6b7e1e082821a8857479177e6..49746c81ad8d28abb3ac85ad82ee9dbc4ce69e81 100644 (file)
@@ -222,7 +222,6 @@ SYSCALL_DEFINE2(setns, int, fd, int, nstype)
 {
        struct task_struct *tsk = current;
        struct nsproxy *new_nsproxy;
-       struct proc_ns *ei;
        struct file *file;
        struct ns_common *ns;
        int err;
@@ -232,8 +231,7 @@ SYSCALL_DEFINE2(setns, int, fd, int, nstype)
                return PTR_ERR(file);
 
        err = -EINVAL;
-       ei = get_proc_ns(file_inode(file));
-       ns = ei->ns;
+       ns = get_proc_ns(file_inode(file));
        if (nstype && (ns->ops->type != nstype))
                goto out;
 
index 4d4acaf7b4985012e2c4eebc3da711d359d74b1e..ce780c722e48ca2bf35534895ef0b7dc0a9541d5 100644 (file)
@@ -337,7 +337,6 @@ EXPORT_SYMBOL_GPL(__put_net);
 
 struct net *get_net_ns_by_fd(int fd)
 {
-       struct proc_ns *ei;
        struct file *file;
        struct ns_common *ns;
        struct net *net;
@@ -346,8 +345,7 @@ struct net *get_net_ns_by_fd(int fd)
        if (IS_ERR(file))
                return ERR_CAST(file);
 
-       ei = get_proc_ns(file_inode(file));
-       ns = ei->ns;
+       ns = get_proc_ns(file_inode(file));
        if (ns->ops == &netns_operations)
                net = get_net(container_of(ns, struct net, ns));
        else