oprofilefs_mkdir() doesn't need superblock argument
authorAl Viro <viro@zeniv.linux.org.uk>
Fri, 19 Jul 2013 11:58:27 +0000 (15:58 +0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 4 Sep 2013 02:52:47 +0000 (22:52 -0400)
it's always equal to ->d_sb of the second argument (parent dentry),
due to either being literally that, or ->d_sb of parent's parent.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/alpha/oprofile/common.c
arch/avr32/oprofile/op_model_avr32.c
arch/mips/oprofile/common.c
arch/powerpc/oprofile/common.c
arch/s390/oprofile/init.c
arch/x86/oprofile/nmi_int.c
arch/x86/oprofile/op_model_amd.c
drivers/oprofile/oprofile_perf.c
drivers/oprofile/oprofile_stats.c
drivers/oprofile/oprofilefs.c
include/linux/oprofile.h

index d79bf68caee259e5ac976438672779c63a0e08c5..a44608b697ace08f6f62c99fae28d961dfe33b03 100644 (file)
@@ -115,7 +115,7 @@ op_axp_create_files(struct dentry *root)
                char buf[4];
 
                snprintf(buf, sizeof buf, "%d", i);
-               dir = oprofilefs_mkdir(root->d_sb, root, buf);
+               dir = oprofilefs_mkdir(root, buf);
 
                oprofilefs_create_ulong(root->d_sb, dir, "enabled", &ctr[i].enabled);
                 oprofilefs_create_ulong(root->d_sb, dir, "event", &ctr[i].event);
index 80a06158da8226db05c9b2c8bf71a1ca71f8ff58..7d920a41b44132651118d44ea4c8cfd2e39557da 100644 (file)
@@ -105,7 +105,7 @@ static int avr32_perf_counter_create_files(struct dentry *root)
 
        for (i = 0; i < NR_counter; i++) {
                snprintf(filename, sizeof(filename), "%u", i);
-               dir = oprofilefs_mkdir(root->d_sb, root, filename);
+               dir = oprofilefs_mkdir(root, filename);
 
                oprofilefs_create_ulong(root->d_sb, dir, "enabled",
                                &counter[i].enabled);
index 7439ae2df05cf1a8f94298a094ecb358d11ea2c8..755caf04656e5098d86127976c9367fc79196635 100644 (file)
@@ -42,7 +42,7 @@ static int op_mips_create_files(struct dentry *root)
                char buf[4];
 
                snprintf(buf, sizeof buf, "%d", i);
-               dir = oprofilefs_mkdir(root->d_sb, root, buf);
+               dir = oprofilefs_mkdir(root, buf);
 
                oprofilefs_create_ulong(root->d_sb, dir, "enabled", &ctr[i].enabled);
                oprofilefs_create_ulong(root->d_sb, dir, "event", &ctr[i].event);
index addc62317f45a2b828450255ece0d58a00cf4b34..60a286b367fcafeedc515fd498bf7de61fdbb7a9 100644 (file)
@@ -160,7 +160,7 @@ static int op_powerpc_create_files(struct dentry *root)
                char buf[4];
 
                snprintf(buf, sizeof buf, "%d", i);
-               dir = oprofilefs_mkdir(root->d_sb, root, buf);
+               dir = oprofilefs_mkdir(root, buf);
 
                oprofilefs_create_ulong(root->d_sb, dir, "enabled", &ctr[i].enabled);
                oprofilefs_create_ulong(root->d_sb, dir, "event", &ctr[i].event);
index 712d57bb783ada595e951eacf6e8608032a43620..5ae06db7ec58a60ca75c9c007e6067c8503279bf 100644 (file)
@@ -350,7 +350,7 @@ static int oprofile_create_hwsampling_files(struct dentry *root)
 {
        struct dentry *dir;
 
-       dir = oprofilefs_mkdir(root->d_sb, root, "timer");
+       dir = oprofilefs_mkdir(root, "timer");
        if (!dir)
                return -EINVAL;
 
@@ -375,7 +375,7 @@ static int oprofile_create_hwsampling_files(struct dentry *root)
                 * and can only be set to 0.
                 */
 
-               dir = oprofilefs_mkdir(root->d_sb, root, "0");
+               dir = oprofilefs_mkdir(root, "0");
                if (!dir)
                        return -EINVAL;
 
@@ -395,7 +395,7 @@ static int oprofile_create_hwsampling_files(struct dentry *root)
                 * space tools.  The /dev/oprofile/hwsampling fs is
                 * provided in that case.
                 */
-               dir = oprofilefs_mkdir(root->d_sb, root, "hwsampling");
+               dir = oprofilefs_mkdir(root, "hwsampling");
                if (!dir)
                        return -EINVAL;
 
index 483f02b8c1a858fefdb66e6666fb39e7d65deb6e..8bb2de6e103c5bd3cff726e79a21475a918d06d9 100644 (file)
@@ -420,7 +420,7 @@ static int nmi_create_files(struct dentry *root)
                        continue;
 
                snprintf(buf,  sizeof(buf), "%d", i);
-               dir = oprofilefs_mkdir(root->d_sb, root, buf);
+               dir = oprofilefs_mkdir(root, buf);
                oprofilefs_create_ulong(root->d_sb, dir, "enabled", &counter_config[i].enabled);
                oprofilefs_create_ulong(root->d_sb, dir, "event", &counter_config[i].event);
                oprofilefs_create_ulong(root->d_sb, dir, "count", &counter_config[i].count);
index 34e16c2fe357a0933e3b868faa0d2e5e8e4055d7..99131435ea32a4d1b3e54a0afc4999d16cc0ca22 100644 (file)
@@ -479,7 +479,7 @@ static int setup_ibs_files(struct dentry *root)
        ibs_config.max_cnt_op = 250000;
 
        if (ibs_caps & IBS_CAPS_FETCHSAM) {
-               dir = oprofilefs_mkdir(root->d_sb, root, "ibs_fetch");
+               dir = oprofilefs_mkdir(root, "ibs_fetch");
                oprofilefs_create_ulong(root->d_sb, dir, "enable",
                                        &ibs_config.fetch_enabled);
                oprofilefs_create_ulong(root->d_sb, dir, "max_count",
@@ -489,7 +489,7 @@ static int setup_ibs_files(struct dentry *root)
        }
 
        if (ibs_caps & IBS_CAPS_OPSAM) {
-               dir = oprofilefs_mkdir(root->d_sb, root, "ibs_op");
+               dir = oprofilefs_mkdir(root, "ibs_op");
                oprofilefs_create_ulong(root->d_sb, dir, "enable",
                                        &ibs_config.op_enabled);
                oprofilefs_create_ulong(root->d_sb, dir, "max_count",
index 52ad942df04e3a0fe9debd69ac8c392749c2dce5..923a245774faa0c1020722fc4c27dd3bcf352844 100644 (file)
@@ -147,7 +147,7 @@ static int oprofile_perf_create_files(struct dentry *root)
                char buf[4];
 
                snprintf(buf, sizeof buf, "%d", i);
-               dir = oprofilefs_mkdir(root->d_sb, root, buf);
+               dir = oprofilefs_mkdir(root, buf);
                oprofilefs_create_ulong(root->d_sb, dir, "enabled", &counter_config[i].enabled);
                oprofilefs_create_ulong(root->d_sb, dir, "event", &counter_config[i].event);
                oprofilefs_create_ulong(root->d_sb, dir, "count", &counter_config[i].count);
index 627dce4783115a6e599e37af3e1eb4878f5ba3cc..8cf2fa9710a4418803dc806a20c8c3eb1940cab6 100644 (file)
@@ -46,14 +46,14 @@ void oprofile_create_stats_files(struct dentry *root)
        char buf[10];
        int i;
 
-       dir = oprofilefs_mkdir(root->d_sb, root, "stats");
+       dir = oprofilefs_mkdir(root, "stats");
        if (!dir)
                return;
 
        for_each_possible_cpu(i) {
                cpu_buf = &per_cpu(op_cpu_buffer, i);
                snprintf(buf, 10, "cpu%d", i);
-               cpudir = oprofilefs_mkdir(root->d_sb, dir, buf);
+               cpudir = oprofilefs_mkdir(dir, buf);
 
                /* Strictly speaking access to these ulongs is racy,
                 * but we can't simply lock them, and they are
index 2e2dd5ca72115a7e95eb2c92455c6f825d882bfe..6a345724637f6fc8f936bd26af0e457dceb66fec 100644 (file)
@@ -211,28 +211,27 @@ int oprofilefs_create_file_perm(struct super_block *sb, struct dentry *root,
 }
 
 
-struct dentry *oprofilefs_mkdir(struct super_block *sb,
-       struct dentry *root, char const *name)
+struct dentry *oprofilefs_mkdir(struct dentry *parent, char const *name)
 {
        struct dentry *dentry;
        struct inode *inode;
 
-       mutex_lock(&root->d_inode->i_mutex);
-       dentry = d_alloc_name(root, name);
+       mutex_lock(&parent->d_inode->i_mutex);
+       dentry = d_alloc_name(parent, name);
        if (!dentry) {
-               mutex_unlock(&root->d_inode->i_mutex);
+               mutex_unlock(&parent->d_inode->i_mutex);
                return NULL;
        }
-       inode = oprofilefs_get_inode(sb, S_IFDIR | 0755);
+       inode = oprofilefs_get_inode(parent->d_sb, S_IFDIR | 0755);
        if (!inode) {
                dput(dentry);
-               mutex_unlock(&root->d_inode->i_mutex);
+               mutex_unlock(&parent->d_inode->i_mutex);
                return NULL;
        }
        inode->i_op = &simple_dir_inode_operations;
        inode->i_fop = &simple_dir_operations;
        d_add(dentry, inode);
-       mutex_unlock(&root->d_inode->i_mutex);
+       mutex_unlock(&parent->d_inode->i_mutex);
        return dentry;
 }
 
index 8a906b4034a769de311e054d2ac8db0e8b088445..957877e7a40852566555b497c842442736a56d9f 100644 (file)
@@ -144,8 +144,7 @@ int oprofilefs_create_ro_atomic(struct super_block * sb, struct dentry * root,
        char const * name, atomic_t * val);
  
 /** create a directory */
-struct dentry * oprofilefs_mkdir(struct super_block * sb, struct dentry * root,
-       char const * name);
+struct dentry *oprofilefs_mkdir(struct dentry *parent, char const *name);
 
 /**
  * Write the given asciz string to the given user buffer @buf, updating *offset