If a inode is a BTRFS_INODE_NODATASUM one, it need not to look for csum items
any more.

Signed-off-by: Liu Bo <liubo2...@cn.fujitsu.com>
---
 fs/btrfs/tree-log.c |   13 ++++++++-----
 1 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c
index 28c3190..ba014ea 100644
--- a/fs/btrfs/tree-log.c
+++ b/fs/btrfs/tree-log.c
@@ -2655,7 +2655,8 @@ static noinline int copy_items(struct btrfs_trans_handle 
*trans,
                               struct inode *inode,
                               struct btrfs_path *dst_path,
                               struct extent_buffer *src,
-                              int start_slot, int nr, int inode_only)
+                              int start_slot, int nr, int inode_only,
+                              int csum)
 {
        unsigned long src_offset;
        unsigned long dst_offset;
@@ -2722,7 +2723,8 @@ static noinline int copy_items(struct btrfs_trans_handle 
*trans,
                 * or deletes of this inode don't have to relog the inode
                 * again
                 */
-               if (btrfs_key_type(ins_keys + i) == BTRFS_EXTENT_DATA_KEY) {
+               if (btrfs_key_type(ins_keys + i) ==
+                                               BTRFS_EXTENT_DATA_KEY && csum) {
                        int found_type;
                        extent = btrfs_item_ptr(src, start_slot + i,
                                                struct btrfs_file_extent_item);
@@ -2836,6 +2838,7 @@ static int btrfs_log_inode(struct btrfs_trans_handle 
*trans,
        int ins_start_slot = 0;
        int ins_nr;
        u64 transid;
+       int csum = (BTRFS_I(inode)->flags & BTRFS_INODE_NODATASUM) ? 0 : 1;
 
        /*
         * We use transid in btrfs_search_forward() as a filter, in order to
@@ -2906,7 +2909,7 @@ filter:
                if (ins_nr) {
                        ret = copy_items(trans, inode, dst_path, src,
                                         ins_start_slot,
-                                        ins_nr, inode_only);
+                                        ins_nr, inode_only, csum);
                        if (ret) {
                                err = ret;
                                goto out_unlock;
@@ -2925,7 +2928,7 @@ next_slot:
                if (ins_nr) {
                        ret = copy_items(trans, inode, dst_path, src,
                                         ins_start_slot,
-                                        ins_nr, inode_only);
+                                        ins_nr, inode_only, csum);
                        if (ret) {
                                err = ret;
                                goto out_unlock;
@@ -2946,7 +2949,7 @@ next_slot:
        if (ins_nr) {
                ret = copy_items(trans, inode, dst_path, src,
                                 ins_start_slot,
-                                ins_nr, inode_only);
+                                ins_nr, inode_only, csum);
                if (ret) {
                        err = ret;
                        goto out_unlock;
-- 
1.6.5.2

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to