From: Zhao Lei <[email protected]>

logical's value should be [em->start, em->start + em->len),
so we need to use
  BUG_ON(em->start > logical || em->start + em->len <= logical);
instead of
  BUG_ON(em->start > logical || em->start + em->len < logical);

Signed-off-by: Zhao Lei <[email protected]>
Signed-off-by: Miao Xie <[email protected]>
---
 fs/btrfs/volumes.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
index 9df8e3f..844c393 100644
--- a/fs/btrfs/volumes.c
+++ b/fs/btrfs/volumes.c
@@ -2598,7 +2598,7 @@ int btrfs_num_copies(struct btrfs_mapping_tree *map_tree, 
u64 logical, u64 len)
        read_unlock(&em_tree->lock);
        BUG_ON(!em);
 
-       BUG_ON(em->start > logical || em->start + em->len < logical);
+       BUG_ON(em->start > logical || em->start + em->len <= logical);
        map = (struct map_lookup *)em->bdev;
        if (map->type & (BTRFS_BLOCK_GROUP_DUP | BTRFS_BLOCK_GROUP_RAID1))
                ret = map->num_stripes;
@@ -2673,7 +2673,7 @@ again:
                BUG();
        }
 
-       BUG_ON(em->start > logical || em->start + em->len < logical);
+       BUG_ON(em->start > logical || em->start + em->len <= logical);
        map = (struct map_lookup *)em->bdev;
        offset = logical - em->start;
 
-- 
1.6.5.2


--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to