Commit 746e2403 authored by Yunlei He's avatar Yunlei He Committed by Jaegeuk Kim

f2fs: add a case of no need to read a page in write begin

If the range we write cover the whole valid data in the last page,
we do not need to read it.
Signed-off-by: default avatarYunlei He <heyunlei@huawei.com>
[Jaegeuk Kim: nullify the remaining area (fix: xfstests/f2fs/001)]
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 7855eba4
...@@ -1715,6 +1715,11 @@ static int f2fs_write_begin(struct file *file, struct address_space *mapping, ...@@ -1715,6 +1715,11 @@ static int f2fs_write_begin(struct file *file, struct address_space *mapping,
if (len == PAGE_SIZE || PageUptodate(page)) if (len == PAGE_SIZE || PageUptodate(page))
return 0; return 0;
if (!(pos & (PAGE_SIZE - 1)) && (pos + len) >= i_size_read(inode)) {
zero_user_segment(page, len, PAGE_SIZE);
return 0;
}
if (blkaddr == NEW_ADDR) { if (blkaddr == NEW_ADDR) {
zero_user_segment(page, 0, PAGE_SIZE); zero_user_segment(page, 0, PAGE_SIZE);
SetPageUptodate(page); SetPageUptodate(page);
...@@ -1768,7 +1773,7 @@ static int f2fs_write_end(struct file *file, ...@@ -1768,7 +1773,7 @@ static int f2fs_write_end(struct file *file,
* let generic_perform_write() try to copy data again through copied=0. * let generic_perform_write() try to copy data again through copied=0.
*/ */
if (!PageUptodate(page)) { if (!PageUptodate(page)) {
if (unlikely(copied != PAGE_SIZE)) if (unlikely(copied != len))
copied = 0; copied = 0;
else else
SetPageUptodate(page); SetPageUptodate(page);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment