diff options
author | Yan Zheng <zheng.yan@oracle.com> | 2008-10-30 14:19:50 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2008-10-30 14:19:50 -0400 |
commit | 6643558db29006825dbb10012b3f8890aca4bcd5 (patch) | |
tree | 0c0f4f7a0011749cda998431828cb9161747b51a /fs/btrfs/extent_io.c | |
parent | 9036c10208e1fc496cef7692ba66a78699b360dc (diff) |
Btrfs: Fix bookend extent race v2
When dropping middle part of an extent, btrfs_drop_extents truncates
the extent at first, then inserts a bookend extent.
Since truncation and insertion can't be done atomically, there is a small
period that the bookend extent isn't in the tree. This causes problem for
functions that search the tree for file extent item. The way to fix this is
lock the range of the bookend extent before truncation.
Signed-off-by: Yan Zheng <zheng.yan@oracle.com>
Diffstat (limited to 'fs/btrfs/extent_io.c')
-rw-r--r-- | fs/btrfs/extent_io.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index 7503bd46819b..65a0583027e9 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c | |||
@@ -946,8 +946,12 @@ int try_lock_extent(struct extent_io_tree *tree, u64 start, u64 end, | |||
946 | 946 | ||
947 | err = set_extent_bit(tree, start, end, EXTENT_LOCKED, 1, | 947 | err = set_extent_bit(tree, start, end, EXTENT_LOCKED, 1, |
948 | &failed_start, mask); | 948 | &failed_start, mask); |
949 | if (err == -EEXIST) | 949 | if (err == -EEXIST) { |
950 | if (failed_start > start) | ||
951 | clear_extent_bit(tree, start, failed_start - 1, | ||
952 | EXTENT_LOCKED, 1, 0, mask); | ||
950 | return 0; | 953 | return 0; |
954 | } | ||
951 | return 1; | 955 | return 1; |
952 | } | 956 | } |
953 | EXPORT_SYMBOL(try_lock_extent); | 957 | EXPORT_SYMBOL(try_lock_extent); |