summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosef Bacik <josef@redhat.com>2011-02-03 02:39:52 +0000
committerChris Mason <chris.mason@oracle.com>2011-02-06 07:13:12 -0500
commit13dbc08987f25d9dba488a34b44b43e3844b027c (patch)
treef1ccabbf2c8bc676be0dacacc3da86c62cf1b19f
parent98d5dc13e7e74b77ca3b4c3cbded9f48d2dbbbb7 (diff)
downloadop-kernel-dev-13dbc08987f25d9dba488a34b44b43e3844b027c.zip
op-kernel-dev-13dbc08987f25d9dba488a34b44b43e3844b027c.tar.gz
Btrfs: make sure search_bitmap finds something in remove_from_bitmap
When we're cleaning up the tree log we need to be able to remove free space from the block group. The problem is if that free space spans bitmaps we would not find the space since we're looking for too many bytes. So make sure the amount of bytes we search for is limited to either the number of bytes we want, or the number of bytes left in the bitmap. This was tested by a user who was hitting the BUG() after search_bitmap. With this patch he can now mount his fs. Thanks, Signed-off-by: Josef Bacik <josef@redhat.com> Signed-off-by: Chris Mason <chris.mason@oracle.com>
-rw-r--r--fs/btrfs/free-space-cache.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c
index a5501ed..a039065 100644
--- a/fs/btrfs/free-space-cache.c
+++ b/fs/btrfs/free-space-cache.c
@@ -1216,6 +1216,7 @@ again:
*/
search_start = *offset;
search_bytes = *bytes;
+ search_bytes = min(search_bytes, end - search_start + 1);
ret = search_bitmap(block_group, bitmap_info, &search_start,
&search_bytes);
BUG_ON(ret < 0 || search_start != *offset);
OpenPOWER on IntegriCloud