Btrfs: Fix compressed checksum fsync log copies
authorChris Mason <chris.mason@oracle.com>
Tue, 9 Dec 2008 00:15:39 +0000 (19:15 -0500)
committerChris Mason <chris.mason@oracle.com>
Tue, 9 Dec 2008 00:15:39 +0000 (19:15 -0500)
The fsync logging code makes sure to onl copy the relevant checksum for each
extent based on the file extent pointers it finds.

But for compressed extents, it needs to copy the checksum for the
entire extent.

Signed-off-by: Chris Mason <chris.mason@oracle.com>
fs/btrfs/file.c
fs/btrfs/tree-log.c

index b5a6a2b6f668b3b4f80e65a2b1299d276618e527..71bfe3a6a4443949772d7f33a388f90e82e6f22c 100644 (file)
@@ -1228,7 +1228,8 @@ int btrfs_sync_file(struct file *file, struct dentry *dentry, int datasync)
        mutex_unlock(&root->fs_info->trans_mutex);
 
        root->fs_info->tree_log_batch++;
-       filemap_fdatawait(inode->i_mapping);
+       filemap_fdatawrite(inode->i_mapping);
+       btrfs_wait_ordered_range(inode, 0, (u64)-1);
        root->fs_info->tree_log_batch++;
 
        /*
index d3f9c2c663c41deb0279270b319b25a7b3d81b25..6ac1b7f72e2a5be331f76f27f1a8ae9e65ca4fb4 100644 (file)
@@ -2610,6 +2610,11 @@ static noinline int copy_items(struct btrfs_trans_handle *trans,
                                u64 cs = btrfs_file_extent_offset(src, extent);
                                u64 cl = btrfs_file_extent_num_bytes(src,
                                                                     extent);;
+                               if (btrfs_file_extent_compression(src,
+                                                                 extent)) {
+                                       cs = 0;
+                                       cl = dl;
+                               }
                                /* ds == 0 is a hole */
                                if (ds != 0) {
                                        ret = btrfs_inc_extent_ref(trans, log,