diff options
| author | Liu Bo <bo.li.liu@oracle.com> | 2013-03-04 16:25:39 +0000 |
|---|---|---|
| committer | Josef Bacik <jbacik@fusionio.com> | 2013-03-04 16:33:23 -0500 |
| commit | 288189471d7e12fb22c37870e151833f438deea8 (patch) | |
| tree | e15070a254c9b16e919c315bee972d90176d8d28 /fs | |
| parent | e1a1267054ea9dcafd01636e52d441f809efad5e (diff) | |
| download | op-kernel-dev-288189471d7e12fb22c37870e151833f438deea8.zip op-kernel-dev-288189471d7e12fb22c37870e151833f438deea8.tar.gz | |
Btrfs: do not BUG_ON in prepare_to_reloc
We can bail out from here gracefully instead of a cold BUG_ON.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/relocation.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c index 7d1654c..9d13786 100644 --- a/fs/btrfs/relocation.c +++ b/fs/btrfs/relocation.c @@ -3731,7 +3731,15 @@ int prepare_to_relocate(struct reloc_control *rc) set_reloc_control(rc); trans = btrfs_join_transaction(rc->extent_root); - BUG_ON(IS_ERR(trans)); + if (IS_ERR(trans)) { + unset_reloc_control(rc); + /* + * extent tree is not a ref_cow tree and has no reloc_root to + * cleanup. And callers are responsible to free the above + * block rsv. + */ + return PTR_ERR(trans); + } btrfs_commit_transaction(trans, rc->extent_root); return 0; } |
