diff options
author | Josef Bacik <jbacik@fusionio.com> | 2012-09-27 17:07:30 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-09 09:15:41 -0400 |
commit | e6138876ad8327250d77291b3262fee356267211 (patch) | |
tree | ffc3fe0a05e0fd7e55b92e3ef8bad42d3c73d68c /fs/btrfs/tree-log.c | |
parent | ce1953325662fa597197ce728e4195582fc21c8d (diff) | |
download | op-kernel-dev-e6138876ad8327250d77291b3262fee356267211.zip op-kernel-dev-e6138876ad8327250d77291b3262fee356267211.tar.gz |
Btrfs: cache extent state when writing out dirty metadata pages
Everytime we write out dirty pages we search for an offset in the tree,
convert the bits in the state, and then when we wait we search for the
offset again and clear the bits. So for every dirty range in the io tree we
are doing 4 rb searches, which is suboptimal. With this patch we are only
doing 2 searches for every cycle (modulo weird things happening). Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs/tree-log.c')
-rw-r--r-- | fs/btrfs/tree-log.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 1d7b348..f4b9e54 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c @@ -2463,7 +2463,8 @@ static void free_log_tree(struct btrfs_trans_handle *trans, while (1) { ret = find_first_extent_bit(&log->dirty_log_pages, - 0, &start, &end, EXTENT_DIRTY | EXTENT_NEW); + 0, &start, &end, EXTENT_DIRTY | EXTENT_NEW, + NULL); if (ret) break; |