On Wed, Dec 11, 2019 at 09:55:04AM -0500, Josef Bacik wrote: > > + /* > > + * We don't need to use reloc tree if: > > + * - No reloc tree > > + * - Relocation not running > > + * - Reloc tree already merged > > + */ > > + if (!root->reloc_root || !rc || test_bit(BTRFS_ROOT_DEAD_RELOC_TREE, > > + &root->state)) > > This is awkward formatting, can we move the test_bit() to the first thing we > check so it's less weird? Then you can add I had the same thought, will move the test_bit on the next line.
