Btrfs: stop committing the transaction so much during relocate
authorJosef Bacik <jbacik@fusionio.com>
Thu, 31 Oct 2013 14:07:19 +0000 (10:07 -0400)
committerChris Mason <chris.mason@fusionio.com>
Tue, 12 Nov 2013 03:08:31 +0000 (22:08 -0500)
commit9e6a0c52b74b2d63a6cdb09cec5eaf66038b218f
tree7582d735115b9bde7527bfc014d10ce8b419b2c6
parent9f23e289edaf1e99f60de3978c50a1c7424f3e92
Btrfs: stop committing the transaction so much during relocate

I noticed with my horrible snapshot excercisor that we were taking forever to
relocate the larger the file system got.  This appeared to be because we were
committing the transaction _constantly_.  There were a few places where we do
braindead things with metadata reservation, like start a transaction and then
try to refill the block rsv, which not only keeps us from committing a
transaction during the enospc stuff, but keeps us from doing some of the harder
flushing work which will make us more likely to need to commit the transaction.
We also were checking the block rsv and committing the transaction if the block
rsv was below a certain threshold, but we were doing this in a place where we
don't actually keep anything in the block rsv so this was always ending up false
so we always committed the transaction in this case.  I tested this to make sure
it didn't break anything, but it takes about 10 hours to get the box to this
state so I don't know how much of an impact it will really make.  Thanks,

Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
fs/btrfs/relocation.c