summaryrefslogtreecommitdiffstats
path: root/fs/btrfs/transaction.c
diff options
context:
space:
mode:
authorJosef Bacik <jbacik@fusionio.com>2013-07-30 16:30:30 -0400
committerChris Mason <chris.mason@fusionio.com>2013-09-01 08:04:41 -0400
commitb6c60c8018c4e9beb2f83fc82c09f9d033766571 (patch)
treef213747ec5f9fb77dc6ecf0893755fa0f5e1c0f3 /fs/btrfs/transaction.c
parentd062d13cf1a100d5632306a28f9a9f54aced1280 (diff)
downloadtalos-op-linux-b6c60c8018c4e9beb2f83fc82c09f9d033766571.tar.gz
talos-op-linux-b6c60c8018c4e9beb2f83fc82c09f9d033766571.zip
Btrfs: change how we queue blocks for backref checking
Previously we only added blocks to the list to have their backrefs checked if the level of the block is right above the one we are searching for. This is because we want to make sure we don't add the entire path up to the root to the lists to make sure we process things one at a time. This assumes that if any blocks in the path to the root are going to be not checked (shared in other words) then they will be in the level right above the current block on up. This isn't quite right though since we can have blocks higher up the list that are shared because they are attached to a reloc root. But we won't add this block to be checked and then later on we will BUG_ON(!upper->checked). So instead keep track of wether or not we've queued a block to be checked in this current search, and if we haven't go ahead and queue it to be checked. This patch fixed the panic I was seeing where we BUG_ON(!upper->checked). Thanks, Signed-off-by: Josef Bacik <jbacik@fusionio.com> Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'fs/btrfs/transaction.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud