btrfs: abort transaction on btrfs_reloc_cow_block()
authorZhaolei <zhaolei@cn.fujitsu.com>
Thu, 6 Aug 2015 13:56:58 +0000 (21:56 +0800)
committerChris Mason <clm@fb.com>
Sun, 9 Aug 2015 14:07:14 +0000 (07:07 -0700)
commit93314e3b64fd2e77237fdba7cfcc0d38dcd05579
tree16a545b8fb3679186e8092414e6c4693ceb795b6
parent147d256e0980e31505d25d721be979d6a8d2148c
btrfs: abort transaction on btrfs_reloc_cow_block()

When btrfs_reloc_cow_block() failed in __btrfs_cow_block(), current
code just return a err-value to caller, but leave new_created extent
buffer exist and locked.

Then subsequent code (in relocate) try to lock above eb again,
and caused deadlock without any dmesg.
(eb lock use wait_event(), so no lockdep message)

It is hard to do recover work in __btrfs_cow_block() at this error
point, but we can abort transaction to avoid deadlock and operate on
unstable state.a

It also helps developer to find wrong place quickly.
(better than a frozen fs without any dmesg before patch)

Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/ctree.c