Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs
[cascardo/linux.git] / fs / xfs / xfs_inode.c
index 8825bcf..624e1df 100644 (file)
@@ -25,6 +25,7 @@
 #include "xfs_trans_resv.h"
 #include "xfs_sb.h"
 #include "xfs_mount.h"
+#include "xfs_defer.h"
 #include "xfs_inode.h"
 #include "xfs_da_format.h"
 #include "xfs_da_btree.h"
@@ -820,7 +821,7 @@ xfs_ialloc(
        ip->i_d.di_nextents = 0;
        ASSERT(ip->i_d.di_nblocks == 0);
 
-       tv = current_fs_time(mp->m_super);
+       tv = current_time(inode);
        inode->i_mtime = tv;
        inode->i_atime = tv;
        inode->i_ctime = tv;
@@ -1122,7 +1123,7 @@ xfs_create(
        struct xfs_inode        *ip = NULL;
        struct xfs_trans        *tp = NULL;
        int                     error;
-       xfs_bmap_free_t         free_list;
+       struct xfs_defer_ops    dfops;
        xfs_fsblock_t           first_block;
        bool                    unlock_dp_on_error = false;
        prid_t                  prid;
@@ -1182,7 +1183,7 @@ xfs_create(
                      XFS_IOLOCK_PARENT | XFS_ILOCK_PARENT);
        unlock_dp_on_error = true;
 
-       xfs_bmap_init(&free_list, &first_block);
+       xfs_defer_init(&dfops, &first_block);
 
        /*
         * Reserve disk quota and the inode.
@@ -1219,7 +1220,7 @@ xfs_create(
        unlock_dp_on_error = false;
 
        error = xfs_dir_createname(tp, dp, name, ip->i_ino,
-                                       &first_block, &free_list, resblks ?
+                                       &first_block, &dfops, resblks ?
                                        resblks - XFS_IALLOC_SPACE_RES(mp) : 0);
        if (error) {
                ASSERT(error != -ENOSPC);
@@ -1253,7 +1254,7 @@ xfs_create(
         */
        xfs_qm_vop_create_dqattach(tp, ip, udqp, gdqp, pdqp);
 
-       error = xfs_bmap_finish(&tp, &free_list, NULL);
+       error = xfs_defer_finish(&tp, &dfops, NULL);
        if (error)
                goto out_bmap_cancel;
 
@@ -1269,7 +1270,7 @@ xfs_create(
        return 0;
 
  out_bmap_cancel:
-       xfs_bmap_cancel(&free_list);
+       xfs_defer_cancel(&dfops);
  out_trans_cancel:
        xfs_trans_cancel(tp);
  out_release_inode:
@@ -1401,7 +1402,7 @@ xfs_link(
        xfs_mount_t             *mp = tdp->i_mount;
        xfs_trans_t             *tp;
        int                     error;
-       xfs_bmap_free_t         free_list;
+       struct xfs_defer_ops    dfops;
        xfs_fsblock_t           first_block;
        int                     resblks;
 
@@ -1452,7 +1453,7 @@ xfs_link(
                        goto error_return;
        }
 
-       xfs_bmap_init(&free_list, &first_block);
+       xfs_defer_init(&dfops, &first_block);
 
        /*
         * Handle initial link state of O_TMPFILE inode
@@ -1464,7 +1465,7 @@ xfs_link(
        }
 
        error = xfs_dir_createname(tp, tdp, target_name, sip->i_ino,
-                                       &first_block, &free_list, resblks);
+                                       &first_block, &dfops, resblks);
        if (error)
                goto error_return;
        xfs_trans_ichgtime(tp, tdp, XFS_ICHGTIME_MOD | XFS_ICHGTIME_CHG);
@@ -1482,9 +1483,9 @@ xfs_link(
        if (mp->m_flags & (XFS_MOUNT_WSYNC|XFS_MOUNT_DIRSYNC))
                xfs_trans_set_sync(tp);
 
-       error = xfs_bmap_finish(&tp, &free_list, NULL);
+       error = xfs_defer_finish(&tp, &dfops, NULL);
        if (error) {
-               xfs_bmap_cancel(&free_list);
+               xfs_defer_cancel(&dfops);
                goto error_return;
        }
 
@@ -1526,7 +1527,7 @@ xfs_itruncate_extents(
 {
        struct xfs_mount        *mp = ip->i_mount;
        struct xfs_trans        *tp = *tpp;
-       xfs_bmap_free_t         free_list;
+       struct xfs_defer_ops    dfops;
        xfs_fsblock_t           first_block;
        xfs_fileoff_t           first_unmap_block;
        xfs_fileoff_t           last_block;
@@ -1562,12 +1563,12 @@ xfs_itruncate_extents(
        ASSERT(first_unmap_block < last_block);
        unmap_len = last_block - first_unmap_block + 1;
        while (!done) {
-               xfs_bmap_init(&free_list, &first_block);
+               xfs_defer_init(&dfops, &first_block);
                error = xfs_bunmapi(tp, ip,
                                    first_unmap_block, unmap_len,
                                    xfs_bmapi_aflag(whichfork),
                                    XFS_ITRUNC_MAX_EXTENTS,
-                                   &first_block, &free_list,
+                                   &first_block, &dfops,
                                    &done);
                if (error)
                        goto out_bmap_cancel;
@@ -1576,7 +1577,7 @@ xfs_itruncate_extents(
                 * Duplicate the transaction that has the permanent
                 * reservation and commit the old transaction.
                 */
-               error = xfs_bmap_finish(&tp, &free_list, ip);
+               error = xfs_defer_finish(&tp, &dfops, ip);
                if (error)
                        goto out_bmap_cancel;
 
@@ -1602,7 +1603,7 @@ out_bmap_cancel:
         * the transaction can be properly aborted.  We just need to make sure
         * we're not holding any resources that we were not when we came in.
         */
-       xfs_bmap_cancel(&free_list);
+       xfs_defer_cancel(&dfops);
        goto out;
 }
 
@@ -1709,7 +1710,7 @@ xfs_inactive_truncate(
        /*
         * Log the inode size first to prevent stale data exposure in the event
         * of a system crash before the truncate completes. See the related
-        * comment in xfs_setattr_size() for details.
+        * comment in xfs_vn_setattr_size() for details.
         */
        ip->i_d.di_size = 0;
        xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
@@ -1743,7 +1744,7 @@ STATIC int
 xfs_inactive_ifree(
        struct xfs_inode *ip)
 {
-       xfs_bmap_free_t         free_list;
+       struct xfs_defer_ops    dfops;
        xfs_fsblock_t           first_block;
        struct xfs_mount        *mp = ip->i_mount;
        struct xfs_trans        *tp;
@@ -1780,8 +1781,8 @@ xfs_inactive_ifree(
        xfs_ilock(ip, XFS_ILOCK_EXCL);
        xfs_trans_ijoin(tp, ip, 0);
 
-       xfs_bmap_init(&free_list, &first_block);
-       error = xfs_ifree(tp, ip, &free_list);
+       xfs_defer_init(&dfops, &first_block);
+       error = xfs_ifree(tp, ip, &dfops);
        if (error) {
                /*
                 * If we fail to free the inode, shut down.  The cancel
@@ -1807,11 +1808,11 @@ xfs_inactive_ifree(
         * Just ignore errors at this point.  There is nothing we can do except
         * to try to keep going. Make sure it's not a silent error.
         */
-       error = xfs_bmap_finish(&tp, &free_list, NULL);
+       error = xfs_defer_finish(&tp, &dfops, NULL);
        if (error) {
-               xfs_notice(mp, "%s: xfs_bmap_finish returned error %d",
+               xfs_notice(mp, "%s: xfs_defer_finish returned error %d",
                        __func__, error);
-               xfs_bmap_cancel(&free_list);
+               xfs_defer_cancel(&dfops);
        }
        error = xfs_trans_commit(tp);
        if (error)
@@ -2367,7 +2368,7 @@ int
 xfs_ifree(
        xfs_trans_t     *tp,
        xfs_inode_t     *ip,
-       xfs_bmap_free_t *flist)
+       struct xfs_defer_ops    *dfops)
 {
        int                     error;
        struct xfs_icluster     xic = { 0 };
@@ -2386,7 +2387,7 @@ xfs_ifree(
        if (error)
                return error;
 
-       error = xfs_difree(tp, ip->i_ino, flist, &xic);
+       error = xfs_difree(tp, ip->i_ino, dfops, &xic);
        if (error)
                return error;
 
@@ -2474,7 +2475,7 @@ xfs_iunpin_wait(
  * directory entry.
  *
  * This is still safe from a transactional point of view - it is not until we
- * get to xfs_bmap_finish() that we have the possibility of multiple
+ * get to xfs_defer_finish() that we have the possibility of multiple
  * transactions in this operation. Hence as long as we remove the directory
  * entry and drop the link count in the first transaction of the remove
  * operation, there are no transactional constraints on the ordering here.
@@ -2489,7 +2490,7 @@ xfs_remove(
        xfs_trans_t             *tp = NULL;
        int                     is_dir = S_ISDIR(VFS_I(ip)->i_mode);
        int                     error = 0;
-       xfs_bmap_free_t         free_list;
+       struct xfs_defer_ops    dfops;
        xfs_fsblock_t           first_block;
        uint                    resblks;
 
@@ -2571,9 +2572,9 @@ xfs_remove(
        if (error)
                goto out_trans_cancel;
 
-       xfs_bmap_init(&free_list, &first_block);
+       xfs_defer_init(&dfops, &first_block);
        error = xfs_dir_removename(tp, dp, name, ip->i_ino,
-                                       &first_block, &free_list, resblks);
+                                       &first_block, &dfops, resblks);
        if (error) {
                ASSERT(error != -ENOENT);
                goto out_bmap_cancel;
@@ -2587,7 +2588,7 @@ xfs_remove(
        if (mp->m_flags & (XFS_MOUNT_WSYNC|XFS_MOUNT_DIRSYNC))
                xfs_trans_set_sync(tp);
 
-       error = xfs_bmap_finish(&tp, &free_list, NULL);
+       error = xfs_defer_finish(&tp, &dfops, NULL);
        if (error)
                goto out_bmap_cancel;
 
@@ -2601,7 +2602,7 @@ xfs_remove(
        return 0;
 
  out_bmap_cancel:
-       xfs_bmap_cancel(&free_list);
+       xfs_defer_cancel(&dfops);
  out_trans_cancel:
        xfs_trans_cancel(tp);
  std_return:
@@ -2662,7 +2663,7 @@ xfs_sort_for_rename(
 static int
 xfs_finish_rename(
        struct xfs_trans        *tp,
-       struct xfs_bmap_free    *free_list)
+       struct xfs_defer_ops    *dfops)
 {
        int                     error;
 
@@ -2673,9 +2674,9 @@ xfs_finish_rename(
        if (tp->t_mountp->m_flags & (XFS_MOUNT_WSYNC|XFS_MOUNT_DIRSYNC))
                xfs_trans_set_sync(tp);
 
-       error = xfs_bmap_finish(&tp, free_list, NULL);
+       error = xfs_defer_finish(&tp, dfops, NULL);
        if (error) {
-               xfs_bmap_cancel(free_list);
+               xfs_defer_cancel(dfops);
                xfs_trans_cancel(tp);
                return error;
        }
@@ -2697,7 +2698,7 @@ xfs_cross_rename(
        struct xfs_inode        *dp2,
        struct xfs_name         *name2,
        struct xfs_inode        *ip2,
-       struct xfs_bmap_free    *free_list,
+       struct xfs_defer_ops    *dfops,
        xfs_fsblock_t           *first_block,
        int                     spaceres)
 {
@@ -2709,14 +2710,14 @@ xfs_cross_rename(
        /* Swap inode number for dirent in first parent */
        error = xfs_dir_replace(tp, dp1, name1,
                                ip2->i_ino,
-                               first_block, free_list, spaceres);
+                               first_block, dfops, spaceres);
        if (error)
                goto out_trans_abort;
 
        /* Swap inode number for dirent in second parent */
        error = xfs_dir_replace(tp, dp2, name2,
                                ip1->i_ino,
-                               first_block, free_list, spaceres);
+                               first_block, dfops, spaceres);
        if (error)
                goto out_trans_abort;
 
@@ -2731,7 +2732,7 @@ xfs_cross_rename(
                if (S_ISDIR(VFS_I(ip2)->i_mode)) {
                        error = xfs_dir_replace(tp, ip2, &xfs_name_dotdot,
                                                dp1->i_ino, first_block,
-                                               free_list, spaceres);
+                                               dfops, spaceres);
                        if (error)
                                goto out_trans_abort;
 
@@ -2758,7 +2759,7 @@ xfs_cross_rename(
                if (S_ISDIR(VFS_I(ip1)->i_mode)) {
                        error = xfs_dir_replace(tp, ip1, &xfs_name_dotdot,
                                                dp2->i_ino, first_block,
-                                               free_list, spaceres);
+                                               dfops, spaceres);
                        if (error)
                                goto out_trans_abort;
 
@@ -2797,10 +2798,10 @@ xfs_cross_rename(
        }
        xfs_trans_ichgtime(tp, dp1, XFS_ICHGTIME_MOD | XFS_ICHGTIME_CHG);
        xfs_trans_log_inode(tp, dp1, XFS_ILOG_CORE);
-       return xfs_finish_rename(tp, free_list);
+       return xfs_finish_rename(tp, dfops);
 
 out_trans_abort:
-       xfs_bmap_cancel(free_list);
+       xfs_defer_cancel(dfops);
        xfs_trans_cancel(tp);
        return error;
 }
@@ -2855,7 +2856,7 @@ xfs_rename(
 {
        struct xfs_mount        *mp = src_dp->i_mount;
        struct xfs_trans        *tp;
-       struct xfs_bmap_free    free_list;
+       struct xfs_defer_ops    dfops;
        xfs_fsblock_t           first_block;
        struct xfs_inode        *wip = NULL;            /* whiteout inode */
        struct xfs_inode        *inodes[__XFS_SORT_INODES];
@@ -2944,13 +2945,13 @@ xfs_rename(
                goto out_trans_cancel;
        }
 
-       xfs_bmap_init(&free_list, &first_block);
+       xfs_defer_init(&dfops, &first_block);
 
        /* RENAME_EXCHANGE is unique from here on. */
        if (flags & RENAME_EXCHANGE)
                return xfs_cross_rename(tp, src_dp, src_name, src_ip,
                                        target_dp, target_name, target_ip,
-                                       &free_list, &first_block, spaceres);
+                                       &dfops, &first_block, spaceres);
 
        /*
         * Set up the target.
@@ -2972,7 +2973,7 @@ xfs_rename(
                 */
                error = xfs_dir_createname(tp, target_dp, target_name,
                                                src_ip->i_ino, &first_block,
-                                               &free_list, spaceres);
+                                               &dfops, spaceres);
                if (error)
                        goto out_bmap_cancel;
 
@@ -3012,7 +3013,7 @@ xfs_rename(
                 */
                error = xfs_dir_replace(tp, target_dp, target_name,
                                        src_ip->i_ino,
-                                       &first_block, &free_list, spaceres);
+                                       &first_block, &dfops, spaceres);
                if (error)
                        goto out_bmap_cancel;
 
@@ -3047,7 +3048,7 @@ xfs_rename(
                 */
                error = xfs_dir_replace(tp, src_ip, &xfs_name_dotdot,
                                        target_dp->i_ino,
-                                       &first_block, &free_list, spaceres);
+                                       &first_block, &dfops, spaceres);
                ASSERT(error != -EEXIST);
                if (error)
                        goto out_bmap_cancel;
@@ -3086,10 +3087,10 @@ xfs_rename(
         */
        if (wip) {
                error = xfs_dir_replace(tp, src_dp, src_name, wip->i_ino,
-                                       &first_block, &free_list, spaceres);
+                                       &first_block, &dfops, spaceres);
        } else
                error = xfs_dir_removename(tp, src_dp, src_name, src_ip->i_ino,
-                                          &first_block, &free_list, spaceres);
+                                          &first_block, &dfops, spaceres);
        if (error)
                goto out_bmap_cancel;
 
@@ -3124,13 +3125,13 @@ xfs_rename(
        if (new_parent)
                xfs_trans_log_inode(tp, target_dp, XFS_ILOG_CORE);
 
-       error = xfs_finish_rename(tp, &free_list);
+       error = xfs_finish_rename(tp, &dfops);
        if (wip)
                IRELE(wip);
        return error;
 
 out_bmap_cancel:
-       xfs_bmap_cancel(&free_list);
+       xfs_defer_cancel(&dfops);
 out_trans_cancel:
        xfs_trans_cancel(tp);
 out_release_wip: