9p: Optimize TCREATE by eliminating a redundant fid clone.
[safe/jmp/linux-2.6] / fs / 9p / vfs_inode.c
index 3ad8455..0ba2db4 100644 (file)
@@ -7,9 +7,8 @@
  *  Copyright (C) 2002 by Ron Minnich <rminnich@lanl.gov>
  *
  *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 2 of the License, or
- *  (at your option) any later version.
+ *  it under the terms of the GNU General Public License version 2
+ *  as published by the Free Software Foundation.
  *
  *  This program is distributed in the hope that it will be useful,
  *  but WITHOUT ANY WARRANTY; without even the implied warranty of
 #include <linux/pagemap.h>
 #include <linux/stat.h>
 #include <linux/string.h>
-#include <linux/smp_lock.h>
 #include <linux/inet.h>
 #include <linux/namei.h>
 #include <linux/idr.h>
+#include <linux/sched.h>
+#include <linux/slab.h>
+#include <net/9p/9p.h>
+#include <net/9p/client.h>
 
-#include "debug.h"
 #include "v9fs.h"
-#include "9p.h"
 #include "v9fs_vfs.h"
 #include "fid.h"
+#include "cache.h"
 
-static struct inode_operations v9fs_dir_inode_operations;
-static struct inode_operations v9fs_dir_inode_operations_ext;
-static struct inode_operations v9fs_file_inode_operations;
-static struct inode_operations v9fs_symlink_inode_operations;
+static const struct inode_operations v9fs_dir_inode_operations;
+static const struct inode_operations v9fs_dir_inode_operations_dotu;
+static const struct inode_operations v9fs_dir_inode_operations_dotl;
+static const struct inode_operations v9fs_file_inode_operations;
+static const struct inode_operations v9fs_file_inode_operations_dotl;
+static const struct inode_operations v9fs_symlink_inode_operations;
+static const struct inode_operations v9fs_symlink_inode_operations_dotl;
 
 /**
  * unixmode2p9mode - convert unix mode bits to plan 9
@@ -59,27 +63,29 @@ static int unixmode2p9mode(struct v9fs_session_info *v9ses, int mode)
        int res;
        res = mode & 0777;
        if (S_ISDIR(mode))
-               res |= V9FS_DMDIR;
-       if (v9ses->extended) {
+               res |= P9_DMDIR;
+       if (v9fs_proto_dotu(v9ses)) {
                if (S_ISLNK(mode))
-                       res |= V9FS_DMSYMLINK;
+                       res |= P9_DMSYMLINK;
                if (v9ses->nodev == 0) {
                        if (S_ISSOCK(mode))
-                               res |= V9FS_DMSOCKET;
+                               res |= P9_DMSOCKET;
                        if (S_ISFIFO(mode))
-                               res |= V9FS_DMNAMEDPIPE;
+                               res |= P9_DMNAMEDPIPE;
                        if (S_ISBLK(mode))
-                               res |= V9FS_DMDEVICE;
+                               res |= P9_DMDEVICE;
                        if (S_ISCHR(mode))
-                               res |= V9FS_DMDEVICE;
+                               res |= P9_DMDEVICE;
                }
 
                if ((mode & S_ISUID) == S_ISUID)
-                       res |= V9FS_DMSETUID;
+                       res |= P9_DMSETUID;
                if ((mode & S_ISGID) == S_ISGID)
-                       res |= V9FS_DMSETGID;
-               if ((mode & V9FS_DMLINK))
-                       res |= V9FS_DMLINK;
+                       res |= P9_DMSETGID;
+               if ((mode & S_ISVTX) == S_ISVTX)
+                       res |= P9_DMSETVTX;
+               if ((mode & P9_DMLINK))
+                       res |= P9_DMLINK;
        }
 
        return res;
@@ -98,34 +104,43 @@ static int p9mode2unixmode(struct v9fs_session_info *v9ses, int mode)
 
        res = mode & 0777;
 
-       if ((mode & V9FS_DMDIR) == V9FS_DMDIR)
+       if ((mode & P9_DMDIR) == P9_DMDIR)
                res |= S_IFDIR;
-       else if ((mode & V9FS_DMSYMLINK) && (v9ses->extended))
+       else if ((mode & P9_DMSYMLINK) && (v9fs_proto_dotu(v9ses)))
                res |= S_IFLNK;
-       else if ((mode & V9FS_DMSOCKET) && (v9ses->extended)
+       else if ((mode & P9_DMSOCKET) && (v9fs_proto_dotu(v9ses))
                 && (v9ses->nodev == 0))
                res |= S_IFSOCK;
-       else if ((mode & V9FS_DMNAMEDPIPE) && (v9ses->extended)
+       else if ((mode & P9_DMNAMEDPIPE) && (v9fs_proto_dotu(v9ses))
                 && (v9ses->nodev == 0))
                res |= S_IFIFO;
-       else if ((mode & V9FS_DMDEVICE) && (v9ses->extended)
+       else if ((mode & P9_DMDEVICE) && (v9fs_proto_dotu(v9ses))
                 && (v9ses->nodev == 0))
                res |= S_IFBLK;
        else
                res |= S_IFREG;
 
-       if (v9ses->extended) {
-               if ((mode & V9FS_DMSETUID) == V9FS_DMSETUID)
+       if (v9fs_proto_dotu(v9ses)) {
+               if ((mode & P9_DMSETUID) == P9_DMSETUID)
                        res |= S_ISUID;
 
-               if ((mode & V9FS_DMSETGID) == V9FS_DMSETGID)
+               if ((mode & P9_DMSETGID) == P9_DMSETGID)
                        res |= S_ISGID;
+
+               if ((mode & P9_DMSETVTX) == P9_DMSETVTX)
+                       res |= S_ISVTX;
        }
 
        return res;
 }
 
-int v9fs_uflags2omode(int uflags)
+/**
+ * v9fs_uflags2omode- convert posix open flags to plan 9 mode bits
+ * @uflags: flags to convert
+ * @extended: if .u extensions are active
+ */
+
+int v9fs_uflags2omode(int uflags, int extended)
 {
        int ret;
 
@@ -133,39 +148,40 @@ int v9fs_uflags2omode(int uflags)
        switch (uflags&3) {
        default:
        case O_RDONLY:
-               ret = V9FS_OREAD;
+               ret = P9_OREAD;
                break;
 
        case O_WRONLY:
-               ret = V9FS_OWRITE;
+               ret = P9_OWRITE;
                break;
 
        case O_RDWR:
-               ret = V9FS_ORDWR;
+               ret = P9_ORDWR;
                break;
        }
 
-       if (uflags & O_EXCL)
-               ret |= V9FS_OEXCL;
-
        if (uflags & O_TRUNC)
-               ret |= V9FS_OTRUNC;
+               ret |= P9_OTRUNC;
 
-       if (uflags & O_APPEND)
-               ret |= V9FS_OAPPEND;
+       if (extended) {
+               if (uflags & O_EXCL)
+                       ret |= P9_OEXCL;
+
+               if (uflags & O_APPEND)
+                       ret |= P9_OAPPEND;
+       }
 
        return ret;
 }
 
 /**
  * v9fs_blank_wstat - helper function to setup a 9P stat structure
- * @v9ses: 9P session info (for determining extended mode)
  * @wstat: structure to initialize
  *
  */
 
-static void
-v9fs_blank_wstat(struct v9fs_wstat *wstat)
+void
+v9fs_blank_wstat(struct p9_wstat *wstat)
 {
        wstat->type = ~0;
        wstat->dev = ~0;
@@ -186,6 +202,39 @@ v9fs_blank_wstat(struct v9fs_wstat *wstat)
        wstat->extension = NULL;
 }
 
+#ifdef CONFIG_9P_FSCACHE
+/**
+ * v9fs_alloc_inode - helper function to allocate an inode
+ * This callback is executed before setting up the inode so that we
+ * can associate a vcookie with each inode.
+ *
+ */
+
+struct inode *v9fs_alloc_inode(struct super_block *sb)
+{
+       struct v9fs_cookie *vcookie;
+       vcookie = (struct v9fs_cookie *)kmem_cache_alloc(vcookie_cache,
+                                                        GFP_KERNEL);
+       if (!vcookie)
+               return NULL;
+
+       vcookie->fscache = NULL;
+       vcookie->qid = NULL;
+       spin_lock_init(&vcookie->lock);
+       return &vcookie->inode;
+}
+
+/**
+ * v9fs_destroy_inode - destroy an inode
+ *
+ */
+
+void v9fs_destroy_inode(struct inode *inode)
+{
+       kmem_cache_free(vcookie_cache, v9fs_inode2cookie(inode));
+}
+#endif
+
 /**
  * v9fs_get_inode - helper function to setup an inode
  * @sb: superblock
@@ -195,117 +244,99 @@ v9fs_blank_wstat(struct v9fs_wstat *wstat)
 
 struct inode *v9fs_get_inode(struct super_block *sb, int mode)
 {
+       int err;
        struct inode *inode;
        struct v9fs_session_info *v9ses = sb->s_fs_info;
 
-       dprintk(DEBUG_VFS, "super block: %p mode: %o\n", sb, mode);
+       P9_DPRINTK(P9_DEBUG_VFS, "super block: %p mode: %o\n", sb, mode);
 
        inode = new_inode(sb);
-       if (inode) {
-               inode->i_mode = mode;
-               inode->i_uid = current->fsuid;
-               inode->i_gid = current->fsgid;
-               inode->i_blksize = sb->s_blocksize;
-               inode->i_blocks = 0;
-               inode->i_rdev = 0;
-               inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME;
-               inode->i_mapping->a_ops = &v9fs_addr_operations;
-
-               switch (mode & S_IFMT) {
-               case S_IFIFO:
-               case S_IFBLK:
-               case S_IFCHR:
-               case S_IFSOCK:
-                       if(!v9ses->extended) {
-                               dprintk(DEBUG_ERROR, "special files without extended mode\n");
-                               return ERR_PTR(-EINVAL);
-                       }
-                       init_special_inode(inode, inode->i_mode,
-                                          inode->i_rdev);
-                       break;
-               case S_IFREG:
+       if (!inode) {
+               P9_EPRINTK(KERN_WARNING, "Problem allocating inode\n");
+               return ERR_PTR(-ENOMEM);
+       }
+
+       inode->i_mode = mode;
+       inode->i_uid = current_fsuid();
+       inode->i_gid = current_fsgid();
+       inode->i_blocks = 0;
+       inode->i_rdev = 0;
+       inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME;
+       inode->i_mapping->a_ops = &v9fs_addr_operations;
+
+       switch (mode & S_IFMT) {
+       case S_IFIFO:
+       case S_IFBLK:
+       case S_IFCHR:
+       case S_IFSOCK:
+               if (!v9fs_proto_dotu(v9ses)) {
+                       P9_DPRINTK(P9_DEBUG_ERROR,
+                                  "special files without extended mode\n");
+                       err = -EINVAL;
+                       goto error;
+               }
+               init_special_inode(inode, inode->i_mode, inode->i_rdev);
+               break;
+       case S_IFREG:
+               if (v9fs_proto_dotl(v9ses)) {
+                       inode->i_op = &v9fs_file_inode_operations_dotl;
+                       inode->i_fop = &v9fs_file_operations_dotl;
+               } else {
                        inode->i_op = &v9fs_file_inode_operations;
                        inode->i_fop = &v9fs_file_operations;
-                       break;
-               case S_IFLNK:
-                       if(!v9ses->extended) {
-                               dprintk(DEBUG_ERROR, "extended modes used w/o 9P2000.u\n");
-                               return ERR_PTR(-EINVAL);
-                       }
-                       inode->i_op = &v9fs_symlink_inode_operations;
-                       break;
-               case S_IFDIR:
-                       inode->i_nlink++;
-                       if(v9ses->extended)
-                               inode->i_op = &v9fs_dir_inode_operations_ext;
-                       else
-                               inode->i_op = &v9fs_dir_inode_operations;
-                       inode->i_fop = &v9fs_dir_operations;
-                       break;
-               default:
-                       dprintk(DEBUG_ERROR, "BAD mode 0x%x S_IFMT 0x%x\n",
-                               mode, mode & S_IFMT);
-                       return ERR_PTR(-EINVAL);
                }
-       } else {
-               eprintk(KERN_WARNING, "Problem allocating inode\n");
-               return ERR_PTR(-ENOMEM);
-       }
-       return inode;
-}
 
-static int
-v9fs_create(struct v9fs_session_info *v9ses, u32 pfid, char *name,
-       u32 perm, u8 mode, u32 *fidp, struct v9fs_qid *qid, u32 *iounit)
-{
-       u32 fid;
-       int err;
-       struct v9fs_fcall *fcall;
+               break;
 
-       fid = v9fs_get_idpool(&v9ses->fidpool);
-       if (fid < 0) {
-               eprintk(KERN_WARNING, "no free fids available\n");
-               return -ENOSPC;
-       }
+       case S_IFLNK:
+               if (!v9fs_proto_dotu(v9ses) && !v9fs_proto_dotl(v9ses)) {
+                       P9_DPRINTK(P9_DEBUG_ERROR, "extended modes used with "
+                                               "legacy protocol.\n");
+                       err = -EINVAL;
+                       goto error;
+               }
 
-       err = v9fs_t_walk(v9ses, pfid, fid, NULL, &fcall);
-       if (err < 0) {
-               PRINT_FCALL_ERROR("clone error", fcall);
-               goto error;
-       }
-       kfree(fcall);
+               if (v9fs_proto_dotl(v9ses))
+                       inode->i_op = &v9fs_symlink_inode_operations_dotl;
+               else
+                       inode->i_op = &v9fs_symlink_inode_operations;
 
-       err = v9fs_t_create(v9ses, fid, name, perm, mode, &fcall);
-       if (err < 0) {
-               PRINT_FCALL_ERROR("create fails", fcall);
+               break;
+       case S_IFDIR:
+               inc_nlink(inode);
+               if (v9fs_proto_dotl(v9ses))
+                       inode->i_op = &v9fs_dir_inode_operations_dotl;
+               else if (v9fs_proto_dotu(v9ses))
+                       inode->i_op = &v9fs_dir_inode_operations_dotu;
+               else
+                       inode->i_op = &v9fs_dir_inode_operations;
+
+               if (v9fs_proto_dotl(v9ses))
+                       inode->i_fop = &v9fs_dir_operations_dotl;
+               else
+                       inode->i_fop = &v9fs_dir_operations;
+
+               break;
+       default:
+               P9_DPRINTK(P9_DEBUG_ERROR, "BAD mode 0x%x S_IFMT 0x%x\n",
+                          mode, mode & S_IFMT);
+               err = -EINVAL;
                goto error;
        }
 
-       if (iounit)
-               *iounit = fcall->params.rcreate.iounit;
-
-       if (qid)
-               *qid = fcall->params.rcreate.qid;
-
-       if (fidp)
-               *fidp = fid;
-
-       kfree(fcall);
-       return 0;
+       return inode;
 
 error:
-       if (fid >= 0)
-               v9fs_put_idpool(fid, &v9ses->fidpool);
-
-       kfree(fcall);
-       return err;
+       iput(inode);
+       return ERR_PTR(err);
 }
 
+/*
 static struct v9fs_fid*
 v9fs_clone_walk(struct v9fs_session_info *v9ses, u32 fid, struct dentry *dentry)
 {
        int err;
-       u32 nfid;
+       int nfid;
        struct v9fs_fid *ret;
        struct v9fs_fcall *fcall;
 
@@ -319,6 +350,9 @@ v9fs_clone_walk(struct v9fs_session_info *v9ses, u32 fid, struct dentry *dentry)
                &fcall);
 
        if (err < 0) {
+               if (fcall && fcall->id == RWALK)
+                       goto clunk_fid;
+
                PRINT_FCALL_ERROR("walk error", fcall);
                v9fs_put_idpool(nfid, &v9ses->fidpool);
                goto error;
@@ -347,39 +381,66 @@ error:
        kfree(fcall);
        return ERR_PTR(err);
 }
+*/
+
+
+/**
+ * v9fs_clear_inode - release an inode
+ * @inode: inode to release
+ *
+ */
+void v9fs_clear_inode(struct inode *inode)
+{
+       filemap_fdatawrite(inode->i_mapping);
+
+#ifdef CONFIG_9P_FSCACHE
+       v9fs_cache_inode_put_cookie(inode);
+#endif
+}
 
-struct inode *
-v9fs_inode_from_fid(struct v9fs_session_info *v9ses, u32 fid,
+/**
+ * v9fs_inode_from_fid - populate an inode by issuing a attribute request
+ * @v9ses: session information
+ * @fid: fid to issue attribute request for
+ * @sb: superblock on which to create inode
+ *
+ */
+
+static struct inode *
+v9fs_inode_from_fid(struct v9fs_session_info *v9ses, struct p9_fid *fid,
        struct super_block *sb)
 {
        int err, umode;
        struct inode *ret;
-       struct v9fs_fcall *fcall;
+       struct p9_wstat *st;
 
        ret = NULL;
-       err = v9fs_t_stat(v9ses, fid, &fcall);
-       if (err) {
-               PRINT_FCALL_ERROR("stat error", fcall);
-               goto error;
-       }
+       st = p9_client_stat(fid);
+       if (IS_ERR(st))
+               return ERR_CAST(st);
 
-       umode = p9mode2unixmode(v9ses, fcall->params.rstat.stat.mode);
+       umode = p9mode2unixmode(v9ses, st->mode);
        ret = v9fs_get_inode(sb, umode);
        if (IS_ERR(ret)) {
                err = PTR_ERR(ret);
-               ret = NULL;
                goto error;
        }
 
-       v9fs_stat2inode(&fcall->params.rstat.stat, ret, sb);
-       kfree(fcall);
+       v9fs_stat2inode(st, ret, sb);
+       ret->i_ino = v9fs_qid2ino(&st->qid);
+
+#ifdef CONFIG_9P_FSCACHE
+       v9fs_vcookie_set_qid(ret, &st->qid);
+       v9fs_cache_inode_get_cookie(ret);
+#endif
+       p9stat_free(st);
+       kfree(st);
+
        return ret;
 
 error:
-       kfree(fcall);
-       if (ret)
-               iput(ret);
-
+       p9stat_free(st);
+       kfree(st);
        return ERR_PTR(err);
 }
 
@@ -393,56 +454,119 @@ error:
 
 static int v9fs_remove(struct inode *dir, struct dentry *file, int rmdir)
 {
-       struct v9fs_fcall *fcall = NULL;
-       struct super_block *sb = NULL;
-       struct v9fs_session_info *v9ses = NULL;
-       struct v9fs_fid *v9fid = NULL;
-       struct inode *file_inode = NULL;
-       int fid = -1;
-       int result = 0;
+       int retval;
+       struct inode *file_inode;
+       struct p9_fid *v9fid;
 
-       dprintk(DEBUG_VFS, "inode: %p dentry: %p rmdir: %d\n", dir, file,
+       P9_DPRINTK(P9_DEBUG_VFS, "inode: %p dentry: %p rmdir: %d\n", dir, file,
                rmdir);
 
        file_inode = file->d_inode;
-       sb = file_inode->i_sb;
-       v9ses = v9fs_inode2v9ses(file_inode);
-       v9fid = v9fs_fid_lookup(file);
+       v9fid = v9fs_fid_clone(file);
+       if (IS_ERR(v9fid))
+               return PTR_ERR(v9fid);
 
-       if (!v9fid) {
-               dprintk(DEBUG_ERROR,
-                       "no v9fs_fid\n");
-               return -EBADF;
+       retval = p9_client_remove(v9fid);
+       if (!retval)
+               drop_nlink(file_inode);
+       return retval;
+}
+
+static int
+v9fs_open_created(struct inode *inode, struct file *file)
+{
+       return 0;
+}
+
+
+/**
+ * v9fs_create - Create a file
+ * @v9ses: session information
+ * @dir: directory that dentry is being created in
+ * @dentry:  dentry that is being created
+ * @extension: 9p2000.u extension string to support devices, etc.
+ * @perm: create permissions
+ * @mode: open mode
+ *
+ */
+static struct p9_fid *
+v9fs_create(struct v9fs_session_info *v9ses, struct inode *dir,
+               struct dentry *dentry, char *extension, u32 perm, u8 mode)
+{
+       int err;
+       char *name;
+       struct p9_fid *dfid, *ofid, *fid;
+       struct inode *inode;
+
+       P9_DPRINTK(P9_DEBUG_VFS, "name %s\n", dentry->d_name.name);
+
+       err = 0;
+       ofid = NULL;
+       fid = NULL;
+       name = (char *) dentry->d_name.name;
+       dfid = v9fs_fid_lookup(dentry->d_parent);
+       if (IS_ERR(dfid)) {
+               err = PTR_ERR(dfid);
+               P9_DPRINTK(P9_DEBUG_VFS, "fid lookup failed %d\n", err);
+               return ERR_PTR(err);
        }
 
-       fid = v9fid->fid;
-       if (fid < 0) {
-               dprintk(DEBUG_ERROR, "inode #%lu, no fid!\n",
-                       file_inode->i_ino);
-               return -EBADF;
+       /* clone a fid to use for creation */
+       ofid = p9_client_walk(dfid, 0, NULL, 1);
+       if (IS_ERR(ofid)) {
+               err = PTR_ERR(ofid);
+               P9_DPRINTK(P9_DEBUG_VFS, "p9_client_walk failed %d\n", err);
+               return ERR_PTR(err);
        }
 
-       result = v9fs_t_remove(v9ses, fid, &fcall);
-       if (result < 0) {
-               PRINT_FCALL_ERROR("remove fails", fcall);
-       } else {
-               v9fs_put_idpool(fid, &v9ses->fidpool);
-               v9fs_fid_destroy(v9fid);
+       err = p9_client_fcreate(ofid, name, perm, mode, extension);
+       if (err < 0) {
+               P9_DPRINTK(P9_DEBUG_VFS, "p9_client_fcreate failed %d\n", err);
+               goto error;
        }
 
-       kfree(fcall);
-       return result;
-}
+       /* now walk from the parent so we can get unopened fid */
+       fid = p9_client_walk(dfid, 1, &name, 1);
+       if (IS_ERR(fid)) {
+               err = PTR_ERR(fid);
+               P9_DPRINTK(P9_DEBUG_VFS, "p9_client_walk failed %d\n", err);
+               fid = NULL;
+               goto error;
+       }
 
-static int
-v9fs_open_created(struct inode *inode, struct file *file)
-{
-       return 0;
+       /* instantiate inode and assign the unopened fid to the dentry */
+       inode = v9fs_inode_from_fid(v9ses, fid, dir->i_sb);
+       if (IS_ERR(inode)) {
+               err = PTR_ERR(inode);
+               P9_DPRINTK(P9_DEBUG_VFS, "inode creation failed %d\n", err);
+               goto error;
+       }
+
+       if (v9ses->cache)
+               dentry->d_op = &v9fs_cached_dentry_operations;
+       else
+               dentry->d_op = &v9fs_dentry_operations;
+
+       d_instantiate(dentry, inode);
+       err = v9fs_fid_add(dentry, fid);
+       if (err < 0)
+               goto error;
+
+       return ofid;
+
+error:
+       if (ofid)
+               p9_client_clunk(ofid);
+
+       if (fid)
+               p9_client_clunk(fid);
+
+       return ERR_PTR(err);
 }
 
 /**
  * v9fs_vfs_create - VFS hook to create files
- * @inode: directory inode that is being deleted
+ * @dir: directory inode that is being created
  * @dentry:  dentry that is being deleted
  * @mode: create permissions
  * @nd: path information
@@ -454,82 +578,54 @@ v9fs_vfs_create(struct inode *dir, struct dentry *dentry, int mode,
                struct nameidata *nd)
 {
        int err;
-       u32 fid, perm, iounit;
+       u32 perm;
        int flags;
        struct v9fs_session_info *v9ses;
-       struct v9fs_fid *dfid, *vfid, *ffid;
-       struct inode *inode;
-       struct v9fs_qid qid;
+       struct p9_fid *fid;
        struct file *filp;
 
-       inode = NULL;
-       vfid = NULL;
+       err = 0;
+       fid = NULL;
        v9ses = v9fs_inode2v9ses(dir);
-       dfid = v9fs_fid_lookup(dentry->d_parent);
        perm = unixmode2p9mode(v9ses, mode);
-
        if (nd && nd->flags & LOOKUP_OPEN)
                flags = nd->intent.open.flags - 1;
        else
                flags = O_RDWR;
 
-       err = v9fs_create(v9ses, dfid->fid, (char *) dentry->d_name.name,
-               perm, v9fs_uflags2omode(flags), &fid, &qid, &iounit);
-
-       if (err)
-               goto error;
-
-       vfid = v9fs_clone_walk(v9ses, dfid->fid, dentry);
-       if (IS_ERR(vfid)) {
-               err = PTR_ERR(vfid);
-               vfid = NULL;
-               goto error;
-       }
-
-       inode = v9fs_inode_from_fid(v9ses, vfid->fid, dir->i_sb);
-       if (IS_ERR(inode)) {
-               err = PTR_ERR(inode);
-               inode = NULL;
+       fid = v9fs_create(v9ses, dir, dentry, NULL, perm,
+                               v9fs_uflags2omode(flags,
+                                               v9fs_proto_dotu(v9ses)));
+       if (IS_ERR(fid)) {
+               err = PTR_ERR(fid);
+               fid = NULL;
                goto error;
        }
 
-       dentry->d_op = &v9fs_dentry_operations;
-       d_instantiate(dentry, inode);
-
+       /* if we are opening a file, assign the open fid to the file */
        if (nd && nd->flags & LOOKUP_OPEN) {
-               ffid = v9fs_fid_create(v9ses, fid);
-               if (!ffid)
-                       return -ENOMEM;
-
                filp = lookup_instantiate_filp(nd, dentry, v9fs_open_created);
                if (IS_ERR(filp)) {
-                       v9fs_fid_destroy(ffid);
-                       return PTR_ERR(filp);
+                       err = PTR_ERR(filp);
+                       goto error;
                }
 
-               ffid->rdir_pos = 0;
-               ffid->rdir_fcall = NULL;
-               ffid->fidopen = 1;
-               ffid->iounit = iounit;
-               ffid->filp = filp;
-               filp->private_data = ffid;
-       }
+               filp->private_data = fid;
+       } else
+               p9_client_clunk(fid);
 
        return 0;
 
 error:
-       if (vfid)
-               v9fs_fid_destroy(vfid);
-
-       if (inode)
-               iput(inode);
+       if (fid)
+               p9_client_clunk(fid);
 
        return err;
 }
 
 /**
  * v9fs_vfs_mkdir - VFS mkdir hook to create a directory
- * @inode:  inode that is being unlinked
+ * @dir:  inode that is being unlinked
  * @dentry: dentry that is being unlinked
  * @mode: mode for new directory
  *
@@ -538,52 +634,22 @@ error:
 static int v9fs_vfs_mkdir(struct inode *dir, struct dentry *dentry, int mode)
 {
        int err;
-       u32 fid, perm;
+       u32 perm;
        struct v9fs_session_info *v9ses;
-       struct v9fs_fid *dfid, *vfid;
-       struct inode *inode;
+       struct p9_fid *fid;
 
-       inode = NULL;
-       vfid = NULL;
+       P9_DPRINTK(P9_DEBUG_VFS, "name %s\n", dentry->d_name.name);
+       err = 0;
        v9ses = v9fs_inode2v9ses(dir);
-       dfid = v9fs_fid_lookup(dentry->d_parent);
        perm = unixmode2p9mode(v9ses, mode | S_IFDIR);
-
-       err = v9fs_create(v9ses, dfid->fid, (char *) dentry->d_name.name,
-               perm, V9FS_OREAD, &fid, NULL, NULL);
-
-       if (err) {
-               dprintk(DEBUG_ERROR, "create error %d\n", err);
-               goto error;
-       }
-
-       err = v9fs_t_clunk(v9ses, fid);
-       if (err) {
-               dprintk(DEBUG_ERROR, "clunk error %d\n", err);
-               goto error;
-       }
-
-       vfid = v9fs_clone_walk(v9ses, dfid->fid, dentry);
-       if (IS_ERR(vfid)) {
-               err = PTR_ERR(vfid);
-               vfid = NULL;
-               goto error;
-       }
-
-       inode = v9fs_inode_from_fid(v9ses, vfid->fid, dir->i_sb);
-       if (IS_ERR(inode)) {
-               err = PTR_ERR(inode);
-               inode = NULL;
-               goto error;
+       fid = v9fs_create(v9ses, dir, dentry, NULL, perm, P9_OREAD);
+       if (IS_ERR(fid)) {
+               err = PTR_ERR(fid);
+               fid = NULL;
        }
 
-       dentry->d_op = &v9fs_dentry_operations;
-       d_instantiate(dentry, inode);
-       return 0;
-
-error:
-       if (vfid)
-               v9fs_fid_destroy(vfid);
+       if (fid)
+               p9_client_clunk(fid);
 
        return err;
 }
@@ -601,97 +667,58 @@ static struct dentry *v9fs_vfs_lookup(struct inode *dir, struct dentry *dentry,
 {
        struct super_block *sb;
        struct v9fs_session_info *v9ses;
-       struct v9fs_fid *dirfid;
-       struct v9fs_fid *fid;
+       struct p9_fid *dfid, *fid;
        struct inode *inode;
-       struct v9fs_fcall *fcall = NULL;
-       int dirfidnum = -1;
-       int newfid = -1;
+       char *name;
        int result = 0;
 
-       dprintk(DEBUG_VFS, "dir: %p dentry: (%s) %p nameidata: %p\n",
+       P9_DPRINTK(P9_DEBUG_VFS, "dir: %p dentry: (%s) %p nameidata: %p\n",
                dir, dentry->d_name.name, dentry, nameidata);
 
+       if (dentry->d_name.len > NAME_MAX)
+               return ERR_PTR(-ENAMETOOLONG);
+
        sb = dir->i_sb;
        v9ses = v9fs_inode2v9ses(dir);
-       dirfid = v9fs_fid_lookup(dentry->d_parent);
-
-       if (!dirfid) {
-               dprintk(DEBUG_ERROR, "no dirfid\n");
-               return ERR_PTR(-EINVAL);
-       }
-
-       dirfidnum = dirfid->fid;
-
-       if (dirfidnum < 0) {
-               dprintk(DEBUG_ERROR, "no dirfid for inode %p, #%lu\n",
-                       dir, dir->i_ino);
-               return ERR_PTR(-EBADF);
-       }
-
-       newfid = v9fs_get_idpool(&v9ses->fidpool);
-       if (newfid < 0) {
-               eprintk(KERN_WARNING, "newfid fails!\n");
-               return ERR_PTR(-ENOSPC);
-       }
+       dfid = v9fs_fid_lookup(dentry->d_parent);
+       if (IS_ERR(dfid))
+               return ERR_CAST(dfid);
 
-       result = v9fs_t_walk(v9ses, dirfidnum, newfid,
-               (char *)dentry->d_name.name, NULL);
-       if (result < 0) {
-               v9fs_put_idpool(newfid, &v9ses->fidpool);
+       name = (char *) dentry->d_name.name;
+       fid = p9_client_walk(dfid, 1, &name, 1);
+       if (IS_ERR(fid)) {
+               result = PTR_ERR(fid);
                if (result == -ENOENT) {
-                       d_add(dentry, NULL);
-                       dprintk(DEBUG_VFS,
-                               "Return negative dentry %p count %d\n",
-                               dentry, atomic_read(&dentry->d_count));
-                       return NULL;
+                       inode = NULL;
+                       goto inst_out;
                }
-               dprintk(DEBUG_ERROR, "walk error:%d\n", result);
-               goto FreeFcall;
-       }
-
-       result = v9fs_t_stat(v9ses, newfid, &fcall);
-       if (result < 0) {
-               dprintk(DEBUG_ERROR, "stat error\n");
-               goto FreeFcall;
-       }
-
-       inode = v9fs_get_inode(sb, p9mode2unixmode(v9ses,
-               fcall->params.rstat.stat.mode));
-
-       if (IS_ERR(inode) && (PTR_ERR(inode) == -ENOSPC)) {
-               eprintk(KERN_WARNING, "inode alloc failes, returns %ld\n",
-                       PTR_ERR(inode));
 
-               result = -ENOSPC;
-               goto FreeFcall;
+               return ERR_PTR(result);
        }
 
-       inode->i_ino = v9fs_qid2ino(&fcall->params.rstat.stat.qid);
-
-       fid = v9fs_fid_create(v9ses, newfid);
-       if (fid == NULL) {
-               dprintk(DEBUG_ERROR, "couldn't insert\n");
-               result = -ENOMEM;
-               goto FreeFcall;
+       inode = v9fs_inode_from_fid(v9ses, fid, dir->i_sb);
+       if (IS_ERR(inode)) {
+               result = PTR_ERR(inode);
+               inode = NULL;
+               goto error;
        }
 
-       result = v9fs_fid_insert(fid, dentry);
+       result = v9fs_fid_add(dentry, fid);
        if (result < 0)
-               goto FreeFcall;
-
-       fid->qid = fcall->params.rstat.stat.qid;
+               goto error;
 
-       dentry->d_op = &v9fs_dentry_operations;
-       v9fs_stat2inode(&fcall->params.rstat.stat, inode, inode->i_sb);
+inst_out:
+       if (v9ses->cache)
+               dentry->d_op = &v9fs_cached_dentry_operations;
+       else
+               dentry->d_op = &v9fs_dentry_operations;
 
        d_add(dentry, inode);
-       kfree(fcall);
-
        return NULL;
 
-      FreeFcall:
-       kfree(fcall);
+error:
+       p9_client_clunk(fid);
+
        return ERR_PTR(result);
 }
 
@@ -732,64 +759,69 @@ static int
 v9fs_vfs_rename(struct inode *old_dir, struct dentry *old_dentry,
                struct inode *new_dir, struct dentry *new_dentry)
 {
-       struct inode *old_inode = old_dentry->d_inode;
-       struct v9fs_session_info *v9ses = v9fs_inode2v9ses(old_inode);
-       struct v9fs_fid *oldfid = v9fs_fid_lookup(old_dentry);
-       struct v9fs_fid *olddirfid =
-           v9fs_fid_lookup(old_dentry->d_parent);
-       struct v9fs_fid *newdirfid =
-           v9fs_fid_lookup(new_dentry->d_parent);
-       struct v9fs_wstat wstat;
-       struct v9fs_fcall *fcall = NULL;
-       int fid = -1;
-       int olddirfidnum = -1;
-       int newdirfidnum = -1;
-       int retval = 0;
-
-       dprintk(DEBUG_VFS, "\n");
-
-       if ((!oldfid) || (!olddirfid) || (!newdirfid)) {
-               dprintk(DEBUG_ERROR, "problem with arguments\n");
-               return -EBADF;
+       struct inode *old_inode;
+       struct v9fs_session_info *v9ses;
+       struct p9_fid *oldfid;
+       struct p9_fid *olddirfid;
+       struct p9_fid *newdirfid;
+       struct p9_wstat wstat;
+       int retval;
+
+       P9_DPRINTK(P9_DEBUG_VFS, "\n");
+       retval = 0;
+       old_inode = old_dentry->d_inode;
+       v9ses = v9fs_inode2v9ses(old_inode);
+       oldfid = v9fs_fid_lookup(old_dentry);
+       if (IS_ERR(oldfid))
+               return PTR_ERR(oldfid);
+
+       olddirfid = v9fs_fid_clone(old_dentry->d_parent);
+       if (IS_ERR(olddirfid)) {
+               retval = PTR_ERR(olddirfid);
+               goto done;
        }
 
-       /* 9P can only handle file rename in the same directory */
-       if (memcmp(&olddirfid->qid, &newdirfid->qid, sizeof(newdirfid->qid))) {
-               dprintk(DEBUG_ERROR, "old dir and new dir are different\n");
-               retval = -EPERM;
-               goto FreeFcallnBail;
+       newdirfid = v9fs_fid_clone(new_dentry->d_parent);
+       if (IS_ERR(newdirfid)) {
+               retval = PTR_ERR(newdirfid);
+               goto clunk_olddir;
        }
 
-       fid = oldfid->fid;
-       olddirfidnum = olddirfid->fid;
-       newdirfidnum = newdirfid->fid;
+       if (v9fs_proto_dotl(v9ses)) {
+               retval = p9_client_rename(oldfid, newdirfid,
+                                       (char *) new_dentry->d_name.name);
+               if (retval != -ENOSYS)
+                       goto clunk_newdir;
+       }
 
-       if (fid < 0) {
-               dprintk(DEBUG_ERROR, "no fid for old file #%lu\n",
-                       old_inode->i_ino);
-               retval = -EBADF;
-               goto FreeFcallnBail;
+       /* 9P can only handle file rename in the same directory */
+       if (memcmp(&olddirfid->qid, &newdirfid->qid, sizeof(newdirfid->qid))) {
+               P9_DPRINTK(P9_DEBUG_ERROR,
+                               "old dir and new dir are different\n");
+               retval = -EXDEV;
+               goto clunk_newdir;
        }
 
        v9fs_blank_wstat(&wstat);
-       wstat.muid = v9ses->name;
+       wstat.muid = v9ses->uname;
        wstat.name = (char *) new_dentry->d_name.name;
+       retval = p9_client_wstat(oldfid, &wstat);
 
-       retval = v9fs_t_wstat(v9ses, fid, &wstat, &fcall);
+clunk_newdir:
+       p9_client_clunk(newdirfid);
 
-      FreeFcallnBail:
-       if (retval < 0)
-               PRINT_FCALL_ERROR("wstat error", fcall);
+clunk_olddir:
+       p9_client_clunk(olddirfid);
 
-       kfree(fcall);
+done:
        return retval;
 }
 
 /**
  * v9fs_vfs_getattr - retrieve file metadata
- * @mnt - mount information
- * @dentry - file to get attributes on
- * @stat - metadata structure to populate
+ * @mnt: mount information
+ * @dentry: file to get attributes on
+ * @stat: metadata structure to populate
  *
  */
 
@@ -797,30 +829,30 @@ static int
 v9fs_vfs_getattr(struct vfsmount *mnt, struct dentry *dentry,
                 struct kstat *stat)
 {
-       struct v9fs_fcall *fcall = NULL;
-       struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dentry->d_inode);
-       struct v9fs_fid *fid = v9fs_fid_lookup(dentry);
-       int err = -EPERM;
-
-       dprintk(DEBUG_VFS, "dentry: %p\n", dentry);
-       if (!fid) {
-               dprintk(DEBUG_ERROR,
-                       "couldn't find fid associated with dentry\n");
-               return -EBADF;
-       }
+       int err;
+       struct v9fs_session_info *v9ses;
+       struct p9_fid *fid;
+       struct p9_wstat *st;
 
-       err = v9fs_t_stat(v9ses, fid->fid, &fcall);
+       P9_DPRINTK(P9_DEBUG_VFS, "dentry: %p\n", dentry);
+       err = -EPERM;
+       v9ses = v9fs_inode2v9ses(dentry->d_inode);
+       if (v9ses->cache == CACHE_LOOSE || v9ses->cache == CACHE_FSCACHE)
+               return simple_getattr(mnt, dentry, stat);
 
-       if (err < 0)
-               dprintk(DEBUG_ERROR, "stat error\n");
-       else {
-               v9fs_stat2inode(&fcall->params.rstat.stat, dentry->d_inode,
-                                 dentry->d_inode->i_sb);
+       fid = v9fs_fid_lookup(dentry);
+       if (IS_ERR(fid))
+               return PTR_ERR(fid);
+
+       st = p9_client_stat(fid);
+       if (IS_ERR(st))
+               return PTR_ERR(st);
+
+       v9fs_stat2inode(st, dentry->d_inode, dentry->d_inode->i_sb);
                generic_fillattr(dentry->d_inode, stat);
-       }
 
-       kfree(fcall);
-       return err;
+       kfree(st);
+       return 0;
 }
 
 /**
@@ -832,19 +864,17 @@ v9fs_vfs_getattr(struct vfsmount *mnt, struct dentry *dentry,
 
 static int v9fs_vfs_setattr(struct dentry *dentry, struct iattr *iattr)
 {
-       struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dentry->d_inode);
-       struct v9fs_fid *fid = v9fs_fid_lookup(dentry);
-       struct v9fs_fcall *fcall = NULL;
-       struct v9fs_wstat wstat;
-       int res = -EPERM;
+       int retval;
+       struct v9fs_session_info *v9ses;
+       struct p9_fid *fid;
+       struct p9_wstat wstat;
 
-       dprintk(DEBUG_VFS, "\n");
-
-       if (!fid) {
-               dprintk(DEBUG_ERROR,
-                       "Couldn't find fid associated with dentry\n");
-               return -EBADF;
-       }
+       P9_DPRINTK(P9_DEBUG_VFS, "\n");
+       retval = -EPERM;
+       v9ses = v9fs_inode2v9ses(dentry->d_inode);
+       fid = v9fs_fid_lookup(dentry);
+       if(IS_ERR(fid))
+               return PTR_ERR(fid);
 
        v9fs_blank_wstat(&wstat);
        if (iattr->ia_valid & ATTR_MODE)
@@ -859,7 +889,7 @@ static int v9fs_vfs_setattr(struct dentry *dentry, struct iattr *iattr)
        if (iattr->ia_valid & ATTR_SIZE)
                wstat.length = iattr->ia_size;
 
-       if (v9ses->extended) {
+       if (v9fs_proto_dotu(v9ses)) {
                if (iattr->ia_valid & ATTR_UID)
                        wstat.n_uid = iattr->ia_uid;
 
@@ -867,16 +897,11 @@ static int v9fs_vfs_setattr(struct dentry *dentry, struct iattr *iattr)
                        wstat.n_gid = iattr->ia_gid;
        }
 
-       res = v9fs_t_wstat(v9ses, fid->fid, &wstat, &fcall);
-
-       if (res < 0)
-               PRINT_FCALL_ERROR("wstat error", fcall);
-
-       kfree(fcall);
-       if (res >= 0)
-               res = inode_setattr(dentry->d_inode, iattr);
+       retval = p9_client_wstat(fid, &wstat);
+       if (retval >= 0)
+               retval = inode_setattr(dentry->d_inode, iattr);
 
-       return res;
+       return retval;
 }
 
 /**
@@ -888,11 +913,12 @@ static int v9fs_vfs_setattr(struct dentry *dentry, struct iattr *iattr)
  */
 
 void
-v9fs_stat2inode(struct v9fs_stat *stat, struct inode *inode,
+v9fs_stat2inode(struct p9_wstat *stat, struct inode *inode,
        struct super_block *sb)
 {
-       int n;
        char ext[32];
+       char tag_name[14];
+       unsigned int i_nlink;
        struct v9fs_session_info *v9ses = sb->s_fs_info;
 
        inode->i_nlink = 1;
@@ -901,25 +927,36 @@ v9fs_stat2inode(struct v9fs_stat *stat, struct inode *inode,
        inode->i_mtime.tv_sec = stat->mtime;
        inode->i_ctime.tv_sec = stat->mtime;
 
-       inode->i_uid = v9ses->uid;
-       inode->i_gid = v9ses->gid;
+       inode->i_uid = v9ses->dfltuid;
+       inode->i_gid = v9ses->dfltgid;
 
-       if (v9ses->extended) {
+       if (v9fs_proto_dotu(v9ses)) {
                inode->i_uid = stat->n_uid;
                inode->i_gid = stat->n_gid;
        }
-
+       if ((S_ISREG(inode->i_mode)) || (S_ISDIR(inode->i_mode))) {
+               if (v9fs_proto_dotu(v9ses) && (stat->extension[0] != '\0')) {
+                       /*
+                        * Hadlink support got added later to
+                        * to the .u extension. So there can be
+                        * server out there that doesn't support
+                        * this even with .u extension. So check
+                        * for non NULL stat->extension
+                        */
+                       strncpy(ext, stat->extension, sizeof(ext));
+                       /* HARDLINKCOUNT %u */
+                       sscanf(ext, "%13s %u", tag_name, &i_nlink);
+                       if (!strncmp(tag_name, "HARDLINKCOUNT", 13))
+                               inode->i_nlink = i_nlink;
+               }
+       }
        inode->i_mode = p9mode2unixmode(v9ses, stat->mode);
        if ((S_ISBLK(inode->i_mode)) || (S_ISCHR(inode->i_mode))) {
                char type = 0;
                int major = -1;
                int minor = -1;
 
-               n = stat->extension.len;
-               if (n > sizeof(ext)-1)
-                       n = sizeof(ext)-1;
-               memmove(ext, stat->extension.str, n);
-               ext[n] = 0;
+               strncpy(ext, stat->extension, sizeof(ext));
                sscanf(ext, "%c %u %u", &type, &major, &minor);
                switch (type) {
                case 'c':
@@ -929,18 +966,19 @@ v9fs_stat2inode(struct v9fs_stat *stat, struct inode *inode,
                case 'b':
                        break;
                default:
-                       dprintk(DEBUG_ERROR, "Unknown special type %c (%.*s)\n",
-                               type, stat->extension.len, stat->extension.str);
+                       P9_DPRINTK(P9_DEBUG_ERROR,
+                               "Unknown special type %c %s\n", type,
+                               stat->extension);
                };
                inode->i_rdev = MKDEV(major, minor);
+               init_special_inode(inode, inode->i_mode, inode->i_rdev);
        } else
                inode->i_rdev = 0;
 
-       inode->i_size = stat->length;
+       i_size_write(inode, stat->length);
 
-       inode->i_blksize = sb->s_blocksize;
-       inode->i_blocks =
-           (inode->i_size + inode->i_blksize - 1) >> sb->s_blocksize_bits;
+       /* not real number of blocks, but 512 byte ones ... */
+       inode->i_blocks = (i_size_read(inode) + 512 - 1) >> 9;
 }
 
 /**
@@ -950,7 +988,7 @@ v9fs_stat2inode(struct v9fs_stat *stat, struct inode *inode,
  * BUG: potential for inode number collisions?
  */
 
-ino_t v9fs_qid2ino(struct v9fs_qid *qid)
+ino_t v9fs_qid2ino(struct p9_qid *qid)
 {
        u64 path = qid->path + 2;
        ino_t i = 0;
@@ -973,86 +1011,40 @@ ino_t v9fs_qid2ino(struct v9fs_qid *qid)
 
 static int v9fs_readlink(struct dentry *dentry, char *buffer, int buflen)
 {
-       int retval = -EPERM;
-
-       struct v9fs_fcall *fcall = NULL;
-       struct v9fs_session_info *v9ses = v9fs_inode2v9ses(dentry->d_inode);
-       struct v9fs_fid *fid = v9fs_fid_lookup(dentry);
-
-       if (!fid) {
-               dprintk(DEBUG_ERROR, "could not resolve fid from dentry\n");
-               retval = -EBADF;
-               goto FreeFcall;
-       }
+       int retval;
 
-       if (!v9ses->extended) {
-               retval = -EBADF;
-               dprintk(DEBUG_ERROR, "not extended\n");
-               goto FreeFcall;
-       }
+       struct v9fs_session_info *v9ses;
+       struct p9_fid *fid;
+       struct p9_wstat *st;
 
-       dprintk(DEBUG_VFS, " %s\n", dentry->d_name.name);
-       retval = v9fs_t_stat(v9ses, fid->fid, &fcall);
+       P9_DPRINTK(P9_DEBUG_VFS, " %s\n", dentry->d_name.name);
+       retval = -EPERM;
+       v9ses = v9fs_inode2v9ses(dentry->d_inode);
+       fid = v9fs_fid_lookup(dentry);
+       if (IS_ERR(fid))
+               return PTR_ERR(fid);
 
-       if (retval < 0) {
-               dprintk(DEBUG_ERROR, "stat error\n");
-               goto FreeFcall;
-       }
+       if (!v9fs_proto_dotu(v9ses))
+               return -EBADF;
 
-       if (!fcall)
-               return -EIO;
+       st = p9_client_stat(fid);
+       if (IS_ERR(st))
+               return PTR_ERR(st);
 
-       if (!(fcall->params.rstat.stat.mode & V9FS_DMSYMLINK)) {
+       if (!(st->mode & P9_DMSYMLINK)) {
                retval = -EINVAL;
-               goto FreeFcall;
+               goto done;
        }
 
        /* copy extension buffer into buffer */
-       if (fcall->params.rstat.stat.extension.len < buflen)
-               buflen = fcall->params.rstat.stat.extension.len;
-
-       memcpy(buffer, fcall->params.rstat.stat.extension.str, buflen - 1);
-       buffer[buflen-1] = 0;
+       strncpy(buffer, st->extension, buflen);
 
-       retval = buflen;
-
-      FreeFcall:
-       kfree(fcall);
+       P9_DPRINTK(P9_DEBUG_VFS,
+               "%s -> %s (%s)\n", dentry->d_name.name, st->extension, buffer);
 
-       return retval;
-}
-
-/**
- * v9fs_vfs_readlink - read a symlink's location
- * @dentry: dentry for symlink
- * @buf: buffer to load symlink location into
- * @buflen: length of buffer
- *
- */
-
-static int v9fs_vfs_readlink(struct dentry *dentry, char __user * buffer,
-                            int buflen)
-{
-       int retval;
-       int ret;
-       char *link = __getname();
-
-       if (buflen > PATH_MAX)
-               buflen = PATH_MAX;
-
-       dprintk(DEBUG_VFS, " dentry: %s (%p)\n", dentry->d_iname, dentry);
-
-       retval = v9fs_readlink(dentry, link, buflen);
-
-       if (retval > 0) {
-               if ((ret = copy_to_user(buffer, link, retval)) != 0) {
-                       dprintk(DEBUG_ERROR, "problem copying to user: %d\n",
-                               ret);
-                       retval = ret;
-               }
-       }
-
-       __putname(link);
+       retval = strnlen(buffer, buflen);
+done:
+       kfree(st);
        return retval;
 }
 
@@ -1068,18 +1060,18 @@ static void *v9fs_vfs_follow_link(struct dentry *dentry, struct nameidata *nd)
        int len = 0;
        char *link = __getname();
 
-       dprintk(DEBUG_VFS, "%s n", dentry->d_name.name);
+       P9_DPRINTK(P9_DEBUG_VFS, "%s n", dentry->d_name.name);
 
        if (!link)
                link = ERR_PTR(-ENOMEM);
        else {
-               len = v9fs_readlink(dentry, link, strlen(link));
+               len = v9fs_readlink(dentry, link, PATH_MAX);
 
                if (len < 0) {
                        __putname(link);
                        link = ERR_PTR(len);
                } else
-                       link[len] = 0;
+                       link[min(len, PATH_MAX-1)] = 0;
        }
        nd_set_link(nd, link);
 
@@ -1090,90 +1082,51 @@ static void *v9fs_vfs_follow_link(struct dentry *dentry, struct nameidata *nd)
  * v9fs_vfs_put_link - release a symlink path
  * @dentry: dentry for symlink
  * @nd: nameidata
+ * @p: unused
  *
  */
 
-static void v9fs_vfs_put_link(struct dentry *dentry, struct nameidata *nd, void *p)
+static void
+v9fs_vfs_put_link(struct dentry *dentry, struct nameidata *nd, void *p)
 {
        char *s = nd_get_link(nd);
 
-       dprintk(DEBUG_VFS, " %s %s\n", dentry->d_name.name, s);
+       P9_DPRINTK(P9_DEBUG_VFS, " %s %s\n", dentry->d_name.name,
+               IS_ERR(s) ? "<error>" : s);
        if (!IS_ERR(s))
                __putname(s);
 }
 
+/**
+ * v9fs_vfs_mkspecial - create a special file
+ * @dir: inode to create special file in
+ * @dentry: dentry to create
+ * @mode: mode to create special file
+ * @extension: 9p2000.u format extension string representing special file
+ *
+ */
+
 static int v9fs_vfs_mkspecial(struct inode *dir, struct dentry *dentry,
        int mode, const char *extension)
 {
-       int err;
-       u32 fid, perm;
+       u32 perm;
        struct v9fs_session_info *v9ses;
-       struct v9fs_fid *dfid, *vfid;
-       struct inode *inode;
-       struct v9fs_fcall *fcall;
-       struct v9fs_wstat wstat;
+       struct p9_fid *fid;
 
-       fcall = NULL;
-       inode = NULL;
-       vfid = NULL;
        v9ses = v9fs_inode2v9ses(dir);
-       dfid = v9fs_fid_lookup(dentry->d_parent);
-       perm = unixmode2p9mode(v9ses, mode);
-
-       if (!v9ses->extended) {
-               dprintk(DEBUG_ERROR, "not extended\n");
+       if (!v9fs_proto_dotu(v9ses)) {
+               P9_DPRINTK(P9_DEBUG_ERROR, "not extended\n");
                return -EPERM;
        }
 
-       err = v9fs_create(v9ses, dfid->fid, (char *) dentry->d_name.name,
-               perm, V9FS_OREAD, &fid, NULL, NULL);
-
-       if (err)
-               goto error;
-
-       err = v9fs_t_clunk(v9ses, fid);
-       if (err)
-               goto error;
-
-       vfid = v9fs_clone_walk(v9ses, dfid->fid, dentry);
-       if (IS_ERR(vfid)) {
-               err = PTR_ERR(vfid);
-               vfid = NULL;
-               goto error;
-       }
-
-       inode = v9fs_inode_from_fid(v9ses, vfid->fid, dir->i_sb);
-       if (IS_ERR(inode)) {
-               err = PTR_ERR(inode);
-               inode = NULL;
-               goto error;
-       }
-
-       /* issue a Twstat */
-       v9fs_blank_wstat(&wstat);
-       wstat.muid = v9ses->name;
-       wstat.extension = (char *) extension;
-       err = v9fs_t_wstat(v9ses, vfid->fid, &wstat, &fcall);
-       if (err < 0) {
-               PRINT_FCALL_ERROR("wstat error", fcall);
-               goto error;
-       }
+       perm = unixmode2p9mode(v9ses, mode);
+       fid = v9fs_create(v9ses, dir, dentry, (char *) extension, perm,
+                                                               P9_OREAD);
+       if (IS_ERR(fid))
+               return PTR_ERR(fid);
 
-       kfree(fcall);
-       dentry->d_op = &v9fs_dentry_operations;
-       d_instantiate(dentry, inode);
+       p9_client_clunk(fid);
        return 0;
-
-error:
-       kfree(fcall);
-       if (vfid)
-               v9fs_fid_destroy(vfid);
-
-       if (inode)
-               iput(inode);
-
-       return err;
-
 }
 
 /**
@@ -1182,15 +1135,15 @@ error:
  * @dentry: dentry for symlink
  * @symname: symlink data
  *
- * See 9P2000.u RFC for more information
+ * See Also: 9P2000.u RFC for more information
  *
  */
 
 static int
 v9fs_vfs_symlink(struct inode *dir, struct dentry *dentry, const char *symname)
 {
-       dprintk(DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name,
-               symname);
+       P9_DPRINTK(P9_DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino,
+                                       dentry->d_name.name, symname);
 
        return v9fs_vfs_mkspecial(dir, dentry, S_IFLNK, symname);
 }
@@ -1203,32 +1156,34 @@ v9fs_vfs_symlink(struct inode *dir, struct dentry *dentry, const char *symname)
  *
  */
 
-/* XXX - lots of code dup'd from symlink and creates,
- * figure out a better reuse strategy
- */
-
 static int
 v9fs_vfs_link(struct dentry *old_dentry, struct inode *dir,
              struct dentry *dentry)
 {
        int retval;
-       struct v9fs_fid *oldfid;
+       struct p9_fid *oldfid;
        char *name;
 
-       dprintk(DEBUG_VFS, " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name,
+       P9_DPRINTK(P9_DEBUG_VFS,
+               " %lu,%s,%s\n", dir->i_ino, dentry->d_name.name,
                old_dentry->d_name.name);
 
-       oldfid = v9fs_fid_lookup(old_dentry);
-       if (!oldfid) {
-               dprintk(DEBUG_ERROR, "can't find oldfid\n");
-               return -EPERM;
-       }
+       oldfid = v9fs_fid_clone(old_dentry);
+       if (IS_ERR(oldfid))
+               return PTR_ERR(oldfid);
 
        name = __getname();
-       sprintf(name, "hardlink(%d)\n", oldfid->fid);
-       retval = v9fs_vfs_mkspecial(dir, dentry, V9FS_DMLINK, name);
+       if (unlikely(!name)) {
+               retval = -ENOMEM;
+               goto clunk_fid;
+       }
+
+       sprintf(name, "%d\n", oldfid->fid);
+       retval = v9fs_vfs_mkspecial(dir, dentry, P9_DMLINK, name);
        __putname(name);
 
+clunk_fid:
+       p9_client_clunk(oldfid);
        return retval;
 }
 
@@ -1237,7 +1192,7 @@ v9fs_vfs_link(struct dentry *old_dentry, struct inode *dir,
  * @dir: inode destination for new link
  * @dentry: dentry for file
  * @mode: mode for creation
- * @dev_t: device associated with special file
+ * @rdev: device associated with special file
  *
  */
 
@@ -1247,13 +1202,16 @@ v9fs_vfs_mknod(struct inode *dir, struct dentry *dentry, int mode, dev_t rdev)
        int retval;
        char *name;
 
-       dprintk(DEBUG_VFS, " %lu,%s mode: %x MAJOR: %u MINOR: %u\n", dir->i_ino,
+       P9_DPRINTK(P9_DEBUG_VFS,
+               " %lu,%s mode: %x MAJOR: %u MINOR: %u\n", dir->i_ino,
                dentry->d_name.name, mode, MAJOR(rdev), MINOR(rdev));
 
        if (!new_valid_dev(rdev))
                return -EINVAL;
 
        name = __getname();
+       if (!name)
+               return -ENOMEM;
        /* build extension */
        if (S_ISBLK(mode))
                sprintf(name, "b %u %u", MAJOR(rdev), MINOR(rdev));
@@ -1261,6 +1219,8 @@ v9fs_vfs_mknod(struct inode *dir, struct dentry *dentry, int mode, dev_t rdev)
                sprintf(name, "c %u %u", MAJOR(rdev), MINOR(rdev));
        else if (S_ISFIFO(mode))
                *name = 0;
+       else if (S_ISSOCK(mode))
+               *name = 0;
        else {
                __putname(name);
                return -EINVAL;
@@ -1272,7 +1232,7 @@ v9fs_vfs_mknod(struct inode *dir, struct dentry *dentry, int mode, dev_t rdev)
        return retval;
 }
 
-static struct inode_operations v9fs_dir_inode_operations_ext = {
+static const struct inode_operations v9fs_dir_inode_operations_dotu = {
        .create = v9fs_vfs_create,
        .lookup = v9fs_vfs_lookup,
        .symlink = v9fs_vfs_symlink,
@@ -1282,14 +1242,15 @@ static struct inode_operations v9fs_dir_inode_operations_ext = {
        .rmdir = v9fs_vfs_rmdir,
        .mknod = v9fs_vfs_mknod,
        .rename = v9fs_vfs_rename,
-       .readlink = v9fs_vfs_readlink,
        .getattr = v9fs_vfs_getattr,
        .setattr = v9fs_vfs_setattr,
 };
 
-static struct inode_operations v9fs_dir_inode_operations = {
+static const struct inode_operations v9fs_dir_inode_operations_dotl = {
        .create = v9fs_vfs_create,
        .lookup = v9fs_vfs_lookup,
+       .symlink = v9fs_vfs_symlink,
+       .link = v9fs_vfs_link,
        .unlink = v9fs_vfs_unlink,
        .mkdir = v9fs_vfs_mkdir,
        .rmdir = v9fs_vfs_rmdir,
@@ -1299,13 +1260,38 @@ static struct inode_operations v9fs_dir_inode_operations = {
        .setattr = v9fs_vfs_setattr,
 };
 
-static struct inode_operations v9fs_file_inode_operations = {
+static const struct inode_operations v9fs_dir_inode_operations = {
+       .create = v9fs_vfs_create,
+       .lookup = v9fs_vfs_lookup,
+       .unlink = v9fs_vfs_unlink,
+       .mkdir = v9fs_vfs_mkdir,
+       .rmdir = v9fs_vfs_rmdir,
+       .mknod = v9fs_vfs_mknod,
+       .rename = v9fs_vfs_rename,
+       .getattr = v9fs_vfs_getattr,
+       .setattr = v9fs_vfs_setattr,
+};
+
+static const struct inode_operations v9fs_file_inode_operations = {
+       .getattr = v9fs_vfs_getattr,
+       .setattr = v9fs_vfs_setattr,
+};
+
+static const struct inode_operations v9fs_file_inode_operations_dotl = {
+       .getattr = v9fs_vfs_getattr,
+       .setattr = v9fs_vfs_setattr,
+};
+
+static const struct inode_operations v9fs_symlink_inode_operations = {
+       .readlink = generic_readlink,
+       .follow_link = v9fs_vfs_follow_link,
+       .put_link = v9fs_vfs_put_link,
        .getattr = v9fs_vfs_getattr,
        .setattr = v9fs_vfs_setattr,
 };
 
-static struct inode_operations v9fs_symlink_inode_operations = {
-       .readlink = v9fs_vfs_readlink,
+static const struct inode_operations v9fs_symlink_inode_operations_dotl = {
+       .readlink = generic_readlink,
        .follow_link = v9fs_vfs_follow_link,
        .put_link = v9fs_vfs_put_link,
        .getattr = v9fs_vfs_getattr,