summaryrefslogtreecommitdiffstats
path: root/net/sunrpc/rpc_pipe.c
diff options
context:
space:
mode:
authorTrond Myklebust <Trond.Myklebust@netapp.com>2009-08-09 15:14:25 -0400
committerTrond Myklebust <Trond.Myklebust@netapp.com>2009-08-09 15:14:25 -0400
commit23ac6581702ac6d029643328a7e6ea3baf834c5e (patch)
treee960a371127a240f17971596e0c456ccc8f01b7e /net/sunrpc/rpc_pipe.c
parent7d217caca5d704e48aa5e59aba0b3ad4c7af4fd2 (diff)
downloadlinux-stable-23ac6581702ac6d029643328a7e6ea3baf834c5e.tar.gz
linux-stable-23ac6581702ac6d029643328a7e6ea3baf834c5e.tar.bz2
linux-stable-23ac6581702ac6d029643328a7e6ea3baf834c5e.zip
SUNRPC: clean up rpc_setup_pipedir()
There is still a little wart or two there: Since we've already got a vfsmount, we might as well pass that in to rpc_create_client_dir. Another point is that if we open code __rpc_lookup_path() here, then we can avoid looking up the entire parent directory path over and over again: it doesn't change. Also get rid of rpc_clnt->cl_pathname, since it has no users... Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/rpc_pipe.c')
-rw-r--r--net/sunrpc/rpc_pipe.c58
1 files changed, 3 insertions, 55 deletions
diff --git a/net/sunrpc/rpc_pipe.c b/net/sunrpc/rpc_pipe.c
index 6d152f69587e..1613d858ba38 100644
--- a/net/sunrpc/rpc_pipe.c
+++ b/net/sunrpc/rpc_pipe.c
@@ -443,42 +443,6 @@ static const struct dentry_operations rpc_dentry_operations = {
.d_delete = rpc_delete_dentry,
};
-static int __rpc_lookup_path(const char *pathname, unsigned flags,
- struct nameidata *nd)
-{
- struct vfsmount *mnt;
-
- if (pathname[0] == '\0')
- return -ENOENT;
-
- mnt = rpc_get_mount();
- if (IS_ERR(mnt)) {
- printk(KERN_WARNING "%s: %s failed to mount "
- "pseudofilesystem \n", __FILE__, __func__);
- return PTR_ERR(mnt);
- }
-
- if (vfs_path_lookup(mnt->mnt_root, mnt, pathname, flags, nd)) {
- printk(KERN_WARNING "%s: %s failed to find path %s\n",
- __FILE__, __func__, pathname);
- rpc_put_mount();
- return -ENOENT;
- }
- return 0;
-}
-
-static int rpc_lookup_parent(const char *pathname, struct nameidata *nd)
-{
- return __rpc_lookup_path(pathname, LOOKUP_PARENT, nd);
-}
-
-static void
-rpc_release_path(struct nameidata *nd)
-{
- path_put(&nd->path);
- rpc_put_mount();
-}
-
static struct inode *
rpc_get_inode(struct super_block *sb, umode_t mode)
{
@@ -889,27 +853,11 @@ EXPORT_SYMBOL_GPL(rpc_unlink);
* information about the client, together with any "pipes" that may
* later be created using rpc_mkpipe().
*/
-struct dentry *rpc_create_client_dir(const char *path,
+struct dentry *rpc_create_client_dir(struct dentry *dentry,
+ struct qstr *name,
struct rpc_clnt *rpc_client)
{
- struct nameidata nd;
- struct dentry *ret;
- struct inode *dir;
-
- ret = ERR_PTR(rpc_lookup_parent(path, &nd));
- if (IS_ERR(ret))
- goto out_err;
- dir = nd.path.dentry->d_inode;
-
- ret = rpc_mkdir_populate(nd.path.dentry, &nd.last,
- S_IRUGO | S_IXUGO, rpc_client);
- rpc_release_path(&nd);
- if (!IS_ERR(ret))
- return ret;
-out_err:
- printk(KERN_WARNING "%s: %s() failed to create directory %s (errno = %ld)\n",
- __FILE__, __func__, path, PTR_ERR(ret));
- return ret;
+ return rpc_mkdir_populate(dentry, name, S_IRUGO | S_IXUGO, rpc_client);
}
/*