diff options
author | Chris Mason <chris.mason@oracle.com> | 2008-07-17 12:54:14 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2008-09-25 11:04:04 -0400 |
commit | f9295749388f82c8d2f485e99c72cd7c7876a99b (patch) | |
tree | 5089b0f723583d262116775b8b9fb5b4b8d291e0 /fs/btrfs/file.c | |
parent | dbe674a99c8af088faa4c95eddaeb271a3140ab6 (diff) |
btrfs_start_transaction: wait for commits in progress to finish
btrfs_commit_transaction has to loop waiting for any writers in the
transaction to finish before it can proceed. btrfs_start_transaction
should be polite and not join a transaction that is in the process
of being finished off.
There are a few places that can't wait, basically the ones doing IO that
might be needed to finish the transaction. For them, btrfs_join_transaction
is added.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/file.c')
-rw-r--r-- | fs/btrfs/file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c index 3e4e5c227c0c..d6505892cd52 100644 --- a/fs/btrfs/file.c +++ b/fs/btrfs/file.c | |||
@@ -251,7 +251,7 @@ static int noinline dirty_and_release_pages(struct btrfs_trans_handle *trans, | |||
251 | end_of_last_block = start_pos + num_bytes - 1; | 251 | end_of_last_block = start_pos + num_bytes - 1; |
252 | 252 | ||
253 | lock_extent(io_tree, start_pos, end_of_last_block, GFP_NOFS); | 253 | lock_extent(io_tree, start_pos, end_of_last_block, GFP_NOFS); |
254 | trans = btrfs_start_transaction(root, 1); | 254 | trans = btrfs_join_transaction(root, 1); |
255 | if (!trans) { | 255 | if (!trans) { |
256 | err = -ENOMEM; | 256 | err = -ENOMEM; |
257 | goto out_unlock; | 257 | goto out_unlock; |