diff options
| author | Yan Zheng | 2008-11-11 15:33:29 +0100 |
|---|---|---|
| committer | Chris Mason | 2008-11-11 15:33:29 +0100 |
| commit | 8247b41ac980d125de8aeba6f33f381056ac0ecb (patch) | |
| tree | 40b8ddab6be8ec6766007b22db9a6b898ff1a960 /fs/btrfs/file.c | |
| parent | Btrfs: empty_size allocation fixes again (diff) | |
| download | kernel-qcow2-linux-8247b41ac980d125de8aeba6f33f381056ac0ecb.tar.gz kernel-qcow2-linux-8247b41ac980d125de8aeba6f33f381056ac0ecb.tar.xz kernel-qcow2-linux-8247b41ac980d125de8aeba6f33f381056ac0ecb.zip | |
Btrfs: Fix starting search offset inside btrfs_drop_extents
btrfs_drop_extents will drop paths and search again when it needs to
force COW of higher nodes. It was using the key it found during the last
search as the offset for the next search.
But, this wasn't always correct. The key could be from before our desired
range, and because we're dropping the path, it is possible for file's items
to change while we do the search again.
The fix here is to make sure we don't search for something smaller than
the offset btrfs_drop_extents was called with.
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 4119f9a95320..934bc094bf17 100644 --- a/fs/btrfs/file.c +++ b/fs/btrfs/file.c @@ -436,7 +436,7 @@ next_slot: goto out; } if (recow) { - search_start = key.offset; + search_start = max(key.offset, start); continue; } if (btrfs_key_type(&key) == BTRFS_EXTENT_DATA_KEY) { |