summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorSahitya Tummala <stummala@codeaurora.org>2018-10-05 10:47:39 +0530
committerJaegeuk Kim <jaegeuk@kernel.org>2018-10-16 09:37:00 -0700
commitc75f2feb80eb8cbb579a3677dce4c165f3102a04 (patch)
tree45eb0768b01ec865e008f4c2d8fd8c6820b6f8a4 /fs
parent3b30eb19dcf1f79d1c3283da8c5f064be6175fbe (diff)
downloadlinux-stable-c75f2feb80eb8cbb579a3677dce4c165f3102a04.tar.gz
linux-stable-c75f2feb80eb8cbb579a3677dce4c165f3102a04.tar.bz2
linux-stable-c75f2feb80eb8cbb579a3677dce4c165f3102a04.zip
f2fs: do not update REQ_TIME in case of error conditions
The REQ_TIME should be updated only in case of success cases as followed at all other places in the file system. Signed-off-by: Sahitya Tummala <stummala@codeaurora.org> Reviewed-by: Chao Yu <yuchao0@huawei.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs')
-rw-r--r--fs/f2fs/dir.c2
-rw-r--r--fs/f2fs/file.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/fs/f2fs/dir.c b/fs/f2fs/dir.c
index c77a58038709..e02db5db62dc 100644
--- a/fs/f2fs/dir.c
+++ b/fs/f2fs/dir.c
@@ -655,9 +655,9 @@ int f2fs_do_tmpfile(struct inode *inode, struct inode *dir)
f2fs_put_page(page, 1);
clear_inode_flag(inode, FI_NEW_INODE);
+ f2fs_update_time(F2FS_I_SB(inode), REQ_TIME);
fail:
up_write(&F2FS_I(inode)->i_sem);
- f2fs_update_time(F2FS_I_SB(inode), REQ_TIME);
return err;
}
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
index b1aaa73e1eeb..543c742f8bd7 100644
--- a/fs/f2fs/file.c
+++ b/fs/f2fs/file.c
@@ -106,6 +106,7 @@ static vm_fault_t f2fs_vm_page_mkwrite(struct vm_fault *vmf)
SetPageUptodate(page);
f2fs_update_iostat(sbi, APP_MAPPED_IO, F2FS_BLKSIZE);
+ f2fs_update_time(sbi, REQ_TIME);
trace_f2fs_vm_page_mkwrite(page, DATA);
out_sem:
@@ -114,7 +115,6 @@ out_sem:
f2fs_balance_fs(sbi, dn.node_changed);
sb_end_pagefault(inode->i_sb);
- f2fs_update_time(sbi, REQ_TIME);
err:
return block_page_mkwrite_return(err);
}