Btrfs: do a full search everytime in btrfs_search_old_slot
authorJosef Bacik <jbacik@fusionio.com>
Tue, 24 Sep 2013 18:09:34 +0000 (14:09 -0400)
committerChris Mason <chris.mason@fusionio.com>
Tue, 12 Nov 2013 02:51:17 +0000 (21:51 -0500)
commitd4b4087c43cc00a196c5be57fac41f41309f1d56
treecdebbe5140bb6c7950622002ba17ac467d3f2b61
parent06ea65a398a2501e94beee3a425d07e1846ff25a
Btrfs: do a full search everytime in btrfs_search_old_slot

While running some snashot aware defrag tests I noticed I was panicing every
once and a while in key_search.  This is because of the optimization that says
if we find a key at slot 0 it will be at slot 0 all the way down the rest of the
tree.  This isn't the case for btrfs_search_old_slot since it will likely replay
changes to a buffer if something has changed since we took our sequence number.
So short circuit this optimization by setting prev_cmp to -1 every time we call
key_search so we will do our normal binary search.  With this patch I am no
longer seeing the panics I was seeing before.  Thanks,

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