diff options
| author |   Josef Bacik <jbacik@fusionio.com> | 2013-10-25 16:13:35 -0400 |
|---|---|---|
| committer |   Chris Mason <chris.mason@fusionio.com> | 2013-11-11 22:07:35 -0500 |
| commit | 0ef8b726075aa6931ddf1c16f5bae043eef184f9 (patch) | |
| tree | 61c819e8541c69043a5d74182ac0f8d0060207e7 /fs/btrfs/relocation.c | |
| parent | Btrfs: stop using vfs_read in send (diff) | |
| download | linux-dev-0ef8b726075aa6931ddf1c16f5bae043eef184f9.tar.xz linux-dev-0ef8b726075aa6931ddf1c16f5bae043eef184f9.zip | |
Btrfs: return an error from btrfs_wait_ordered_range
I noticed that if the free space cache has an error writing out it's data it
won't actually error out, it will just carry on. This is because it doesn't
check the return value of btrfs_wait_ordered_range, which didn't actually return
anything. So fix this in order to keep us from making free space cache look
valid when it really isnt. Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'fs/btrfs/relocation.c')
| -rw-r--r-- | fs/btrfs/relocation.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c index a945374993f7..729c91e151a3 100644 --- a/fs/btrfs/relocation.c +++ b/fs/btrfs/relocation.c @@ -4257,7 +4257,12 @@ int btrfs_relocate_block_group(struct btrfs_root *extent_root, u64 group_start) rc->extents_found); if (rc->stage == MOVE_DATA_EXTENTS && rc->found_file_extent) { - btrfs_wait_ordered_range(rc->data_inode, 0, (u64)-1); + ret = btrfs_wait_ordered_range(rc->data_inode, 0, + (u64)-1); + if (ret) { + err = ret; + goto out; + } invalidate_mapping_pages(rc->data_inode->i_mapping, 0, -1); rc->stage = UPDATE_DATA_PTRS; |