Merge tag 'xfs-5.3-merge-12' of git://git.kernel.org/pub/scm/fs/xfs/xfs-linux

Pull xfs updates from Darrick Wong:
 "In this release there are a significant amounts of consolidations and
  cleanups in the log code; restructuring of the log to issue struct
  bios directly; new bulkstat ioctls to return v5 fs inode information
  (and fix all the padding problems of the old ioctl); the beginnings of
  multithreaded inode walks (e.g. quotacheck); and a reduction in memory
  usage in the online scrub code leading to reduced runtimes.

   - Refactor inode geometry calculation into a single structure instead
     of open-coding pieces everywhere.

   - Add online repair to build options.

   - Remove unnecessary function call flags and functions.

   - Claim maintainership of various loose xfs documentation and header
     files.

   - Use struct bio directly for log buffer IOs instead of struct
     xfs_buf.

   - Reduce log item boilerplate code requirements.

   - Merge log item code spread across too many files.

   - Further distinguish between log item commits and cancellations.

   - Various small cleanups to the ag small allocator.

   - Support cgroup-aware writeback

   - libxfs refactoring for mkfs cleanup

   - Remove unneeded #includes

   - Fix a memory allocation miscalculation in the new log bio code

   - Fix bisection problems

   - Fix a crash in ioend processing caused by tripping over freeing of
     preallocated transactions

   - Split out a generic inode walk mechanism from the bulkstat code,
     hook up all the internal users to use the walking code, then clean
     up bulkstat to serve only the bulkstat ioctls.

   - Add a multithreaded iwalk implementation to speed up quotacheck on
     fast storage with many CPUs.

   - Remove unnecessary return values in logging teardown functions.

   - Supplement the bstat and inogrp structures with new bulkstat and
     inumbers structures that have all the fields we need for v5
     filesystem features and none of the padding problems of their
     predecessors.

   - Wire up new ioctls that use the new structures with a much simpler
     bulk_ireq structure at the head instead of the pointerhappy mess we
     had before.

   - Enable userspace to constrain bulkstat returns to a single AG or a
     single special inode so that we can phase out a lot of geometry
     guesswork in userspace.

   - Reduce memory consumption and zeroing overhead in extended
     attribute scrub code.

   - Fix some behavioral regressions in the new bulkstat backend code.

   - Fix some behavioral regressions in the new log bio code"

* tag 'xfs-5.3-merge-12' of git://git.kernel.org/pub/scm/fs/xfs/xfs-linux: (100 commits)
  xfs: chain bios the right way around in xfs_rw_bdev
  xfs: bump INUMBERS cursor correctly in xfs_inumbers_walk
  xfs: don't update lastino for FSBULKSTAT_SINGLE
  xfs: online scrub needn't bother zeroing its temporary buffer
  xfs: only allocate memory for scrubbing attributes when we need it
  xfs: refactor attr scrub memory allocation function
  xfs: refactor extended attribute buffer pointer functions
  xfs: attribute scrub should use seen_enough to pass error values
  xfs: allow single bulkstat of special inodes
  xfs: specify AG in bulk req
  xfs: wire up the v5 inumbers ioctl
  xfs: wire up new v5 bulkstat ioctls
  xfs: introduce v5 inode group structure
  xfs: introduce new v5 bulkstat structure
  xfs: rename bulkstat functions
  xfs: remove various bulk request typedef usage
  fs: xfs: xfs_log: Change return type from int to void
  xfs: poll waiting for quotacheck
  xfs: multithreaded iwalk implementation
  xfs: refactor INUMBERS to use iwalk functions
  ...

1 files changed, 25 insertions, 18 deletions

diff --git a/fs/xfs/xfs_trans.c b/fs/xfs/xfs_trans.c
index 912b42f5fe4a..d42a68d8313b 100644
--- a/fs/xfs/xfs_trans.c
+++ b/fs/xfs/xfs_trans.c
@@ -11,7 +11,6 @@
 #include "xfs_log_format.h"
 #include "xfs_trans_resv.h"
 #include "xfs_mount.h"
-#include "xfs_inode.h"
 #include "xfs_extent_busy.h"
 #include "xfs_quota.h"
 #include "xfs_trans.h"
@@ -264,9 +263,7 @@ xfs_trans_alloc(
          * GFP_NOFS allocation context so that we avoid lockdep false positives
          * by doing GFP_KERNEL allocations inside sb_start_intwrite().
          */
-        tp = kmem_zone_zalloc(xfs_trans_zone,
-                (flags & XFS_TRANS_NOFS) ? KM_NOFS : KM_SLEEP);
-
+        tp = kmem_zone_zalloc(xfs_trans_zone, KM_SLEEP);
         if (!(flags & XFS_TRANS_NO_WRITECOUNT))
                 sb_start_intwrite(mp->m_super);
 
@@ -452,7 +449,7 @@ xfs_trans_apply_sb_deltas(
         xfs_buf_t       *bp;
         int             whole = 0;
 
-        bp = xfs_trans_getsb(tp, tp->t_mountp, 0);
+        bp = xfs_trans_getsb(tp, tp->t_mountp);
         sbp = XFS_BUF_TO_SBP(bp);
 
         /*
@@ -767,10 +764,9 @@ xfs_trans_del_item(
 }
 
 /* Detach and unlock all of the items in a transaction */
-void
+static void
 xfs_trans_free_items(
         struct xfs_trans        *tp,
-        xfs_lsn_t               commit_lsn,
         bool                    abort)
 {
         struct xfs_log_item     *lip, *next;
@@ -779,11 +775,10 @@ xfs_trans_free_items(
 
         list_for_each_entry_safe(lip, next, &tp->t_items, li_trans) {
                 xfs_trans_del_item(lip);
-                if (commit_lsn != NULLCOMMITLSN)
-                        lip->li_ops->iop_committing(lip, commit_lsn);
                 if (abort)
                         set_bit(XFS_LI_ABORTED, &lip->li_flags);
-                lip->li_ops->iop_unlock(lip);
+                if (lip->li_ops->iop_release)
+                        lip->li_ops->iop_release(lip);
         }
 }
 
@@ -804,7 +799,8 @@ xfs_log_item_batch_insert(
         for (i = 0; i < nr_items; i++) {
                 struct xfs_log_item *lip = log_items[i];
 
-                lip->li_ops->iop_unpin(lip, 0);
+                if (lip->li_ops->iop_unpin)
+                        lip->li_ops->iop_unpin(lip, 0);
         }
 }
 
@@ -815,7 +811,7 @@ xfs_log_item_batch_insert(
  *
  * If we are called with the aborted flag set, it is because a log write during
  * a CIL checkpoint commit has failed. In this case, all the items in the
- * checkpoint have already gone through iop_commited and iop_unlock, which
+ * checkpoint have already gone through iop_committed and iop_committing, which
  * means that checkpoint commit abort handling is treated exactly the same
  * as an iclog write error even though we haven't started any IO yet. Hence in
  * this case all we need to do is iop_committed processing, followed by an
@@ -833,7 +829,7 @@ xfs_trans_committed_bulk(
         struct xfs_ail          *ailp,
         struct xfs_log_vec      *log_vector,
         xfs_lsn_t               commit_lsn,
-        int                     aborted)
+        bool                    aborted)
 {
 #define LOG_ITEM_BATCH_SIZE     32
         struct xfs_log_item     *log_items[LOG_ITEM_BATCH_SIZE];
@@ -852,7 +848,16 @@ xfs_trans_committed_bulk(
 
                 if (aborted)
                         set_bit(XFS_LI_ABORTED, &lip->li_flags);
-                item_lsn = lip->li_ops->iop_committed(lip, commit_lsn);
+
+                if (lip->li_ops->flags & XFS_ITEM_RELEASE_WHEN_COMMITTED) {
+                        lip->li_ops->iop_release(lip);
+                        continue;
+                }
+
+                if (lip->li_ops->iop_committed)
+                        item_lsn = lip->li_ops->iop_committed(lip, commit_lsn);
+                else
+                        item_lsn = commit_lsn;
 
                 /* item_lsn of -1 means the item needs no further processing */
                 if (XFS_LSN_CMP(item_lsn, (xfs_lsn_t)-1) == 0)
@@ -864,7 +869,8 @@ xfs_trans_committed_bulk(
                  */
                 if (aborted) {
                         ASSERT(XFS_FORCED_SHUTDOWN(ailp->ail_mount));
-                        lip->li_ops->iop_unpin(lip, 1);
+                        if (lip->li_ops->iop_unpin)
+                                lip->li_ops->iop_unpin(lip, 1);
                         continue;
                 }
 
@@ -882,7 +888,8 @@ xfs_trans_committed_bulk(
                                 xfs_trans_ail_update(ailp, lip, item_lsn);
                         else
                                 spin_unlock(&ailp->ail_lock);
-                        lip->li_ops->iop_unpin(lip, 0);
+                        if (lip->li_ops->iop_unpin)
+                                lip->li_ops->iop_unpin(lip, 0);
                         continue;
                 }
 
@@ -998,7 +1005,7 @@ out_unreserve:
                 tp->t_ticket = NULL;
         }
         current_restore_flags_nested(&tp->t_pflags, PF_MEMALLOC_NOFS);
-        xfs_trans_free_items(tp, NULLCOMMITLSN, !!error);
+        xfs_trans_free_items(tp, !!error);
         xfs_trans_free(tp);
 
         XFS_STATS_INC(mp, xs_trans_empty);
@@ -1060,7 +1067,7 @@ xfs_trans_cancel(
         /* mark this thread as no longer being in a transaction */
         current_restore_flags_nested(&tp->t_pflags, PF_MEMALLOC_NOFS);
 
-        xfs_trans_free_items(tp, NULLCOMMITLSN, dirty);
+        xfs_trans_free_items(tp, dirty);
         xfs_trans_free(tp);
 }