Commit 4126c06e authored by Allison Henderson's avatar Allison Henderson

xfs: Reverse apply 72b97ea4

Originally we added this patch to help modularize the attr code in
preparation for delayed attributes and the state machine it requires.
However, later reviews found that this slightly alters the transaction
handling as the helper function is ambiguous as to whether the
transaction is diry or clean.  This may cause a dirty transaction to be
included in the next roll, where previously it had not.  To preserve the
existing code flow, we reverse apply this commit.
Signed-off-by: default avatarAllison Henderson <allison.henderson@oracle.com>
Reviewed-by: default avatarBrian Foster <bfoster@redhat.com>
Reviewed-by: default avatarChandan Babu R <chandanrlinux@gmail.com>
Reviewed-by: default avatarDarrick J. Wong <djwong@kernel.org>
parent 0fe0bbe0
...@@ -1214,24 +1214,6 @@ int xfs_attr_node_removename_setup( ...@@ -1214,24 +1214,6 @@ int xfs_attr_node_removename_setup(
return 0; return 0;
} }
STATIC int
xfs_attr_node_remove_rmt(
struct xfs_da_args *args,
struct xfs_da_state *state)
{
int error = 0;
error = xfs_attr_rmtval_remove(args);
if (error)
return error;
/*
* Refill the state structure with buffers, the prior calls released our
* buffers.
*/
return xfs_attr_refillstate(state);
}
/* /*
* Remove a name from a B-tree attribute list. * Remove a name from a B-tree attribute list.
* *
...@@ -1260,7 +1242,15 @@ xfs_attr_node_removename( ...@@ -1260,7 +1242,15 @@ xfs_attr_node_removename(
* overflow the maximum size of a transaction and/or hit a deadlock. * overflow the maximum size of a transaction and/or hit a deadlock.
*/ */
if (args->rmtblkno > 0) { if (args->rmtblkno > 0) {
error = xfs_attr_node_remove_rmt(args, state); error = xfs_attr_rmtval_remove(args);
if (error)
goto out;
/*
* Refill the state structure with buffers, the prior calls
* released our buffers.
*/
error = xfs_attr_refillstate(state);
if (error) if (error)
goto out; goto out;
} }
......
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