Commit d3db90b0 authored by Al Viro's avatar Al Viro

__rpc_lookup_create_exclusive: pass string instead of qstr

... and use d_hash_and_lookup() instead of open-coding it, for fsck sake...
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent a95e691f
...@@ -656,13 +656,12 @@ static int __rpc_rmpipe(struct inode *dir, struct dentry *dentry) ...@@ -656,13 +656,12 @@ static int __rpc_rmpipe(struct inode *dir, struct dentry *dentry)
} }
static struct dentry *__rpc_lookup_create_exclusive(struct dentry *parent, static struct dentry *__rpc_lookup_create_exclusive(struct dentry *parent,
struct qstr *name) const char *name)
{ {
struct dentry *dentry; struct qstr q = QSTR_INIT(name, strlen(name));
struct dentry *dentry = d_hash_and_lookup(parent, &q);
dentry = d_lookup(parent, name);
if (!dentry) { if (!dentry) {
dentry = d_alloc(parent, name); dentry = d_alloc(parent, &q);
if (!dentry) if (!dentry)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
} }
...@@ -689,8 +688,7 @@ static void __rpc_depopulate(struct dentry *parent, ...@@ -689,8 +688,7 @@ static void __rpc_depopulate(struct dentry *parent,
for (i = start; i < eof; i++) { for (i = start; i < eof; i++) {
name.name = files[i].name; name.name = files[i].name;
name.len = strlen(files[i].name); name.len = strlen(files[i].name);
name.hash = full_name_hash(name.name, name.len); dentry = d_hash_and_lookup(parent, &name);
dentry = d_lookup(parent, &name);
if (dentry == NULL) if (dentry == NULL)
continue; continue;
...@@ -732,12 +730,7 @@ static int rpc_populate(struct dentry *parent, ...@@ -732,12 +730,7 @@ static int rpc_populate(struct dentry *parent,
mutex_lock(&dir->i_mutex); mutex_lock(&dir->i_mutex);
for (i = start; i < eof; i++) { for (i = start; i < eof; i++) {
struct qstr q; dentry = __rpc_lookup_create_exclusive(parent, files[i].name);
q.name = files[i].name;
q.len = strlen(files[i].name);
q.hash = full_name_hash(q.name, q.len);
dentry = __rpc_lookup_create_exclusive(parent, &q);
err = PTR_ERR(dentry); err = PTR_ERR(dentry);
if (IS_ERR(dentry)) if (IS_ERR(dentry))
goto out_bad; goto out_bad;
...@@ -774,13 +767,11 @@ static struct dentry *rpc_mkdir_populate(struct dentry *parent, ...@@ -774,13 +767,11 @@ static struct dentry *rpc_mkdir_populate(struct dentry *parent,
int (*populate)(struct dentry *, void *), void *args_populate) int (*populate)(struct dentry *, void *), void *args_populate)
{ {
struct dentry *dentry; struct dentry *dentry;
struct qstr q = QSTR_INIT(name, strlen(name));
struct inode *dir = parent->d_inode; struct inode *dir = parent->d_inode;
int error; int error;
q.hash = full_name_hash(q.name, q.len);
mutex_lock_nested(&dir->i_mutex, I_MUTEX_PARENT); mutex_lock_nested(&dir->i_mutex, I_MUTEX_PARENT);
dentry = __rpc_lookup_create_exclusive(parent, &q); dentry = __rpc_lookup_create_exclusive(parent, name);
if (IS_ERR(dentry)) if (IS_ERR(dentry))
goto out; goto out;
error = __rpc_mkdir(dir, dentry, mode, NULL, private); error = __rpc_mkdir(dir, dentry, mode, NULL, private);
...@@ -843,7 +834,6 @@ struct dentry *rpc_mkpipe_dentry(struct dentry *parent, const char *name, ...@@ -843,7 +834,6 @@ struct dentry *rpc_mkpipe_dentry(struct dentry *parent, const char *name,
struct dentry *dentry; struct dentry *dentry;
struct inode *dir = parent->d_inode; struct inode *dir = parent->d_inode;
umode_t umode = S_IFIFO | S_IRUSR | S_IWUSR; umode_t umode = S_IFIFO | S_IRUSR | S_IWUSR;
struct qstr q;
int err; int err;
if (pipe->ops->upcall == NULL) if (pipe->ops->upcall == NULL)
...@@ -851,12 +841,8 @@ struct dentry *rpc_mkpipe_dentry(struct dentry *parent, const char *name, ...@@ -851,12 +841,8 @@ struct dentry *rpc_mkpipe_dentry(struct dentry *parent, const char *name,
if (pipe->ops->downcall == NULL) if (pipe->ops->downcall == NULL)
umode &= ~S_IWUGO; umode &= ~S_IWUGO;
q.name = name;
q.len = strlen(name);
q.hash = full_name_hash(q.name, q.len),
mutex_lock_nested(&dir->i_mutex, I_MUTEX_PARENT); mutex_lock_nested(&dir->i_mutex, I_MUTEX_PARENT);
dentry = __rpc_lookup_create_exclusive(parent, &q); dentry = __rpc_lookup_create_exclusive(parent, name);
if (IS_ERR(dentry)) if (IS_ERR(dentry))
goto out; goto out;
err = __rpc_mkpipe_dentry(dir, dentry, umode, &rpc_pipe_fops, err = __rpc_mkpipe_dentry(dir, dentry, umode, &rpc_pipe_fops,
...@@ -1063,9 +1049,7 @@ struct dentry *rpc_d_lookup_sb(const struct super_block *sb, ...@@ -1063,9 +1049,7 @@ struct dentry *rpc_d_lookup_sb(const struct super_block *sb,
const unsigned char *dir_name) const unsigned char *dir_name)
{ {
struct qstr dir = QSTR_INIT(dir_name, strlen(dir_name)); struct qstr dir = QSTR_INIT(dir_name, strlen(dir_name));
return d_hash_and_lookup(sb->s_root, &dir);
dir.hash = full_name_hash(dir.name, dir.len);
return d_lookup(sb->s_root, &dir);
} }
EXPORT_SYMBOL_GPL(rpc_d_lookup_sb); EXPORT_SYMBOL_GPL(rpc_d_lookup_sb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment