diff options
author | Josef Bacik <josef@redhat.com> | 2011-04-11 15:45:29 -0400 |
---|---|---|
committer | Josef Bacik <josef@redhat.com> | 2011-04-11 20:43:52 -0400 |
commit | 13c5a93e7005d7dae0b6d070d25203593e692d13 (patch) | |
tree | ecbc7205618940bbba3f0d9ad000462a5364f763 /fs/btrfs/disk-io.c | |
parent | 93a54bc4c28a125978cddbe2db9e347391e3522d (diff) |
Btrfs: avoid taking the trans_mutex in btrfs_end_transaction
I've been working on making our O_DIRECT latency not suck and I noticed we were
taking the trans_mutex in btrfs_end_transaction. So to do this we convert
num_writers and use_count to atomic_t's and just decrement them in
btrfs_end_transaction. Instead of deleting the transaction from the trans list
in put_transaction we do that in btrfs_commit_transaction() since that's the
only time it actually needs to be removed from the list. Thanks,
Signed-off-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'fs/btrfs/disk-io.c')
-rw-r--r-- | fs/btrfs/disk-io.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index a272bfd74ea0..ef6865c17cd6 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c | |||
@@ -3136,7 +3136,7 @@ static int btrfs_cleanup_transaction(struct btrfs_root *root) | |||
3136 | btrfs_destroy_pinned_extent(root, | 3136 | btrfs_destroy_pinned_extent(root, |
3137 | root->fs_info->pinned_extents); | 3137 | root->fs_info->pinned_extents); |
3138 | 3138 | ||
3139 | t->use_count = 0; | 3139 | atomic_set(&t->use_count, 0); |
3140 | list_del_init(&t->list); | 3140 | list_del_init(&t->list); |
3141 | memset(t, 0, sizeof(*t)); | 3141 | memset(t, 0, sizeof(*t)); |
3142 | kmem_cache_free(btrfs_transaction_cachep, t); | 3142 | kmem_cache_free(btrfs_transaction_cachep, t); |