Btrfs balance can easily hit BUG_ON in these places, but we want
to it bail out gracefully after we force the whole filesystem to
readonly. So we use btrfs_std_error hook in place of BUG_ON.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
while (1) {
progress++;
trans = btrfs_start_transaction(rc->extent_root, 0);
while (1) {
progress++;
trans = btrfs_start_transaction(rc->extent_root, 0);
+ if (IS_ERR(trans)) {
+ err = PTR_ERR(trans);
+ trans = NULL;
+ break;
+ }
restart:
if (update_backref_cache(trans, &rc->backref_cache)) {
btrfs_end_transaction(trans, rc->extent_root);
restart:
if (update_backref_cache(trans, &rc->backref_cache)) {
btrfs_end_transaction(trans, rc->extent_root);
return ret;
trans = btrfs_start_transaction(root, 0);
return ret;
trans = btrfs_start_transaction(root, 0);
+ if (IS_ERR(trans)) {
+ ret = PTR_ERR(trans);
+ btrfs_std_error(root->fs_info, ret);
+ return ret;
+ }
unset_balance_control(fs_info);
ret = del_balance_item(fs_info->tree_root);
unset_balance_control(fs_info);
ret = del_balance_item(fs_info->tree_root);
+ if (ret)
+ btrfs_std_error(fs_info, ret);
atomic_set(&fs_info->mutually_exclusive_operation_running, 0);
}
atomic_set(&fs_info->mutually_exclusive_operation_running, 0);
}