diff options
author | Mark Fasheh <mfasheh@suse.de> | 2011-09-08 20:29:00 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2012-03-21 20:45:39 -0400 |
commit | 1dd4602fa74273c28b3577a58aa389f330e9a0dc (patch) | |
tree | c506007911ac9aa210a7e2b66a19ea49da72b8a5 /fs/btrfs/volumes.c | |
parent | 2cdcecbc153c222fae1be6f8ddb320b29e3a5200 (diff) |
btrfs: Remove BUG_ON from __btrfs_alloc_chunk()
We BUG_ON() error from add_extent_mapping(), but that error looks pretty
easy to bubble back up - as far as I can tell there have not been any
permanent modifications to fs state at that point.
Signed-off-by: Mark Fasheh <mfasheh@suse.de>
Diffstat (limited to 'fs/btrfs/volumes.c')
-rw-r--r-- | fs/btrfs/volumes.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index 281ff354f64f..f002973959d0 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c | |||
@@ -3327,8 +3327,9 @@ static int __btrfs_alloc_chunk(struct btrfs_trans_handle *trans, | |||
3327 | write_lock(&em_tree->lock); | 3327 | write_lock(&em_tree->lock); |
3328 | ret = add_extent_mapping(em_tree, em); | 3328 | ret = add_extent_mapping(em_tree, em); |
3329 | write_unlock(&em_tree->lock); | 3329 | write_unlock(&em_tree->lock); |
3330 | BUG_ON(ret); | ||
3331 | free_extent_map(em); | 3330 | free_extent_map(em); |
3331 | if (ret) | ||
3332 | goto error; | ||
3332 | 3333 | ||
3333 | ret = btrfs_make_block_group(trans, extent_root, 0, type, | 3334 | ret = btrfs_make_block_group(trans, extent_root, 0, type, |
3334 | BTRFS_FIRST_CHUNK_TREE_OBJECTID, | 3335 | BTRFS_FIRST_CHUNK_TREE_OBJECTID, |