]> err.no Git - linux-2.6/blobdiff - fs/xfs/linux-2.6/xfs_iops.c
Merge master.kernel.org:/home/rmk/linux-2.6-arm
[linux-2.6] / fs / xfs / linux-2.6 / xfs_iops.c
index 93b9e6e43f20a105661d659943719768af8cd8e5..af487437bd7e7e6c0de33b94389a567e0ce93b7f 100644 (file)
@@ -106,7 +106,7 @@ xfs_ichgtime(
        xfs_inode_t     *ip,
        int             flags)
 {
-       struct inode    *inode = LINVFS_GET_IP(XFS_ITOV(ip));
+       struct inode    *inode = vn_to_inode(XFS_ITOV(ip));
        timespec_t      tv;
 
        nanotime(&tv);
@@ -202,7 +202,7 @@ xfs_validate_fields(
        struct inode    *ip,
        struct vattr    *vattr)
 {
-       vnode_t         *vp = LINVFS_GET_VP(ip);
+       vnode_t         *vp = vn_from_inode(ip);
        int             error;
 
        vattr->va_mask = XFS_AT_NLINK|XFS_AT_SIZE|XFS_AT_NBLOCKS;
@@ -228,7 +228,7 @@ xfs_init_security(
        struct vnode    *vp,
        struct inode    *dir)
 {
-       struct inode    *ip = LINVFS_GET_IP(vp);
+       struct inode    *ip = vn_to_inode(vp);
        size_t          length;
        void            *value;
        char            *name;
@@ -277,7 +277,7 @@ xfs_cleanup_inode(
         * xfs_init_security we must back out.
         * ENOSPC can hit here, among other things.
         */
-       teardown.d_inode = LINVFS_GET_IP(vp);
+       teardown.d_inode = vn_to_inode(vp);
        teardown.d_name = dentry->d_name;
 
        if (S_ISDIR(mode))
@@ -295,8 +295,8 @@ xfs_vn_mknod(
        dev_t           rdev)
 {
        struct inode    *ip;
-       vattr_t         *vattr;
-       vnode_t         *vp = NULL, *dvp = LINVFS_GET_VP(dir);
+       vattr_t         vattr = { 0 };
+       vnode_t         *vp = NULL, *dvp = vn_from_inode(dir);
        xfs_acl_t       *default_acl = NULL;
        attrexists_t    test_default_acl = _ACL_DEFAULT_EXISTS;
        int             error;
@@ -308,13 +308,8 @@ xfs_vn_mknod(
        if (unlikely(!sysv_valid_dev(rdev) || MAJOR(rdev) & ~0x1ff))
                return -EINVAL;
 
-       vattr = kmalloc(sizeof(*vattr), GFP_KERNEL);
-       if (unlikely(!vattr))
-               return -ENOMEM;
-
        if (unlikely(test_default_acl && test_default_acl(dvp))) {
                if (!_ACL_ALLOC(default_acl)) {
-                       kfree(vattr);
                        return -ENOMEM;
                }
                if (!_ACL_GET_DEFAULT(dvp, default_acl)) {
@@ -326,20 +321,19 @@ xfs_vn_mknod(
        if (IS_POSIXACL(dir) && !default_acl && xfs_has_fs_struct(current))
                mode &= ~current->fs->umask;
 
-       memset(vattr, 0, sizeof(*vattr));
-       vattr->va_mask = XFS_AT_TYPE|XFS_AT_MODE;
-       vattr->va_mode = mode;
+       vattr.va_mask = XFS_AT_TYPE|XFS_AT_MODE;
+       vattr.va_mode = mode;
 
        switch (mode & S_IFMT) {
        case S_IFCHR: case S_IFBLK: case S_IFIFO: case S_IFSOCK:
-               vattr->va_rdev = sysv_encode_dev(rdev);
-               vattr->va_mask |= XFS_AT_RDEV;
+               vattr.va_rdev = sysv_encode_dev(rdev);
+               vattr.va_mask |= XFS_AT_RDEV;
                /*FALLTHROUGH*/
        case S_IFREG:
-               VOP_CREATE(dvp, dentry, vattr, &vp, NULL, error);
+               VOP_CREATE(dvp, dentry, &vattr, &vp, NULL, error);
                break;
        case S_IFDIR:
-               VOP_MKDIR(dvp, dentry, vattr, &vp, NULL, error);
+               VOP_MKDIR(dvp, dentry, &vattr, &vp, NULL, error);
                break;
        default:
                error = EINVAL;
@@ -354,7 +348,7 @@ xfs_vn_mknod(
 
        if (unlikely(default_acl)) {
                if (!error) {
-                       error = _ACL_INHERIT(vp, vattr, default_acl);
+                       error = _ACL_INHERIT(vp, &vattr, default_acl);
                        if (!error)
                                VMODIFY(vp);
                        else
@@ -365,16 +359,15 @@ xfs_vn_mknod(
 
        if (likely(!error)) {
                ASSERT(vp);
-               ip = LINVFS_GET_IP(vp);
+               ip = vn_to_inode(vp);
 
                if (S_ISCHR(mode) || S_ISBLK(mode))
                        ip->i_rdev = rdev;
                else if (S_ISDIR(mode))
-                       xfs_validate_fields(ip, vattr);
+                       xfs_validate_fields(ip, &vattr);
                d_instantiate(dentry, ip);
-               xfs_validate_fields(dir, vattr);
+               xfs_validate_fields(dir, &vattr);
        }
-       kfree(vattr);
        return -error;
 }
 
@@ -403,7 +396,7 @@ xfs_vn_lookup(
        struct dentry   *dentry,
        struct nameidata *nd)
 {
-       struct vnode    *vp = LINVFS_GET_VP(dir), *cvp;
+       struct vnode    *vp = vn_from_inode(dir), *cvp;
        int             error;
 
        if (dentry->d_name.len >= MAXNAMELEN)
@@ -417,7 +410,7 @@ xfs_vn_lookup(
                return NULL;
        }
 
-       return d_splice_alias(LINVFS_GET_IP(cvp), dentry);
+       return d_splice_alias(vn_to_inode(cvp), dentry);
 }
 
 STATIC int
@@ -429,28 +422,23 @@ xfs_vn_link(
        struct inode    *ip;    /* inode of guy being linked to */
        vnode_t         *tdvp;  /* target directory for new name/link */
        vnode_t         *vp;    /* vp of name being linked */
-       vattr_t         *vattr;
+       vattr_t         vattr;
        int             error;
 
        ip = old_dentry->d_inode;       /* inode being linked to */
        if (S_ISDIR(ip->i_mode))
                return -EPERM;
 
-       vattr = kmalloc(sizeof(*vattr), GFP_KERNEL);
-       if (unlikely(!vattr))
-               return -ENOMEM;
-
-       tdvp = LINVFS_GET_VP(dir);
-       vp = LINVFS_GET_VP(ip);
+       tdvp = vn_from_inode(dir);
+       vp = vn_from_inode(ip);
 
        VOP_LINK(tdvp, vp, dentry, NULL, error);
        if (likely(!error)) {
                VMODIFY(tdvp);
                VN_HOLD(vp);
-               xfs_validate_fields(ip, vattr);
+               xfs_validate_fields(ip, &vattr);
                d_instantiate(dentry, ip);
        }
-       kfree(vattr);
        return -error;
 }
 
@@ -461,22 +449,17 @@ xfs_vn_unlink(
 {
        struct inode    *inode;
        vnode_t         *dvp;   /* directory containing name to remove */
-       vattr_t         *vattr;
+       vattr_t         vattr;
        int             error;
 
-       vattr = kmalloc(sizeof(*vattr), GFP_KERNEL);
-       if (unlikely(!vattr))
-               return -ENOMEM;
-
        inode = dentry->d_inode;
-       dvp = LINVFS_GET_VP(dir);
+       dvp = vn_from_inode(dir);
 
        VOP_REMOVE(dvp, dentry, NULL, error);
        if (likely(!error)) {
-               xfs_validate_fields(dir, vattr);        /* size needs update */
-               xfs_validate_fields(inode, vattr);
+               xfs_validate_fields(dir, &vattr);       /* size needs update */
+               xfs_validate_fields(inode, &vattr);
        }
-       kfree(vattr);
        return -error;
 }
 
@@ -487,35 +470,31 @@ xfs_vn_symlink(
        const char      *symname)
 {
        struct inode    *ip;
-       vattr_t         *vattr;
+       vattr_t         vattr = { 0 };
        vnode_t         *dvp;   /* directory containing name of symlink */
        vnode_t         *cvp;   /* used to lookup symlink to put in dentry */
        int             error;
 
-       dvp = LINVFS_GET_VP(dir);
+       dvp = vn_from_inode(dir);
        cvp = NULL;
 
-       vattr = kmalloc(sizeof(*vattr), GFP_KERNEL);
-       if (unlikely(!vattr))
-               return -ENOMEM;
-
-       memset(vattr, 0, sizeof(*vattr));
-       vattr->va_mode = S_IFLNK |
+       vattr.va_mode = S_IFLNK |
                (irix_symlink_mode ? 0777 & ~current->fs->umask : S_IRWXUGO);
-       vattr->va_mask = XFS_AT_TYPE|XFS_AT_MODE;
+       vattr.va_mask = XFS_AT_TYPE|XFS_AT_MODE;
 
        error = 0;
-       VOP_SYMLINK(dvp, dentry, vattr, (char *)symname, &cvp, NULL, error);
+       VOP_SYMLINK(dvp, dentry, &vattr, (char *)symname, &cvp, NULL, error);
        if (likely(!error && cvp)) {
                error = xfs_init_security(cvp, dir);
                if (likely(!error)) {
-                       ip = LINVFS_GET_IP(cvp);
+                       ip = vn_to_inode(cvp);
                        d_instantiate(dentry, ip);
-                       xfs_validate_fields(dir, vattr);
-                       xfs_validate_fields(ip, vattr);
+                       xfs_validate_fields(dir, &vattr);
+                       xfs_validate_fields(ip, &vattr);
+               } else {
+                       xfs_cleanup_inode(dvp, cvp, dentry, 0);
                }
        }
-       kfree(vattr);
        return -error;
 }
 
@@ -525,20 +504,15 @@ xfs_vn_rmdir(
        struct dentry   *dentry)
 {
        struct inode    *inode = dentry->d_inode;
-       vnode_t         *dvp = LINVFS_GET_VP(dir);
-       vattr_t         *vattr;
+       vnode_t         *dvp = vn_from_inode(dir);
+       vattr_t         vattr;
        int             error;
 
-       vattr = kmalloc(sizeof(*vattr), GFP_KERNEL);
-       if (unlikely(!vattr))
-               return -ENOMEM;
-
        VOP_RMDIR(dvp, dentry, NULL, error);
        if (likely(!error)) {
-               xfs_validate_fields(inode, vattr);
-               xfs_validate_fields(dir, vattr);
+               xfs_validate_fields(inode, &vattr);
+               xfs_validate_fields(dir, &vattr);
        }
-       kfree(vattr);
        return -error;
 }
 
@@ -552,25 +526,20 @@ xfs_vn_rename(
        struct inode    *new_inode = ndentry->d_inode;
        vnode_t         *fvp;   /* from directory */
        vnode_t         *tvp;   /* target directory */
-       vattr_t         *vattr;
+       vattr_t         vattr;
        int             error;
 
-       vattr = kmalloc(sizeof(*vattr), GFP_KERNEL);
-       if (unlikely(!vattr))
-               return -ENOMEM;
-
-       fvp = LINVFS_GET_VP(odir);
-       tvp = LINVFS_GET_VP(ndir);
+       fvp = vn_from_inode(odir);
+       tvp = vn_from_inode(ndir);
 
        VOP_RENAME(fvp, odentry, tvp, ndentry, NULL, error);
        if (likely(!error)) {
                if (new_inode)
-                       xfs_validate_fields(new_inode, vattr);
-               xfs_validate_fields(odir, vattr);
+                       xfs_validate_fields(new_inode, &vattr);
+               xfs_validate_fields(odir, &vattr);
                if (ndir != odir)
-                       xfs_validate_fields(ndir, vattr);
+                       xfs_validate_fields(ndir, &vattr);
        }
-       kfree(vattr);
        return -error;
 }
 
@@ -606,7 +575,7 @@ xfs_vn_follow_link(
                return NULL;
        }
 
-       vp = LINVFS_GET_VP(dentry->d_inode);
+       vp = vn_from_inode(dentry->d_inode);
 
        iov.iov_base = link;
        iov.iov_len = MAXPATHLEN;
@@ -649,7 +618,7 @@ xfs_vn_permission(
        int             mode,
        struct nameidata *nd)
 {
-       vnode_t         *vp = LINVFS_GET_VP(inode);
+       vnode_t         *vp = vn_from_inode(inode);
        int             error;
 
        mode <<= 6;             /* convert from linux to vnode access bits */
@@ -667,7 +636,7 @@ xfs_vn_getattr(
        struct kstat    *stat)
 {
        struct inode    *inode = dentry->d_inode;
-       vnode_t         *vp = LINVFS_GET_VP(inode);
+       vnode_t         *vp = vn_from_inode(inode);
        int             error = 0;
 
        if (unlikely(vp->v_flag & VMODIFIED))
@@ -684,7 +653,7 @@ xfs_vn_setattr(
 {
        struct inode    *inode = dentry->d_inode;
        unsigned int    ia_valid = attr->ia_valid;
-       vnode_t         *vp = LINVFS_GET_VP(inode);
+       vnode_t         *vp = vn_from_inode(inode);
        vattr_t         vattr = { 0 };
        int             flags = 0;
        int             error;
@@ -750,7 +719,7 @@ xfs_vn_setxattr(
        size_t          size,
        int             flags)
 {
-       vnode_t         *vp = LINVFS_GET_VP(dentry->d_inode);
+       vnode_t         *vp = vn_from_inode(dentry->d_inode);
        char            *attr = (char *)name;
        attrnames_t     *namesp;
        int             xflags = 0;
@@ -780,7 +749,7 @@ xfs_vn_getxattr(
        void            *data,
        size_t          size)
 {
-       vnode_t         *vp = LINVFS_GET_VP(dentry->d_inode);
+       vnode_t         *vp = vn_from_inode(dentry->d_inode);
        char            *attr = (char *)name;
        attrnames_t     *namesp;
        int             xflags = 0;
@@ -809,7 +778,7 @@ xfs_vn_listxattr(
        char                    *data,
        size_t                  size)
 {
-       vnode_t                 *vp = LINVFS_GET_VP(dentry->d_inode);
+       vnode_t                 *vp = vn_from_inode(dentry->d_inode);
        int                     error, xflags = ATTR_KERNAMELS;
        ssize_t                 result;
 
@@ -828,7 +797,7 @@ xfs_vn_removexattr(
        struct dentry   *dentry,
        const char      *name)
 {
-       vnode_t         *vp = LINVFS_GET_VP(dentry->d_inode);
+       vnode_t         *vp = vn_from_inode(dentry->d_inode);
        char            *attr = (char *)name;
        attrnames_t     *namesp;
        int             xflags = 0;