Commit 8804630e authored by Brian Foster's avatar Brian Foster Committed by Darrick J. Wong

xfs: defer agfl frees from directory op transactions

Directory operations can perform block allocations as entries are
added/removed from directories. Defer AGFL block frees from the
remaining directory operation transactions. This covers the hard
link, remove and rename operations.
Signed-off-by: default avatarBrian Foster <bfoster@redhat.com>
Reviewed-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
parent 8b922f0e
...@@ -1464,6 +1464,7 @@ xfs_link( ...@@ -1464,6 +1464,7 @@ xfs_link(
} }
xfs_defer_init(&dfops, &first_block); xfs_defer_init(&dfops, &first_block);
tp->t_agfl_dfops = &dfops;
/* /*
* Handle initial link state of O_TMPFILE inode * Handle initial link state of O_TMPFILE inode
...@@ -2661,6 +2662,7 @@ xfs_remove( ...@@ -2661,6 +2662,7 @@ xfs_remove(
goto out_trans_cancel; goto out_trans_cancel;
xfs_defer_init(&dfops, &first_block); xfs_defer_init(&dfops, &first_block);
tp->t_agfl_dfops = &dfops;
error = xfs_dir_removename(tp, dp, name, ip->i_ino, error = xfs_dir_removename(tp, dp, name, ip->i_ino,
&first_block, &dfops, resblks); &first_block, &dfops, resblks);
if (error) { if (error) {
...@@ -3028,6 +3030,7 @@ xfs_rename( ...@@ -3028,6 +3030,7 @@ xfs_rename(
} }
xfs_defer_init(&dfops, &first_block); xfs_defer_init(&dfops, &first_block);
tp->t_agfl_dfops = &dfops;
/* RENAME_EXCHANGE is unique from here on. */ /* RENAME_EXCHANGE is unique from here on. */
if (flags & RENAME_EXCHANGE) if (flags & RENAME_EXCHANGE)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment