diff options
| author | Brian Foster <bfoster@redhat.com> | 2018-07-12 08:26:07 +0300 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2018-07-12 08:26:07 +0300 |
| commit | 6aa6718439ac6907b3c34cbe7ac2c71408f84f6e (patch) | |
| tree | b496b18ad3c609eb50f942666dbdebef76096c51 /fs/xfs/xfs_inode.c | |
| parent | 8a749386498327c98c2792e19eeeaa21dafc6954 (diff) | |
| download | linux-6aa6718439ac6907b3c34cbe7ac2c71408f84f6e.tar.xz | |
xfs: rename xfs_trans ->t_agfl_dfops to ->t_dfops
The ->t_agfl_dfops field is currently used to defer agfl block frees
from associated transaction contexts. While all known problematic
contexts have already been updated to use ->t_agfl_dfops, the
broader goal is defer agfl frees from all callers that already use a
deferred operations structure. Further, the transaction field
facilitates a good amount of code clean up where the transaction and
dfops have historically been passed down through the stack
separately.
Rename the field to something more generic to prepare to use it as
such throughout XFS. This patch does not change behavior.
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'fs/xfs/xfs_inode.c')
| -rw-r--r-- | fs/xfs/xfs_inode.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c index 5df4de666cc1..9c5fbc94cf14 100644 --- a/fs/xfs/xfs_inode.c +++ b/fs/xfs/xfs_inode.c @@ -1196,7 +1196,7 @@ xfs_create( unlock_dp_on_error = true; xfs_defer_init(&dfops, &first_block); - tp->t_agfl_dfops = &dfops; + tp->t_dfops = &dfops; /* * Reserve disk quota and the inode. @@ -1452,7 +1452,7 @@ xfs_link( } xfs_defer_init(&dfops, &first_block); - tp->t_agfl_dfops = &dfops; + tp->t_dfops = &dfops; /* * Handle initial link state of O_TMPFILE inode @@ -1813,7 +1813,7 @@ xfs_inactive_ifree( xfs_trans_ijoin(tp, ip, 0); xfs_defer_init(&dfops, &first_block); - tp->t_agfl_dfops = &dfops; + tp->t_dfops = &dfops; error = xfs_ifree(tp, ip, &dfops); if (error) { /* @@ -2659,7 +2659,7 @@ xfs_remove( goto out_trans_cancel; xfs_defer_init(&dfops, &first_block); - tp->t_agfl_dfops = &dfops; + tp->t_dfops = &dfops; error = xfs_dir_removename(tp, dp, name, ip->i_ino, &first_block, &dfops, resblks); if (error) { @@ -3027,7 +3027,7 @@ xfs_rename( } xfs_defer_init(&dfops, &first_block); - tp->t_agfl_dfops = &dfops; + tp->t_dfops = &dfops; /* RENAME_EXCHANGE is unique from here on. */ if (flags & RENAME_EXCHANGE) |