Btrfs: make sure search_bitmap finds something in remove_from_bitmap
authorJosef Bacik <josef@redhat.com>
Thu, 3 Feb 2011 02:39:52 +0000 (02:39 +0000)
committerChris Mason <chris.mason@oracle.com>
Sun, 6 Feb 2011 12:13:12 +0000 (07:13 -0500)
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>
fs/btrfs/free-space-cache.c

index a5501ed..a039065 100644 (file)
@@ -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);