f2fs: fix to update page flag
authorChao Yu <chao2.yu@samsung.com>
Wed, 8 Jul 2015 10:24:38 +0000 (18:24 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Tue, 4 Aug 2015 21:09:57 +0000 (14:09 -0700)
This patch fixes to update page flag (e.g. Uptodate/cold flag) in
->write_begin.

Otherwise, page will be non-uptodate when we try to write entire
page, and cold data flag in page will not be clean when gced page
is being rewritten.

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

index c9d0f8b06d15438b721309b7f705877a7e6b21c9..de55c088948f7bb09522328dc0e47b3f6b59a360 100644 (file)
@@ -1790,8 +1790,10 @@ put_next:
        f2fs_put_dnode(&dn);
        f2fs_unlock_op(sbi);
 
-       if ((len == PAGE_CACHE_SIZE) || PageUptodate(page))
-               return 0;
+       if (len == PAGE_CACHE_SIZE)
+               goto out_update;
+       if (PageUptodate(page))
+               goto out_clear;
 
        f2fs_wait_on_page_writeback(page, DATA);
 
@@ -1801,7 +1803,7 @@ put_next:
 
                /* Reading beyond i_size is simple: memset to zero */
                zero_user_segments(page, 0, start, end, PAGE_CACHE_SIZE);
-               goto out;
+               goto out_update;
        }
 
        if (dn.data_blkaddr == NEW_ADDR) {
@@ -1839,8 +1841,9 @@ put_next:
                        }
                }
        }
-out:
+out_update:
        SetPageUptodate(page);
+out_clear:
        clear_cold_data(page);
        return 0;