xfs: defer frees from common inode allocation paths
Inode allocation can require block allocation for physical inode chunk allocation, inode btree record insertion, and/or directory block allocation for entry insertion. Any of these block allocation requests can require AGFL fixups prior to the actual allocation. Update the common file creation transacions to defer AGFL frees from these contexts to avoid too much log reservation consumption per-transaction. Since these transactions are already passed down through the btree cursors and da_args structure, this simply requires to attach dfops to the transaction. Note that this covers tr_create, tr_mkdir and tr_symlink. Other transactions such as tr_create_tmpfile do not already make use of deferred operations and so are left alone for the time being. Signed-off-by: Brian Foster <bfoster@redhat.com> Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
This commit is contained in:
parent
658f8f9511
commit
8b922f0e6a
|
@ -1208,6 +1208,7 @@ xfs_create(
|
|||
unlock_dp_on_error = true;
|
||||
|
||||
xfs_defer_init(&dfops, &first_block);
|
||||
tp->t_agfl_dfops = &dfops;
|
||||
|
||||
/*
|
||||
* Reserve disk quota and the inode.
|
||||
|
|
|
@ -259,6 +259,7 @@ xfs_symlink(
|
|||
* bmapi or the directory create code.
|
||||
*/
|
||||
xfs_defer_init(&dfops, &first_block);
|
||||
tp->t_agfl_dfops = &dfops;
|
||||
|
||||
/*
|
||||
* Allocate an inode for the symlink.
|
||||
|
|
Loading…
Reference in New Issue