btrfs: Move error handling of btrfs_start_dirty_block_groups closer to call site
authorNikolay Borisov <nborisov@suse.com>
Fri, 9 Feb 2018 09:30:18 +0000 (11:30 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 26 Mar 2018 13:09:36 +0000 (15:09 +0200)
commitf9cacae3145a07c8a2b699f18824df0cf7778431
tree3173a3692b4dfb120261ec75196e23c04c4ab7ea
parent7ef2d6a7228f54f337a6fcb2e40de41bcc826ef2
btrfs: Move error handling of btrfs_start_dirty_block_groups closer to call site

Even though btrfs_start_dirty_block_groups is fairly in the beginning of
btrfs_commit_transaction outside of the critical section defined by the
transaction states it can only be run by a single comitter. In other
words it defines its own critical section thanks to the
BTRFS_TRANS_DIRTY_BG run flag and ro_block_group_mutex. However, its
error handling is outside of this critical section which is a bit
counter-intuitive. So move the error handling righ after the function is
executed and let the sole runner of dirty block groups handle the return
value. No functional changes.

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/transaction.c