f2fs: fix to reset variable correctlly
authorFan Li <fanofcode.li@samsung.com>
Tue, 15 Dec 2015 09:02:41 +0000 (17:02 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Wed, 16 Dec 2015 16:58:04 +0000 (08:58 -0800)
f2fs_map_blocks will set m_flags and m_len to 0, so we don't need to
reset m_flags ourselves, but have to reset m_len to correct value
before use it again.

Signed-off-by: Fan li <fanofcode.li@samsung.com>
Reviewed-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/file.c

index c6d909e9661e98c3376c1e4e7c7290e2934be424..1f5892f38018951d53c702d5c48161fe24ba1fb0 100644 (file)
@@ -1700,7 +1700,6 @@ static int f2fs_defragment_range(struct f2fs_sb_info *sbi,
        }
 
        map.m_lblk = pg_start;
-       map.m_len = pg_end - pg_start;
 
        /*
         * lookup mapping info in dnode page cache, skip defragmenting if all
@@ -1708,14 +1707,13 @@ static int f2fs_defragment_range(struct f2fs_sb_info *sbi,
         * in logical blocks.
         */
        while (map.m_lblk < pg_end) {
-               map.m_flags = 0;
+               map.m_len = pg_end - map.m_lblk;
                err = f2fs_map_blocks(inode, &map, 0, F2FS_GET_BLOCK_READ);
                if (err)
                        goto out;
 
                if (!(map.m_flags & F2FS_MAP_FLAGS)) {
                        map.m_lblk++;
-                       map.m_len--;
                        continue;
                }
 
@@ -1726,7 +1724,6 @@ static int f2fs_defragment_range(struct f2fs_sb_info *sbi,
                blk_end = map.m_pblk + map.m_len;
 
                map.m_lblk += map.m_len;
-               map.m_len = pg_end - map.m_lblk;
        }
 
        if (!fragmented)
@@ -1752,14 +1749,13 @@ static int f2fs_defragment_range(struct f2fs_sb_info *sbi,
                int cnt = 0;
 
 do_map:
-               map.m_flags = 0;
+               map.m_len = pg_end - map.m_lblk;
                err = f2fs_map_blocks(inode, &map, 0, F2FS_GET_BLOCK_READ);
                if (err)
                        goto clear_out;
 
                if (!(map.m_flags & F2FS_MAP_FLAGS)) {
                        map.m_lblk++;
-                       map.m_len--;
                        continue;
                }
 
@@ -1784,7 +1780,6 @@ do_map:
                }
 
                map.m_lblk = idx;
-               map.m_len = pg_end - idx;
 
                if (idx < pg_end && cnt < blk_per_seg)
                        goto do_map;