diff options
author | Brian Foster <bfoster@redhat.com> | 2018-08-01 10:20:33 -0400 |
---|---|---|
committer | Darrick J. Wong <darrick.wong@oracle.com> | 2018-08-03 02:05:14 -0400 |
commit | 60f31a609ed3d28791acb2bc24188cb7e2259176 (patch) | |
tree | 70672520f8148d5a6f7d2d80f1ad9d1dfa60fcc7 /fs/xfs/xfs_trans.c | |
parent | 7dbddbaccd189e63c39c9e22c728c4548b9893bb (diff) |
xfs: clean out superfluous dfops dop params/vars
The dfops code still passes around the xfs_defer_ops pointer
superfluously in a few places. Clean this up wherever the
transaction will suffice.
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'fs/xfs/xfs_trans.c')
-rw-r--r-- | fs/xfs/xfs_trans.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_trans.c b/fs/xfs/xfs_trans.c index ae3c875a14e5..b0130b21f4de 100644 --- a/fs/xfs/xfs_trans.c +++ b/fs/xfs/xfs_trans.c | |||
@@ -929,7 +929,7 @@ __xfs_trans_commit( | |||
929 | * Finish deferred items on final commit. Only permanent transactions | 929 | * Finish deferred items on final commit. Only permanent transactions |
930 | * should ever have deferred ops. | 930 | * should ever have deferred ops. |
931 | */ | 931 | */ |
932 | WARN_ON_ONCE(xfs_defer_has_unfinished_work(tp->t_dfops) && | 932 | WARN_ON_ONCE(xfs_defer_has_unfinished_work(tp) && |
933 | !(tp->t_flags & XFS_TRANS_PERM_LOG_RES)); | 933 | !(tp->t_flags & XFS_TRANS_PERM_LOG_RES)); |
934 | if (!regrant && (tp->t_flags & XFS_TRANS_PERM_LOG_RES)) { | 934 | if (!regrant && (tp->t_flags & XFS_TRANS_PERM_LOG_RES)) { |
935 | error = xfs_defer_finish_noroll(&tp); | 935 | error = xfs_defer_finish_noroll(&tp); |