diff options
| author | Josef Bacik <jbacik@fusionio.com> | 2012-09-27 23:07:30 +0200 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-09 15:15:41 +0200 |
| commit | e6138876ad8327250d77291b3262fee356267211 (patch) | |
| tree | ffc3fe0a05e0fd7e55b92e3ef8bad42d3c73d68c /fs/btrfs/tree-log.c | |
| parent | Btrfs: do not hold the file extent leaf locked when adding extent item (diff) | |
| download | linux-e6138876ad8327250d77291b3262fee356267211.tar.xz linux-e6138876ad8327250d77291b3262fee356267211.zip | |
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 1d7b34844323..f4b9e54b1da2 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; |