]> Pileus Git - ~andy/linux/commitdiff
don't pass superblock to hypfs_vm_create_files()
authorAl Viro <viro@zeniv.linux.org.uk>
Fri, 19 Jul 2013 13:03:49 +0000 (17:03 +0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 4 Sep 2013 02:52:48 +0000 (22:52 -0400)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/s390/hypfs/hypfs.h
arch/s390/hypfs/hypfs_vm.c
arch/s390/hypfs/inode.c

index f41e0ef7fdf94578166eafee6d55df6a6f946390..41a57ceda9289d9a204082a0141fd72370c2e249 100644 (file)
@@ -37,7 +37,7 @@ extern int hypfs_diag_create_files(struct super_block *sb, struct dentry *root);
 /* VM Hypervisor */
 extern int hypfs_vm_init(void);
 extern void hypfs_vm_exit(void);
-extern int hypfs_vm_create_files(struct super_block *sb, struct dentry *root);
+extern int hypfs_vm_create_files(struct dentry *root);
 
 /* debugfs interface */
 struct hypfs_dbfs_file;
index f364dcf77e8efe528267a57582eb8ea60604856e..a7c5b57663e5c846de39d546f01734088ad4085a 100644 (file)
@@ -107,16 +107,15 @@ static void diag2fc_free(const void *data)
        vfree(data);
 }
 
-#define ATTRIBUTE(sb, dir, name, member) \
+#define ATTRIBUTE(dir, name, member) \
 do { \
        void *rc; \
-       rc = hypfs_create_u64(sb, dir, name, member); \
+       rc = hypfs_create_u64(dir->d_sb, dir, name, member); \
        if (IS_ERR(rc)) \
                return PTR_ERR(rc); \
 } while(0)
 
-static int hpyfs_vm_create_guest(struct super_block *sb,
-                                struct dentry *systems_dir,
+static int hpyfs_vm_create_guest(struct dentry *systems_dir,
                                 struct diag2fc_data *data)
 {
        char guest_name[NAME_LEN + 1] = {};
@@ -130,46 +129,46 @@ static int hpyfs_vm_create_guest(struct super_block *sb,
        memcpy(guest_name, data->guest_name, NAME_LEN);
        EBCASC(guest_name, NAME_LEN);
        strim(guest_name);
-       guest_dir = hypfs_mkdir(sb, systems_dir, guest_name);
+       guest_dir = hypfs_mkdir(systems_dir->d_sb, systems_dir, guest_name);
        if (IS_ERR(guest_dir))
                return PTR_ERR(guest_dir);
-       ATTRIBUTE(sb, guest_dir, "onlinetime_us", data->el_time);
+       ATTRIBUTE(guest_dir, "onlinetime_us", data->el_time);
 
        /* logical cpu information */
-       cpus_dir = hypfs_mkdir(sb, guest_dir, "cpus");
+       cpus_dir = hypfs_mkdir(guest_dir->d_sb, guest_dir, "cpus");
        if (IS_ERR(cpus_dir))
                return PTR_ERR(cpus_dir);
-       ATTRIBUTE(sb, cpus_dir, "cputime_us", data->used_cpu);
-       ATTRIBUTE(sb, cpus_dir, "capped", capped_value);
-       ATTRIBUTE(sb, cpus_dir, "dedicated", dedicated_flag);
-       ATTRIBUTE(sb, cpus_dir, "count", data->vcpus);
-       ATTRIBUTE(sb, cpus_dir, "weight_min", data->cpu_min);
-       ATTRIBUTE(sb, cpus_dir, "weight_max", data->cpu_max);
-       ATTRIBUTE(sb, cpus_dir, "weight_cur", data->cpu_shares);
+       ATTRIBUTE(cpus_dir, "cputime_us", data->used_cpu);
+       ATTRIBUTE(cpus_dir, "capped", capped_value);
+       ATTRIBUTE(cpus_dir, "dedicated", dedicated_flag);
+       ATTRIBUTE(cpus_dir, "count", data->vcpus);
+       ATTRIBUTE(cpus_dir, "weight_min", data->cpu_min);
+       ATTRIBUTE(cpus_dir, "weight_max", data->cpu_max);
+       ATTRIBUTE(cpus_dir, "weight_cur", data->cpu_shares);
 
        /* memory information */
-       mem_dir = hypfs_mkdir(sb, guest_dir, "mem");
+       mem_dir = hypfs_mkdir(guest_dir->d_sb, guest_dir, "mem");
        if (IS_ERR(mem_dir))
                return PTR_ERR(mem_dir);
-       ATTRIBUTE(sb, mem_dir, "min_KiB", data->mem_min_kb);
-       ATTRIBUTE(sb, mem_dir, "max_KiB", data->mem_max_kb);
-       ATTRIBUTE(sb, mem_dir, "used_KiB", data->mem_used_kb);
-       ATTRIBUTE(sb, mem_dir, "share_KiB", data->mem_share_kb);
+       ATTRIBUTE(mem_dir, "min_KiB", data->mem_min_kb);
+       ATTRIBUTE(mem_dir, "max_KiB", data->mem_max_kb);
+       ATTRIBUTE(mem_dir, "used_KiB", data->mem_used_kb);
+       ATTRIBUTE(mem_dir, "share_KiB", data->mem_share_kb);
 
        /* samples */
-       samples_dir = hypfs_mkdir(sb, guest_dir, "samples");
+       samples_dir = hypfs_mkdir(guest_dir->d_sb, guest_dir, "samples");
        if (IS_ERR(samples_dir))
                return PTR_ERR(samples_dir);
-       ATTRIBUTE(sb, samples_dir, "cpu_using", data->cpu_use_samp);
-       ATTRIBUTE(sb, samples_dir, "cpu_delay", data->cpu_delay_samp);
-       ATTRIBUTE(sb, samples_dir, "mem_delay", data->page_wait_samp);
-       ATTRIBUTE(sb, samples_dir, "idle", data->idle_samp);
-       ATTRIBUTE(sb, samples_dir, "other", data->other_samp);
-       ATTRIBUTE(sb, samples_dir, "total", data->total_samp);
+       ATTRIBUTE(samples_dir, "cpu_using", data->cpu_use_samp);
+       ATTRIBUTE(samples_dir, "cpu_delay", data->cpu_delay_samp);
+       ATTRIBUTE(samples_dir, "mem_delay", data->page_wait_samp);
+       ATTRIBUTE(samples_dir, "idle", data->idle_samp);
+       ATTRIBUTE(samples_dir, "other", data->other_samp);
+       ATTRIBUTE(samples_dir, "total", data->total_samp);
        return 0;
 }
 
-int hypfs_vm_create_files(struct super_block *sb, struct dentry *root)
+int hypfs_vm_create_files(struct dentry *root)
 {
        struct dentry *dir, *file;
        struct diag2fc_data *data;
@@ -181,38 +180,38 @@ int hypfs_vm_create_files(struct super_block *sb, struct dentry *root)
                return PTR_ERR(data);
 
        /* Hpervisor Info */
-       dir = hypfs_mkdir(sb, root, "hyp");
+       dir = hypfs_mkdir(root->d_sb, root, "hyp");
        if (IS_ERR(dir)) {
                rc = PTR_ERR(dir);
                goto failed;
        }
-       file = hypfs_create_str(sb, dir, "type", "z/VM Hypervisor");
+       file = hypfs_create_str(root->d_sb, dir, "type", "z/VM Hypervisor");
        if (IS_ERR(file)) {
                rc = PTR_ERR(file);
                goto failed;
        }
 
        /* physical cpus */
-       dir = hypfs_mkdir(sb, root, "cpus");
+       dir = hypfs_mkdir(root->d_sb, root, "cpus");
        if (IS_ERR(dir)) {
                rc = PTR_ERR(dir);
                goto failed;
        }
-       file = hypfs_create_u64(sb, dir, "count", data->lcpus);
+       file = hypfs_create_u64(root->d_sb, dir, "count", data->lcpus);
        if (IS_ERR(file)) {
                rc = PTR_ERR(file);
                goto failed;
        }
 
        /* guests */
-       dir = hypfs_mkdir(sb, root, "systems");
+       dir = hypfs_mkdir(root->d_sb, root, "systems");
        if (IS_ERR(dir)) {
                rc = PTR_ERR(dir);
                goto failed;
        }
 
        for (i = 0; i < count; i++) {
-               rc = hpyfs_vm_create_guest(sb, dir, &(data[i]));
+               rc = hpyfs_vm_create_guest(dir, &(data[i]));
                if (rc)
                        goto failed;
        }
index 7a539f4f5e30b776527c5426eabfbefead4a0f34..70627216ab66afba1946aa6f5d6f618a196cc62c 100644 (file)
@@ -193,7 +193,7 @@ static ssize_t hypfs_aio_write(struct kiocb *iocb, const struct iovec *iov,
        }
        hypfs_delete_tree(sb->s_root);
        if (MACHINE_IS_VM)
-               rc = hypfs_vm_create_files(sb, sb->s_root);
+               rc = hypfs_vm_create_files(sb->s_root);
        else
                rc = hypfs_diag_create_files(sb, sb->s_root);
        if (rc) {
@@ -302,7 +302,7 @@ static int hypfs_fill_super(struct super_block *sb, void *data, int silent)
        if (!root_dentry)
                return -ENOMEM;
        if (MACHINE_IS_VM)
-               rc = hypfs_vm_create_files(sb, root_dentry);
+               rc = hypfs_vm_create_files(root_dentry);
        else
                rc = hypfs_diag_create_files(sb, root_dentry);
        if (rc)