Btrfs: only drop modified extents if we logged the whole inode
authorJosef Bacik <jbacik@fusionio.com>
Wed, 13 Nov 2013 01:54:09 +0000 (20:54 -0500)
committerChris Mason <chris.mason@fusionio.com>
Thu, 21 Nov 2013 01:41:32 +0000 (20:41 -0500)
If we fsync, seek and write, rename and then fsync again we will lose the
modified hole extent because the rename will drop all of the modified extents
since we didn't do the fast search.  We need to only drop the modified extents
if we didn't do the fast search and we were logging the entire inode as we don't
need them anymore, otherwise this is being premature.  Thanks,

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

index b0c7134162fb386581a3013d0a21bd6b64581312..9f7fc51ca334864b72336e127d786047dfb1f5de 100644 (file)
@@ -3802,7 +3802,7 @@ log_extents:
                        err = ret;
                        goto out_unlock;
                }
-       } else {
+       } else if (inode_only == LOG_INODE_ALL) {
                struct extent_map_tree *tree = &BTRFS_I(inode)->extent_tree;
                struct extent_map *em, *n;