kernfs: move the last knowledge of sysfs out from kernfs
authorJianyu Zhan <nasa4836@gmail.com>
Sat, 26 Apr 2014 07:40:28 +0000 (15:40 +0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 3 Jun 2014 15:11:18 +0000 (08:11 -0700)
There is still one residue of sysfs remaining: the sb_magic
SYSFS_MAGIC. However this should be kernfs user specific,
so this patch moves it out. Kerrnfs user should specify their
magic number while mouting.

Signed-off-by: Jianyu Zhan <nasa4836@gmail.com>
Acked-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/kernfs/mount.c
fs/sysfs/mount.c
include/linux/kernfs.h
kernel/cgroup.c

index 6a5f04ac87040d85179420d254d528789ddcf1d6..95dcd1d558bb00e1f92b5dae3143a448d1160745 100644 (file)
@@ -62,7 +62,7 @@ struct kernfs_root *kernfs_root_from_sb(struct super_block *sb)
        return NULL;
 }
 
-static int kernfs_fill_super(struct super_block *sb)
+static int kernfs_fill_super(struct super_block *sb, unsigned long magic)
 {
        struct kernfs_super_info *info = kernfs_info(sb);
        struct inode *inode;
@@ -70,7 +70,7 @@ static int kernfs_fill_super(struct super_block *sb)
 
        sb->s_blocksize = PAGE_CACHE_SIZE;
        sb->s_blocksize_bits = PAGE_CACHE_SHIFT;
-       sb->s_magic = SYSFS_MAGIC;
+       sb->s_magic = magic;
        sb->s_op = &kernfs_sops;
        sb->s_time_gran = 1;
 
@@ -131,6 +131,7 @@ const void *kernfs_super_ns(struct super_block *sb)
  * @fs_type: file_system_type of the fs being mounted
  * @flags: mount flags specified for the mount
  * @root: kernfs_root of the hierarchy being mounted
+ * @magic: file system specific magic number
  * @new_sb_created: tell the caller if we allocated a new superblock
  * @ns: optional namespace tag of the mount
  *
@@ -142,8 +143,8 @@ const void *kernfs_super_ns(struct super_block *sb)
  * The return value can be passed to the vfs layer verbatim.
  */
 struct dentry *kernfs_mount_ns(struct file_system_type *fs_type, int flags,
-                              struct kernfs_root *root, bool *new_sb_created,
-                              const void *ns)
+                               struct kernfs_root *root, unsigned long magic,
+                               bool *new_sb_created, const void *ns)
 {
        struct super_block *sb;
        struct kernfs_super_info *info;
@@ -166,7 +167,7 @@ struct dentry *kernfs_mount_ns(struct file_system_type *fs_type, int flags,
                *new_sb_created = !sb->s_root;
 
        if (!sb->s_root) {
-               error = kernfs_fill_super(sb);
+               error = kernfs_fill_super(sb, magic);
                if (error) {
                        deactivate_locked_super(sb);
                        return ERR_PTR(error);
index 8794423f7efbe2c90408ba9dfa6cbb7e55e6c3d9..8a49486bf30c9859a5474f3cc5055bc7b9917de4 100644 (file)
@@ -13,6 +13,7 @@
 #define DEBUG
 
 #include <linux/fs.h>
+#include <linux/magic.h>
 #include <linux/mount.h>
 #include <linux/init.h>
 #include <linux/user_namespace.h>
@@ -38,7 +39,8 @@ static struct dentry *sysfs_mount(struct file_system_type *fs_type,
        }
 
        ns = kobj_ns_grab_current(KOBJ_NS_TYPE_NET);
-       root = kernfs_mount_ns(fs_type, flags, sysfs_root, &new_sb, ns);
+       root = kernfs_mount_ns(fs_type, flags, sysfs_root,
+                               SYSFS_MAGIC, &new_sb, ns);
        if (IS_ERR(root) || !new_sb)
                kobj_ns_drop(KOBJ_NS_TYPE_NET, ns);
        return root;
index ca1be5c9136c4557de511143b4c9b36e1eec3734..52bf5677db0bdce87d5cbb690fa438ef8499d6f9 100644 (file)
@@ -297,8 +297,8 @@ void kernfs_notify(struct kernfs_node *kn);
 
 const void *kernfs_super_ns(struct super_block *sb);
 struct dentry *kernfs_mount_ns(struct file_system_type *fs_type, int flags,
-                              struct kernfs_root *root, bool *new_sb_created,
-                              const void *ns);
+                              struct kernfs_root *root, unsigned long magic,
+                              bool *new_sb_created, const void *ns);
 void kernfs_kill_sb(struct super_block *sb);
 
 void kernfs_init(void);
@@ -391,7 +391,8 @@ static inline const void *kernfs_super_ns(struct super_block *sb)
 
 static inline struct dentry *
 kernfs_mount_ns(struct file_system_type *fs_type, int flags,
-               struct kernfs_root *root, bool *new_sb_created, const void *ns)
+               struct kernfs_root *root, unsigned long magic,
+               bool *new_sb_created, const void *ns)
 { return ERR_PTR(-ENOSYS); }
 
 static inline void kernfs_kill_sb(struct super_block *sb) { }
@@ -449,9 +450,11 @@ static inline int kernfs_rename(struct kernfs_node *kn,
 
 static inline struct dentry *
 kernfs_mount(struct file_system_type *fs_type, int flags,
-            struct kernfs_root *root, bool *new_sb_created)
+               struct kernfs_root *root, unsigned long magic,
+               bool *new_sb_created)
 {
-       return kernfs_mount_ns(fs_type, flags, root, new_sb_created, NULL);
+       return kernfs_mount_ns(fs_type, flags, root,
+                               magic, new_sb_created, NULL);
 }
 
 #endif /* __LINUX_KERNFS_H */
index 3f1ca934a2378495e5129dbe807bfc7111f53e8b..ceee0c54c6a4a7845dc31a18689fec5fd0abcb83 100644 (file)
@@ -33,6 +33,7 @@
 #include <linux/init_task.h>
 #include <linux/kernel.h>
 #include <linux/list.h>
+#include <linux/magic.h>
 #include <linux/mm.h>
 #include <linux/mutex.h>
 #include <linux/mount.h>
@@ -1604,7 +1605,8 @@ out_unlock:
        if (ret)
                return ERR_PTR(ret);
 
-       dentry = kernfs_mount(fs_type, flags, root->kf_root, &new_sb);
+       dentry = kernfs_mount(fs_type, flags, root->kf_root,
+                               CGROUP_SUPER_MAGIC, &new_sb);
        if (IS_ERR(dentry) || !new_sb)
                cgroup_put(&root->cgrp);
        return dentry;